On 14/09/2012 01:21, Brad Wetmore wrote:
Netbean's automatic formatting does a pretty good job with new code.
However, I think the general advice is to not change existing code
just because. When you're dealing with multiple release families, it
makes the merges much more difficult.
Brad
One think that Paul Sandoz suggested recently is that we should have a
NB template that folks can use to avoid some discussions/debates on
styles. It would be great for someone to run with that, the hard part is
of course that it will be impossible to get agreement. Personally I find
NB's defaults okay but there are several cases where its indenting is
horrible.
Anyway, the main advice I think is to keep things locally consistent
where possible. Also major refactoring or formatting in a bug fix is a
royal pain for reviewers.
-Alan
