st

commit 742a41d6554ce1848f45ea89cb83d62c7e881352 (patch)
parent f36dd277a2e71e99624fe26365757242d56259c0
Author: alp@alexpilon.ca <alp@alexpilon.ca>
Date:   Tue, 21 Apr 2015 16:27:51 +0200

Make build shut up about system() without return value check.

    st.c:1321:2: warning: ignoring return value of function declared with warn_unused_result attribute [-Wunused-result]
            system(cmd);
            ^~~~~~ ~~~

Debatable whether an error here should case exit(EXIT_FAILURE). Just
preserving the existing behaviour for now.

Diffstat:
M
st.c
 | 
3
++-

1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/st.c b/st.c
@@ -1276,7 +1276,8 @@ stty(void)
 		siz-= n + 1;
 	}
 	*q = '\0';
-	system(cmd);
+	if (system(cmd) != 0)
+	    perror("Couldn't call stty");
 }
 
 void