aBothe / Mono-D

D Add-In for MonoDevelop
http://wiki.dlang.org/Mono-D
Other
113 stars 26 forks source link

Ident after \n in AttributeList \n Declaration #572

Closed CZDanol closed 9 years ago

CZDanol commented 9 years ago

Hey,

could you please add automatic indenting to this case? So the code would look:

public void foo(); @safe void test();

instead of

public void foo(); @safe void test();

Thanks :)

aBothe commented 9 years ago

Now that I think about it again, there once has been a reason why I decided to let declarations be unindented. What does the official D style guide say concerning this question?

CZDanol commented 9 years ago

If you mean this page: http://dlang.org/dstyle.html then nothing. I am not aware of any other pages about this topic.

aBothe commented 9 years ago

https://github.com/D-Programming-Language/phobos/blob/master/std/base64.d#L136 That's the reason. I think I've handled this topic back a year ago or so, because someone actually complained that it's not following some style guides or phobos-compliancies or whatever

CZDanol commented 9 years ago

Oh, okay, then :)

aBothe commented 9 years ago

I'd recommend then to put accesibility modifiers (private,public,,) into the same line, UDAs into separate ones, so this issue won't even occur + you're coding like the guys from phobos :)