summaryrefslogtreecommitdiff
path: root/util.c
diff options
context:
space:
mode:
authorHiltjo Posthuma <hiltjo@codemadness.org>2021-01-09 14:58:40 +0100
committerHiltjo Posthuma <hiltjo@codemadness.org>2021-01-09 14:58:40 +0100
commit6403eadff853f470a859317f605dd306eb427bba (patch)
tree154c4945fa9058a6bc9c323bf20c147c37693624 /util.c
parentd95499b2d8e777fcb82fd3637044c3e70118a06b (diff)
printutf8pad: small code-style/clarify changes
Diffstat (limited to 'util.c')
-rw-r--r--util.c18
1 files changed, 10 insertions, 8 deletions
diff --git a/util.c b/util.c
index 955332b..2e4110f 100644
--- a/util.c
+++ b/util.c
@@ -234,27 +234,29 @@ printutf8pad(FILE *fp, const char *s, size_t len, int pad)
{
wchar_t wc;
size_t col = 0, i, slen;
- int rl, siz, w;
+ int inc, rl, w;
if (!len)
return;
slen = strlen(s);
- for (i = 0; i < slen; i += siz) {
- siz = 1;
+ for (i = 0; i < slen; i += inc) {
+ inc = 1;
if ((unsigned char)s[i] < 32) {
continue; /* skip control characters */
} else if ((unsigned char)s[i] >= 127) {
- rl = siz = mbtowc(&wc, s + i, slen - i < 4 ? slen - i : 4);
+ rl = mbtowc(&wc, s + i, slen - i < 4 ? slen - i : 4);
if (rl < 0) {
mbtowc(NULL, NULL, 0); /* reset state */
- siz = 1; /* next byte */
+ inc = 1; /* next byte */
w = 1; /* replacement char is one width */
} else if ((w = wcwidth(wc)) == -1) {
continue;
+ } else {
+ inc = rl;
}
- if (col + w > len || (col + w == len && s[i + siz])) {
+ if (col + w > len || (col + w == len && s[i + inc])) {
fputs("\xe2\x80\xa6", fp); /* ellipsis */
col++;
break;
@@ -263,10 +265,10 @@ printutf8pad(FILE *fp, const char *s, size_t len, int pad)
col++;
continue;
}
- fwrite(&s[i], 1, siz, fp);
+ fwrite(&s[i], 1, rl, fp);
col += w;
} else {
- /* simple ASCII character */
+ /* optimization: simple ASCII character */
if (col + 1 > len || (col + 1 == len && s[i + 1])) {
fputs("\xe2\x80\xa6", fp); /* ellipsis */
col++;