summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChristian Göttsche <cgzones@googlemail.com>2021-09-02 22:32:40 +0200
committerBenBE <BenBE@geshi.org>2021-09-04 09:48:53 +0200
commit7891cdc55265a3435bd0d9b4ed7fc5b9b9c937ee (patch)
tree20c4ff72b3b8ee5a1392daf60d07506ef9fe1599
parentb9fdf1c2a1a00867d68ff66ace5c4ceb978561e3 (diff)
Reduce variable scope
Also avoid declaring variables of different type, pointer and array, in the same line.
-rw-r--r--Header.c3
-rw-r--r--Settings.c3
2 files changed, 4 insertions, 2 deletions
diff --git a/Header.c b/Header.c
index ba7c3aaf..a88a472c 100644
--- a/Header.c
+++ b/Header.c
@@ -86,8 +86,9 @@ static void Header_addMeterByName(Header* this, const char* name, MeterModeId mo
if (paren) {
int ok = sscanf(paren, "(%10u)", &param); // CPUMeter
if (!ok) {
- char* end, dynamic[32] = {0};
+ char dynamic[32] = {0};
if (sscanf(paren, "(%30s)", dynamic)) { // DynamicMeter
+ char* end;
if ((end = strrchr(dynamic, ')')) == NULL)
return; // htoprc parse failure
*end = '\0';
diff --git a/Settings.c b/Settings.c
index 7ef314d0..44e91a47 100644
--- a/Settings.c
+++ b/Settings.c
@@ -134,8 +134,9 @@ static void Settings_readFields(Settings* settings, const char* line) {
}
// Dynamically-defined columns are always stored by-name.
- char* end, dynamic[32] = {0};
+ char dynamic[32] = {0};
if (sscanf(ids[i], "Dynamic(%30s)", dynamic)) {
+ char* end;
if ((end = strrchr(dynamic, ')')) == NULL)
continue;
*end = '\0';

© 2014-2024 Faster IT GmbH | imprint | privacy policy