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 2018/07/10 18:52:43 UTC

svn commit: r1835596 - /pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/LosslessFactory.java

Author: tilman
Date: Tue Jul 10 18:52:43 2018
New Revision: 1835596

URL: http://svn.apache.org/viewvc?rev=1835596&view=rev
Log:
PDFBOX-4184: fix heuristics

Modified:
    pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/LosslessFactory.java

Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/LosslessFactory.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/LosslessFactory.java?rev=1835596&r1=1835595&r2=1835596&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/LosslessFactory.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/LosslessFactory.java Tue Jul 10 18:52:43 2018
@@ -663,7 +663,8 @@ public final class LosslessFactory
             long sum = 0;
             for (byte aDataRawRowSub : dataRawRowSub)
             {
-                sum += aDataRawRowSub & 0xFF;
+                // https://www.w3.org/TR/PNG-Encoders.html#E.Filter-selection
+                sum += Math.abs(aDataRawRowSub);
             }
             return sum;
         }