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

[HB] Fix apply_lookup() loop 

Part of Bug 595539 - Regressions in rendering certain Thai sequences with
OpenType font
This commit is contained in:
Behdad Esfahbod 2009-09-21 13:43:54 -04:00
parent c0ab43c058
commit f9c0a2dad0
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/hb-ot-layout-gsubgpos-private.hh
View File

@ -219,7 +219,7 @@ static inline bool apply_lookup (APPLY_ARG_DEF,
/* TODO We don't support lookupRecord arrays that are not increasing: /* TODO We don't support lookupRecord arrays that are not increasing:
* Should be easy for in_place ones at least. */ * Should be easy for in_place ones at least. */
for (unsigned int i = 0; i < count; i++) for (unsigned int i = 0; i < count; /* NOP */)
{ {
while (_hb_ot_layout_skip_mark (context->face, IN_CURINFO (), lookup_flag, NULL)) while (_hb_ot_layout_skip_mark (context->face, IN_CURINFO (), lookup_flag, NULL))
{ {
@ -238,6 +238,7 @@ static inline bool apply_lookup (APPLY_ARG_DEF,
lookupRecord++; lookupRecord++;
lookupCount--; lookupCount--;
/* Err, this is wrong if the lookup jumped over some glyphs */
i += buffer->in_pos - old_pos; i += buffer->in_pos - old_pos;
if (HB_UNLIKELY (buffer->in_pos == end)) if (HB_UNLIKELY (buffer->in_pos == end))
return true; return true;