[MPlayer-cvslog] r34475 - in trunk: DOCS/xml/de/skin.xml DOCS/xml/en/skin.xml gui/app.c gui/app.h gui/interface.c gui/ui/actions.c gui/ui/gtk/menu.c gui/ui/main.c gui/ui/pixmaps/cd.xpm gui/ui/pixmaps/dvd.xpm gui/ui...

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sat Dec 31 17:22:51 CET 2011


On Sat, Dec 31, 2011 at 01:38:53PM +0100, ib wrote:
> Author: ib
> Date: Sat Dec 31 13:38:52 2011
> New Revision: 34475
> 
> Log:
> Add audio CD playback support to the X11/GTK GUI.
> 
> (The Win32 GUI already had this support. Add missing - although already
> used - user event evPlayCD here as well as internal event ivSetCDTrack
> - although it's unused - for consistency.)

Could I convince you to work on removing those ifdefs (both CDDA, VCD
and probably more)?
By what I can tell they save maybe 1kB out of 12 MB (as in, 0.01%)
from the compiled binary, that just isn't worth the amount of code mess they add.


More information about the MPlayer-cvslog mailing list