stat: remove terse v2 blank lines with description not set
authorVincent Fu <vincent.fu@wdc.com>
Wed, 15 May 2019 20:05:03 +0000 (16:05 -0400)
committerJens Axboe <axboe@kernel.dk>
Thu, 16 May 2019 14:42:45 +0000 (08:42 -0600)
The documentation says that the job description will appear on a second
line if this is set. If it is not set there will be an empty line.
Eliminate the empty line when the description is not set.

Signed-off-by: Jens Axboe <axboe@kernel.dk>
stat.c

diff --git a/stat.c b/stat.c
index 2bc21dad371770311c344d63d6c5483975aa11e0..bf87917c2956eb60729fa9ea95ac93b1cbe1ed50 100644 (file)
--- a/stat.c
+++ b/stat.c
@@ -1244,12 +1244,13 @@ static void show_thread_status_terse_all(struct thread_stat *ts,
        /* Additional output if continue_on_error set - default off*/
        if (ts->continue_on_error)
                log_buf(out, ";%llu;%d", (unsigned long long) ts->total_err_count, ts->first_error);
        /* Additional output if continue_on_error set - default off*/
        if (ts->continue_on_error)
                log_buf(out, ";%llu;%d", (unsigned long long) ts->total_err_count, ts->first_error);
-       if (ver == 2)
-               log_buf(out, "\n");
 
        /* Additional output if description is set */
 
        /* Additional output if description is set */
-       if (strlen(ts->description))
+       if (strlen(ts->description)) {
+               if (ver == 2)
+                       log_buf(out, "\n");
                log_buf(out, ";%s", ts->description);
                log_buf(out, ";%s", ts->description);
+       }
 
        log_buf(out, "\n");
 }
 
        log_buf(out, "\n");
 }