From: Behdad Esfahbod Date: Sat, 4 May 2013 20:01:20 +0000 (-0400) Subject: [OTLayout] Add note about collect_glyphs() and recursive lookups X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=76ea563673d24ae1673a5aa3a21da6014479d433;p=platform%2Fupstream%2FlibHarfBuzzSharp.git [OTLayout] Add note about collect_glyphs() and recursive lookups --- diff --git a/src/hb-ot-layout-gsubgpos-private.hh b/src/hb-ot-layout-gsubgpos-private.hh index f46b378..5a5bce3 100644 --- a/src/hb-ot-layout-gsubgpos-private.hh +++ b/src/hb-ot-layout-gsubgpos-private.hh @@ -158,7 +158,13 @@ struct hb_collect_glyphs_context_t /* Note that GPOS sets recurse_func to NULL already, so it doesn't get * past the previous check. For GSUB, we only want to collect the output - * glyphs in the recursion. If output is not requested, we can go home now. */ + * glyphs in the recursion. If output is not requested, we can go home now. + * + * Note further, that the above is not exactly correct. A recursed lookup + * is allowed to match input that is not matched in the context, but that's + * not how most fonts are built. It's possible to relax that and recurse + * with all sets here if it proves to be an issue. + */ if (output == hb_set_get_empty ()) return HB_VOID;