Hi,
On Fri, Aug 3, 2012 at 2:14 AM, Måns Rullgård <m...@mansr.com> wrote:
> "Ronald S. Bultje" <rsbul...@gmail.com> writes:
>> + } else if (!av_strncasecmp(nptr, "0x", 2) ||
>> + !av_strncasecmp(nptr, "-0x", 3) ||
>> + !av_strncasecmp(nptr, "+0x", 3)) {
>> + return strtol(nptr, endptr, 16);
>
> This will not handle non-integers, exponents, or values out of range for
> long.
>
>> + } else {
>> + return strtod(nptr, endptr);
>> + }
>> +}
Can non-integers or exponents follow 0x? I don't think so. Note the
sequence: if 0x..., use strtol(), else use native strtod().
Ronald
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
