[MPlayer-dev-eng] [PATCH] double click support on Mac OSX
Nicolas Plourde
nicolas.plourde at gmail.com
Sat Aug 18 14:11:43 CEST 2007
On 18-Aug-07, at 7:04 AM, Ulion wrote:
> 2007/8/18, Reimar Döffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de>:
>> Hello,
>> On Sat, Aug 18, 2007 at 04:16:59PM +0800, Ulion wrote:
>>> 2007/8/18, Reimar Döffinger <Reimar.Doeffinger at stud.uni-
>>> karlsruhe.de>:
>>>> If it works (could not even test compilation), attached patch
>>>> might be a
>>>> better way for 1. + 2.
>>>>
>>> It seems I sent a similar patch with your one, most part of code
>>> is same.
>>>
>>> as the difference, I think the "other mouse events" should take us
>>> more mouse buttons, why not fix the code to support up to 10 mouse
>>> buttons instead of the only first three of them?
>>
>> Of course, especially since it simplifies the code. But it has
>> nothing
>> to do with making double-click work, so it definitely should not
>> be in
>> the same patch (see point 8 of DOCS/tech/patches.txt).
>> Now what I need first of all is either an okay for a specific
>> patch or a
>> general go-ahead for something similar to this from our maintainer
>> for
>> this code, so I CC him...
>>
>> Greetings,
>> Reimar Döffinger
>
> yes, I agree, the patch is mainly for making double click work.
> I only hope the double click fix in either patch can be made into
> the svn asap:)
> PS: your patch missed the declare part in .h file, if commit, should
> also remember to add it.
>
Im testing it and should submit soon.
====
Nicolas Plourde <nicolas.plourde at gmail.com>
More information about the MPlayer-dev-eng
mailing list