fixup: just check the result

Sjors · Sjors · commit fdb04c21066a · 2025-06-28T11:19:54.000+02:00
diff --git a/src/ipc/interfaces.cpp b/src/ipc/interfaces.cpp
@@ -32,7 +32,10 @@ std::string g_ignore_ctrl_c;
 
 void HandleCtrlC(int)
 {
-    (void)write(STDOUT_FILENO, g_ignore_ctrl_c.data(), g_ignore_ctrl_c.size());
+    ssize_t result = write(STDOUT_FILENO, g_ignore_ctrl_c.data(), g_ignore_ctrl_c.size());
+    // Casting unused result to (void) does not work in gcc
+    // https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66425
+    if (!result) return;
 }
 #endif
 

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,10 @@ std::string g_ignore_ctrl_c;`
`32`	`32`
`33`	`33`	`void HandleCtrlC(int)`
`34`	`34`	`{`
`35`		`- (void)write(STDOUT_FILENO, g_ignore_ctrl_c.data(), g_ignore_ctrl_c.size());`
	`35`	`+ ssize_t result = write(STDOUT_FILENO, g_ignore_ctrl_c.data(), g_ignore_ctrl_c.size());`
	`36`	`+ // Casting unused result to (void) does not work in gcc`
	`37`	`+ // https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66425`
	`38`	`+ if (!result) return;`
`36`	`39`	`}`
`37`	`40`	`#endif`
`38`	`41`