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 2015/01/10 22:59:24 UTC

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

Author: tilman
Date: Sat Jan 10 21:59:24 2015
New Revision: 1650815

URL: http://svn.apache.org/r1650815
Log:
PDFBOX-2576: the result of int divide by int is always an int

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=1650815&r1=1650814&r2=1650815&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 Sat Jan 10 21:59:24 2015
@@ -238,9 +238,9 @@ public final class PDIndexed extends PDS
         int numComponents = baseColorSpace.getNumberOfComponents();
 
         // some tables are too short
-        if (lookupData.length / numComponents < maxIndex + 1f)
+        if (lookupData.length / numComponents < maxIndex + 1)
         {
-            maxIndex = (int)Math.floor(lookupData.length / numComponents) - 1;
+            maxIndex = lookupData.length / numComponents - 1;
         }
         actualMaxIndex = maxIndex;  // TODO "actual" is ugly, tidy this up