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 2019/05/10 08:24:52 UTC
svn commit: r1859068 -
/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSWriter.java
Author: tilman
Date: Fri May 10 08:24:52 2019
New Revision: 1859068
URL: http://svn.apache.org/viewvc?rev=1859068&view=rev
Log:
PDFBOX-4540: don't construct new decoder each time, as suggested by Jonathan
Modified:
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSWriter.java
Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSWriter.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSWriter.java?rev=1859068&r1=1859067&r2=1859068&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSWriter.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdfwriter/COSWriter.java Fri May 10 08:24:52 2019
@@ -891,15 +891,13 @@ public class COSWriter implements ICOSVi
{
actual = ((COSObject)obj).getObject();
}
- COSObjectKey key = null;
- if( actual != null )
+ // PDFBOX-4540: because objectKeys is accessible from outside, it is possible
+ // that a COSObject obj is already in the objectKeys map.
+ COSObjectKey key = objectKeys.get(obj);
+ if( key == null && actual != null )
{
key = objectKeys.get(actual);
}
- if( key == null )
- {
- key = objectKeys.get(obj);
- }
if (key == null)
{
setNumber(getNumber()+1);