[rebase] split off gps and cava modules into run-time loaded shared libraries

include/AAppIconLabel.hpp

@@ -10,8 +10,9 @@ namespace waybar {
 class AAppIconLabel : public AIconLabel {
  public:
   AAppIconLabel(const Json::Value& config, const std::string& name, const std::string& id,
-                const std::string& format, uint16_t interval = 0, bool ellipsize = false,
-                bool enable_click = false, bool enable_scroll = false);
+                const std::string& format, std::mutex& reap_mtx, std::list<pid_t>& reap,
+                uint16_t interval = 0, bool ellipsize = false, bool enable_click = false,
+                bool enable_scroll = false);
   virtual ~AAppIconLabel() = default;
   auto update() -> void override;
 

include/AIconLabel.hpp

@@ -10,8 +10,9 @@ namespace waybar {
 class AIconLabel : public ALabel {
  public:
   AIconLabel(const Json::Value& config, const std::string& name, const std::string& id,
-             const std::string& format, uint16_t interval = 0, bool ellipsize = false,
-             bool enable_click = false, bool enable_scroll = false);
+             const std::string& format, std::mutex& reap_mtx, std::list<pid_t>& reap,
+             uint16_t interval = 0, bool ellipsize = false, bool enable_click = false,
+             bool enable_scroll = false);
   virtual ~AIconLabel() = default;
   auto update() -> void override;
 

include/ALabel.hpp

@@ -11,8 +11,8 @@ namespace waybar {
 class ALabel : public AModule {
  public:
   ALabel(const Json::Value&, const std::string&, const std::string&, const std::string& format,
-         uint16_t interval = 0, bool ellipsize = false, bool enable_click = false,
-         bool enable_scroll = false);
+         std::mutex& reap_mtx, std::list<pid_t>& reap, uint16_t interval = 0,
+         bool ellipsize = false, bool enable_click = false, bool enable_scroll = false);
   virtual ~ALabel() = default;
   auto update() -> void override;
   virtual std::string getIcon(uint16_t, const std::string& alt = "", uint16_t max = 0);

include/AModule.hpp

@@ -25,11 +25,14 @@ class AModule : public IModule {
 
   bool expandEnabled() const;
 
+  std::mutex& reap_mtx;
+  std::list<pid_t>& reap;
+
  protected:
   // Don't need to make an object directly
   // Derived classes are able to use it
-  AModule(const Json::Value&, const std::string&, const std::string&, bool enable_click = false,
-          bool enable_scroll = false);
+  AModule(const Json::Value&, const std::string&, const std::string&, std::mutex& reap_mtx,
+          std::list<pid_t>& reap, bool enable_click = false, bool enable_scroll = false);
 
   enum SCROLL_DIR { NONE, UP, DOWN, LEFT, RIGHT };
 

include/ASlider.hpp

@@ -7,7 +7,8 @@ namespace waybar {
 
 class ASlider : public AModule {
  public:
-  ASlider(const Json::Value& config, const std::string& name, const std::string& id);
+  ASlider(const Json::Value& config, const std::string& name, const std::string& id,
+          std::mutex& reap_mtx, std::list<pid_t>& reap);
   virtual void onValueChanged();
 
  protected:
@@ -16,4 +17,4 @@ class ASlider : public AModule {
   Gtk::Scale scale_;
 };
 
-}  // namespace waybar
+}  // namespace waybar

include/IModule.hpp

@@ -1,6 +1,10 @@
 #pragma once
 
 #include <gtkmm/widget.h>
+#include <sys/types.h>
+
+#include <list>
+#include <mutex>
 
 namespace waybar {
 

include/factory.hpp

@@ -1,5 +1,6 @@
 #pragma once
 
+#include <dlfcn.h>
 #include <json/json.h>
 
 #include <AModule.hpp>

include/modules/backlight.hpp

@@ -17,7 +17,7 @@ namespace waybar::modules {
 
 class Backlight : public ALabel {
  public:
-  Backlight(const std::string&, const Json::Value&);
+  Backlight(const std::string&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   virtual ~Backlight() = default;
   auto update() -> void override;
 

include/modules/backlight_slider.hpp

@@ -9,7 +9,7 @@ namespace waybar::modules {
 
 class BacklightSlider : public ASlider {
  public:
-  BacklightSlider(const std::string&, const Json::Value&);
+  BacklightSlider(const std::string&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   virtual ~BacklightSlider() = default;
 
   void update() override;
@@ -21,4 +21,4 @@ class BacklightSlider : public ASlider {
   util::BacklightBackend backend;
 };
 
-}  // namespace waybar::modules
+}  // namespace waybar::modules

include/modules/battery.hpp

@@ -24,7 +24,8 @@ namespace fs = std::filesystem;
 
 class Battery : public ALabel {
  public:
-  Battery(const std::string&, const waybar::Bar&, const Json::Value&);
+  Battery(const std::string&, const waybar::Bar&, const Json::Value&, std::mutex&,
+          std::list<pid_t>&);
   virtual ~Battery();
   auto update() -> void override;
 

include/modules/bluetooth.hpp

@@ -44,7 +44,7 @@ class Bluetooth : public ALabel {
   };
 
  public:
-  Bluetooth(const std::string&, const Json::Value&);
+  Bluetooth(const std::string&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   virtual ~Bluetooth() = default;
   auto update() -> void override;
 

include/modules/cava/cavaGLSL.hpp

@@ -9,7 +9,7 @@ namespace waybar::modules::cava {
 
 class CavaGLSL final : public AModule, public Gtk::GLArea {
  public:
-  CavaGLSL(const std::string&, const Json::Value&);
+  CavaGLSL(const std::string&, const Json::Value&, std::mutex& reap_mtx, std::list<pid_t>& reap);
   ~CavaGLSL() = default;
 
  private:

include/modules/cava/cavaRaw.hpp

@@ -7,7 +7,7 @@ namespace waybar::modules::cava {
 
 class Cava final : public ALabel, public sigc::trackable {
  public:
-  Cava(const std::string&, const Json::Value&);
+  Cava(const std::string&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   ~Cava() = default;
   auto doAction(const std::string& name) -> void override;
 

include/modules/cffi.hpp

@@ -29,7 +29,7 @@ struct wbcffi_config_entry {
 
 class CFFI : public AModule {
  public:
-  CFFI(const std::string&, const std::string&, const Json::Value&);
+  CFFI(const std::string&, const std::string&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   virtual ~CFFI();
 
   virtual auto refresh(int signal) -> void override;

include/modules/clock.hpp

@@ -15,7 +15,7 @@ enum class WS { LEFT, RIGHT, HIDDEN };
 
 class Clock final : public ALabel {
  public:
-  Clock(const std::string&, const Json::Value&);
+  Clock(const std::string&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   virtual ~Clock() = default;
   auto update() -> void override;
   auto doAction(const std::string&) -> void override;

include/modules/cpu.hpp

@@ -16,7 +16,7 @@ namespace waybar::modules {
 
 class Cpu : public ALabel {
  public:
-  Cpu(const std::string&, const Json::Value&);
+  Cpu(const std::string&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   virtual ~Cpu() = default;
   auto update() -> void override;
 

include/modules/cpu_frequency.hpp

@@ -16,7 +16,7 @@ namespace waybar::modules {
 
 class CpuFrequency : public ALabel {
  public:
-  CpuFrequency(const std::string&, const Json::Value&);
+  CpuFrequency(const std::string&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   virtual ~CpuFrequency() = default;
   auto update() -> void override;
 

include/modules/cpu_usage.hpp

@@ -16,7 +16,7 @@ namespace waybar::modules {
 
 class CpuUsage : public ALabel {
  public:
-  CpuUsage(const std::string&, const Json::Value&);
+  CpuUsage(const std::string&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   virtual ~CpuUsage() = default;
   auto update() -> void override;
 

include/modules/custom.hpp

@@ -14,7 +14,8 @@ namespace waybar::modules {
 
 class Custom : public ALabel {
  public:
-  Custom(const std::string&, const std::string&, const Json::Value&, const std::string&);
+  Custom(const std::string&, const std::string&, const Json::Value&, const std::string&,
+         std::mutex&, std::list<pid_t>&);
   virtual ~Custom();
   auto update() -> void override;
   void refresh(int /*signal*/) override;

include/modules/disk.hpp

@@ -13,7 +13,7 @@ namespace waybar::modules {
 
 class Disk : public ALabel {
  public:
-  Disk(const std::string&, const Json::Value&);
+  Disk(const std::string&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   virtual ~Disk() = default;
   auto update() -> void override;
 

include/modules/dwl/tags.hpp

@@ -12,7 +12,7 @@ namespace waybar::modules::dwl {
 
 class Tags : public waybar::AModule {
  public:
-  Tags(const std::string&, const waybar::Bar&, const Json::Value&);
+  Tags(const std::string&, const waybar::Bar&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   virtual ~Tags();
 
   // Handlers for wayland events

include/modules/dwl/window.hpp

@@ -13,7 +13,8 @@ namespace waybar::modules::dwl {
 
 class Window : public AAppIconLabel, public sigc::trackable {
  public:
-  Window(const std::string&, const waybar::Bar&, const Json::Value&);
+  Window(const std::string&, const waybar::Bar&, const Json::Value&, std::mutex&,
+         std::list<pid_t>&);
   ~Window();
 
   void handle_layout(const uint32_t layout);

include/modules/ext/workspace_manager.hpp

@@ -20,7 +20,8 @@ class Workspace;
 
 class WorkspaceManager final : public AModule {
  public:
-  WorkspaceManager(const std::string& id, const waybar::Bar& bar, const Json::Value& config);
+  WorkspaceManager(const std::string& id, const waybar::Bar& bar, const Json::Value& config,
+                   std::mutex&, std::list<pid_t>&);
   ~WorkspaceManager() override;
   void register_manager(wl_registry* registry, uint32_t name, uint32_t version);
   void remove_workspace_group(uint32_t id);

include/modules/gamemode.hpp

@@ -17,7 +17,7 @@ namespace waybar::modules {
 
 class Gamemode : public AModule {
  public:
-  Gamemode(const std::string&, const Json::Value&);
+  Gamemode(const std::string&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   virtual ~Gamemode();
   auto update() -> void override;
 

include/modules/gps.hpp

@@ -16,7 +16,7 @@ namespace waybar::modules {
 
 class Gps : public ALabel {
  public:
-  Gps(const std::string&, const Json::Value&);
+  Gps(const std::string&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   virtual ~Gps();
   auto update() -> void override;
 

include/modules/hyprland/language.hpp

@@ -13,7 +13,8 @@ namespace waybar::modules::hyprland {
 
 class Language : public waybar::ALabel, public EventHandler {
  public:
-  Language(const std::string&, const waybar::Bar&, const Json::Value&);
+  Language(const std::string&, const waybar::Bar&, const Json::Value&, std::mutex&,
+           std::list<pid_t>&);
   virtual ~Language();
 
   auto update() -> void override;

include/modules/hyprland/submap.hpp

@@ -13,7 +13,8 @@ namespace waybar::modules::hyprland {
 
 class Submap : public waybar::ALabel, public EventHandler {
  public:
-  Submap(const std::string&, const waybar::Bar&, const Json::Value&);
+  Submap(const std::string&, const waybar::Bar&, const Json::Value&, std::mutex&,
+         std::list<pid_t>&);
   ~Submap() override;
 
   auto update() -> void override;

include/modules/hyprland/window.hpp

@@ -13,7 +13,8 @@ namespace waybar::modules::hyprland {
 
 class Window : public waybar::AAppIconLabel, public EventHandler {
  public:
-  Window(const std::string&, const waybar::Bar&, const Json::Value&);
+  Window(const std::string&, const waybar::Bar&, const Json::Value&, std::mutex&,
+         std::list<pid_t>&);
   ~Window() override;
 
   auto update() -> void override;

include/modules/hyprland/windowcount.hpp

@@ -12,7 +12,8 @@ namespace waybar::modules::hyprland {
 
 class WindowCount : public waybar::AAppIconLabel, public EventHandler {
  public:
-  WindowCount(const std::string&, const waybar::Bar&, const Json::Value&);
+  WindowCount(const std::string&, const waybar::Bar&, const Json::Value&, std::mutex&,
+              std::list<pid_t>&);
   ~WindowCount() override;
 
   auto update() -> void override;

include/modules/hyprland/workspaces.hpp

@@ -30,7 +30,8 @@ class Workspaces;
 
 class Workspaces : public AModule, public EventHandler {
  public:
-  Workspaces(const std::string&, const waybar::Bar&, const Json::Value&);
+  Workspaces(const std::string&, const waybar::Bar&, const Json::Value&, std::mutex&,
+             std::list<pid_t>&);
   ~Workspaces() override;
   void update() override;
   void init();

include/modules/idle_inhibitor.hpp

@@ -12,7 +12,8 @@ class IdleInhibitor : public ALabel {
   sigc::connection timeout_;
 
  public:
-  IdleInhibitor(const std::string&, const waybar::Bar&, const Json::Value&);
+  IdleInhibitor(const std::string&, const waybar::Bar&, const Json::Value&, std::mutex&,
+                std::list<pid_t>&);
   virtual ~IdleInhibitor();
   auto update() -> void override;
   static std::list<waybar::AModule*> modules;

include/modules/image.hpp

@@ -16,7 +16,7 @@ namespace waybar::modules {
 
 class Image : public AModule {
  public:
-  Image(const std::string&, const Json::Value&);
+  Image(const std::string&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   virtual ~Image() = default;
   auto update() -> void override;
   void refresh(int /*signal*/) override;

include/modules/inhibitor.hpp

@@ -11,7 +11,8 @@ namespace waybar::modules {
 
 class Inhibitor : public ALabel {
  public:
-  Inhibitor(const std::string&, const waybar::Bar&, const Json::Value&);
+  Inhibitor(const std::string&, const waybar::Bar&, const Json::Value&, std::mutex&,
+            std::list<pid_t>&);
   virtual ~Inhibitor();
   auto update() -> void override;
   auto activated() -> bool;

include/modules/jack.hpp

@@ -13,7 +13,7 @@ namespace waybar::modules {
 
 class JACK : public ALabel {
  public:
-  JACK(const std::string&, const Json::Value&);
+  JACK(const std::string&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   virtual ~JACK() = default;
   auto update() -> void override;
 

include/modules/keyboard_state.hpp

@@ -19,7 +19,8 @@ namespace waybar::modules {
 
 class KeyboardState : public AModule {
  public:
-  KeyboardState(const std::string&, const waybar::Bar&, const Json::Value&);
+  KeyboardState(const std::string&, const waybar::Bar&, const Json::Value&, std::mutex&,
+                std::list<pid_t>&);
   virtual ~KeyboardState();
   auto update() -> void override;
 

include/modules/load.hpp

@@ -16,7 +16,7 @@ namespace waybar::modules {
 
 class Load : public ALabel {
  public:
-  Load(const std::string&, const Json::Value&);
+  Load(const std::string&, const Json::Value&, std::mutex&, std::list<pid_t>&);
   virtual ~Load() = default;
   auto update() -> void override;