treewide: clang-format

include/AModule.hpp

@@ -15,7 +15,7 @@ class AModule : public IModule {
 
   virtual ~AModule();
   auto update() -> void override;
-  virtual auto refresh(int) -> void {};
+  virtual auto refresh(int) -> void{};
   operator Gtk::Widget &() override;
   auto doAction(const std::string &name) -> void override;
 

include/modules/clock.hpp

@@ -51,8 +51,8 @@ class Clock final : public ALabel {
   day cldBaseDay_{0};            // calendar Cached day. Is used when today is changing(midnight)
   std::string cldText_{""};      // calendar text to print
   CldMode cldMode_{CldMode::MONTH};
-  auto get_calendar(const year_month_day& today, const year_month_day& ymd,
+  auto get_calendar(const year_month_day& today, const year_month_day& ymd, const time_zone* tz)
-                    const time_zone* tz) -> const std::string;
+      -> const std::string;
 
   // get local time zone
   auto local_zone() -> const time_zone*;

include/util/clara.hpp

@@ -622,8 +622,8 @@ inline auto convertInto(std::string const &source, bool &target) -> ParserResult
 }
 #ifdef CLARA_CONFIG_OPTIONAL_TYPE
 template <typename T>
-inline auto convertInto(std::string const &source,
+inline auto convertInto(std::string const &source, CLARA_CONFIG_OPTIONAL_TYPE<T> &target)
-                        CLARA_CONFIG_OPTIONAL_TYPE<T> &target) -> ParserResult {
+    -> ParserResult {
   T temp;
   auto result = convertInto(source, temp);
   if (result) target = std::move(temp);
@@ -751,8 +751,8 @@ class ParserBase {
  public:
   virtual ~ParserBase() = default;
   virtual auto validate() const -> Result { return Result::ok(); }
-  virtual auto parse(std::string const &exeName,
+  virtual auto parse(std::string const &exeName, TokenStream const &tokens) const
-                     TokenStream const &tokens) const -> InternalParseResult = 0;
+      -> InternalParseResult = 0;
   virtual auto cardinality() const -> size_t { return 1; }
 
   auto parse(Args const &args) const -> InternalParseResult {
@@ -1098,8 +1098,8 @@ struct Parser : ParserBase {
 
   using ParserBase::parse;
 
-  auto parse(std::string const &exeName,
+  auto parse(std::string const &exeName, TokenStream const &tokens) const
-             TokenStream const &tokens) const -> InternalParseResult override {
+      -> InternalParseResult override {
     struct ParserInfo {
       ParserBase const *parser = nullptr;
       size_t count = 0;

include/util/format.hpp

@@ -6,7 +6,7 @@
 class pow_format {
  public:
   pow_format(long long val, std::string&& unit, bool binary = false)
-      : val_(val), unit_(unit), binary_(binary) {};
+      : val_(val), unit_(unit), binary_(binary){};
 
   long long val_;
   std::string unit_;

src/modules/bluetooth.cpp

@@ -49,8 +49,8 @@ auto getBoolProperty(GDBusProxy* proxy, const char* property_name) -> bool {
   return false;
 }
 
-auto getOptionalStringProperty(GDBusProxy* proxy,
+auto getOptionalStringProperty(GDBusProxy* proxy, const char* property_name)
-                               const char* property_name) -> std::optional<std::string> {
+    -> std::optional<std::string> {
   auto gvar = g_dbus_proxy_get_cached_property(proxy, property_name);
   if (gvar) {
     std::string property_value = g_variant_get_string(gvar, NULL);
@@ -115,8 +115,7 @@ waybar::modules::Bluetooth::Bluetooth(const std::string& id, const Json::Value&
   g_signal_connect(manager_.get(), "object-removed", G_CALLBACK(onObjectRemoved), this);
   g_signal_connect(manager_.get(), "interface-proxy-properties-changed",
                    G_CALLBACK(onInterfaceProxyPropertiesChanged), this);
-  g_signal_connect(manager_.get(), "interface-added", G_CALLBACK(onInterfaceAddedOrRemoved),
+  g_signal_connect(manager_.get(), "interface-added", G_CALLBACK(onInterfaceAddedOrRemoved), this);
-                   this);
   g_signal_connect(manager_.get(), "interface-removed", G_CALLBACK(onInterfaceAddedOrRemoved),
                    this);
 
@@ -283,14 +282,12 @@ auto waybar::modules::Bluetooth::update() -> void {
   ALabel::update();
 }
 
-auto waybar::modules::Bluetooth::onObjectAdded(GDBusObjectManager* manager,
+auto waybar::modules::Bluetooth::onObjectAdded(GDBusObjectManager* manager, GDBusObject* object,
-                                               GDBusObject* object,
                                                gpointer user_data) -> void {
   ControllerInfo info;
   Bluetooth* bt = static_cast<Bluetooth*>(user_data);
 
-  if (!bt->cur_controller_.has_value() &&
+  if (!bt->cur_controller_.has_value() && bt->getControllerProperties(object, info) &&
-      bt->getControllerProperties(object, info) &&
       (!bt->config_["controller-alias"].isString() ||
        bt->config_["controller-alias"].asString() == info.alias)) {
     bt->cur_controller_ = std::move(info);
@@ -298,8 +295,7 @@ auto waybar::modules::Bluetooth::onObjectAdded(GDBusObjectManager* manager,
   }
 }
 
-auto waybar::modules::Bluetooth::onObjectRemoved(GDBusObjectManager* manager,
+auto waybar::modules::Bluetooth::onObjectRemoved(GDBusObjectManager* manager, GDBusObject* object,
-                                                 GDBusObject* object,
                                                  gpointer user_data) -> void {
   Bluetooth* bt = static_cast<Bluetooth*>(user_data);
   GDBusProxy* proxy_controller;
@@ -311,7 +307,6 @@ auto waybar::modules::Bluetooth::onObjectRemoved(GDBusObjectManager* manager,
   proxy_controller = G_DBUS_PROXY(g_dbus_object_get_interface(object, "org.bluez.Adapter1"));
 
   if (proxy_controller != NULL) {
-
     std::string object_path = g_dbus_object_get_object_path(object);
 
     if (object_path == bt->cur_controller_->path) {
@@ -350,8 +345,8 @@ auto waybar::modules::Bluetooth::onInterfaceAddedOrRemoved(GDBusObjectManager* m
 
 auto waybar::modules::Bluetooth::onInterfaceProxyPropertiesChanged(
     GDBusObjectManagerClient* manager, GDBusObjectProxy* object_proxy, GDBusProxy* interface_proxy,
-    GVariant* changed_properties, const gchar* const* invalidated_properties,
+    GVariant* changed_properties, const gchar* const* invalidated_properties, gpointer user_data)
-    gpointer user_data) -> void {
+    -> void {
   std::string interface_name = g_dbus_proxy_get_interface_name(interface_proxy);
   std::string object_path = g_dbus_object_get_object_path(G_DBUS_OBJECT(object_proxy));
 
@@ -400,8 +395,8 @@ auto waybar::modules::Bluetooth::getDeviceBatteryPercentage(GDBusObject* object)
   return std::nullopt;
 }
 
-auto waybar::modules::Bluetooth::getDeviceProperties(GDBusObject* object,
+auto waybar::modules::Bluetooth::getDeviceProperties(GDBusObject* object, DeviceInfo& device_info)
-                                                     DeviceInfo& device_info) -> bool {
+    -> bool {
   GDBusProxy* proxy_device = G_DBUS_PROXY(g_dbus_object_get_interface(object, "org.bluez.Device1"));
 
   if (proxy_device != NULL) {
@@ -467,8 +462,9 @@ auto waybar::modules::Bluetooth::findCurController() -> std::optional<Controller
   return controller_info;
 }
 
-auto waybar::modules::Bluetooth::findConnectedDevices(
+auto waybar::modules::Bluetooth::findConnectedDevices(const std::string& cur_controller_path,
-    const std::string& cur_controller_path, std::vector<DeviceInfo>& connected_devices) -> void {
+                                                      std::vector<DeviceInfo>& connected_devices)
+    -> void {
   GList* objects = g_dbus_object_manager_get_objects(manager_.get());
   for (GList* l = objects; l != NULL; l = l->next) {
     GDBusObject* object = G_DBUS_OBJECT(l->data);

src/modules/clock.cpp

@@ -197,8 +197,8 @@ const unsigned cldRowsInMonth(const year_month& ym, const weekday& firstdow) {
   return 2u + ceil<weeks>((weekday{ym / 1} - firstdow) + ((ym / last).day() - day{0})).count();
 }
 
-auto cldGetWeekForLine(const year_month& ym, const weekday& firstdow,
+auto cldGetWeekForLine(const year_month& ym, const weekday& firstdow, const unsigned line)
-                       const unsigned line) -> const year_month_weekday {
+    -> const year_month_weekday {
   unsigned index{line - 2};
   if (weekday{ym / 1} == firstdow) ++index;
   return ym / firstdow[index];

src/modules/cpu_usage/linux.cpp

@@ -16,7 +16,8 @@ std::vector<std::tuple<size_t, size_t>> waybar::modules::CpuUsage::parseCpuinfo(
     }
     std::stringstream sline(line.substr(5));
     std::vector<size_t> times;
-    for (size_t time = 0; sline >> time; times.push_back(time));
+    for (size_t time = 0; sline >> time; times.push_back(time))
+      ;
 
     size_t idle_time = 0;
     size_t total_time = 0;

src/modules/dwl/tags.cpp

@@ -53,7 +53,7 @@ static void set_layout(void *data, zdwl_ipc_output_v2 *zdwl_output_v2, uint32_t
   // Intentionally empty
 }
 
-static void appid(void *data, zdwl_ipc_output_v2 *zdwl_output_v2, const char *appid) {
+static void appid(void *data, zdwl_ipc_output_v2 *zdwl_output_v2, const char *appid){
     // Intentionally empty
 };
 

src/modules/wlr/workspace_manager.cpp

@@ -118,8 +118,8 @@ auto WorkspaceManager::sort_workspaces() -> void {
   }
 }
 
-auto WorkspaceManager::register_manager(wl_registry *registry, uint32_t name,
+auto WorkspaceManager::register_manager(wl_registry *registry, uint32_t name, uint32_t version)
-                                        uint32_t version) -> void {
+    -> void {
   if (workspace_manager_) {
     spdlog::warn("Register workspace manager again although already registered!");
     return;

test/utils/SafeSignal.cpp

@@ -71,7 +71,7 @@ struct TestObject {
   unsigned copied = 0;
   unsigned moved = 0;
 
-  TestObject(const T& v) : value(v) {};
+  TestObject(const T& v) : value(v){};
   ~TestObject() = default;
 
   TestObject(const TestObject& other)