bug-ncurses
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Mouse button handling


From: Thomas Dickey
Subject: Re: Mouse button handling
Date: Sat, 03 Sep 2011 19:14:30 -0400
User-agent: Mutt/1.5.18 (2008-05-17)

On Sat, Sep 03, 2011 at 06:13:20PM -0400, Thomas Dickey wrote:
> On Mon, Aug 29, 2011 at 05:29:37PM -0400, Thomas Dickey wrote:
> > On Mon, 29 Aug 2011, Damien Guibouret wrote:
> >
> >> Hello,
> >>
> >> Loading the ncurses 5.9 version last week to try it in replacement to the 
> >> old
> >> 5.4 I am still using, I've found some differences into mouse management 
> >> that
> >> breaks the behaviour in some cases. I take a look at the sources and did 
> >> some
> >> changes that I think necessary. All the changes I did are into
> >> ncurses/base/lib_mouse.c file (I used 5.9 released version as base) and are
> >> into the attached file with the following explanations.
> >
> > thanks (will review/etc)
> 
> I'm about halfway through investigating this, won't finish today...
> 
> (as a general rule, I isolate the change that broke it, and then
> see how the patch relates to that - looks like something between 5.7/5.8).

The change was this one:

20100102
        + modify getmouse() to act as its documentation implied, returning on
          each call the preceding event until none are left.  When no more
          events remain, it will return ERR.

The reason for it was this:

   modify getmouse() so it peels off the latest event, and checks before
   returning data that it is indeed valid.  This is prompted by changenote
   for Alexander's "le" where he was trying to purge the mouse queue by looping
   on getmouse until it returned an ERR.  It never did that, but modifying it
   seems to work consistently

(I agree it's not correct - hence your bug report, but to fix it properly,
I have to take into account the reason for the previous change.  So I'm
studying it... ;-)

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

Attachment: signature.asc
Description: Digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]