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/02/27 16:51:26 UTC

svn commit: r1854471 - /pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/PDFCloneUtility.java

Author: tilman
Date: Wed Feb 27 16:51:26 2019
New Revision: 1854471

URL: http://svn.apache.org/viewvc?rev=1854471&view=rev
Log:
PDFBOX-4475: remove code line that is called again immediately after (but keep the lines where the later call is not immediate)

Modified:
    pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/PDFCloneUtility.java

Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/PDFCloneUtility.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/PDFCloneUtility.java?rev=1854471&r1=1854470&r2=1854471&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/PDFCloneUtility.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/multipdf/PDFCloneUtility.java Wed Feb 27 16:51:26 2019
@@ -102,13 +102,11 @@ public class PDFCloneUtility
           else if( base instanceof COSObjectable && !(base instanceof COSBase) )
           {
               retval = cloneForNewDocument( ((COSObjectable)base).getCOSObject() );
-              clonedVersion.put( base, retval );
           }
           else if( base instanceof COSObject )
           {
               COSObject object = (COSObject)base;
               retval = cloneForNewDocument( object.getObject() );
-              clonedVersion.put( base, retval );
           }
           else if( base instanceof COSArray )
           {
@@ -119,7 +117,6 @@ public class PDFCloneUtility
                   newArray.add( cloneForNewDocument( array.get( i ) ) );
               }
               retval = newArray;
-              clonedVersion.put( base, retval );
           }
           else if( base instanceof COSStream )
           {
@@ -182,7 +179,6 @@ public class PDFCloneUtility
           if (!(base instanceof COSBase))
           {
               cloneMerge(base.getCOSObject(), target.getCOSObject());
-              clonedVersion.put(base, retval);
           }
           else if( base instanceof COSObject )
           {
@@ -194,7 +190,6 @@ public class PDFCloneUtility
               {
                   cloneMerge(((COSObject) base).getObject(), target);
               }
-              clonedVersion.put( base, retval );
           }
           else if( base instanceof COSArray )
           {
@@ -203,7 +198,6 @@ public class PDFCloneUtility
               {
                   ((COSArray)target).add( cloneForNewDocument( array.get( i ) ) );
               }
-              clonedVersion.put( base, retval );
           }
           else if( base instanceof COSStream )
           {