diff options
author | Christian Göttsche <cgzones@googlemail.com> | 2020-12-18 15:49:37 +0100 |
---|---|---|
committer | cgzones <cgzones@googlemail.com> | 2020-12-19 21:13:32 +0100 |
commit | c150e4bde9080799c7c9f115565c8324faa953be (patch) | |
tree | 57c82c679afd66967f8bfaf45c61a495ba245bd9 /BatteryMeter.c | |
parent | 9f68c8d34142746a16ea3883d736a0955977e625 (diff) |
Enable -Wformat=2
Now that the global format variable Process_pidFormat is gone, enable
the compiler warning -Wformat=2.
Diffstat (limited to 'BatteryMeter.c')
-rw-r--r-- | BatteryMeter.c | 29 |
1 files changed, 13 insertions, 16 deletions
diff --git a/BatteryMeter.c b/BatteryMeter.c index 4836809b..2c808c72 100644 --- a/BatteryMeter.c +++ b/BatteryMeter.c @@ -35,24 +35,21 @@ static void BatteryMeter_updateValues(Meter* this, char* buffer, size_t len) { this->values[0] = percent; - const char *onAcText, *onBatteryText, *unknownText; - - unknownText = "%.1f%%"; - if (this->mode == TEXT_METERMODE) { - onAcText = "%.1f%% (Running on A/C)"; - onBatteryText = "%.1f%% (Running on battery)"; - } else { - onAcText = "%.1f%%(A/C)"; - onBatteryText = "%.1f%%(bat)"; + const char* text; + switch (isOnAC) { + case AC_PRESENT: + text = this->mode == TEXT_METERMODE ? " (Running on A/C)" : "(A/C)"; + break; + case AC_ABSENT: + text = this->mode == TEXT_METERMODE ? " (Running on battery)" : "(bat)"; + break; + case AC_ERROR: + default: + text = ""; + break; } - if (isOnAC == AC_PRESENT) { - xSnprintf(buffer, len, onAcText, percent); - } else if (isOnAC == AC_ABSENT) { - xSnprintf(buffer, len, onBatteryText, percent); - } else { - xSnprintf(buffer, len, unknownText, percent); - } + xSnprintf(buffer, len, "%.1f%%%s", percent, text); } const MeterClass BatteryMeter_class = { |