[MPlayer-cvslog] r28602 - trunk/libvo/vo_xvmc.c

Diego Biurrun diego at biurrun.de
Sun Feb 15 19:22:49 CET 2009


On Sun, Feb 15, 2009 at 07:13:47PM +0100, Reimar Döffinger wrote:
> On Sun, Feb 15, 2009 at 06:38:51PM +0100, diego wrote:
> > 
> > Log:
> > Remove unnecessary #ifdef around internal #include.
> 
> Just be careful, you never know if one if those Gui headers includes
> some Gtk stuff, then that #ifdef would be necessary (I nearly made that
> mistake with the libavcodec/xmvc.h include).

Yes, I'm aware of this, but thanks for reminding everybody.

A good way to verify this is to have a look at the .d file.  If it
does not include GTK/whatever stuff while the GUI/whatever is enabled,
then it's safe to remove the #ifdef.

Diego



More information about the MPlayer-cvslog mailing list