On 10/01/14 21:16, Anton Khirnov wrote: > > On Wed, 8 Jan 2014 03:25:37 +0100, Luca Barbato <[email protected]> wrote: >> From: Michael Niedermayer <[email protected]> >> >> Signed-off-by: Luca Barbato <[email protected]> >> --- >> libavformat/mxfdec.c | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c >> index 8777965..7bca23d 100644 >> --- a/libavformat/mxfdec.c >> +++ b/libavformat/mxfdec.c >> @@ -864,6 +864,11 @@ static int mxf_read_generic_descriptor(void *arg, >> AVIOContext *pb, int tag, int >> default: >> /* Private uid used by SONY C0023S01.mxf */ >> if (IS_KLV_KEY(uid, mxf_sony_mpeg4_extradata)) { >> + if (descriptor->extradata) >> + av_log(NULL, AV_LOG_WARNING, > > NULL? > Please no more av_logs to NULL, we have plenty of those already. >
I'll drop it for now, mxf can be part of the next sprint, this code is quite ugly. lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
