[MPlayer-dev-eng] [PATCH] move mousewheel-down event supression to fifo.c

Mate Miklos mtmkls at freemail.hu
Sun Jan 21 15:29:01 CET 2007


On 2007 January 18 Thursday 20:48, Reimar Döffinger wrote:
> Hello,
> mousewheel down events are supressed because they tend to get stuck
> (only space for 10 events in our queue). Instead of having this code
> twice in x11_common.c and vo_sdl.c this patch moves it to fifo.c.
> That should make both the reason for this "hack" more clear and maybe
> make it easier to create a proper solution.
>
> Greetings,
> Reimar Döffinger

Hi,

Mouse wheel is button 4 & 5, not 3 & 4 IMHO. I didn't notice this the first 
time I saw this line of code...

And the two extra spaces in the second line make the code prettier :)

Pls correct (fifo.c):

-  if (code < (MOUSE_BTN3 | MP_KEY_DOWN) || code > (MOUSE_BTN4 | MP_KEY_DOWN))
+  if (code < (MOUSE_BTN4 | MP_KEY_DOWN) || code > (MOUSE_BTN5 | MP_KEY_DOWN))
-   mplayer_put_key_internal(code);
+     mplayer_put_key_internal(code);

Miklos



More information about the MPlayer-dev-eng mailing list