At 2017-11-05T15:09:41+0100, Bertrand Garrigues wrote:
>    if (len == 0)
> -    return NULL;
> +    goto end;

We're kind of giving a hostage to fortune here, aren't we?

https://en.wikipedia.org/wiki/Unreachable_code#goto_fail_bug

Can I beg a pair of braces there in case someone in the future decides
that loop needs to do just _one_ more tiny little thing?

-- 
Regards,
Branden

Attachment: signature.asc
Description: PGP signature

Reply via email to