[HarfBuzz] Potential bug

Behdad Esfahbod behdad at behdad.org
Mon Sep 29 15:35:36 PDT 2008


Pushed this and a GSUB fix.

behdad

Peter Hunter wrote:
> In HarfBuzz-GPOS.c:: Position_CursiveChain
> Is this curious sequence
> 
>    HB_UInt   i, j;
>    HB_Position positions = buffer->positions;
> 
>    /* First handle all left-to-right connections */
>    for (j = 0; j < buffer->in_length; j--)
>    {...}
> 
> As j is an unsigned int, this loop can only ever be executed once, surely the j-- should be j++ ?
> 
> Peter Hunter
> _______________________________________________
> HarfBuzz mailing list
> HarfBuzz at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/harfbuzz
> 



More information about the HarfBuzz mailing list