On Tue, Apr 01, 2014 at 10:26:20AM +0530, Nidhi Makhijani wrote:
> --- a/libavformat/rdt.c
> +++ b/libavformat/rdt.c
> @@ -521,7 +523,8 @@ static PayloadContext *
> rdt_new_context (void)
> {
> PayloadContext *rdt = av_mallocz(sizeof(PayloadContext));
> -
> + if (!rdt)
> + return rdt;
> int ret = avformat_open_input(&rdt->rmctx, "", &ff_rdt_demuxer, NULL);
You never bothered to compile, much less test, this. That's bad.
Patches you submit should pass the testsuite.
hint: Run "make" and "make check" and look out for warnings in the files
you modify.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
