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 2021/05/21 06:26:23 UTC
svn commit: r1890058 -
/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/JPEGFactory.java
Author: tilman
Date: Fri May 21 06:26:22 2021
New Revision: 1890058
URL: http://svn.apache.org/viewvc?rev=1890058&view=rev
Log:
PDFBOX-5196, PDFBOX-5137: use numFrameComponents instead of numScanComponents to get component count, as suggested by Matej Risek and Stéphane Vallée; closes #110
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/JPEGFactory.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/JPEGFactory.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/JPEGFactory.java?rev=1890058&r1=1890057&r2=1890058&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/JPEGFactory.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/image/JPEGFactory.java Fri May 21 06:26:22 2021
@@ -202,12 +202,12 @@ public final class JPEGFactory
try
{
XPath xpath = XPathFactory.newInstance().newXPath();
- String numScanComponents = xpath.evaluate("markerSequence/sos/@numScanComponents", root);
- if (numScanComponents.isEmpty())
+ String numFrameComponents = xpath.evaluate("markerSequence/sof/@numFrameComponents", root);
+ if (numFrameComponents.isEmpty())
{
return 0;
}
- return Integer.parseInt(numScanComponents);
+ return Integer.parseInt(numFrameComponents);
}
catch (NumberFormatException | XPathExpressionException ex)
{