chiark / gitweb /
xtitle.c: Ignore boring return codes better.
authorMark Wooding <mdw@distorted.org.uk>
Thu, 29 Mar 2012 18:50:37 +0000 (19:50 +0100)
committerMark Wooding <mdw@distorted.org.uk>
Thu, 29 Mar 2012 18:50:37 +0000 (19:50 +0100)
I'm not convinced that ignoring the write code when reading the
current title is clever, but that code is hopelessly broken anyway.

xtitle.c

index 29607cb7188117111c91b9f08dae1e24b0c8cc9f..a6696d315951fd67652f8e540b8a04a2a7c487a6 100644 (file)
--- a/xtitle.c
+++ b/xtitle.c
@@ -6,13 +6,14 @@
 #include <string.h>
 #include <fcntl.h>
 
+#define IGNORE(x) do if (x); while (0)
+
 int main(int argc, char *argv[])
 {
   int query = 0;
   int fd;
   int openned = 0;
   int i;
-  int hunoz;
 
   for (;;) {
     i = getopt(argc, argv, "q");
@@ -52,13 +53,13 @@ int main(int argc, char *argv[])
 
   if (!query) {
     char sp = ' ';
-    hunoz = write(fd, "\33]0;", 4);
+    IGNORE(write(fd, "\33]0;", 4));
     for (i = optind; i < argc; i++) {
-      hunoz = write(fd, argv[i], strlen(argv[i]));
+      IGNORE(write(fd, argv[i], strlen(argv[i])));
       if (i < argc - 1)
-       hunoz = write(fd, &sp, 1);
+       IGNORE(write(fd, &sp, 1));
     }
-    hunoz = write(fd, "\7", 2);
+    IGNORE(write(fd, "\7", 2));
   } else {
     struct termios o, n;
     char hack;
@@ -72,7 +73,7 @@ int main(int argc, char *argv[])
     n.c_cflag &= ~(CSIZE|PARENB);
     n.c_cflag |= CS8;
     tcsetattr(fd, TCSAFLUSH, &n);
-    hunoz = write(fd, "\33[21t", 5);
+    IGNORE(write(fd, "\33[21t", 5));
 
     while (state != -1) {
       if (read(fd, &hack, 1) < 1)