opt: test HAVE_SYS_TERMIOS_H via #if
authorRusty Russell <rusty@rustcorp.com.au>
Thu, 6 Feb 2014 03:51:40 +0000 (14:21 +1030)
committerRusty Russell <rusty@rustcorp.com.au>
Thu, 6 Feb 2014 03:51:40 +0000 (14:21 +1030)
As ccanlint warns.  Also, test TIOCGWINSZ before ioctl, rather than the
header directly since it's a little orthogonal.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
ccan/opt/usage.c

index ef33f9188dd304f41b12f19c6bbd29b3f8a69849..ff0545591ed4050a5153aa5c8d6db2041a1295e4 100644 (file)
@@ -1,6 +1,6 @@
 /* Licensed under GPLv3+ - see LICENSE file for details */
 #include <ccan/opt/opt.h>
-#ifdef HAVE_SYS_TERMIOS_H
+#if HAVE_SYS_TERMIOS_H
 #include <sys/ioctl.h>
 #include <sys/termios.h> /* Required on Solaris for struct winsize */
 #endif
@@ -24,7 +24,8 @@ static unsigned int get_columns(void)
 
        if (env)
                ws_col = atoi(env);
-#ifdef HAVE_SYS_TERMIOS_H
+
+#ifdef TIOCGWINSZ
        if (!ws_col)
        {
                struct winsize w;