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

Art Clarke aclarke
Mon Feb 9 06:48:25 CET 2009


On Sun, Feb 8, 2009 at 3:00 PM, Stefano Sabatini
<stefano.sabatini-lala at poste.it> wrote:
>> I wonder if I should bump micro/minor, maybe there are some
>> compatibility issues with some compilers...
>
> Applied.

A little late now, but you should at least bump minor;  This breaks
code for swscale users who previously passed in int values (like, oh,
us).

We ran into the problem because we wanted to disguise the PixelFormat
dependency in other objects and hence pass-around an opaque int; we
just didn't cast it when passing back into SWSContext calls because we
didn't have to (until this morning when our builds started breaking).

It's a simple fix in users' code, but I think it is an API change
(yes, for the better, but a change none the less).

- Art

-- 
http://www.xuggle.com/
xu?ggle (z?' gl) v. To freely encode, decode, and experience audio and video.

Use Xuggle to get the power of FFMPEG in Java.




More information about the ffmpeg-devel mailing list