summaryrefslogtreecommitdiff
path: root/lib/termtable.c
diff options
context:
space:
mode:
authorQuentin Young <qlyoung@cumulusnetworks.com>2017-06-16 19:04:57 +0000
committerQuentin Young <qlyoung@cumulusnetworks.com>2017-06-16 19:04:57 +0000
commit94b989402f6a4f4ea00c3a044deb7d910c4e3f45 (patch)
tree6a6f972df446109624281c4c0a7459a18ab123a2 /lib/termtable.c
parent62e4232010330e515630399838288f6281d04de6 (diff)
lib: termtable coverity warnings
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>
Diffstat (limited to 'lib/termtable.c')
-rw-r--r--lib/termtable.c8
1 files changed, 3 insertions, 5 deletions
diff --git a/lib/termtable.c b/lib/termtable.c
index fb16e5dba1..283fa173d8 100644
--- a/lib/termtable.c
+++ b/lib/termtable.c
@@ -123,15 +123,13 @@ static struct ttable_cell *ttable_insert_row_va(struct ttable *tt, int i,
assert(i >= -1 && i < tt->nrows);
char *res, *orig, *section;
- const char *f;
struct ttable_cell *row;
int col = 0;
int ncols = 0;
/* count how many columns we have */
- f = format;
- for (; f[ncols]; f[ncols] == '|' ? ncols++ : *f++)
- ;
+ for (int i = 0; format[i]; i++)
+ ncols += !!(format[i] == '|');
ncols++;
if (tt->ncols == 0)
@@ -338,7 +336,7 @@ char *ttable_dump(struct ttable *tt, const char *newline)
/* calculate number of lines en total */
nlines = tt->nrows;
nlines += tt->style.border.top_on ? 1 : 0;
- nlines += tt->style.border.bottom_on ? 1 : 1; // makes life easier
+ nlines += 1; // tt->style.border.bottom_on ? 1 : 1; makes life easier
for (int i = 0; i < tt->nrows; i++) {
/* if leftmost cell has top / bottom border, whole row does */
nlines += tt->table[i][0].style.border.top_on ? 1 : 0;