Hi all,

no vote yet, but let me add some comments:

Line length (section Sun code conventions 4.1)

I would suggest to exclude this point and allow longer lines than 80
chars. The argument about terminals is no longer valid nowadays and
with typical screen resolutions even lines with 150 chars do fit well.
Especially breaking up long log messages into several lines makes the
code much harder to read in my opinion.

Intend size

I have done some testing based on the suggested code formatter
configuration. In general the  4 spaces intend works best for simple
code segments while the 2 spaces indent feels better for more complex
parts with a lot of hierarchy levels.
As long we do not stick to the 80 char line length I do not have any
preference. In case we decide to respect the 80 char line length I do
have a strong preference for using only 2 space intends.

License Headers

A short google search for "mvn plugin license" showed up several
options for plugin that are able to manage license headers. I suggest
to use one of those.

best
Rupert Westenthaler

Reply via email to