Github user vrozov commented on the pull request:
https://github.com/apache/incubator-apex-core/pull/125#issuecomment-150053367
The code style is not in stone. It needs to be convenient for developers to
follow and adhere. Both static and synchronized() designate block of code and
it seems that IntelliJ formats both the same way and there is no separate
setting for static and/or synchronized().
I can't find reference to K&R style regarding class/method scope. Here is
reference to [google code style](
https://google.github.io/styleguide/javaguide.html#s4.1.2-blocks-k-r-style).
Note that braces follow the Kernighan and Ritchie style ("Egyptian brackets")
for nonempty blocks and **block-like constructs**:
* No line break before the opening brace.
* Line break after the opening brace.
* Line break before the closing brace.
* Line break after the closing brace if that brace terminates a statement
or the body of a method, constructor or named class. For example, there is no
line break after the brace if it is followed by else or a comma.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
