[hackers] [st] Move calls to selsnap into selnormalize || Ivan Delalande

From: <git_AT_suckless.org>
Date: Fri, 22 Aug 2014 16:05:38 +0200

commit c490a60b804f467490cd4d8275a181dc37edef9f
Author: Ivan Delalande <colona_AT_ycc.fr>
Date: Wed Aug 20 21:14:08 2014 +0200

    Move calls to selsnap into selnormalize
    
    This simplifies getbuttoninfo() and bpress(), and fixes a bug which made word
    snapping behave incorrectly when a delimiter was at the beginning or end of
    line.
    
    Signed-off-by: Roberto E. Vargas Caballero <k0ga_AT_shike2.com>

diff --git a/st.c b/st.c
index 6165119..01ab962 100644
--- a/st.c
+++ b/st.c
_AT_@ -682,6 +682,9 @@ selnormalize(void) {
         sel.nb.y = MIN(sel.ob.y, sel.oe.y);
         sel.ne.y = MAX(sel.ob.y, sel.oe.y);
 
+ selsnap(sel.snap, &sel.nb.x, &sel.nb.y, -1);
+ selsnap(sel.snap, &sel.ne.x, &sel.ne.y, +1);
+
         /* expand selection over line breaks */
         if (sel.type == SEL_RECTANGULAR)
                 return;
_AT_@ -777,15 +780,6 @@ getbuttoninfo(XEvent *e) {
 
         sel.oe.x = x2col(e->xbutton.x);
         sel.oe.y = y2row(e->xbutton.y);
-
- if(sel.ob.y < sel.oe.y
- || (sel.ob.y == sel.oe.y && sel.ob.x < sel.oe.x)) {
- selsnap(sel.snap, &sel.ob.x, &sel.ob.y, -1);
- selsnap(sel.snap, &sel.oe.x, &sel.oe.y, +1);
- } else {
- selsnap(sel.snap, &sel.oe.x, &sel.oe.y, -1);
- selsnap(sel.snap, &sel.ob.x, &sel.ob.y, +1);
- }
         selnormalize();
 
         sel.type = SEL_REGULAR;
_AT_@ -900,8 +894,6 @@ bpress(XEvent *e) {
                 } else {
                         sel.snap = 0;
                 }
- selsnap(sel.snap, &sel.ob.x, &sel.ob.y, -1);
- selsnap(sel.snap, &sel.oe.x, &sel.oe.y, +1);
                 selnormalize();
 
                 /*
Received on Fri Aug 22 2014 - 16:05:38 CEST

This archive was generated by hypermail 2.3.0 : Fri Aug 22 2014 - 16:12:09 CEST