You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by le...@apache.org on 2013/03/11 20:29:59 UTC

svn commit: r1455294 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDNameTreeNode.java

Author: lehmi
Date: Mon Mar 11 19:29:59 2013
New Revision: 1455294

URL: http://svn.apache.org/r1455294
Log:
PDFBOX-1380: fixed a remaining issue with the Names values

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDNameTreeNode.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDNameTreeNode.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDNameTreeNode.java?rev=1455294&r1=1455293&r2=1455294&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDNameTreeNode.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/common/PDNameTreeNode.java Mon Mar 11 19:29:59 2013
@@ -153,15 +153,20 @@ public class PDNameTreeNode implements C
             for (PDNameTreeNode kidsNode : kids)
                 kidsNode.setParent(this);
             node.setItem( COSName.KIDS, COSArrayList.converterToCOSArray( kids ) );
-            calculateLimits();
+            // root nodes with kids don't have Names
+            if (isRootNode())
+            {
+                node.setItem(COSName.NAMES, null);
+            }
         }
         else 
         {
-            // Remove Names and Limits if there are no kids
-            node.setItem(COSName.NAMES, null);
-            node.setItem(COSName.LIMITS, null);
+            // remove kids
             node.setItem(COSName.KIDS, null);
+            // remove Limits 
+            node.setItem(COSName.LIMITS, null);
         }
+        calculateLimits();
     }
 
     private void calculateLimits()