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 [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
