[FFmpeg-devel] [PATCH 05/17] lavfi: rename all_channel_layouts.h to all_channel_layouts.inc.

Stefano Sabatini stefasab at gmail.com
Thu May 10 00:32:58 CEST 2012


On date Wednesday 2012-05-09 13:17:29 +0200, Clément Bœsch encoded:
> On Wed, May 09, 2012 at 12:36:07PM +0200, Michael Niedermayer wrote:
> > On Wed, May 09, 2012 at 10:01:30AM +0200, Clément Bœsch wrote:
> > > all_channel_layouts.h is not a real header file and breaks make checkheaders.
> > > ---
> > >  libavfilter/all_channel_layouts.h   |   68 -----------------------------------
> > >  libavfilter/all_channel_layouts.inc |   68 +++++++++++++++++++++++++++++++++++
> > 
> > why not add it to SKIPHEADERS ?
> > 
> 
> I wanted to avoid exceptions. Also this file is clearly not a valid c or
> header file so I though it was a rename for the better. If people prefer
> making use of SKIPHEADERS, I don't mind much. Stefano, you were the author
> of this file, any opinion?

No opinion. Ideally we should get rid of the file once we remove the
<=8 channels limitation.
-- 
FFmpeg = Freak & Foolish Majestic Ponderous Elitist Gadget


More information about the ffmpeg-devel mailing list