[FFmpeg-devel] [libav-devel] [ffmpeg-devel] [PATCH 1/2] lavfi: make AVFilteFormats use int64_t lists to support channel layouts.

Mina Nagy Zaki mnzaki at gmail.com
Thu Jun 9 15:59:23 CEST 2011


On Thursday 09 June 2011 17:36:16 Stefano Sabatini wrote:
> On date Thursday 2011-06-09 13:25:25 +0300, Mina Nagy Zaki encoded:
> > The list type was changed to int64_t to be able to hold
> > channel layouts.
> > 
> > Usage of avfilter_make_format_list for PixelFromats/[AV]SampleFormats
> > had to be changed to use int64_t[] instead of enums, as they are 32bit.
> 
> I discussed this with Mina and this looked like the best solution for
> avoiding separate int/int64_t functions. If you have reasons to think
> there are better solutions, please comment.
> 
> [...]
> 
> > diff --git a/libavfilter/vsrc_movie.c b/libavfilter/vsrc_movie.c
> > index 6c5c831..883e665 100644
> > --- a/libavfilter/vsrc_movie.c
> > +++ b/libavfilter/vsrc_movie.c
> > @@ -201,7 +201,7 @@ static av_cold void uninit(AVFilterContext *ctx)
> > 
> >  static int query_formats(AVFilterContext *ctx)
> >  {
> >  
> >      MovieContext *movie = ctx->priv;
> > 
> > -    enum PixelFormat pix_fmts[] = { movie->codec_ctx->pix_fmt,
> > PIX_FMT_NONE };
> > +    int64_t pix_fmts[] = { movie->codec_ctx->pix_fmt, PIX_FMT_NONE };
> 
> Mangled?

Stupid mail agent decided to add a \n for line wrapping. I have attached the 
patch this time instead of copy/paste.

-- 
Mina
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lavfi-make-AVFilteFormats-use-int64_t-lists-to-suppo.patch
Type: text/x-patch
Size: 15031 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110609/02711461/attachment.bin>


More information about the ffmpeg-devel mailing list