Merge pull request #2366 from zjeffer/fix/build-warnings

Fixed sway IPC compile warnings
Alexays · Jul 30, 2023 · 8882826 · 8882826
2 parents b09f6cc + 28635c1
commit 8882826
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/modules/sway/ipc/client.cpp b/src/modules/sway/ipc/client.cpp
@@ -1,6 +1,7 @@
 #include "modules/sway/ipc/client.hpp"
 
 #include <fcntl.h>
+#include <spdlog/spdlog.h>
 
 #include <stdexcept>
 
@@ -17,12 +18,16 @@ Ipc::~Ipc() {
 
   if (fd_ > 0) {
     // To fail the IPC header
-    write(fd_, "close-sway-ipc", 14);
+    if (write(fd_, "close-sway-ipc", 14) == -1) {
+      spdlog::error("Failed to close sway IPC");
+    }
     close(fd_);
     fd_ = -1;
   }
   if (fd_event_ > 0) {
-    write(fd_event_, "close-sway-ipc", 14);
+    if (write(fd_event_, "close-sway-ipc", 14) == -1) {
+      spdlog::error("Failed to close sway IPC event handler");
+    }
     close(fd_event_);
     fd_event_ = -1;
   }