Hi all, When choosing the selection, we really don't want to do a "line select" in vertical border cases, since noticing the exact cursor position at the border is very difficult. Such behaviour is highly annoying ("skippy") and UNWANTED: commented out relevant parts. (besides, X11 doesn't do it that way either!) We could choose to add a configuration option for that behaviour, but given that X11 doesn't do it that way (or is it even configurable there?), I don't think it's worth it. Also, some spelling fixes. Greetings, Andreas Mohr Index: README =================================================================== RCS file: /data/cvs/gpm/README,v retrieving revision 1.6 diff -u -r1.6 README --- README 24 Dec 2002 23:24:47 -0000 1.6 +++ README 16 Mar 2003 16:56:54 -0000 @@ -31,7 +31,7 @@ =========== MAINTENANCE As of 1.19.4, gpm is officially maintained again. Most of the README is written -by Alessandro Rubine, only minor changes are made by me. +by Alessandro Rubini, only minor changes are made by me. =========== MAILING LIST @@ -128,7 +128,7 @@ msc" The program goes to the background by itself, unless - it is run with debuging enabled (see the -D option in the + it is run with debugging enabled (see the -D option in the documentation). ========== DOCS Index: src/gpm.c =================================================================== RCS file: /data/cvs/gpm/src/gpm.c,v retrieving revision 1.4 diff -u -r1.4 gpm.c --- src/gpm.c 30 Nov 2002 13:20:09 -0000 1.4 +++ src/gpm.c 16 Mar 2003 16:56:59 -0000 @@ -259,8 +259,13 @@ if (event->buttons==GPM_B_LEFT) { if (event->margin) /* fix margins */ switch(event->margin) { - case GPM_TOP: x2=1; y2++; break; - case GPM_BOT: x2=maxx; y2--; break; + /* we really don't want to do a "line select" in vertical + * border cases, since noticing the exact cursor position + * at the border is very difficult. Such behaviour is + * highly annoying ("skippy") and UNWANTED: commented out + * relevant parts. (besides, X11 doesn't do it either!) */ + case GPM_TOP: /* x2=1; */ y2++; break; + case GPM_BOT: /* x2=maxx; */ y2--; break; case GPM_RGT: x2--; break; case GPM_LFT: y2<=y1 ? x2++ : (x2=maxx, y2--); break; } @@ -1036,7 +1041,7 @@ gpm_report(GPM_PR_DEBUG,GPM_MESS_SELECT_TIMES,pending); -/*....................................... manage graphic mode */ +/*....................................... manage graphics mode */ /* * Be sure to be in text mode. This used to be before select, @@ -1059,7 +1064,7 @@ /*....................................... got mouse, process event */ /* * Well, actually, run a loop to maintain inlining of functions without - * lenghtening the file. This is not too clean a code, but it works.... + * lengthening the file. This is not too clean a code, but it works.... */ for (i=1; i <= 1+opt_double; i++) { Index: src/startup.c =================================================================== RCS file: /data/cvs/gpm/src/startup.c,v retrieving revision 1.2 diff -u -r1.2 startup.c --- src/startup.c 24 Dec 2002 23:24:48 -0000 1.2 +++ src/startup.c 16 Mar 2003 16:56:59 -0000 @@ -145,7 +145,7 @@ if (setsid() < 0) gpm_report(GPM_PR_OOPS,GPM_MESS_SETSID_FAILED); } - /* damon init: check whether we run or not, display message */ + /* daemon init: check whether we run or not, display message */ check_uniqueness(); gpm_report(GPM_PR_INFO,GPM_MESS_STARTED); -- Help prevent Information Technology Fascism! - before it's too late... http://www.againsttcpa.com