On Tue, Jul 19, 2022 at 04:38:34PM +0200, nenesses wrote:
> -static size_t
> -utf8validate(long *u, size_t i)
> -{
> -     if (!BETWEEN(*u, utfmin[i], utfmax[i]) || BETWEEN(*u, 0xD800, 0xDFFF))
> -             *u = UTF_INVALID;
> -     for (i = 1; *u > utfmax[i]; ++i)
> -             ;
> -     return i;
> -}

I wonder if the return value was ever used or was it just premature
functionality. Either ways, the patch looks good to me, but I'm not the
maintainer here.

- NRK

Reply via email to