From 67c730293885684a954632a70fe041abd2288685 Mon Sep 17 00:00:00 2001
From: Alex <alexisr245@gmail.com>
Date: Sat, 18 Sep 2021 13:50:16 +0200
Subject: [PATCH] Revert "Add CPU usage for every core"

---
 include/modules/cpu.hpp    | 11 +++++------
 man/waybar-cpu.5.scd       | 22 ++--------------------
 src/modules/cpu/common.cpp | 32 +++++++++++---------------------
 3 files changed, 18 insertions(+), 47 deletions(-)

diff --git a/include/modules/cpu.hpp b/include/modules/cpu.hpp
index 1fe1199b..866b5af7 100644
--- a/include/modules/cpu.hpp
+++ b/include/modules/cpu.hpp
@@ -1,7 +1,6 @@
 #pragma once
 
 #include <fmt/format.h>
-#include <fmt/args.h>
 #include <cstdint>
 #include <fstream>
 #include <numeric>
@@ -20,11 +19,11 @@ class Cpu : public ALabel {
   auto update() -> void;
 
  private:
-  double                                         getCpuLoad();
-  std::tuple<std::vector<uint16_t>, std::string> getCpuUsage();
-  std::tuple<float, float, float>                getCpuFrequency();
-  std::vector<std::tuple<size_t, size_t>>        parseCpuinfo();
-  std::vector<float>                             parseCpuFrequencies();
+  double                                  getCpuLoad();
+  std::tuple<uint16_t, std::string>       getCpuUsage();
+  std::tuple<float, float, float>         getCpuFrequency();
+  std::vector<std::tuple<size_t, size_t>> parseCpuinfo();
+  std::vector<float>                      parseCpuFrequencies();
 
   std::vector<std::tuple<size_t, size_t>> prev_times_;
 
diff --git a/man/waybar-cpu.5.scd b/man/waybar-cpu.5.scd
index 2e0d6c71..679ca2ce 100644
--- a/man/waybar-cpu.5.scd
+++ b/man/waybar-cpu.5.scd
@@ -82,9 +82,7 @@ The *cpu* module displays the current cpu utilization.
 
 *{load}*: Current cpu load.
 
-*{usage}*: Current overall cpu usage.
-
-*{usage*{n}*}*: Current cpu core n usage. Cores are numbered from zero, so first core will be {usage0} and 4th will be {usage3}.
+*{usage}*: Current cpu usage.
 
 *{avg_frequency}*: Current cpu average frequency (based on all cores) in GHz.
 
@@ -92,13 +90,7 @@ The *cpu* module displays the current cpu utilization.
 
 *{min_frequency}*: Current cpu min frequency (based on the core with the lowest frequency) in GHz.
 
-*{icon}*: Icon for overall cpu usage.
-
-*{icon*{n}*}*: Icon for cpu core n usage. Use like {icon0}.
-
-# EXAMPLES
-
-Basic configuration:
+# EXAMPLE
 
 ```
 "cpu": {
@@ -108,16 +100,6 @@ Basic configuration:
 }
 ```
 
-Cpu usage per core rendered as icons:
-
-```
-"cpu": {
-	"interval": 1,
-	"format": "{icon0}{icon1}{icon2}{icon3} {usage:>2}% ",
-	"format-icons": ["▁", "▂", "▃", "▄", "▅", "▆", "▇", "█"],
-},
-```
-
 # STYLE
 
 - *#cpu*
diff --git a/src/modules/cpu/common.cpp b/src/modules/cpu/common.cpp
index 5869e13f..767cde92 100644
--- a/src/modules/cpu/common.cpp
+++ b/src/modules/cpu/common.cpp
@@ -17,8 +17,7 @@ auto waybar::modules::Cpu::update() -> void {
     label_.set_tooltip_text(tooltip);
   }
   auto format = format_;
-  auto total_usage = cpu_usage.empty() ? 0 : cpu_usage[0];
-  auto state = getState(total_usage);
+  auto state = getState(cpu_usage);
   if (!state.empty() && config_["format-" + state].isString()) {
     format = config_["format-" + state].asString();
   }
@@ -28,22 +27,13 @@ auto waybar::modules::Cpu::update() -> void {
   } else {
     event_box_.show();
     auto icons = std::vector<std::string>{state};
-    fmt::dynamic_format_arg_store<fmt::format_context> store;
-    store.push_back(fmt::arg("load", cpu_load));
-    store.push_back(fmt::arg("load", cpu_load));
-    store.push_back(fmt::arg("usage", total_usage));
-    store.push_back(fmt::arg("icon", getIcon(total_usage, icons)));
-    store.push_back(fmt::arg("max_frequency", max_frequency));
-    store.push_back(fmt::arg("min_frequency", min_frequency));
-    store.push_back(fmt::arg("avg_frequency", avg_frequency));
-    for (size_t i = 1; i < cpu_usage.size(); ++i) {
-	    auto core_i = i - 1;
-	    auto core_format = fmt::format("usage{}", core_i);
-	    store.push_back(fmt::arg(core_format.c_str(), cpu_usage[i]));
-	    auto icon_format = fmt::format("icon{}", core_i);
-	    store.push_back(fmt::arg(icon_format.c_str(), getIcon(cpu_usage[i], icons)));
-    }
-    label_.set_markup(fmt::vformat(format, store));
+    label_.set_markup(fmt::format(format,
+                                  fmt::arg("load", cpu_load),
+                                  fmt::arg("usage", cpu_usage),
+                                  fmt::arg("icon", getIcon(cpu_usage, icons)),
+                                  fmt::arg("max_frequency", max_frequency),
+                                  fmt::arg("min_frequency", min_frequency),
+                                  fmt::arg("avg_frequency", avg_frequency)));
   }
 
   // Call parent update
@@ -58,14 +48,14 @@ double waybar::modules::Cpu::getCpuLoad() {
   throw std::runtime_error("Can't get Cpu load");
 }
 
-std::tuple<std::vector<uint16_t>, std::string> waybar::modules::Cpu::getCpuUsage() {
+std::tuple<uint16_t, std::string> waybar::modules::Cpu::getCpuUsage() {
   if (prev_times_.empty()) {
     prev_times_ = parseCpuinfo();
     std::this_thread::sleep_for(std::chrono::milliseconds(100));
   }
   std::vector<std::tuple<size_t, size_t>> curr_times = parseCpuinfo();
   std::string                             tooltip;
-  std::vector<uint16_t>                   usage;
+  uint16_t                                usage = 0;
   for (size_t i = 0; i < curr_times.size(); ++i) {
     auto [curr_idle, curr_total] = curr_times[i];
     auto [prev_idle, prev_total] = prev_times_[i];
@@ -73,11 +63,11 @@ std::tuple<std::vector<uint16_t>, std::string> waybar::modules::Cpu::getCpuUsage
     const float delta_total = curr_total - prev_total;
     uint16_t    tmp = 100 * (1 - delta_idle / delta_total);
     if (i == 0) {
+      usage = tmp;
       tooltip = fmt::format("Total: {}%", tmp);
     } else {
       tooltip = tooltip + fmt::format("\nCore{}: {}%", i - 1, tmp);
     }
-    usage.push_back(tmp);
   }
   prev_times_ = curr_times;
   return {usage, tooltip};