As someone reviewing code I don't want to be distracted by alternative styles for placing curly braces

[pdurbin]

All of the developers at IQSS use Netbeans to edit Dataverse code and at http://guides.dataverse.org/en/4.9.2/developers/coding-style.html#format-code-you-changed-with-netbeans we kindly ask contributors to format the code they changed (not all the code in the file they touched) with Netbeans. That way, people doing code review are not distracted by differences of style and can focus on how the behavior of the code is changing.

In the spirit of small chunks, I'd like to update the dev guide to specifically address placement of curly braces. I don't know how to express this in Checkstyle (or even what it's called), but here are the settings Netbeans uses for braces:

[pameyer]

This looks like what astyle calls "attached" or "java" braces (http://astyle.sourceforge.net/astyle.html#_style=java). Does that sound reasonable to folks?

[pdurbin]

@pameyer --style=java / --style=attach is quite close but can it be combined with --add-braces / -j which means "add braces to unbraced one line conditional statements"?

[pameyer]

@pdurbin Probably, but I'll find out for sure.

[pdurbin]

@pameyer thanks for pull request #4995! I added a bit more and moved it to QA.