[FFmpeg-devel] GOP_START_CODE definition conflict

Michael Niedermayer michaelni at gmx.at
Fri Oct 21 23:00:28 CEST 2011

On Fri, Oct 21, 2011 at 04:48:54PM +0000, C Chatterjee wrote:
> Found an issue with MPEG1/2 and MPEG4 definitions.
> In particular we have:
> libavcodec\mpegvideo.h(75):#define GOP_START_CODE          0x000001b8
> which is the MPEG1/2 definition of GOP Start code.
> These are redefined in:
> libavformat\mpegvideodec.c(27):#define GOP_START_CODE          0x000001b8
> why?
> #define SEQ_START_CODE          0x000001b3
> #define GOP_START_CODE          0x000001b8
> #define PICTURE_START_CODE      0x00000100
> #define SLICE_START_CODE        0x00000101
> Also we have:
> libavcodec\mpeg4video.h(58):#define GOP_STARTCODE        0x1B3
> which is the MPEG4 definition.
> Although the subtle _ difference exits, it will be easier to name the latter say GVOP_START_CODE ie Group of VOP startcode.
> Also we should take out the redefinition in libavformat\mpegvideodec.c.

I agree
can you send a patch ?

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
-------------- 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/20111021/f2249bf1/attachment.asc>

More information about the ffmpeg-devel mailing list