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

[kerx] Fix peculiar indexing that was needed previously

Browse Source 
Not needed now that we use GPOS attachment for cursive kerx.
This commit is contained in:
Behdad Esfahbod 2018-11-07 15:44:40 -05:00
parent 6ee6cd93d8
commit ea579f9ccc
1 changed files with 7 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/hb-aat-layout-kerx-table.hh
View File

@ -286,23 +286,18 @@ struct KerxSubTableFormat1
/* From Apple 'kern' spec: /* From Apple 'kern' spec:
* "Each pops one glyph from the kerning stack and applies the kerning value to it. * "Each pops one glyph from the kerning stack and applies the kerning value to it.
* The end of the list is marked by an odd value... */ * The end of the list is marked by an odd value... */
unsigned int i;
for (i = 0; i < depth; i++)
if (actions[i] & 1)
{
i++;
break;
}
unsigned int tuple_count = table->header.tuple_count (); unsigned int tuple_count = table->header.tuple_count ();
tuple_count = tuple_count ? tuple_count : 1; tuple_count = tuple_count ? tuple_count : 1;
for (; i; i--) bool last = false;
while (!last && depth--)
{ {
unsigned int idx = stack[depth - i]; unsigned int idx = stack[depth];
int v = *actions;
actions += tuple_count;
if (idx >= buffer->len) continue; if (idx >= buffer->len) continue;
int v = actions[(i - 1) * tuple_count]; /* "The end of the list is marked by an odd value..." */
last = v & 1;
/* "The end of the list is marked by an odd value..." Ignore it. */
v &= ~1; v &= ~1;
hb_glyph_position_t &o = buffer->pos[idx]; hb_glyph_position_t &o = buffer->pos[idx];
@ -355,7 +350,6 @@ struct KerxSubTableFormat1
} }
} }
} }
depth = 0;
} }
return true; return true;