Flaws in my opinion as a dm, and player are that they bring in an interesting aspect of taking something away, and giving something, but I am angry as a dm tht the wizard will take noncombatant, and suffer a -2 to melee attack, and that is something most mages will barely every use. I would allow if they took a flaw that affected tem directly such as frail for a mage, and lose more of their low hit points. Fighters could take murkey eyed, and things like that. As a player I love the idea of more roleplaying, and many other things.