Re: Mouse button handling

[Thomas Dickey]

On Fri, Sep 16, 2011 at 09:44:39PM +0200, Damien Guibouret wrote:
> I have found a problem in what I sent previously: when merging clicks 
> into doubleclicks and tripleclicks, I remove the check that clicks were 
> done with same button. So here is the changes to set it back.

I'm missing something: the patch appears to modify a later version
of your patch from a week ago.  (The "merge = FALSE" assignment doesn't
appear in that context before or after the previous patch). 

> Regards,
>
> Damien

> --- ncurses-5.9.patch/ncurses/base/lib_mouse.c        2011-09-16 
> 05:50:19.000000000 +0200
> +++ ncurses-5.9/ncurses/base/lib_mouse.c      2011-09-16 20:58:31.771697056 
> +0200
> @@ -1253,6 +1253,7 @@
>                               merge = FALSE;
>                               for (b = 1; b <= MAX_BUTTONS; ++b) {
>                                       if ((sp->_mouse_mask & 
> MASK_DOUBLE_CLICK(b))
> +                                                     && (ep->bstate & 
> MASK_CLICK(b))
>                                                       && (next->bstate & 
> MASK_CLICK(b))) {
>                                               next->bstate &= ~MASK_CLICK(b);
>                                               next->bstate |= 
> MASK_DOUBLE_CLICK(b);
> @@ -1269,6 +1270,7 @@
>                               merge = FALSE;
>                               for (b = 1; b <= MAX_BUTTONS; ++b) {
>                                       if ((sp->_mouse_mask & 
> MASK_TRIPLE_CLICK(b))
> +                                                     && (ep->bstate & 
> MASK_DOUBLE_CLICK(b))
>                                                       && (next->bstate & 
> MASK_CLICK(b))) {
>                                               next->bstate &= ~MASK_CLICK(b);
>                                               next->bstate |= 
> MASK_TRIPLE_CLICK(b);

-- 
Thomas E. Dickey <address@hidden>
http://invisible-island.net
ftp://invisible-island.net

signature.asc
Description: Digital signature