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/08/28 03:18:51 UTC

svn commit: r1903714 - /pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForBengali.java

Author: tilman
Date: Sun Aug 28 03:18:51 2022
New Revision: 1903714

URL: http://svn.apache.org/viewvc?rev=1903714&view=rev
Log:
PDFBOX-4892: optimize, as suggested by Valery Bokov

Modified:
    pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForBengali.java

Modified: pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForBengali.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForBengali.java?rev=1903714&r1=1903713&r2=1903714&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForBengali.java (original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/gsub/GsubWorkerForBengali.java Sun Aug 28 03:18:51 2022
@@ -127,10 +127,10 @@ public class GsubWorkerForBengali implem
         for (int index = 1; index < originalGlyphIds.size(); index++)
         {
             int glyphId = originalGlyphIds.get(index);
-            if (beforeAndAfterSpanGlyphIds.containsKey(glyphId))
+            BeforeAndAfterSpanComponent beforeAndAfterSpanComponent =
+                    beforeAndAfterSpanGlyphIds.get(glyphId);
+            if (beforeAndAfterSpanComponent != null)
             {
-                BeforeAndAfterSpanComponent beforeAndAfterSpanComponent = beforeAndAfterSpanGlyphIds
-                        .get(glyphId);
                 int previousGlyphId = originalGlyphIds.get(index - 1);
                 repositionedGlyphIds.set(index, previousGlyphId);
                 repositionedGlyphIds.set(index - 1,