Author: tilman
Date: Wed Oct 22 09:58:32 2025
New Revision: 1929273

Log:
PDFBOX-6085: rename variables so that similar code looks even more similar

Modified:
   
pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java

Modified: 
pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java
==============================================================================
--- 
pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java
  Wed Oct 22 09:58:27 2025        (r1929272)
+++ 
pdfbox/branches/3.0/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java
  Wed Oct 22 09:58:32 2025        (r1929273)
@@ -515,10 +515,10 @@ public final class TTFSubsetter
                 }
 
                 long lastOff = 0L;
-                for (Integer glyphId : glyphIds)
+                for (Integer gid : glyphIds)
                 {
-                    long offset = offsets[glyphId];
-                    long len = offsets[glyphId + 1] - offset;
+                    long offset = offsets[gid];
+                    long length = offsets[gid + 1] - offset;
                     isResult = is.skip(offset - lastOff);
                     
                     if (Long.compare(isResult, offset - lastOff) != 0)
@@ -526,12 +526,12 @@ public final class TTFSubsetter
                         LOG.debug("Tried skipping " + (offset - lastOff) + " 
bytes but skipped only " + isResult + " bytes");
                     }
 
-                    byte[] buf = new byte[(int)len];
+                    byte[] buf = new byte[(int) length];
                     isResult = is.read(buf);
 
-                    if (Long.compare(isResult, len) != 0)
+                    if (Long.compare(isResult, length) != 0)
                     {
-                        LOG.debug("Tried reading " + len + " bytes but only " 
+ isResult + " bytes read");
+                        LOG.debug("Tried reading " + length + " bytes but only 
" + isResult + " bytes read");
                     }
                     
                     // rewrite glyphIds for compound glyphs
@@ -581,7 +581,7 @@ public final class TTFSubsetter
                         while ((flags & 1 << 5) != 0); // MORE_COMPONENTS
 
                     }
-                    lastOff = offsets[glyphId + 1];
+                    lastOff = offsets[gid + 1];
                 }
             }
             hasNested = glyphIdsToAdd != null;

Reply via email to