On 31/12/14 19:13, Derek Buitenhuis wrote:
On 12/31/2014 6:11 PM, Luca Barbato wrote:
next_packet, premux_packet and predecode_packet implement a linked list
queue in a quite confusing way.
Again another item for a later set "clarify the names so people do not
get so confused".
That wasn't my point, though.
The old code guarantees that next_packet is set to NULL upon malloc failure,
and the new code does not. I am wondering if that guarantee is provided
elsewhere.
If you expect to call that function again probably is better to keep it
as is.
If you do not expect to call that function again, you do not care since
that variable is used only there.
If I'm not missing something glaringly obvious. (again, still not
completely sane from the experience).
lu
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
