We had a discussion about this long ago on the developers mailing list.  We can discuss changing the default indentation in `core/contrib/reindent.pl` and then apply it.  As long as we're consistent... I know @nickanderson prefers different indentation for instance, and I agree this indentation requires line breaks.