On Fri, 28 Feb 2014, Janne Grunau wrote:

On 2014-02-28 12:29:01 +0200, Martin Storsjö wrote:
Copying data in chunks of 1 KB is a little wasteful.

64 KB should still easily fit on the stack, so there's no need
to allocate it dynamically.

not that I care but I do not see an advantage of avoiding a additional
heap allocation.

Not anything else than keeping the patch smaller - if someone wants to run this on a system with less than 64 KB stack I guess we could change it to a dynamic allocation, but until then this probably works just as well.

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

Reply via email to