You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ms...@apache.org on 2018/02/19 07:23:43 UTC
svn commit: r1824717 -
/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSWriter.java
Author: msahyoun
Date: Mon Feb 19 07:23:43 2018
New Revision: 1824717
URL: http://svn.apache.org/viewvc?rev=1824717&view=rev
Log:
PDFBOX-4071: simplify code, test no longer neccesary as idArray is never null
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSWriter.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSWriter.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSWriter.java?rev=1824717&r1=1824716&r2=1824717&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSWriter.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSWriter.java Mon Feb 19 07:23:43 2018
@@ -1319,7 +1319,7 @@ public class COSWriter implements ICOSVi
COSDocument cosDoc = pdDocument.getDocument();
COSDictionary trailer = cosDoc.getTrailer();
- COSArray idArray = null;
+ COSArray idArray;
boolean missingID = true;
COSBase base = trailer.getDictionaryObject(COSName.ID);
if (base instanceof COSArray)
@@ -1334,11 +1334,6 @@ public class COSWriter implements ICOSVi
{
idArray = new COSArray();
}
- // check for an existing documentID
- if (idArray != null && idArray.size() == 2)
- {
- missingID = false;
- }
if( missingID || incrementalUpdate)
{
MessageDigest md5;