[FFmpeg-devel] [PATCH] lavfi: add ff_all_channel_layouts internal symbol
stefasab at gmail.com
Fri Sep 16 01:24:19 CEST 2011
On date Tuesday 2011-09-13 20:56:27 +0200, Michael Niedermayer encoded:
> On Tue, Sep 13, 2011 at 05:46:21PM +0200, Stefano Sabatini wrote:
> > On date Monday 2011-09-12 22:44:54 +0200, Stefano Sabatini encoded:
> > > On date Monday 2011-09-12 03:19:17 +0200, Michael Niedermayer encoded:
> > > > On Sun, Sep 11, 2011 at 11:00:06AM +0200, Stefano Sabatini wrote:
> > > [...]
> > > > > a huge list of channel layouts, some may be not supported by
> > > > > libavfilter due to the unsupported number of channels
> > > > >
> > > > > and such obscure definitions quickly lead to bugs/hacks.
> > > > >
> > > > > That's why I prefer compile-time generation.
> > > >
> > > > btw its 72 formats, thats 72 lines of code, the email above is 82 lines
> > > > it just doesnt seem enough for me to generate at compiletime (with
> > > > more code)
> > > > either way, if you want it at compile time iam perfectly fine with
> > > > that but i expect you to maintain it. (that is if changes elsewhere
> > > > break it for example)
> > >
> > > I'm fine with the statically defined list if that's easier and simpler
> > > to maintain, what I want to avoid is special casing and extra
> > > (sometimes unexpected) complexity deriving from it.
> > Check patch.
> fine with me
FFmpeg = Fast Frenzy MultiPurpose Enhancing Gangster
More information about the ffmpeg-devel