[FFmpeg-cvslog] r20874 - trunk/libavcodec/aac.c

Robert Swain robert.swain
Thu Dec 17 07:52:04 CET 2009


On 16/12/09 13:27, Carl Eugen Hoyos wrote:
> David Conrad<lessen42<at>  gmail.com>  writes:
>
>> This breaks gcc and clang probably because they assume that an enum cannot
>> have a value other than what's listed, and so assume type>= TYPE_SCE is
>> always true.
>
> Reverted.
>
>> The attached patch fixes it.
>
> If Robert accepts this, I will commit again (if nobody objects).

Assuming it works and fixes both the originally noticed issue and 
doesn't break compilation for commonly used compilers aside from icc, 
it's fine. :)

Regards,
Rob



More information about the ffmpeg-cvslog mailing list