On 31/12/14 19:07, Derek Buitenhuis wrote:
On 12/31/2014 5:30 PM, Luca Barbato wrote:
-    *stream->next_packet     =
-    pkt_desc                 = av_mallocz(sizeof(PacketDesc));
+    pkt_desc = av_mallocz(sizeof(PacketDesc));
+    if (!pkt_desc)
+        return AVERROR(ENOMEM);

Is *stream->next_packet already guaranteed to be NULL here?

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".

lu

_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to