[FFmpeg-devel] [PATCH 1/2] avfilter: Dry run mode for list merging

Michael Niedermayer michaelni at gmx.at
Tue Jul 23 19:31:26 CEST 2013


On Mon, Jul 22, 2013 at 12:16:10PM +0200, Nicolas George wrote:
> Le quartidi 4 thermidor, an CCXXI, Michael Niedermayer a écrit :
> >  AVFilterFormats *ff_merge_formats(AVFilterFormats *a, AVFilterFormats *b,
> > -                                  enum AVMediaType type)
> > +                                  enum AVMediaType type, int dry_run)
> >  {
> 
> Wouldn't it be simpler to have
> 
> ff_try_merge_formats(a, b, type);
> 
> that will allocate a copy of a and b with dummy back-references and pass it
> to ff_merge_formats?

implemented & pushed this way


> 
> For reference, the problem with the channel layouts is that the algorithm
> currently overwrites its inputs. It does so only when success is certain
> (exccept for OOM), but that will not do for dry-run.
> 
> Regards,
> 
> -- 
>   Nicolas George



> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 3
"Rare item" - "Common item with rare defect or maybe just a lie"
"Professional" - "'Toy' made in china, not functional except as doorstop"
"Experts will know" - "The seller hopes you are not an expert"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130723/a30fa9a1/attachment.asc>


More information about the ffmpeg-devel mailing list