diff options
author | Hiltjo Posthuma <hiltjo@codemadness.org> | 2021-01-09 14:58:40 +0100 |
---|---|---|
committer | Hiltjo Posthuma <hiltjo@codemadness.org> | 2021-01-09 14:58:40 +0100 |
commit | 6403eadff853f470a859317f605dd306eb427bba (patch) | |
tree | 154c4945fa9058a6bc9c323bf20c147c37693624 | |
parent | d95499b2d8e777fcb82fd3637044c3e70118a06b (diff) |
printutf8pad: small code-style/clarify changes
-rw-r--r-- | util.c | 18 |
1 files changed, 10 insertions, 8 deletions
@@ -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++; |