On 28.02.2023 16:03, Zhao Zhili wrote:
I can move the check from here to close(). assert is helpful if it can catch bugs during development. It doesn't help much here since this is a rare case which doesn't triggered during development.
Crashing with an assertion is still much better than dereferencing a NULL pointer.
I'll have a look at the calling function in a bit, to see if it can be refactored to never run into the chance of calling this function when not needed.
_______________________________________________ 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".
