Skip to content

Commit 06e357f

Browse files
committed
PDFBOX-5660: refactor, as suggested by Valery Bokov, closes #373
git-svn-id: https://svn.apache.org/repos/asf/pdfbox/trunk@1930555 13f79535-47bb-0310-9956-ffa450edef68
1 parent 28423e1 commit 06e357f

File tree

1 file changed

+14
-8
lines changed

1 file changed

+14
-8
lines changed

pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDColorSpace.java

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -245,20 +245,26 @@ private static PDColorSpace createFromCOSObject(COSObject colorSpace, PDResource
245245
throws IOException
246246
{
247247
PDColorSpace cs;
248-
if (resources != null && resources.getResourceCache() != null)
248+
if (resources != null)
249249
{
250250
ResourceCache resourceCache = resources.getResourceCache();
251-
cs = resourceCache.getColorSpace(colorSpace);
252-
if (cs != null)
251+
if (resourceCache != null)
253252
{
254-
return cs;
253+
cs = resourceCache.getColorSpace(colorSpace);
254+
if (cs != null)
255+
{
256+
return cs;
257+
}
258+
}
259+
cs = create(colorSpace.getObject(), resources);
260+
if (resourceCache != null && cs != null)
261+
{
262+
resourceCache.put(colorSpace, cs);
255263
}
256264
}
257-
cs = create(colorSpace.getObject(), resources);
258-
if (resources != null && resources.getResourceCache() != null && cs != null)
265+
else
259266
{
260-
ResourceCache resourceCache = resources.getResourceCache();
261-
resourceCache.put(colorSpace, cs);
267+
cs = create(colorSpace.getObject();
262268
}
263269
return cs;
264270
}

0 commit comments

Comments
 (0)