[MPlayer-cvslog] r33848 - trunk/libmpcodecs/ve_xvid4.c

Ivan Kalvachev ikalvachev at gmail.com
Sat Jul 9 01:51:25 CEST 2011


On 7/8/11, diego <subversion at mplayerhq.hu> wrote:
> Author: diego
> Date: Fri Jul  8 01:06:29 2011
> New Revision: 33848
>
> Log:
> ve_xvid: Add libavutil/rational.h #include for AVRational et al.
>
> Modified:
>    trunk/libmpcodecs/ve_xvid4.c
>
> Modified: trunk/libmpcodecs/ve_xvid4.c
> ==============================================================================
> --- trunk/libmpcodecs/ve_xvid4.c	Thu Jul  7 19:37:58 2011	(r33847)
> +++ trunk/libmpcodecs/ve_xvid4.c	Fri Jul  8 01:06:29 2011	(r33848)
> @@ -56,6 +56,7 @@
>  #include <assert.h>
>
>  #include "m_option.h"
> +#include "libavutil/rational.h"
>  #include "libavutil/avutil.h"

That's redundant because avutil.h is supposed to include it all.
It just pollutes the code.

Also, I'd appreciate it if you post patches before committing to files
I maintain.


More information about the MPlayer-cvslog mailing list