[hackers] [st] Copy non set positions as spaces || "Roberto E. Vargas Caballero"

From: <hg_AT_suckless.org>
Date: Mon, 17 Sep 2012 22:46:40 +0200 (CEST)

changeset: 315:522c577a2962
user: "Roberto E. Vargas Caballero" <k0ga_AT_shike2.com>
date: Mon Sep 17 22:13:09 2012 +0200
files: st.c
description:
Copy non set positions as spaces
st selection don't insert in the selection position whose value is not
set. This is correct for the positions in the end of the line, but cause
some problems in the beginning. For example echo -e 'a\tb' will print in the
screen:

a b

but after selecting and copying in some place you get:

ab

because positions from 1 to 7 don't have any value. This patch deals all
positions without value as blank (even at the end of the line).

---
 st.c |   17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)
diff -r 8a10729d93c9 -r 522c577a2962 st.c
--- a/st.c	Mon Sep 17 22:11:28 2012 +0200
+++ b/st.c	Mon Sep 17 22:13:09 2012 +0200
_AT_@ -596,14 +596,17 @@
 		/* append every set & selected glyph to the selection */
 		for(y = 0; y < term.row; y++) {
 			for(x = 0; x < term.col; x++) {
-				is_selected = selected(x, y);
-				if((term.line[y][x].state & GLYPH_SET) && is_selected) {
-					int size = utf8size(term.line[y][x].c);
-					memcpy(ptr, term.line[y][x].c, size);
-					ptr += size;
-				}
+				int size;
+				char *p;
+				Glyph *gp = &term.line[y][x];
+
+				if(!(is_selected = selected(x, y)))
+					continue;
+				p = (gp->state & GLYPH_SET) ? gp->c : " ";
+				size = utf8size(p);
+				memcpy(ptr, p, size);
+				ptr += size;
 			}
-
 			/* \n at the end of every selected line except for the last one */
 			if(is_selected && y < sel.e.y)
 				*ptr++ = '\n';

Received on Mon Sep 17 2012 - 22:46:40 CEST