Le quintidi 15 vendémiaire, an CCXXIV, Ronald S. Bultje a écrit : > > // inlink supports any channel layout > > layouts = ff_all_channel_counts(); > > - ff_channel_layouts_ref(layouts, &inlink->out_channel_layouts); > > + if (!layouts) > > + return AVERROR(ENOMEM); > > + if ((ret = ff_channel_layouts_ref(layouts, > &inlink->out_channel_layouts)) < 0) { > > + ff_channel_layouts_unref(&layouts); > > + return ret; > > + }
> I already feel fully inadequate to review this patch :-D. But this looks > weird. What do you find weird in this change? (What I find weird is that your MUA exhibits the same bogus quoted lines re-wrapping behaviour than outlook more than ten years ago.) > My proposal is that someone familiar with avfilter review this. > Nicolas, Clement, Stefano, Michael, Paul, (I probably forgot some people,) > can one of you volunteer? I have looked at the changes for the files I know. I will progressively look at the rest of the changes and post here if I find something not right, but I can not guarantee I will not miss something. Several pairs of eyes are always better. Regards, -- Nicolas George
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel