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 2020/02/09 14:48:25 UTC
svn commit: r1873832 -
/pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java
Author: tilman
Date: Sun Feb 9 14:48:25 2020
New Revision: 1873832
URL: http://svn.apache.org/viewvc?rev=1873832&view=rev
Log:
PDFBOX-4071: SonarQube fix
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
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java?rev=1873832&r1=1873831&r2=1873832&view=diff
==============================================================================
--- pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java (original)
+++ pdfbox/branches/2.0/fontbox/src/main/java/org/apache/fontbox/ttf/TTFSubsetter.java Sun Feb 9 14:48:25 2020
@@ -705,7 +705,7 @@ public final class TTFSubsetter
// encoding record
writeUint16(out, CmapTable.PLATFORM_WINDOWS); // platformID
writeUint16(out, CmapTable.ENCODING_WIN_UNICODE_BMP); // platformSpecificID
- writeUint32(out, 4 * 2 + 4); // offset
+ writeUint32(out, 12); // offset 4 * 2 + 4
// build Format 4 subtable (Unicode BMP)
Iterator<Entry<Integer, Integer>> it = uniToGID.entrySet().iterator();
@@ -896,7 +896,7 @@ public final class TTFSubsetter
if (glyphId <= lastgid)
{
// copy width and lsb
- offset = glyphId * 4;
+ offset = glyphId * 4l;
lastOffset = copyBytes(is, bos, offset, lastOffset, 4);
}
else
@@ -906,14 +906,14 @@ public final class TTFSubsetter
// one time only: copy width from lastgid, whose width applies
// to all later glyphs
needLastGidWidth = false;
- offset = lastgid * 4;
+ offset = lastgid * 4l;
lastOffset = copyBytes(is, bos, offset, lastOffset, 2);
// then go on with lsb from actual glyph (lsb are individual even in monotype fonts)
}
// copy lsb only, as we are beyond numOfHMetrics
- offset = h.getNumberOfHMetrics() * 4 + (glyphId - h.getNumberOfHMetrics()) * 2;
+ offset = h.getNumberOfHMetrics() * 4l + (glyphId - h.getNumberOfHMetrics()) * 2l;
lastOffset = copyBytes(is, bos, offset, lastOffset, 2);
}
}