On Mon, 2012-03-19 at 20:55 -0700, Ronald S. Bultje wrote:
> Shouldn't avpkt->size be unsigned also?

Unsigned types cause problems for normal code. Making the type unsigned
would break tests like
"if (size_left - avpkt->size < min_left)"
If you want to support packets larger than 2 GiB, then a 64-bit signed
type would be a better idea.

BTW the casts in the original patch are redundant; the function return
value is automatically cast to the return type of the function. Unless
the intent is to emphasize that sizes larger than 4 GiB won't work.


_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to