scope_guard/modules: Rename scope_guard to ScopeGuard

Using pascal case for the class name keeps it more consistent with the majority of the other class names.
Alexays · Oct 23, 2023 · c5b1703 · c5b1703
1 parent ffeeed8
commit c5b1703
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 15 deletions.
diff --git a/include/util/scope_guard.hpp b/include/util/scope_guard.hpp
@@ -5,14 +5,14 @@
 namespace waybar::util {
 
 template <typename Func>
-class scope_guard {
+class ScopeGuard {
  public:
-  explicit scope_guard(Func&& exit_function) : f{std::forward<Func>(exit_function)} {}
-  scope_guard(const scope_guard&) = delete;
-  scope_guard(scope_guard&&) = default;
-  scope_guard& operator=(const scope_guard&) = delete;
-  scope_guard& operator=(scope_guard&&) = default;
-  ~scope_guard() { f(); }
+  explicit ScopeGuard(Func&& exit_function) : f{std::forward<Func>(exit_function)} {}
+  ScopeGuard(const ScopeGuard&) = delete;
+  ScopeGuard(ScopeGuard&&) = default;
+  ScopeGuard& operator=(const ScopeGuard&) = delete;
+  ScopeGuard& operator=(ScopeGuard&&) = default;
+  ~ScopeGuard() { f(); }
 
  private:
   Func f;

diff --git a/src/modules/bluetooth.cpp b/src/modules/bluetooth.cpp
@@ -14,7 +14,7 @@ using GDBusManager = std::unique_ptr<GDBusObjectManager, void (*)(GDBusObjectMan
 
 auto generateManager() -> GDBusManager {
   GError* error = nullptr;
-  waybar::util::scope_guard error_deleter([error]() {
+  waybar::util::ScopeGuard error_deleter([error]() {
     if (error) {
       g_error_free(error);
     }

diff --git a/src/modules/custom.cpp b/src/modules/custom.cpp
@@ -59,7 +59,7 @@ void waybar::modules::Custom::continuousWorker() {
   }
   thread_ = [this, cmd] {
     char* buff = nullptr;
-    waybar::util::scope_guard buff_deleter([buff]() {
+    waybar::util::ScopeGuard buff_deleter([buff]() {
       if (buff) {
         free(buff);
       }

diff --git a/src/modules/mpris/mpris.cpp b/src/modules/mpris/mpris.cpp
@@ -119,7 +119,7 @@ Mpris::Mpris(const std::string& id, const Json::Value& config)
   }
 
   GError* error = nullptr;
-  waybar::util::scope_guard error_deleter([error]() {
+  waybar::util::ScopeGuard error_deleter([error]() {
     if (error) {
       g_error_free(error);
     }
@@ -482,7 +482,7 @@ auto Mpris::getPlayerInfo() -> std::optional<PlayerInfo> {
   }
 
   GError* error = nullptr;
-  waybar::util::scope_guard error_deleter([error]() {
+  waybar::util::ScopeGuard error_deleter([error]() {
     if (error) {
       g_error_free(error);
     }
@@ -580,7 +580,7 @@ auto Mpris::getPlayerInfo() -> std::optional<PlayerInfo> {
 
 bool Mpris::handleToggle(GdkEventButton* const& e) {
   GError* error = nullptr;
-  waybar::util::scope_guard error_deleter([error]() {
+  waybar::util::ScopeGuard error_deleter([error]() {
     if (error) {
       g_error_free(error);
     }

diff --git a/src/modules/sni/host.cpp b/src/modules/sni/host.cpp
@@ -59,7 +59,7 @@ void Host::nameVanished(const Glib::RefPtr<Gio::DBus::Connection>& conn, const G
 
 void Host::proxyReady(GObject* src, GAsyncResult* res, gpointer data) {
   GError* error = nullptr;
-  waybar::util::scope_guard error_deleter([error]() {
+  waybar::util::ScopeGuard error_deleter([error]() {
     if (error != nullptr) {
       g_error_free(error);
     }
@@ -81,7 +81,7 @@ void Host::proxyReady(GObject* src, GAsyncResult* res, gpointer data) {
 
 void Host::registerHost(GObject* src, GAsyncResult* res, gpointer data) {
   GError* error = nullptr;
-  waybar::util::scope_guard error_deleter([error]() {
+  waybar::util::ScopeGuard error_deleter([error]() {
     if (error != nullptr) {
       g_error_free(error);
     }

diff --git a/src/modules/sni/watcher.cpp b/src/modules/sni/watcher.cpp
@@ -31,7 +31,7 @@ Watcher::~Watcher() {
 
 void Watcher::busAcquired(const Glib::RefPtr<Gio::DBus::Connection>& conn, Glib::ustring name) {
   GError* error = nullptr;
-  waybar::util::scope_guard error_deleter([error]() {
+  waybar::util::ScopeGuard error_deleter([error]() {
     if (error) {
       g_error_free(error);
     }