[FFmpeg-cvslog] r8985 - in trunk: configure libavcodec/Makefile libavcodec/cabac.h libavcodec/h264.c libavcodec/h264.h libavcodec/h264_parser.c libavcodec/h264_parser.h

Diego Biurrun diego
Fri May 11 09:26:59 CEST 2007


On Fri, May 11, 2007 at 12:26:44AM +0200, aurel wrote:
> 
> Log:
> split h264.c to move parser in its own file

I had an identical change in my local tree, oh well ...

> Added:
>    trunk/libavcodec/h264.h
>       - copied, changed from r8983, /trunk/libavcodec/h264.c
>    trunk/libavcodec/h264_parser.c
>       - copied, changed from r8983, /trunk/libavcodec/h264.c
>    trunk/libavcodec/h264_parser.h
>       - copied, changed from r8983, /trunk/libavcodec/h264.c

Copying h264_parser.h is overkill IMO, they basically just share the
license header.

> --- trunk/libavcodec/h264.c	(original)
> +++ trunk/libavcodec/h264.c	Fri May 11 00:26:44 2007
> @@ -30,7 +30,9 @@
>  #include "dsputil.h"
>  #include "avcodec.h"
>  #include "mpegvideo.h"
> +#include "h264.h"
>  #include "h264data.h"
> +#include "h264_parser.h"
>  #include "golomb.h"
>  
>  #include "cabac.h"

I think the #includes that are in h264.h can be removed from the .c file
now.

Diego




More information about the ffmpeg-cvslog mailing list