[FFmpeg-devel] [PATCH] Do not override pix_fmt in rawdec.c

Michael Niedermayer michaelni
Mon Jun 7 12:54:13 CEST 2010

On Mon, Jun 07, 2010 at 11:01:23AM +0200, Tomas H?rdin wrote:
> Hi
> The attached patch adds a check for pix_fmt == PIX_FMT_NONE before
> attempting to guess it near rawdec.c:75. This enables demuxers to set it
> directly if known in advance (for example mxfdec.c).
> Patch passes regtests.

violates api
     * Pixel format, see PIX_FMT_xxx.
     * - encoding: Set by user.
     * - decoding: Set by libavcodec.
this doesnt say set by "user" aka libavformat/app
    enum PixelFormat pix_fmt;

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100607/14f22590/attachment.pgp>

More information about the ffmpeg-devel mailing list