diff options
author | Ingo Schwarze <schwarze@openbsd.org> | 2010-10-02 15:15:55 +0000 |
---|---|---|
committer | Ingo Schwarze <schwarze@openbsd.org> | 2010-10-02 15:15:55 +0000 |
commit | 4d2382996b261e8be35cad8e75ba96e1aa68435c (patch) | |
tree | 23ac72f231b46bbee327e7b46855e29c3cf31c38 /term.c | |
parent | 1bba1dfa3e70cbb4cfa27b8456bf1b1cd1b03ffe (diff) | |
download | mandoc-4d2382996b261e8be35cad8e75ba96e1aa68435c.tar.gz |
style cleanup, no functional change:
* make the initial maxvis/mmax calculation easier to understand
* where real, non-indexing casts happen, make them explicit
* avoid a few lint warnings that can easily be fixed
* remove one needless LINTED comment
"I like this" kristaps@
Diffstat (limited to 'term.c')
-rw-r--r-- | term.c | 36 |
1 files changed, 14 insertions, 22 deletions
@@ -146,24 +146,21 @@ term_flushln(struct termp *p) * an indentation, but can be, for tagged lists or columns, a * small set of values. */ - - assert(p->offset < p->rmargin); - - maxvis = (int)(p->rmargin - p->offset) - p->overstep < 0 ? - /* LINTED */ - 0 : p->rmargin - p->offset - p->overstep; - mmax = (int)(p->maxrmargin - p->offset) - p->overstep < 0 ? - /* LINTED */ - 0 : p->maxrmargin - p->offset - p->overstep; + assert (p->rmargin > p->offset); + dv = p->rmargin - p->offset; + maxvis = (int)dv > p->overstep ? dv - (size_t)p->overstep : 0; + dv = p->maxrmargin - p->offset; + mmax = (int)dv > p->overstep ? dv - (size_t)p->overstep : 0; bp = TERMP_NOBREAK & p->flags ? mmax : maxvis; /* * Indent the first line of a paragraph. */ - vbl = p->flags & TERMP_NOLPAD ? 0 : p->offset; + vbl = p->flags & TERMP_NOLPAD ? (size_t)0 : p->offset; - vis = vend = i = 0; + vis = vend = 0; + i = 0; while (i < (int)p->col) { /* @@ -184,7 +181,6 @@ term_flushln(struct termp *p) * space is printed according to regular spacing rules). */ - /* LINTED */ for (j = i, jhy = 0; j < (int)p->col; j++) { if ((j && ' ' == p->buf[j]) || '\t' == p->buf[j]) break; @@ -223,8 +219,7 @@ term_flushln(struct termp *p) /* Remove the p->overstep width. */ - bp += (int)/* LINTED */ - p->overstep; + bp += (size_t)p->overstep; p->overstep = 0; } @@ -238,7 +233,7 @@ term_flushln(struct termp *p) j = i; while (' ' == p->buf[i]) i++; - dv = (i - j) * (*p->width)(p, ' '); + dv = (size_t)(i - j) * (*p->width)(p, ' '); vbl += dv; vend += dv; break; @@ -287,8 +282,7 @@ term_flushln(struct termp *p) if (TERMP_HANG & p->flags) { /* We need one blank after the tag. */ - p->overstep = /* LINTED */ - vis - maxvis + (*p->width)(p, ' '); + p->overstep = (int)(vis - maxvis + (*p->width)(p, ' ')); /* * Behave exactly the same way as groff: @@ -302,8 +296,7 @@ term_flushln(struct termp *p) */ if (p->overstep >= -1) { assert((int)maxvis + p->overstep >= 0); - /* LINTED */ - maxvis += p->overstep; + maxvis += (size_t)p->overstep; } else p->overstep = 0; @@ -311,9 +304,8 @@ term_flushln(struct termp *p) return; /* Right-pad. */ - if (maxvis > vis + /* LINTED */ - ((TERMP_TWOSPACE & p->flags) ? - (*p->width)(p, ' ') : 0)) { + if (maxvis > vis + + ((TERMP_TWOSPACE & p->flags) ? (*p->width)(p, ' ') : 0)) { p->viscol += maxvis - vis; (*p->advance)(p, maxvis - vis); vis += (maxvis - vis); |