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

Stefano Sabatini stefasab at gmail.com
Wed Feb 6 01:05:16 CET 2013


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).
-- 
FFmpeg = Free Foolish Maxi Portentous Empowered Generator


More information about the ffmpeg-devel mailing list