[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