aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRoberto E. Vargas Caballero <k0ga@shike2.com>2014-08-15 14:48:16 +0200
committerRoberto E. Vargas Caballero <k0ga@shike2.com>2014-08-15 14:48:16 +0200
commit6530025bcaf3a65083667a93ae50035bd7137bae (patch)
treef0cef72fd0b335c8f36939aac3c03b812c668975
parentd4a17316d33f3c5a0017d7fe6e7e174883ccaa97 (diff)
downloadst-6530025bcaf3a65083667a93ae50035bd7137bae.tar.gz
st-6530025bcaf3a65083667a93ae50035bd7137bae.tar.bz2
Fix portability problem in techo()
ISCONTROL chechks if a value is between 0 and 0x1f or between 0x80 and 0x9f. Char signess depends of architecture and compiler, so in some environment the second case is always false (and wrong), Techo() calls ISCONTROL with a char variable, whose type cannot be changed because tpuc() expects a pointer to char, so the solution is to insert a cast in the call to ISCONTROL.
-rw-r--r--st.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/st.c b/st.c
index 814f3f5..d8ff30b 100644
--- a/st.c
+++ b/st.c
@@ -2311,13 +2311,13 @@ techo(char *buf, int len) {
for(; len > 0; buf++, len--) {
char c = *buf;
- if(ISCONTROL(c)) { /* control code */
+ if(ISCONTROL((uchar) c)) { /* control code */
if(c & 0x80) {
c &= 0x7f;
tputc("^", 1);
tputc("[", 1);
} else if(c != '\n' && c != '\r' && c != '\t') {
- c ^= '\x40';
+ c ^= 0x40;
tputc("^", 1);
}
tputc(&c, 1);