You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ja...@apache.org on 2016/08/10 18:34:38 UTC

svn commit: r1755796 - in /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering: GlyphCache.java PageDrawer.java

Author: jahewson
Date: Wed Aug 10 18:34:38 2016
New Revision: 1755796

URL: http://svn.apache.org/viewvc?rev=1755796&view=rev
Log:
PDFBOX-3459: don't cache twice

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/GlyphCache.java
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/GlyphCache.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/GlyphCache.java?rev=1755796&r1=1755795&r2=1755796&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/GlyphCache.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/GlyphCache.java Wed Aug 10 18:34:38 2016
@@ -42,11 +42,6 @@ final class GlyphCache
     {
         this.font = font;
     }
-
-    public void put(int code, GeneralPath path)
-    {
-        cache.put(code, path);
-    }
     
     public GeneralPath getPathForCharacterCode(int code)
     {

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java?rev=1755796&r1=1755795&r2=1755796&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/rendering/PageDrawer.java Wed Aug 10 18:34:38 2016
@@ -334,14 +334,7 @@ public class PageDrawer extends PDFGraph
             glyphCaches.put(font, cache);
         }
         
-        // cache glyph path if is not already cache
         GeneralPath path = cache.getPathForCharacterCode(code);
-        if (path == null)
-        {
-            path = vectorFont.getNormalizedPath(code);
-            cache.put(code, path);
-        }
-        
         drawGlyph(path, font, code, displacement, at);
     }