[REEF-360] Fix violations / reconfigure checkstyle checks in "Blocks" category

• Type: Sub-task
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.12
• Component/s: All
• Assignee: Mariia Mykhailova
• Reporter: Mariia Mykhailova
• Created: 10/Jun/15 23:02
• Updated: 17/Jun/15 01:36
• Resolved: 17/Jun/15 01:36

Description

1. LeftCurly ({ must be on the end of line) and NeedBraces ({} around code blocks): fix
2. AvoidNestedBlocks (prohibit "blocks used freely in code"): we have several violations, but some of them seem legit because they introduce local variables:
   • org/apache/reef/util/MemoryUtils.java, line 101
   • org/apache/reef/examples/suspend/SuspendDriver.java, line 280
   • org/apache/reef/webserver/HttpServerReefEventHandler.java, line 122
     The check can be configured to ignore blocks in case in a switch statement, but not the first one.
3. *EmptyBlock (prohibit blocks without any statements): we have multiple violations, majority of them due to empty catch clauses. The check can be reconfigured to allow blocks with comments only, or to enforce the check for only an explicit subset of block types (excluding catch blocks); it feels weird to add some kind of empty statement just for the sake of it being present.