From e0cdcb6e30efbf56a862b0b4dfb7baa5677bf35d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pedro=20C=C3=B4rte-Real?= <pedro@pedrocr.net>
Date: Thu, 26 Nov 2020 02:16:57 +0000
Subject: [PATCH] Handle charging above 100% gracefully

When calibrating a battery it's possible to go above 100%. Handle that
gracefully by just presenting the battery as full and 100%.
---
 src/modules/battery.cpp | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/src/modules/battery.cpp b/src/modules/battery.cpp
index 93fcc8bd..02e05b79 100644
--- a/src/modules/battery.cpp
+++ b/src/modules/battery.cpp
@@ -115,6 +115,11 @@ const std::tuple<uint8_t, float, std::string> waybar::modules::Battery::getInfos
       time_remaining = (float)total_energy / total_power;
     } else if (status == "Charging" && total_power != 0) {
       time_remaining = -(float)(total_energy_full - total_energy) / total_power;
+      if (time_remaining > 0.0f) {
+        // If we've turned positive it means the battery is past 100% and so
+        // just report that as no time remaining
+        time_remaining = 0.0f;
+      }
     }
     float capacity = ((float)total_energy * 100.0f / (float) total_energy_full);
     // Handle full-at
@@ -127,6 +132,12 @@ const std::tuple<uint8_t, float, std::string> waybar::modules::Battery::getInfos
         }
       }
     }
+    if (capacity > 100.f) {
+      // This can happen when the battery is calibrating and goes above 100%
+      // Handle it gracefully by clamping at 100% and presenting it as full
+      capacity = 100.f;
+      status = "Full";
+    }
     return {capacity, time_remaining, status};
   } catch (const std::exception& e) {
     spdlog::error("Battery: {}", e.what());