summaryrefslogtreecommitdiff
path: root/usr.bin/less
diff options
context:
space:
mode:
authorIngo Schwarze <schwarze@cvs.openbsd.org>2019-03-01 16:41:38 +0000
committerIngo Schwarze <schwarze@cvs.openbsd.org>2019-03-01 16:41:38 +0000
commit51b117799e9ea8f0dc09fc07892f02b56c9e4fe6 (patch)
treee0a882c151ace22e754df06336c1048d61fc4b6f /usr.bin/less
parentfe17616be87bf4ad45ef6942bd002c6b78eab169 (diff)
de-obfuscate by expanding three useless macros; no functional change;
tweak and OK millert@
Diffstat (limited to 'usr.bin/less')
-rw-r--r--usr.bin/less/line.c46
1 files changed, 22 insertions, 24 deletions
diff --git a/usr.bin/less/line.c b/usr.bin/less/line.c
index a702290b7ef..acc9e604ce7 100644
--- a/usr.bin/less/line.c
+++ b/usr.bin/less/line.c
@@ -484,10 +484,6 @@ is_ansi_middle(LWCHAR ch)
/*
* Append a character and attribute to the line buffer.
*/
-#define STORE_CHAR(ch, a, rep, pos) \
- if (store_char((ch), (a), (rep), (pos))) \
- return (1)
-
static int
store_char(LWCHAR ch, char a, char *rep, off_t pos)
{
@@ -584,10 +580,6 @@ store_char(LWCHAR ch, char a, char *rep, off_t pos)
* Append a tab to the line buffer.
* Store spaces to represent the tab.
*/
-#define STORE_TAB(a, pos) \
- if (store_tab((a), (pos))) \
- return (1)
-
static int
store_tab(int attr, off_t pos)
{
@@ -609,15 +601,12 @@ store_tab(int attr, off_t pos)
return (1);
do {
- STORE_CHAR(' ', attr, " ", pos);
+ if (store_char(' ', attr, " ", pos))
+ return (1);
} while (--to_tab > 0);
return (0);
}
-#define STORE_PRCHAR(c, pos) \
- if (store_prchar((c), (pos))) \
- return (1)
-
static int
store_prchar(char c, off_t pos)
{
@@ -637,7 +626,8 @@ store_prchar(char c, off_t pos)
return (1);
for (; *s != 0; s++) {
- STORE_CHAR(*s, AT_BINARY, NULL, pos);
+ if (store_char(*s, AT_BINARY, NULL, pos))
+ return (1);
}
return (0);
}
@@ -647,10 +637,10 @@ flush_mbc_buf(off_t pos)
{
int i;
- for (i = 0; i < mbc_buf_index; i++)
+ for (i = 0; i < mbc_buf_index; i++) {
if (store_prchar(mbc_buf[i], pos))
return (mbc_buf_index - i);
-
+ }
return (0);
}
@@ -771,9 +761,11 @@ do_append(LWCHAR ch, char *rep, off_t pos)
if (curr <= lmargin ||
column <= lmargin ||
(attr[curr - 1] & (AT_ANSI|AT_BINARY))) {
- STORE_PRCHAR('\b', pos);
+ if (store_prchar('\b', pos))
+ return (1);
} else if (bs_mode == BS_NORMAL) {
- STORE_CHAR(ch, AT_NORMAL, NULL, pos);
+ if (store_char(ch, AT_NORMAL, NULL, pos))
+ return (1);
} else if (bs_mode == BS_SPECIAL) {
overstrike = backc();
}
@@ -842,7 +834,8 @@ do_append(LWCHAR ch, char *rep, off_t pos)
goto do_control_char;
case BS_NORMAL:
case BS_SPECIAL:
- STORE_TAB(a, pos);
+ if (store_tab(a, pos))
+ return (1);
break;
}
} else if ((!utf_mode || is_ascii_char(ch)) && control_char((char)ch)) {
@@ -852,9 +845,11 @@ do_control_char:
/*
* Output as a normal character.
*/
- STORE_CHAR(ch, AT_NORMAL, rep, pos);
+ if (store_char(ch, AT_NORMAL, rep, pos))
+ return (1);
} else {
- STORE_PRCHAR((char)ch, pos);
+ if (store_prchar(ch, pos))
+ return (1);
}
} else if (utf_mode && ctldisp != OPT_ON && is_ubin_char(ch)) {
char *s;
@@ -865,10 +860,13 @@ do_control_char:
pwidth(' ', binattr, 0) + attr_ewidth(binattr) > sc_width)
return (1);
- for (; *s != 0; s++)
- STORE_CHAR(*s, AT_BINARY, NULL, pos);
+ for (; *s != 0; s++) {
+ if (store_char(*s, AT_BINARY, NULL, pos))
+ return (1);
+ }
} else {
- STORE_CHAR(ch, a, rep, pos);
+ if (store_char(ch, a, rep, pos))
+ return (1);
}
return (0);
}