svn commit: r1930557 - pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color

[tilman]

Author: tilman
Date: Sun Dec 14 13:55:40 2025
New Revision: 1930557

Log:
PDFBOX-5660: refactor, as suggested by Valery Bokov, closes #373

Modified:
   
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColorSpace.java

Modified: 
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColorSpace.java
==============================================================================
--- 
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColorSpace.java
 Sun Dec 14 13:55:36 2025        (r1930556)
+++ 
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColorSpace.java
 Sun Dec 14 13:55:40 2025        (r1930557)
@@ -245,20 +245,26 @@ public abstract class PDColorSpace imple
             throws IOException
     {
         PDColorSpace cs;
-        if (resources != null && resources.getResourceCache() != null)
+        if (resources != null)
         {
             ResourceCache resourceCache = resources.getResourceCache();
-            cs = resourceCache.getColorSpace(colorSpace);
-            if (cs != null)
+            if (resourceCache != null)
             {
-                return cs;
+                cs = resourceCache.getColorSpace(colorSpace);
+                if (cs != null)
+                {
+                    return cs;
+                }
+            }
+            cs = create(colorSpace.getObject(), resources);
+            if (resourceCache != null && cs != null)
+            {
+                resourceCache.put(colorSpace, cs);
             }
         }
-        cs = create(colorSpace.getObject(), resources);
-        if (resources != null && resources.getResourceCache() != null && cs != 
null)
+        else
         {
-            ResourceCache resourceCache = resources.getResourceCache();
-            resourceCache.put(colorSpace, cs);
+            cs = create(colorSpace.getObject());
         }
         return cs;
     }