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

2014-06-06 Thread Roberto E. Vargas Caballero
> Yeah, you’re right, yours is definitely better, I’m totally fine with > it! Applied! -- Roberto E. Vargas Caballero

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

2014-06-05 Thread Colona
On Thu, Jun 05, 2014 at 09:19:49PM +0200, Roberto E. Vargas Caballero wrote: > 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. Yeah, you’re right

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

2014-06-05 Thread Roberto E. Vargas Caballero
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 Date: Thu Jun 5 06:32:01 2014 +0200

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

2014-06-04 Thread Colona
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. --- st.c | 40 +--- 1 file changed, 17 insertions(+), 23 deletions(-) diff --git a/st.c b/st.c index e0