Zbigniew Jędrzejewski-Szmek 62fe94
From c6828d2794fcd00b6b0d2a040301c90ad424d14f Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 62fe94
From: Tom Gundersen <teg@jklm.no>
Zbigniew Jędrzejewski-Szmek 62fe94
Date: Tue, 30 Sep 2014 11:37:28 +0200
Zbigniew Jędrzejewski-Szmek 62fe94
Subject: [PATCH] shared: util - use nicer idiom to silence Coverity
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
Suggested by Zbigniew.
Zbigniew Jędrzejewski-Szmek 62fe94
---
Zbigniew Jędrzejewski-Szmek 62fe94
 src/shared/util.c | 9 ++-------
Zbigniew Jędrzejewski-Szmek 62fe94
 1 file changed, 2 insertions(+), 7 deletions(-)
Zbigniew Jędrzejewski-Szmek 62fe94
Zbigniew Jędrzejewski-Szmek 62fe94
diff --git a/src/shared/util.c b/src/shared/util.c
Zbigniew Jędrzejewski-Szmek 62fe94
index ec33fc1263..ebacee55b1 100644
Zbigniew Jędrzejewski-Szmek 62fe94
--- a/src/shared/util.c
Zbigniew Jędrzejewski-Szmek 62fe94
+++ b/src/shared/util.c
Zbigniew Jędrzejewski-Szmek 62fe94
@@ -3272,13 +3272,8 @@ unsigned columns(void) {
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         c = 0;
Zbigniew Jędrzejewski-Szmek 62fe94
         e = getenv("COLUMNS");
Zbigniew Jędrzejewski-Szmek 62fe94
-        if (e) {
Zbigniew Jędrzejewski-Szmek 62fe94
-                int r;
Zbigniew Jędrzejewski-Szmek 62fe94
-
Zbigniew Jędrzejewski-Szmek 62fe94
-                r = safe_atoi(e, &c);
Zbigniew Jędrzejewski-Szmek 62fe94
-                if (r < 0) {}
Zbigniew Jędrzejewski-Szmek 62fe94
-                        /* do nothing, we fall back to c = 0 */
Zbigniew Jędrzejewski-Szmek 62fe94
-        }
Zbigniew Jędrzejewski-Szmek 62fe94
+        if (e)
Zbigniew Jędrzejewski-Szmek 62fe94
+                (void) safe_atoi(e, &c);
Zbigniew Jędrzejewski-Szmek 62fe94
 
Zbigniew Jędrzejewski-Szmek 62fe94
         if (c <= 0)
Zbigniew Jędrzejewski-Szmek 62fe94
                 c = fd_columns(STDOUT_FILENO);