refactor(clang-tidy): Apply fixes

include/components/builder.hpp

@@ -32,7 +32,7 @@ class builder {
   void node(string str, bool add_space = false);
   void node(string str, int font_index, bool add_space = false);
   void node(const label_t& label, bool add_space = false);
-  void node_repeat(string str, size_t n, bool add_space = false);
+  void node_repeat(const string& str, size_t n, bool add_space = false);
   void node_repeat(const label_t& label, size_t n, bool add_space = false);
   void offset(int pixels = 0);
   void space(int width = DEFAULT_SPACING);

include/utils/string.hpp

@@ -16,9 +16,9 @@ namespace string_util {
   string upper(const string& s);
   string lower(const string& s);
   bool compare(const string& s1, const string& s2);
-  string replace(const string& haystack, const string& needle, const string& replacement, size_t start = 0,
+  string replace(const string& haystack, const string& needle, const string& reply, size_t start = 0,
       size_t end = string::npos);
-  string replace_all(const string& haystack, const string& needle, const string& replacement, size_t start = 0,
+  string replace_all(const string& haystack, const string& needle, const string& reply, size_t start = 0,
       size_t end = string::npos);
   string squeeze(const string& haystack, char needle);
   string strip(const string& haystack, char needle);

include/x11/tray.hpp

@@ -130,7 +130,7 @@ class tray_manager : public xpp::event::sink<evt::expose, evt::visibility_notify
   void track_selection_owner(xcb_window_t owner);
   void process_docking_request(xcb_window_t win);
 
-  void bar_visibility_change(bool state);
+  void bar_visibility_change(bool visible);
 
   int16_t calculate_x(uint16_t width) const;
   int16_t calculate_y() const;

include/x11/xkb.hpp

@@ -61,7 +61,7 @@ class keyboard {
       : layouts(forward<decltype(layouts)>(layouts_)), indicators(forward<decltype(indicators)>(indicators_)) {}
 
   const indicator& get(const indicator::type& i) const;
-  void set(uint32_t indicator_state);
+  void set(uint32_t state);
   bool on(const indicator::type&) const;
 
   const string group_name(size_t index = 0) const;

src/components/builder.cpp

@@ -250,7 +250,7 @@ void builder::node(const label_t& label, bool add_space) {
 /**
  * Repeat text string n times
  */
-void builder::node_repeat(string str, size_t n, bool add_space) {
+void builder::node_repeat(const string& str, size_t n, bool add_space) {
   string text;
   while (n--) {
     text += str;

src/components/eventloop.cpp

@@ -1,8 +1,8 @@
 #include <csignal>
 
 #include "components/eventloop.hpp"
-#include "components/types.hpp"
 #include "components/signals.hpp"
+#include "components/types.hpp"
 #include "utils/string.hpp"
 #include "utils/time.hpp"
 #include "x11/color.hpp"

src/components/screen.cpp

@@ -2,11 +2,11 @@
 #include <thread>
 
 #include "components/config.hpp"
+#include "components/eventloop.hpp"
 #include "components/logger.hpp"
 #include "components/screen.hpp"
-#include "components/types.hpp"
 #include "components/signals.hpp"
-#include "components/eventloop.hpp"
+#include "components/types.hpp"
 #include "x11/connection.hpp"
 #include "x11/randr.hpp"
 #include "x11/winspec.hpp"

src/modules/bspwm.cpp

@@ -15,10 +15,12 @@ namespace {
 
   uint32_t make_mask(bspwm_state s1, bspwm_state s2 = bspwm_state::NONE) {
     uint32_t mask{0U};
-    if (static_cast<uint32_t>(s1))
+    if (static_cast<uint32_t>(s1)) {
       mask |= 1 << (static_cast<uint32_t>(s1) - 1);
-    if (static_cast<uint32_t>(s2))
+    }
+    if (static_cast<uint32_t>(s2)) {
       mask |= 1 << (static_cast<uint32_t>(s2) - 1);
+    }
     return mask;
   }
 
@@ -327,16 +329,21 @@ namespace modules {
         auto label = m_statelabels.at(workspace_mask)->clone();
 
         if (!m_monitors.back()->focused) {
-          if (m_statelabels[make_mask(state::DIMMED)])
+          if (m_statelabels[make_mask(state::DIMMED)]) {
             label->replace_defined_values(m_statelabels[make_mask(state::DIMMED)]);
-          if (workspace_mask & make_mask(state::EMPTY))
+          }
+          if (workspace_mask & make_mask(state::EMPTY)) {
             label->replace_defined_values(m_statelabels[make_mask(state::DIMMED, state::EMPTY)]);
-          if (workspace_mask & make_mask(state::OCCUPIED))
+          }
+          if (workspace_mask & make_mask(state::OCCUPIED)) {
             label->replace_defined_values(m_statelabels[make_mask(state::DIMMED, state::OCCUPIED)]);
-          if (workspace_mask & make_mask(state::FOCUSED))
+          }
+          if (workspace_mask & make_mask(state::FOCUSED)) {
             label->replace_defined_values(m_statelabels[make_mask(state::DIMMED, state::FOCUSED)]);
-          if (workspace_mask & make_mask(state::URGENT))
+          }
+          if (workspace_mask & make_mask(state::URGENT)) {
             label->replace_defined_values(m_statelabels[make_mask(state::DIMMED, state::URGENT)]);
+          }
         }
 
         label->reset_tokens();

src/modules/meta/base.cpp

@@ -1,8 +1,8 @@
 #include <utility>
 
 #include "components/builder.hpp"
-#include "modules/meta/base.hpp"
 #include "drawtypes/label.hpp"
+#include "modules/meta/base.hpp"
 
 POLYBAR_NS
 

src/modules/xworkspaces.cpp

@@ -3,9 +3,9 @@
 #include "drawtypes/iconset.hpp"
 #include "drawtypes/label.hpp"
 #include "modules/xworkspaces.hpp"
+#include "utils/math.hpp"
 #include "x11/atoms.hpp"
 #include "x11/connection.hpp"
-#include "utils/math.hpp"
 
 #include "modules/meta/base.inl"
 #include "modules/meta/static_module.inl"

src/x11/tray.cpp

@@ -677,7 +677,6 @@ void tray_manager::notify_clients_delayed(chrono::duration<double, std::milli> d
     this_thread::sleep_for(delay);
     notify_clients();
   });
-
 }
 
 /**