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 2018/04/27 13:37:10 UTC

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

Author: tilman
Date: Fri Apr 27 13:37:09 2018
New Revision: 1830353

URL: http://svn.apache.org/viewvc?rev=1830353&view=rev
Log:
PDFBOX-4189: fix SonarQube pet peeve (variable hides a field)

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=1830353&r1=1830352&r2=1830353&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 Fri Apr 27 13:37:09 2018
@@ -202,16 +202,16 @@ public class GsubWorkerForBengali implem
 
     private Map<Integer, BeforeAndAfterSpanComponent> getBeforeAndAfterSpanGlyphIds()
     {
-        Map<Integer, BeforeAndAfterSpanComponent> beforeAndAfterSpanGlyphIds = new HashMap<>();
+        Map<Integer, BeforeAndAfterSpanComponent> result = new HashMap<>();
 
         for (BeforeAndAfterSpanComponent beforeAndAfterSpanComponent : BEFORE_AND_AFTER_SPAN_CHARS)
         {
-            beforeAndAfterSpanGlyphIds.put(
+            result.put(
                     getGlyphId(beforeAndAfterSpanComponent.originalCharacter),
                     beforeAndAfterSpanComponent);
         }
 
-        return Collections.unmodifiableMap(beforeAndAfterSpanGlyphIds);
+        return Collections.unmodifiableMap(result);
     }
 
     /**