[vlc-devel] [PATCH] MacOS: manage the mouse events to allow DVD navigation.

Pierre d'Herbemont pdherbemont at free.fr
Thu Jul 29 14:09:48 CEST 2010


On Wed, Jul 28, 2010 at 12:10 AM, Sebastien Zwickert <dilaroga at gmail.com> wrote:
> Hi,

Hello,

That's something great, however:
What if the libvlc application wants to handle mouse events itself,
and is not interested in DVD functionality? The patch is not
forwarding event at all, so there is no way to disable it easily. I
know that we are suffering from the fact that libvlccore and its
plugins do not say if the mouse events will be handled or not. Still,
we need to at least keep an option at the very least.

>From what I remember the mouseMoved thing won't work if you don't
first set acceptsMouseMoved events.
Might be wrong though, but you surely could do some factorization or
use vout_display_PlacePicture in mouseMoved:.

And there are some minor style issues such as:
> +- (BOOL) acceptsFirstResponder
no space before accepts

>+    if ( !vd || ![self mouse:curPoint inRect:[self bounds]] )
no space after ( and before ).
...

Pierre.



More information about the vlc-devel mailing list