"AIS, Informofico" <i...@ais-sanmarino.org> wrote:

Hi,

> Gladly I will do my best to explain the patch for xsane. You (JB)
> write that "it doesn't do much" but the problem was (as I pointed out
> in the Ubuntu bug discussion in Launchpad) that the
> "GdkEventMotion.state" value originally was masked (AND-ed) with
> values such as GDK_Num_Lock (0xff7f) or GDK_Caps_Lock (0xffe5). These
> values are */not/* bit masks, they are enumerated values for key
> codes. So it makes no sense to use them for bitwise operations. That
> they worked certainly was not pure coincidence but maybe a result of
> trial and error.

Thank you very much for the research and explanations behind your patch,
it's crystal clear and very appreciated!

I'll apply the patch in the next revision, won't take long.

JB.

-- 
 Julien BLACHE <jbla...@debian.org>  |  Debian, because code matters more 
 Debian & GNU/Linux Developer        |       <http://www.debian.org>
 Public key available on <http://www.jblache.org> - KeyID: F5D6 5169 
 GPG Fingerprint : 935A 79F1 C8B3 3521 FD62 7CC7 CD61 4FD7 F5D6 5169 



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to