cleanup findbugs report

git-svn-id: https://svn.code.sf.net/p/jackcess/code/jackcess/trunk@961 f203690c-595d-4dc9-a70b-905162fa7fd2

2 files changed, 28 insertions, 7 deletions

diff --git a/src/main/java/com/healthmarketscience/jackcess/impl/OleUtil.java b/src/main/java/com/healthmarketscience/jackcess/impl/OleUtil.java
index b1dbdb1..d8a2336 100644
--- a/src/main/java/com/healthmarketscience/jackcess/impl/OleUtil.java
+++ b/src/main/java/com/healthmarketscience/jackcess/impl/OleUtil.java
@@ -107,7 +107,7 @@ public class OleUtil
   /**
    * Creates a new OlBlob instance using the given information.
    */
-  public static OleBlob createBlob(OleBlob.Builder oleBuilder)
+  public static OleBlob createBlob(Builder oleBuilder)
     throws IOException
   {
     try {
@@ -183,7 +183,7 @@ public class OleUtil
     }
   }
 
-  private static byte[] writePackageHeader(OleBlob.Builder oleBuilder,
+  private static byte[] writePackageHeader(Builder oleBuilder,
                                            long contentLen) {
 
     byte[] prettyNameBytes = getZeroTermStrBytes(oleBuilder.getPrettyName());
@@ -229,7 +229,7 @@ public class OleUtil
     return headerBytes;
   }
 
-  private static byte[] writePackageStreamHeader(OleBlob.Builder oleBuilder) {
+  private static byte[] writePackageStreamHeader(Builder oleBuilder) {
 
     byte[] fileNameBytes = getZeroTermStrBytes(oleBuilder.getFileName());
     byte[] filePathBytes = getZeroTermStrBytes(oleBuilder.getFilePath());
@@ -264,7 +264,7 @@ public class OleUtil
     return headerBytes;
   }
 
-  private static byte[] writePackageStreamFooter(OleBlob.Builder oleBuilder) {
+  private static byte[] writePackageStreamFooter(Builder oleBuilder) {
 
     // note, these are _not_ zero terminated
     byte[] fileNameBytes = oleBuilder.getFileName().getBytes(OLE_UTF_CHARSET);
@@ -299,19 +299,19 @@ public class OleUtil
 
     // read outer package header
     int headerSize = bb.getShort();
-    int objType = bb.getInt();
+    /* int objType = */ bb.getInt();
     int prettyNameLen = bb.getShort();
     int classNameLen = bb.getShort();
     int prettyNameOff = bb.getShort();
     int classNameOff = bb.getShort();       
-    int objSize = bb.getInt();
+    /* int objSize = */ bb.getInt();
     String prettyName = readStr(bb, prettyNameOff, prettyNameLen);
     String className = readStr(bb, classNameOff, classNameLen);
     bb.position(headerSize);
 
     // read ole header
     int oleVer = bb.getInt();
-    int format = bb.getInt();
+    /* int format = */ bb.getInt();
 
     if(oleVer != OLE_VERSION) {
       return new UnknownContentImpl(blob);
diff --git a/src/site/pmd/custom-ruleset.xml b/src/site/pmd/custom-ruleset.xml
new file mode 100644
index 0000000..14cfd4f
--- /dev/null
+++ b/src/site/pmd/custom-ruleset.xml
@@ -0,0 +1,21 @@
+<?xml version="1.0"?>
+
+<ruleset name="Custom"
+    xmlns="http://pmd.sourceforge.net/ruleset/2.0.0"
+    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+    xsi:schemaLocation="http://pmd.sourceforge.net/ruleset/2.0.0 http://pmd.sourceforge.net/ruleset_2_0_0.xsd">
+  <description>
+Custom ruleset.
+  </description>
+
+  <rule ref="rulesets/java/basic.xml">
+    <exclude name="UselessParentheses"/>
+    <exclude name="CollapsibleIfStatements"/>
+  </rule>
+  <rule ref="rulesets/java/unusedcode.xml">
+    <exclude name="UnusedModifier"/>
+  </rule>
+  <rule ref="rulesets/java/imports.xml">
+    <exclude name="UnusedImports"/>
+  </rule>
+</ruleset>