[FFmpeg-devel] [PATCH] ffplay: change type for channels and channel_layout AudioParams fields

Stefano Sabatini stefasab at gmail.com
Mon Feb 25 00:02:54 CET 2013


On date Wednesday 2013-02-06 01:05:16 +0100, Stefano Sabatini encoded:
> On date Monday 2013-02-04 10:06:53 +0100, Nicolas George encoded:
> > Le quintidi 15 pluviôse, an CCXXI, Stefano Sabatini a écrit :
> > > Switch from int to int64_t, to better reflect the type of the
> > > corresponding fields in lavc/lavfi.
> > 
> > ok for channel_layouts, but don't you think that 1<<31 is enough for the
> > number of channels?
> 
> Looks like I messed-it up in 23fc4dd6e7e150ea163a867dfaee2062ade90b74.
> 
> I suppose I can still favor int over int64_t, ideally we should make
> it consistent all over the place and set it back to int (theoretically
> since we used accessors it should not involve ABI breaks).

Fixed and pushed.
-- 
FFmpeg = Fast and Furious Mind-dumbing Proud Ecumenical Gnome


More information about the ffmpeg-devel mailing list