Hi All: I only hope that this will not turn into a bike shedding thread...
Commons Configuration is one of the few components we have that uses the formatting rule (enforced by Checkstyle) where braces must be on separate lines. In the age of lambdas, this is, IMO, lame (a technical term ;-) for example: public static final ConfigurationConsumer<ConfigurationException> DEFAULT_INCLUDE_LISTENER = e -> { throw e; }; public static final ConfigurationConsumer<ConfigurationException> NOOP_INCLUDE_LISTENER = e -> { // noop }; Instead of: public static final ConfigurationConsumer<ConfigurationException> DEFAULT_INCLUDE_LISTENER = e -> { throw e; }; public static final ConfigurationConsumer<ConfigurationException> NOOP_INCLUDE_LISTENER = e -> { /* noop */ }; I propose a reformatting to use the "{ on the same line" which means that blocks go from: if (test) { // this } else { // that } to: if (test) { // this } else { // that } and so on. Gary