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

Diego Biurrun diego at biurrun.de
Sun Sep 12 00:52:33 CEST 2010


On Wed, Sep 08, 2010 at 09:10:49PM +0200, Reimar Döffinger wrote:
> On Wed, Sep 08, 2010 at 08:16:06PM +0200, diego wrote:
> > 
> > Log:
> > Add proper include instead of declaring the vo_pts variable extern.
> > 
> > --- 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 "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?

Sure, done.

Diego


More information about the MPlayer-cvslog mailing list