[Ffmpeg-devel] [PATCH] THP Demuxer (Summer of Code qualification task)
Michael Niedermayer
michaelni
Fri Mar 30 13:34:55 CEST 2007
Hi
On Fri, Mar 30, 2007 at 11:14:24AM +0200, Baptiste Coudurier wrote:
> Hi
>
> Marco Gerards wrote:
> > Index: libavcodec/avcodec.h
> > ===================================================================
> > --- libavcodec/avcodec.h (revision 8540)
> > +++ libavcodec/avcodec.h (working copy)
> > @@ -248,6 +248,7 @@
> > CODEC_ID_MUSEPACK7,
> > CODEC_ID_MLP,
> > CODEC_ID_GSM_MS, /* As found in WAV */
> > + CODEC_ID_THP,
>
> Put it at the end of "video" codec ids.
>
> > /* subtitle codecs */
> > CODEC_ID_DVD_SUBTITLE= 0x17000,
> > @@ -255,6 +256,7 @@
> >
> > CODEC_ID_MPEG2TS= 0x20000, /* _FAKE_ codec to indicate a raw MPEG2 transport
> > stream (only used by libavformat) */
> > +
> > };
>
> cosmetics.
nice review, the patch looks pretty good except these 2 points ...
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070330/ac19ae3b/attachment.pgp>
More information about the ffmpeg-devel
mailing list