Marmelator/pdf.js.mirror
1
0
Fork 0
mirror of https://github.com/mozilla/pdf.js.git synced 2026-07-13 10:35:47 +02:00
Code Issues Packages Projects Releases Wiki Activity

Use a DataView when building the "hmtx" TrueType table (PR 21072 follow-up)

Browse Source
This commit is contained in:
Jonas Jenwald 2026-04-13 11:47:22 +02:00
parent 652700dac6
commit aaaf143156
1 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/core/fonts.js
View File

@ -327,13 +327,15 @@ function safeString16(value) {
return String.fromCharCode((value >> 8) & 0xff, value & 0xff); return String.fromCharCode((value >> 8) & 0xff, value & 0xff);
} }
function ensureInt16(v) { function setInt16(view, pos, val) {
if (typeof PDFJSDev === "undefined" || PDFJSDev.test("TESTING")) { if (typeof PDFJSDev === "undefined" || PDFJSDev.test("TESTING")) {
assert( assert(
typeof v === "number" && Math.abs(v) < 2 ** 16, typeof val === "number" && Math.abs(val) < 2 ** 16,
`ensureInt16: Unexpected input "${v}".` `setInt16: Unexpected input "${val}".`
); );
} }
view.setInt16(pos, val);
return pos + 2;
} }
function isTrueTypeFile(file) { function isTrueTypeFile(file) {
@ -3310,11 +3312,12 @@ class Font {
// Horizontal metrics // Horizontal metrics
builder.addTable( builder.addTable(
"hmtx", "hmtx",
(function fontFieldsHmtx() { (function fontTableHmtx() {
const charstrings = font.charstrings; const charstrings = font.charstrings;
const cffWidths = font.cff?.widths ?? null; const cffWidths = font.cff?.widths ?? null;
const data = new Uint8Array(numGlyphs * 4); const data = new Uint8Array(numGlyphs * 4),
view = new DataView(data.buffer);
// Fake .notdef (width=0 and lsb=0) first, skip redundant assignment. // Fake .notdef (width=0 and lsb=0) first, skip redundant assignment.
let pos = 4; let pos = 4;
@ -3325,11 +3328,8 @@ class Font {
} else if (cffWidths) { } else if (cffWidths) {
width = Math.ceil(cffWidths[i] || 0); width = Math.ceil(cffWidths[i] || 0);
} }
ensureInt16(width); pos = setInt16(view, pos, width);
data[pos++] = (width >> 8) & 0xff; pos += 2; // Use lsb=0, skip redundant assignment.
data[pos++] = width & 0xff;
// Use lsb=0, skip redundant assignment.
pos += 2;
} }
return data; return data;
})() })()