[MPlayer-dev-eng] [PATCH] Adding the Half Size to gmplayer menu

Pierre Marc Dumuid pierre.dumuid at adelaide.edu.au
Mon Jan 17 07:17:02 CET 2005


Here is a patch to add the missing "Half size" menu item.
On a side note, as well as the pixmap, Gui/mplayer/pixmaps/half.xpm
Though I only know english, hence was unable to add the other languages..

I was rather curious when looking at main_patch/Gui/app.h, the following 
arrangement:

#define evNone              0
#define evPlay              1
#define evStop              2
#define evPause             3
#define evPrev              6
#define evNext              7
#define evLoad              8
#define evEqualizer         9
#define evPlayList          10
#define evIconify           11
#define evAbout             12
#define evLoadPlay          13
#define evPreferences       14
#define evSkinBrowser       15
#define evPlaySwitchToPause 16
#define evPauseSwitchToPlay 17

#define evBackward10sec     18
#define evForward10sec      19
#define evBackward1min      20
#define evForward1min       21
#define evBackward10min     22
#define evForward10min      23

....

Considering that one might want to add a feature to the first section, 
then that are meant to increment all the numbers all the way down to 
maintain neatness.  I guess a few tricks in emacs could do this, but I 
thought it'd be easier to implement it as:

#define evNone              0
#define evPlay              101
#define evStop              102
#define evPause             103
#define evPrev              106
#define evNext              107
#define evLoad              108
#define evEqualizer         109
#define evPlayList          110
#define evIconify           111
#define evAbout             112
#define evLoadPlay          113
#define evPreferences       114
#define evSkinBrowser       115
#define evPlaySwitchToPause 116
#define evPauseSwitchToPlay 117

#define evBackward10sec     218
#define evForward10sec      219
#define evBackward1min      220
#define evForward1min       221
#define evBackward10min     222
#define evForward10min      223

....

etc. whereby addition of an event, only means re-arranging that section, 
and the ability of if (event) is still maintained since evNone = 0

Any thought on this??

Regards,
Pierre Dumuid

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20050117/87eca96b/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: half.xpm
Type: image/x-xpixmap
Size: 2628 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20050117/87eca96b/attachment.xpm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mplayer_addhalfsizetomenu.patch
Type: text/x-patch
Size: 5990 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20050117/87eca96b/attachment.bin>


More information about the MPlayer-dev-eng mailing list