From f1e7777d17273bc479b7f0565ecb33803a72e39d Mon Sep 17 00:00:00 2001 From: mtlcl410 Date: Sat, 7 Nov 2026 09:04:28 +0800 Subject: [PATCH] M2: name both families in the non-embedded font tests' comments (review) Co-Authored-By: Claude Opus 5.5 Claude-Session: https://claude.ai/code/session_01Pq7HyDBXygTf7jcqdsjcfn --- crates/render/src/raster/tests.rs | 3 ++- crates/render/src/raster/tests/non_embedded_arial.rs | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/crates/render/src/raster/tests.rs b/crates/render/src/raster/tests.rs index 8d6bad0437..0c8ed987f3 100644 --- a/crates/render/src/raster/tests.rs +++ b/crates/render/src/raster/tests.rs @@ -8199,7 +8199,8 @@ mod icc_profile_cache; // A symbolic TrueType font draws through its (3, 0) cmap subtable first (#4156). mod symbolic_truetype_cmap; -// An Arial the PDF doesn't embed draws the glyphs its codes name (#5889). +// An Arial (#5889) or Times New Roman (#6449) the PDF doesn't embed draws the glyphs its codes +// name. mod non_embedded_arial; /// Render `pdf`'s first page at 72 dpi with no time limit beyond the test's own. diff --git a/crates/render/src/raster/tests/non_embedded_arial.rs b/crates/render/src/raster/tests/non_embedded_arial.rs index a75ff1bce8..42a990638a 100644 --- a/crates/render/src/raster/tests/non_embedded_arial.rs +++ b/crates/render/src/raster/tests/non_embedded_arial.rs @@ -20,7 +20,7 @@ struct Font<'a> { base_font: &'a str, /// The descendant's `/Subtype`. subtype: &'a str, - /// `true`: the codes are 1, 2, … and a `/CIDToGIDMap` stream gives each its Arial glyph. + /// `true`: the codes are 1, 2, … and a `/CIDToGIDMap` stream gives each its glyph in the author's font. /// `false`: `/CIDToGIDMap /Identity`, and the codes are the glyph indexes themselves. mapped: bool, to_unicode: bool,