[FFmpeg-cvslog] r17715 - trunk/doc/general.texi

[Michael Niedermayer]

On Mon, Mar 02, 2009 at 07:29:07AM +0100, Diego Biurrun wrote:
> On Mon, Mar 02, 2009 at 09:08:19AM +0200, Kostya wrote:
> > On Mon, Mar 02, 2009 at 07:04:37AM +0100, Diego Biurrun wrote:
> > > On Mon, Mar 02, 2009 at 06:08:58AM +0100, diego wrote:
> > > > 
> > > > Log:
> > > > Decoding of PAM, PBM, PGM, PGMYUV and PPM image formats is not supported.
> > > 
> > > Hmm, this seems to be wrong, i.e. FFmpeg does support decoding these
> > > formats.  However libavcodec/allcodecs.c just lists encoders for them.
> > > What am I missing?
> > 
> > The fact all of those are PNM incarnations that are handled by single decoder.
> 
> I figured out as much by now:
> 
> #if CONFIG_PAM_ENCODER
> AVCodec pam_encoder = {
>     "pam",
>     CODEC_TYPE_VIDEO,
>     CODEC_ID_PAM,
>     sizeof(PNMContext),
>     common_init,
>     pam_encode_frame,
>     NULL, //encode_end,
>     pnm_decode_frame,
>     .pix_fmts= (enum PixelFormat[]){PIX_FMT_RGB24, PIX_FMT_RGB32, PIX_FMT_GRAY8, PIX_FMT_MONOWHITE, PIX_FMT_NONE},
>     .long_name= NULL_IF_CONFIG_SMALL("PAM (Portable AnyMap) image"),
> };
> #endif // CONFIG_PAM_ENCODER
> 
> This carries both encoding and decoding functions.  Doesn't this at least
> imply that they cannot be disabled separately?

yes, spliting it is welcome though i doubt its wort it except for consistency

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- 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-cvslog/attachments/20090302/e9456591/attachment.pgp>