FindBugs

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/codec/trunk@1744746 13f79535-47bb-0310-9956-ffa450edef68

Author: sebbASF

src/main/java/org/apache/commons/codec/cli/Digest.java

@@ -119,7 +119,10 @@ private void run(String prefix, final MessageDigest messageDigest) throws IOExce
             if (file.isFile()) {
                 println(prefix, DigestUtils.digest(messageDigest, file), source);
             } else if (file.isDirectory()) {
-                run(prefix, messageDigest, file.listFiles());
+                final File[] listFiles = file.listFiles();
+                if (listFiles != null) {
+                    run(prefix, messageDigest, listFiles);
+                }
             } else {
                 // use the default charset for the command-line parameter
                 final byte[] bytes = source.getBytes(Charset.defaultCharset());

src/main/java/org/apache/commons/codec/digest/Md5Crypt.java

@@ -186,7 +186,7 @@ public static String md5Crypt(final byte[] keyBytes, final String salt, final St
         } else {
             final Pattern p = Pattern.compile("^" + prefix.replace("$", "\\$") + "([\\.\\/a-zA-Z0-9]{1,8}).*");
             final Matcher m = p.matcher(salt);
-            if (m == null || !m.find()) {
+            if (!m.find()) {
                 throw new IllegalArgumentException("Invalid salt value: " + salt);
             }
             saltString = m.group(1);

src/main/java/org/apache/commons/codec/digest/PureJavaCrc32C.java

@@ -90,7 +90,7 @@ public void update(byte[] b, int off, int len) {
       case 2: localCrc = (localCrc >>> 8) ^ T[T8_0_start + ((localCrc ^ b[off++]) & 0xff)];
       case 1: localCrc = (localCrc >>> 8) ^ T[T8_0_start + ((localCrc ^ b[off++]) & 0xff)];
       default:
-        /* nothing */
+        break; // satisfy Findbugs
     }
 
     // Publish crc out to object

src/main/java/org/apache/commons/codec/digest/Sha2Crypt.java

@@ -143,7 +143,7 @@ private static String sha2Crypt(final byte[] keyBytes, final String salt, final
         }
 
         final Matcher m = SALT_PATTERN.matcher(salt);
-        if (m == null || !m.find()) {
+        if (!m.find()) {
             throw new IllegalArgumentException("Invalid salt value: " + salt);
         }
         if (m.group(3) != null) {

src/main/java/org/apache/commons/codec/language/MatchRatingApproachEncoder.java

@@ -185,9 +185,9 @@ int getMinRating(final int sumLength) {
 
         if (sumLength <= FOUR) {
             minRating = FIVE;
-        } else if (sumLength >= FIVE && sumLength <= SEVEN) {
+        } else if (sumLength <= SEVEN) { // aready know it is at least 5
             minRating = FOUR;
-        } else if (sumLength >= EIGHT && sumLength <= ELEVEN) {
+        } else if (sumLength <= ELEVEN) { // aready know it is at least 8
             minRating = THREE;
         } else if (sumLength == TWELVE) {
             minRating = TWO;