You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2013/11/01 19:51:20 UTC

svn commit: r1538032 - /commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/xpm/XpmImageParser.java

Author: ggregory
Date: Fri Nov  1 18:51:20 2013
New Revision: 1538032

URL: http://svn.apache.org/r1538032
Log:
Statement unnecessarily nested within else clause.

Modified:
    commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/xpm/XpmImageParser.java

Modified: commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/xpm/XpmImageParser.java
URL: http://svn.apache.org/viewvc/commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/xpm/XpmImageParser.java?rev=1538032&r1=1538031&r2=1538032&view=diff
==============================================================================
--- commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/xpm/XpmImageParser.java (original)
+++ commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/xpm/XpmImageParser.java Fri Nov  1 18:51:20 2013
@@ -364,9 +364,8 @@ public class XpmImageParser extends Imag
             loadColorNames();
             if (colorNames.containsKey(color)) {
                 return (colorNames.get(color)).intValue();
-            } else {
-                return 0x00000000;
             }
+            return 0x00000000;
         }
     }
     
@@ -645,9 +644,8 @@ public class XpmImageParser extends Imag
             final char zeroes[] = new char[6 - hex.length()];
             Arrays.fill(zeroes, '0');
             return "#" + new String(zeroes) + hex;
-        } else {
-            return "#" + hex;
         }
+        return "#" + hex;
     }
 
     @Override