icsharpcode/NRefactory — pull request #384 by DavidKarlas (2014-02-22 06:22:22+00:00)
Title: Fixing Constructor Initializer

Mostly what code does can be seen from unit tests...
I tested all possible scenarios I could think of(adding multiple bank lines...) and it always worked as expected.
I talked to @mkrueger what default should be in MonoDevelop he said DoNotCare.
Visual studio has Before=NewLine and After=SameLine so I did so in CreateKRStyle(is this correct placement)?
A bit extra code explaining...
ForceSpaces are not needed any more since PlaceOnNewLine takes care of that.
Always setting curIndent.Push(IndentType.Block); doesn't hurt if its all in same line but works very well for anything(colon or initializer) that is in new line.
TestConstructorInitializerColonDontCare always set DoNotCare in case Mono default change... Is this Ok or should we test defaults aswell?