Sonar fixes

constant name to match the regular expression '^[A-Z][A-Z0-9]*(_[A-Z0-9]+)*$'
reorder "final static" -> "static final"

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1885613 13f79535-47bb-0310-9956-ffa450edef68

2 files changed, 3 insertions, 7 deletions

diff --git a/src/scratchpad/testcases/org/apache/poi/hwpf/HWPFTestDataSamples.java b/src/scratchpad/testcases/org/apache/poi/hwpf/HWPFTestDataSamples.java
index ea5a499111..b4efb828d3 100644
--- a/src/scratchpad/testcases/org/apache/poi/hwpf/HWPFTestDataSamples.java
+++ b/src/scratchpad/testcases/org/apache/poi/hwpf/HWPFTestDataSamples.java
@@ -30,8 +30,7 @@ import org.apache.poi.util.POILogger;
 
 public class HWPFTestDataSamples {
 
-    private static final POILogger logger = POILogFactory
-            .getLogger( HWPFTestDataSamples.class );
+    private static final POILogger LOG = POILogFactory.getLogger( HWPFTestDataSamples.class );
 
     public static HWPFDocument openSampleFile(String sampleFileName) {
         try {
@@ -66,7 +65,7 @@ public class HWPFTestDataSamples {
                 final long endUnzip = System.currentTimeMillis();
                 byte[] byteArray = baos.toByteArray();
 
-                logger.log(POILogger.DEBUG, "Unzipped in ",
+                LOG.log(POILogger.DEBUG, "Unzipped in ",
                         Long.valueOf(endUnzip - start), " ms -- ",
                         Long.valueOf(byteArray.length), " byte(s)");
 
@@ -74,7 +73,7 @@ public class HWPFTestDataSamples {
                 HWPFDocument doc = new HWPFDocument(bais);
                 final long endParse = System.currentTimeMillis();
 
-                logger.log(POILogger.DEBUG, "Parsed in ",
+                LOG.log(POILogger.DEBUG, "Parsed in ",
                         Long.valueOf(endParse - start), " ms");
 
                 return doc;
diff --git a/src/scratchpad/testcases/org/apache/poi/hwpf/usermodel/TestBugs.java b/src/scratchpad/testcases/org/apache/poi/hwpf/usermodel/TestBugs.java
index f1d1cb578f..c30d80169f 100644
--- a/src/scratchpad/testcases/org/apache/poi/hwpf/usermodel/TestBugs.java
+++ b/src/scratchpad/testcases/org/apache/poi/hwpf/usermodel/TestBugs.java
@@ -63,9 +63,6 @@ import org.w3c.dom.Document;
  * against HWPF
  */
 class TestBugs {
-
-    private static final POILogger logger = POILogFactory.getLogger(TestBugs.class);
-
     private static void assertEqualsIgnoreNewline(String expected, String actual) {
         String newExpected = expected.replaceAll("\r\n", "\n")
             .replaceAll("\r", "\n").trim();