[MPlayer-cvslog] r32780 - in trunk: libao2/ao_dxr2.c libvo/vo_dxr2.c
Tobias Diedrich
ranma at tdiedrich.de
Thu Jan 13 18:35:13 CET 2011
Diego Biurrun wrote:
> Tobias Diedrich wrote:
> > Duplicate prototype (ao_dxr2), missing prototypes and unused
> > variable (vo_dxr2) IIRC.
>
> You should paste the warnings into the log message.
Log message is updated now.
> > > In any case this forward declaration should be replaced by a header.
> >
> > I wanted to keep the changes minimal, I'm not sure it's worth
> > spending more time cleaning up this part of the code.
> > After all it was broken for about 6 Months and no one noticed.
> > I myself no longer have the hardware to test.
> > At least now it compiles again and doesn't give compiler warnings.
>
> I valiantly fought to move all extern and forward declarations into
> proper header files and would hate to see the effort subverted.
Seems like you missed these. :)
I just saw that there already is an vo_dxr2.h which you created, but
it never had these in it and also wasn't included from vo_dxr2.c,
only from ao_dxr2.c. ;)
--
Tobias PGP: http://8ef7ddba.uguu.de
More information about the MPlayer-cvslog
mailing list