You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2014/12/12 08:38:07 UTC

svn commit: r1644844 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDIndexed.java

Author: tilman
Date: Fri Dec 12 07:38:07 2014
New Revision: 1644844

URL: http://svn.apache.org/r1644844
Log:
PDFBOX-2561: don't multiply index with 255

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDIndexed.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDIndexed.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDIndexed.java?rev=1644844&r1=1644843&r2=1644844&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDIndexed.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDIndexed.java Fri Dec 12 07:38:07 2014
@@ -147,9 +147,9 @@ public final class PDIndexed extends PDS
         {
             throw new IllegalArgumentException("Indexed color spaces must have one color value");
         }
-
+        
         // scale and clamp input value
-        int index = Math.round(255f * value[0]);
+        int index = Math.round(value[0]);
         index = Math.max(index, 0);
         index = Math.min(index, actualMaxIndex);
 
@@ -197,7 +197,7 @@ public final class PDIndexed extends PDS
     }
 
     // returns "hival" array element
-    private int gettHival()
+    private int getHival()
     {
         return ((COSNumber) array.getObject(2)).intValue();
     }
@@ -234,7 +234,7 @@ public final class PDIndexed extends PDS
     private void readColorTable() throws IOException
     {
         byte[] lookupData = getLookupData();
-        int maxIndex = Math.min(gettHival(), 255);
+        int maxIndex = Math.min(getHival(), 255);
         int numComponents = baseColorSpace.getNumberOfComponents();
 
         // some tables are too short
@@ -274,10 +274,11 @@ public final class PDIndexed extends PDS
         array.set(2, high);
     }
 
+    @Override
     public String toString()
     {
         return "Indexed{base:" + baseColorSpace + " " +
-                "hival:" + gettHival() + " " +
+                "hival:" + getHival() + " " +
                 "lookup:(" + colorTable.length + " entries)}";
     }
 }