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

Mike Scheutzow mike.scheutzow at alcatel-lucent.com
Thu Jun 9 17:58:19 CEST 2011


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.


This change appears to make things much less type-safe.

What was the original reason to change 'enum SampleFormat' to int64_t?


Mike Scheutzow


More information about the ffmpeg-devel mailing list