[FFmpeg-devel] [PATCH] y41p encoder and decoder

[Michael Niedermayer]

On Mon, Jan 02, 2012 at 03:37:29AM +0100, Michael Niedermayer wrote:
> On Sun, Jan 01, 2012 at 11:18:57PM +0000, Paul B Mahol wrote:
> > On 1/1/12, Michael Niedermayer <michaelni at gmx.at> wrote:
> > [...]
> > >> diff --git a/libavcodec/allcodecs.c b/libavcodec/allcodecs.c
> > >> index 1f47aee..7582d3e 100644
> > >> --- a/libavcodec/allcodecs.c
> > >> +++ b/libavcodec/allcodecs.c
> > >> @@ -85,6 +85,7 @@ void avcodec_register_all(void)
> > >>      REGISTER_DECODER (BINK, bink);
> > >>      REGISTER_ENCDEC  (BMP, bmp);
> > >>      REGISTER_DECODER (BMV_VIDEO, bmv_video);
> > >> +    REGISTER_ENCDEC  (Y41P, y41p);
> > >>      REGISTER_DECODER (C93, c93);
> > >
> > > nit: they are alphabetically ordered
> > 
> > Fixed.
> > >
> > >>      REGISTER_DECODER (CAVS, cavs);
> > >>      REGISTER_DECODER (CDGRAPHICS, cdgraphics);
> > >> diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
> > >> index 690ea38..c306933 100644
> > >> --- a/libavcodec/avcodec.h
> > >> +++ b/libavcodec/avcodec.h
> > >> @@ -255,6 +255,7 @@ enum CodecID {
> > >>      CODEC_ID_VBLE,
> > >>      CODEC_ID_DXTORY,
> > >>      CODEC_ID_V410,
> > >> +    CODEC_ID_Y41P,
> > >>      CODEC_ID_UTVIDEO = 0x800,
> > >>      CODEC_ID_ESCAPE130  = MKBETAG('E','1','3','0'),
> > >
> > > please use MKBETAG('Y','4','1','P'), so we can keep ABI compatibility
> > > with forks that append different codecs there.
> > >
> > 
> > Fixed.
> > >
> > >>
> > >> diff --git a/libavcodec/version.h b/libavcodec/version.h
> > >> index 1a6e330..064da05 100644
> > >> --- a/libavcodec/version.h
> > >> +++ b/libavcodec/version.h
> > >> @@ -21,7 +21,7 @@
> > >>  #define AVCODEC_VERSION_H
> > >>
> > >>  #define LIBAVCODEC_VERSION_MAJOR 53
> > >> -#define LIBAVCODEC_VERSION_MINOR 49
> > >> +#define LIBAVCODEC_VERSION_MINOR 50
> > >>  #define LIBAVCODEC_VERSION_MICRO 101
> > >                                       ^
> > > when increasing minor, micro should be reset to the base (100)
> > 
> > Fixed.
> 
> applied, thanks

btw, you might want to add yourself in MAINTAINERS for y41p

[...]


-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Old school: Use the lowest level language in which you can solve the problem
            conveniently.
New school: Use the highest level language in which the latest supercomputer
            can solve the problem without the user falling asleep waiting.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120102/6fe7d769/attachment.asc>