From 52e9a71d578c5171bbb0f4bfc1b70841c3270328 Mon Sep 17 00:00:00 2001 From: Behdad Esfahbod Date: Mon, 21 Sep 2009 13:58:56 -0400 Subject: [PATCH] [HB] Add comment --- src/hb-ot-layout-gsubgpos-private.hh | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/hb-ot-layout-gsubgpos-private.hh b/src/hb-ot-layout-gsubgpos-private.hh index fd9c783..8b9b1cd 100644 --- a/src/hb-ot-layout-gsubgpos-private.hh +++ b/src/hb-ot-layout-gsubgpos-private.hh @@ -219,6 +219,11 @@ static inline bool apply_lookup (APPLY_ARG_DEF, /* TODO We don't support lookupRecord arrays that are not increasing: * Should be easy for in_place ones at least. */ + + /* Note: If sublookup is reverse, i will underflow after the first loop + * and we jump out of it. Not entirely disastrous. So we don't check + * for reverse lookup here. + */ for (unsigned int i = 0; i < count; /* NOP */) { while (_hb_ot_layout_skip_mark (context->face, IN_CURINFO (), lookup_flag, NULL)) -- 2.7.4