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 2022/04/22 13:17:00 UTC

svn commit: r1900158 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/documentinterchange/logicalstructure/PDUserAttributeObject.java

Author: tilman
Date: Fri Apr 22 13:17:00 2022
New Revision: 1900158

URL: http://svn.apache.org/viewvc?rev=1900158&view=rev
Log:
PDFBOX-4892: update state only if needed, as suggested by Valery Bokov

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/documentinterchange/logicalstructure/PDUserAttributeObject.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/documentinterchange/logicalstructure/PDUserAttributeObject.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/documentinterchange/logicalstructure/PDUserAttributeObject.java?rev=1900158&r1=1900157&r2=1900158&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/documentinterchange/logicalstructure/PDUserAttributeObject.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/documentinterchange/logicalstructure/PDUserAttributeObject.java Fri Apr 22 13:17:00 2022
@@ -111,8 +111,10 @@ public class PDUserAttributeObject exten
             return;
         }
         COSArray p = getCOSObject().getCOSArray(COSName.P);
-        p.remove(userProperty.getCOSObject());
-        this.notifyChanged();
+        if (p.remove(userProperty.getCOSObject()))
+        {
+            this.notifyChanged();
+        }
     }
 
     /**