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 2025/10/22 10:05:36 UTC
svn commit: r1929278 - pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf
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);
}
}