Re: [dev] [st] [PATCH] Refactor selsnap SNAP_WORD.

From: Roberto E. Vargas Caballero <k0ga_AT_shike2.com>
Date: Thu, 5 Jun 2014 21:19:49 +0200

What do you think about this slightly different patch? For my eyes was
too hard see the line break in the brackets of and array, and I think was
not a good idea to have '(direction > 0)' repeated.

commit 5159d55c631cd1179bd2f872859d4540668745b9
Author: Colona <colona_AT_ycc.fr>
Date: Thu Jun 5 06:32:01 2014 +0200

    Refactor selsnap SNAP_WORD.
    
    Refactor the SNAP_WORD part in selsnap, and fix a bug that caused the word
    delimiters to be ignored if it was at the very beginning or end of a wrapped
    line.
    
    Signed-off-by: Roberto E. Vargas Caballero <k0ga_AT_shike2.com>

diff --git a/st.c b/st.c
index 8b65450..13226d8 100644
--- a/st.c
+++ b/st.c
_AT_@ -703,6 +703,9 @@ selected(int x, int y) {
 
 void
 selsnap(int mode, int *x, int *y, int direction) {
+ int newx, newy, xt, yt;
+ Glyph *gp;
+
         switch(mode) {
         case SNAP_WORD:
                 /*
_AT_@ -710,36 +713,31 @@ selsnap(int mode, int *x, int *y, int direction) {
                  * beginning of a line.
                  */
                 for(;;) {
- if(direction < 0 && *x <= 0) {
- if(*y > 0 && term.line[*y - 1][term.col-1].mode
- & ATTR_WRAP) {
- *y -= 1;
- *x = term.col-1;
- } else {
+ newx = *x + direction;
+ newy = *y;
+ if(!BETWEEN(newx, 0, term.col - 1)) {
+ newy += direction;
+ newx = (newx + term.col) % term.col;
+ if (!BETWEEN(newy, 0, term.row - 1))
                                         break;
- }
- }
- if(direction > 0 && *x >= term.col-1) {
- if(*y < term.row-1 && term.line[*y][*x].mode
- & ATTR_WRAP) {
- *y += 1;
- *x = 0;
- } else {
+
+ if(direction > 0)
+ yt = *y, xt = *x;
+ else
+ yt = newy, xt = newx;
+ if(!(term.line[yt][xt].mode & ATTR_WRAP))
                                         break;
- }
                         }
 
- if(term.line[*y][*x+direction].mode & ATTR_WDUMMY) {
- *x += direction;
- continue;
- }
+ if (newx >= tlinelen(newy))
+ break;
 
- if(*x >= tlinelen(*y) || strchr(worddelimiters,
- term.line[*y][*x+direction].c[0])) {
+ gp = &term.line[newy][newx];
+ if (!(gp->mode & ATTR_WDUMMY) && strchr(worddelimiters, gp->c[0]))
                                 break;
- }
 
- *x += direction;
+ *x = newx;
+ *y = newy;
                 }
                 break;
         case SNAP_LINE:
-- 
Roberto E. Vargas Caballero
Received on Thu Jun 05 2014 - 21:19:49 CEST

This archive was generated by hypermail 2.3.0 : Thu Jun 05 2014 - 21:24:14 CEST