[MPlayer-cvslog] r32081 - in trunk/libao2: ao_dxr2.c ao_ivtv.c ao_mpegpes.c ao_pcm.c ao_v4l2.c

[Reimar Döffinger]

On Wed, Sep 08, 2010 at 08:16:06PM +0200, diego wrote:
> Author: diego
> Date: Wed Sep  8 20:16:06 2010
> New Revision: 32081
> 
> Log:
> Add proper include instead of declaring the vo_pts variable extern.
> 
> Modified:
>    trunk/libao2/ao_dxr2.c
>    trunk/libao2/ao_ivtv.c
>    trunk/libao2/ao_mpegpes.c
>    trunk/libao2/ao_pcm.c
>    trunk/libao2/ao_v4l2.c
> 
> Modified: trunk/libao2/ao_dxr2.c
> ==============================================================================
> --- trunk/libao2/ao_dxr2.c	Wed Sep  8 20:02:38 2010	(r32080)
> +++ trunk/libao2/ao_dxr2.c	Wed Sep  8 20:16:06 2010	(r32081)
> @@ -34,6 +34,7 @@
>  #include "audio_out_internal.h"
>  #include "libaf/af_format.h"
>  #include "libmpdemux/mpeg_packetizer.h"
> +#include "libvo/video_out.h"

Including a huge libvo header from inside libao is not exactly beautiful...
I won't object since the vo_pts hack is really ugly in the first place,
but I can't say I really like it either.
Maybe it would be a good case for just adding a comment that this include
is (only) there for vo_pts?