Author: damjan Date: Sat Jan 28 18:41:17 2017 New Revision: 1780731 URL: http://svn.apache.org/viewvc?rev=1780731&view=rev Log: Match XPM color names case insensitively.
Patch by: me 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=1780731&r1=1780730&r2=1780731&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 Sat Jan 28 18:41:17 2017 @@ -36,6 +36,7 @@ import java.io.PrintWriter; import java.util.ArrayList; import java.util.Arrays; import java.util.HashMap; +import java.util.Locale; import java.util.Map; import java.util.Map.Entry; import java.util.Properties; @@ -91,7 +92,7 @@ public class XpmImageParser extends Imag final int green = Integer.parseInt(line.substring(4, 7).trim()); final int blue = Integer.parseInt(line.substring(8, 11).trim()); final String colorName = line.substring(11).trim(); - colors.put(colorName, 0xff000000 | (red << 16) + colors.put(colorName.toLowerCase(Locale.ENGLISH), 0xff000000 | (red << 16) | (green << 8) | blue); } catch (final NumberFormatException nfe) { throw new ImageReadException("Couldn't parse color in rgb.txt", nfe); @@ -353,8 +354,9 @@ public class XpmImageParser extends Imag return 0x00000000; } else { loadColorNames(); - if (colorNames.containsKey(color)) { - return colorNames.get(color); + final String colorLowercase = color.toLowerCase(Locale.ENGLISH); + if (colorNames.containsKey(colorLowercase)) { + return colorNames.get(colorLowercase); } return 0x00000000; }