walkero
/
harfbuzz
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

[subset] Re-introduce size threshold in choosing unicode collection method. 

Threshold is needed since the unicodes set might be an inverted set.
This commit is contained in:
Garret Rieger 2022-05-04 21:22:26 +00:00
parent 42c54eba83
commit 7cb36e4222
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/hb-subset-plan.cc
View File

@ -302,10 +302,12 @@ _populate_unicodes_to_retain (const hb_set_t *unicodes,
{
OT::cmap::accelerator_t cmap (plan->source);
if (glyphs->is_empty ())
unsigned size_threshold = plan->source->get_num_glyphs ();
if (glyphs->is_empty () && unicodes->get_population () < size_threshold)
{
// This is approach to collection is faster, but can only be used if glyphs
// are not being explicitly added to the subset.
// are not being explicitly added to the subset and the input unicodes set is
// not excessively large (eg. an inverted set).
plan->unicode_to_new_gid_list.alloc (unicodes->get_population ());
for (hb_codepoint_t cp : *unicodes)
{
@ -326,8 +328,10 @@ _populate_unicodes_to_retain (const hb_set_t *unicodes,
// them with cmap entries.
hb_map_t unicode_glyphid_map;
cmap.collect_mapping (hb_set_get_empty (), &unicode_glyphid_map);
plan->unicode_to_new_gid_list.alloc (unicodes->get_population ()
+ glyphs->get_population ());
plan->unicode_to_new_gid_list.alloc (hb_min(unicodes->get_population ()
+ glyphs->get_population (),
unicode_glyphid_map.get_population ()));
for (hb_pair_t<hb_codepoint_t, hb_codepoint_t> cp_gid :
+ unicode_glyphid_map.iter ())