summaryrefslogtreecommitdiff
path: root/src/utils.cpp
diff options
context:
space:
mode:
authorGravatar Franz König <Arlon1@users.noreply.github.com> 2017-08-29 01:03:24 +0200
committerGravatar Franz König <Arlon1@users.noreply.github.com> 2017-08-29 01:03:24 +0200
commit7455313cf730cba1470b9b4ab5fe3554ff19c628 (patch)
tree9dd95b9f93efb7e76773c74a3a0e727f0948acb6 /src/utils.cpp
parent930605cac684e351e58c797fe4d376d50a16f5de (diff)
downloadnewsboat-7455313cf730cba1470b9b4ab5fe3554ff19c628.tar.gz
newsboat-7455313cf730cba1470b9b4ab5fe3554ff19c628.tar.zst
newsboat-7455313cf730cba1470b9b4ab5fe3554ff19c628.zip
improved code quality according to Minoru's advice
Diffstat (limited to '')
-rw-r--r--src/utils.cpp7
1 files changed, 2 insertions, 5 deletions
diff --git a/src/utils.cpp b/src/utils.cpp
index 4f8bfed0..47aeefbd 100644
--- a/src/utils.cpp
+++ b/src/utils.cpp
@@ -1101,12 +1101,9 @@ int utils::run_interactively(
LOG(level::DEBUG, "%s: couldn't create a child process", caller);
} else if (status == 127) {
LOG(level::DEBUG, "%s: couldn't run shell", caller);
+ } else if (status != 0) {
+ LOG(level::DEBUG, "%s: cmd = %s, returned %d", caller, command, status);
}
- if (status != 0) {
- LOG(level::DEBUG, "utils::run_interactively: "
- "%s: cmd = %s, returned %d", caller, command, status);
- }
-
return status;
}