Fix CS warnings in sal-clustering-commons and enable enforcement
authorTom Pantelis <tpanteli@brocade.com>
Wed, 12 Oct 2016 20:28:13 +0000 (16:28 -0400)
committerTom Pantelis <tpanteli@brocade.com>
Fri, 21 Oct 2016 11:36:17 +0000 (11:36 +0000)
commit89bc78a1622608c9374e861580a13fbdcf9a14f5
treea7151623d0b79f1bdadd6a6e07a7da1d7b656288
parent488954d6dada1f5963cd5f4d4ebb19264575275f
Fix CS warnings in sal-clustering-commons and enable enforcement

Fixed checkstyle warnings and enabled enforcement. Most of the
warnings/changes were for:
 - white space before if/for/while/catch
 - white space before beginning brace
 - line too long
 - illegal catching of Exception (suppressed)
 - variable name too short
 - indentation
 - local vars/params hiding a field
 - putting overloaded methods close to one another
 - remove unused vars
 - convert functional interfaces to lambdas (eclipse save action)
 - empty catch block - added comment or Throwables.propagate as appropriate
 - missing period after first sentence in javadoc
 - missing first sentence in javadoc
 - adding final for locals declared too far from first usage

Change-Id: I0a6690f97820e8fb670f209221d8e4e2f1cf5d8b
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>
14 files changed:
node/NormalizedNodeToNodeCodecTest.java
node/utils/NodeIdentifierFactoryTest.java
node/utils/PathUtilsTest.java
node/utils/QNameFactoryTest.java
node/utils/serialization/NormalizedNodeSerializerTest.java
node/utils/serialization/PathArgumentSerializerTest.java
node/utils/serialization/ValueSerializerTest.java
node/utils/serialization/ValueTypeTest.java
node/utils/stream/NormalizedNodeStreamReaderWriterTest.java
node/utils/stream/SampleNormalizedNodeSerializable.java
node/utils/stream/ValueTypesTest.java
node/utils/transformer/NormalizedNodePrunerTest.java
util/InstanceIdentifierUtilsTest.java
util/TestModel.java