[FFmpeg-devel] [PATCH] CrystalHD decoder support v3

Luca Barbato lu_zero
Tue Feb 1 18:18:58 CET 2011


On 02/01/2011 05:59 PM, Philip Langdale wrote:
> 
> On Tue, 1 Feb 2011 11:51:34 +0100, Benoit Fouet <benoit.fouet at free.fr>
> wrote:
>>
>> Even nicer, IMO, would be to initialize when declaring:
>>     BC_INPUT_FORMAT format = {
>>         .FGTEnable   = FALSE,
>>         .Progressive = TRUE,
>>         .OptFlags    = 0x80000000 | vdecFrameRate59_94 | 0x40,
>>         .width       = avctx->width,
>>         .height      = avctx->height,
>>     };
>>
> 
> I have no personal problem with this but doesn't this make the code
> gcc specific?

It's C99 and it's what ffmpeg uses =)

> Doesn't ffmpeg notionally support other compilers?

many C99 compilers build ffmpeg just fine or at least the non-inline-asm
part of it, clang seems to be fine with that as well.

lu

-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero




More information about the ffmpeg-devel mailing list