[vlc-devel] commit: libvlc: add mouse events as libvlc_MediaPlayerEvents ( Jean-Paul Saman )

Jean-Paul Saman jpsaman at gmail.com
Thu Feb 11 16:17:50 CET 2010


2010/2/11 Rémi Denis-Courmont <remi at remlab.net>:
>
> On Thu, 11 Feb 2010 12:54:43 +0100, Jean-Paul Saman <jpsaman at gmail.com>
> wrote:
>>> Attaching mouse-object to vout_thread_t seems a bad idea to me as it
>>> used only by the logo filter.
>>>  IMO, it should be moved to the logo filter instead.
>>
>> But how do I obtain the logo filter object then? Without
>> vlc_object_find_name() ??
>
> In my opinion, the question is wrong to begin with. No matter the
> mechanism, it will be at least as bad as vlc_object_find_name(). The
> premise that the logo object must be found is flawed.
> vlc_object_find_name() sucks. Fundamentally. If it were just an
> implementation bug, I would already have fixed. But its design is wrong,
> and that cannot be fixed short of using another design.
>
Can I conclude that you do not like Laurents suggestion?

Kind regards,
Jean-Paul Saman



More information about the vlc-devel mailing list