[OTLayout] Reuse apply context for recursion
diff --git a/src/hb-ot-layout-gpos-table.hh b/src/hb-ot-layout-gpos-table.hh index 873060f..e0b8913 100644 --- a/src/hb-ot-layout-gpos-table.hh +++ b/src/hb-ot-layout-gpos-table.hh
@@ -1537,8 +1537,13 @@ { const GPOS &gpos = *(hb_ot_layout_from_face (c->face)->gpos); const PosLookup &l = gpos.get_lookup (lookup_index); + unsigned int saved_lookup_props = c->lookup_props; + unsigned int saved_property = c->property; c->set_lookup (l); - return l.apply_once (c); + bool ret = l.apply_once (c); + c->lookup_props = saved_lookup_props; + c->property = saved_property; + return ret; }
diff --git a/src/hb-ot-layout-gsub-table.hh b/src/hb-ot-layout-gsub-table.hh index 690b2d5..ddbb3b5 100644 --- a/src/hb-ot-layout-gsub-table.hh +++ b/src/hb-ot-layout-gsub-table.hh
@@ -1402,8 +1402,13 @@ { const GSUB &gsub = *(hb_ot_layout_from_face (c->face)->gsub); const SubstLookup &l = gsub.get_lookup (lookup_index); + unsigned int saved_lookup_props = c->lookup_props; + unsigned int saved_property = c->property; c->set_lookup (l); - return l.apply_once (c); + bool ret = l.apply_once (c); + c->lookup_props = saved_lookup_props; + c->property = saved_property; + return ret; }
diff --git a/src/hb-ot-layout-gsubgpos-private.hh b/src/hb-ot-layout-gsubgpos-private.hh index 00a4bd4..0b1bc68 100644 --- a/src/hb-ot-layout-gsubgpos-private.hh +++ b/src/hb-ot-layout-gsubgpos-private.hh
@@ -233,10 +233,10 @@ if (unlikely (nesting_level_left == 0 || !recurse_func)) return default_return_value (); - /* TODO Reuse context. */ - hb_apply_context_t new_c (*this); - new_c.nesting_level_left--; - return recurse_func (&new_c, lookup_index); + nesting_level_left--; + recurse_func (this, lookup_index); + nesting_level_left++; + return default_return_value (); } hb_font_t *font;