Author: tilman
Date: Wed Oct 22 10:05:32 2025
New Revision: 1929277
Log:
PDFBOX-6085: rename variables so that similar code looks even more similar
Modified:
pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java
Modified:
pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java
Wed Oct 22 10:03:41 2025 (r1929276)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java
Wed Oct 22 10:05:32 2025 (r1929277)
@@ -942,13 +942,13 @@ public final class TTFSubsetter
}
long lastOffset = 0;
- for (Integer glyphId : glyphIds)
+ for (Integer gid : glyphIds)
{
// offset in original file
long offset;
- if (glyphId <= lastgid)
+ if (gid <= lastgid)
{
- if (invisibleGlyphIds.contains(glyphId))
+ if (invisibleGlyphIds.contains(gid))
{
// force zero width (no change to last offset)
// 4 bytes total, 2 bytes each for: advance width = 0,
left side bearing = 0
@@ -957,7 +957,7 @@ public final class TTFSubsetter
else
{
// copy width and lsb
- offset = glyphId * 4l;
+ offset = gid * 4l;
lastOffset = copyBytes(is, bos, offset, lastOffset, 4);
}
}
@@ -975,7 +975,7 @@ public final class TTFSubsetter
}
// copy lsb only, as we are beyond numOfHMetrics
- offset = h.getNumberOfHMetrics() * 4l + (glyphId -
h.getNumberOfHMetrics()) * 2l;
+ offset = h.getNumberOfHMetrics() * 4l + (gid -
h.getNumberOfHMetrics()) * 2l;
lastOffset = copyBytes(is, bos, offset, lastOffset, 2);
}
}