Author: tilman
Date: Wed Oct 22 10:05:36 2025
New Revision: 1929278
Log:
PDFBOX-6085: rename variables so that similar code looks even more similar
Modified:
pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java
Modified:
pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java
==============================================================================
---
pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java
Wed Oct 22 10:05:32 2025 (r1929277)
+++
pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java
Wed Oct 22 10:05:36 2025 (r1929278)
@@ -914,13 +914,13 @@ public final class TTFSubsetter
{
is.skip(hm.getOffset());
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
@@ -929,7 +929,7 @@ public final class TTFSubsetter
else
{
// copy width and lsb
- offset = glyphId * 4l;
+ offset = gid * 4l;
lastOffset = copyBytes(is, bos, offset, lastOffset, 4);
}
}
@@ -947,7 +947,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);
}
}