[FFmpeg-devel] [PATCH] Explicitely declare {dst, src}Format sws_get*Context() params as enum PixelFormat

Aurelien Jacobs aurel
Mon Feb 9 23:59:29 CET 2009


Art Clarke wrote:

> > so if i configure gcc to error out on a comment that contains "teh"
> > then we will consider adding such typo requireing a bump?
> 
> slightly different problem here; prior to the original checkin valid C
> code like this (apologies, but not double checking):
> 
> int foo = PIX_FMT_YUV420P;
> ...
> struct SwsContext * s = sws_getContext(10,20,foo,20,20,foo,0,0,0);
> 
> was valid.  now it no longer is.

It is still valid and still compiles fine !
It don't even generate a warning with -Wall.

Aurel




More information about the ffmpeg-devel mailing list