Github user violetagg commented on the issue:
https://github.com/apache/tomcat/pull/56
> I noticed instances of the code where curly braces were omitted for
single-statement ifs,
Yep, we know that. If you take a look at the history you will see that
typically when we prepare changes first make a change with formatting if we see
that it is needed and then implement the change.
We use
- the standard Eclipse formatting
- spaces only
- lines up to 100 chars for code and 80 chars for javadoc, comments etc.
- some additional settings for the compiler and javadoc
http://svn.apache.org/repos/asf/tomcat/trunk/res/ide-support/eclipse/java-compiler-errors-warnings.txt
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
