Fix the computation of unitsPerEm when the fontMatrix has some negative coefficients

It's a follow-up of #18253.
This commit is contained in:
Calixte Denizet 2024-06-24 16:38:17 +02:00
parent 6784124a74
commit 42bb2b0737
4 changed files with 10 additions and 2 deletions

View File

@ -799,7 +799,7 @@ function createOS2Table(properties, charstrings, override) {
const unitsPerEm =
override.unitsPerEm ||
(properties.fontMatrix
? 1 / Math.max(...properties.fontMatrix.slice(0, 4))
? 1 / Math.max(...properties.fontMatrix.slice(0, 4).map(Math.abs))
: 1000);
// if the font units differ to the PDF glyph space units
@ -3199,7 +3199,7 @@ class Font {
}
const unitsPerEm = properties.fontMatrix
? 1 / Math.max(...properties.fontMatrix.slice(0, 4))
? 1 / Math.max(...properties.fontMatrix.slice(0, 4).map(Math.abs))
: 1000;
const builder = new OpenTypeFileBuilder("\x4F\x54\x54\x4F");

View File

@ -650,3 +650,4 @@
!issue17998.pdf
!pdfjs_wikipedia.pdf
!bug1539074.pdf
!bug1539074.1.pdf

BIN
test/pdfs/bug1539074.1.pdf Executable file

Binary file not shown.

View File

@ -10098,6 +10098,13 @@
"rounds": 1,
"type": "eq"
},
{
"id": "bug1539074_1",
"file": "pdfs/bug1539074.1.pdf",
"md5": "d15c49142fda433323d3d35f2762cd33",
"rounds": 1,
"type": "eq"
},
{
"id": "bug1903731",
"file": "pdfs/bug1903731.pdf",