James Almer: > On 9/20/2019 5:39 PM, Andreas Rheinhardt wrote: >> From: James Almer <jamr...@gmail.com> >> >> Demuxers may have allocated a packet before encountering an error and >> aborting. > > Fun thing about this, the doxy states that packets should be freed on > error before returning from the demuxer, so technically the apng demuxer > is misbehaving. > > Still, i think the generic code doing some cleaning is a good idea. But > of course, if people prefer to enforce the expected behavior from > documentation, we could replace the unref call introduced by this patch > with an assert. > IMO this change is very good and I have already begun work on removing unnecessary av_packet_unref() calls in various demuxers [1]. I'll add changes to the doxy when I'm done.
- Andreas [1]: https://github.com/mkver/FFmpeg/commit/1930edf0e225fc541ffacbdff1921d4743e998cc _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".