*: code shrink by adding a wrapper around very common tcsetattr(0, TCSANOW, xx) op
authorDenis Vlasenko <vda.linux@googlemail.com>
Wed, 5 Nov 2008 13:20:58 +0000 (13:20 -0000)
committerDenis Vlasenko <vda.linux@googlemail.com>
Wed, 5 Nov 2008 13:20:58 +0000 (13:20 -0000)
function                                             old     new   delta
tcsetattr_stdin_TCSANOW                                -      14     +14
set_sane_term                                        116     113      -3
top_main                                            1277    1273      -4
make_new_session                                     421     415      -6
rawmode                                              133     126      -7
reset_term                                            18      10      -8
die                                                   43      35      -8
cookmode                                              62      54      -8
vlock_main                                           425     415     -10
read_line_input                                     3165    3155     -10
bb_askpass                                           357     347     -10
fsck_minix_main                                     3079    3065     -14
getty_main                                          2375    2332     -43
------------------------------------------------------------------------------
(add/remove: 1/0 grow/shrink: 0/12 up/down: 14/-131)         Total: -117 bytes

13 files changed:
e2fsprogs/old_e2fsprogs/e2fsck.c
editors/vi.c
include/libbb.h
init/init.c
libbb/bb_askpass.c
libbb/get_console.c
libbb/lineedit.c
libbb/xfuncs.c
loginutils/getty.c
loginutils/vlock.c
networking/telnetd.c
procps/top.c
util-linux/fsck_minix.c

index 61a30f4872a5583d4193a896854601b33bd8b8ea..741536c1c58a702d8535f7e98c6c73c1f0b4aee7 100644 (file)
@@ -12281,7 +12281,7 @@ static int ask_yn(const char * string, int def)
        tmp.c_lflag &= ~(ICANON | ECHO);
        tmp.c_cc[VMIN] = 1;
        tmp.c_cc[VTIME] = 0;
-       tcsetattr (0, TCSANOW, &tmp);
+       tcsetattr_stdin_TCSANOW(&tmp);
 #endif
 
        if (def == 1)
@@ -12297,7 +12297,7 @@ static int ask_yn(const char * string, int def)
                        break;
                if (c == 3) {
 #ifdef HAVE_TERMIOS_H
-                       tcsetattr (0, TCSANOW, &termios);
+                       tcsetattr_stdin_TCSANOW(&termios);
 #endif
                        if (e2fsck_global_ctx &&
                            e2fsck_global_ctx->flags & E2F_FLAG_SETJMP_OK) {
@@ -12323,7 +12323,7 @@ static int ask_yn(const char * string, int def)
        else
                puts ("no\n");
 #ifdef HAVE_TERMIOS_H
-       tcsetattr (0, TCSANOW, &termios);
+       tcsetattr_stdin_TCSANOW(&termios);
 #endif
        return def;
 }
index 93e5647569fe8a7d5c8964682bc2376148eed739..92c069de05931755c7e25a1656381374ef36c508 100644 (file)
@@ -2106,13 +2106,13 @@ static void rawmode(void)
        term_vi.c_cc[VMIN] = 1;
        term_vi.c_cc[VTIME] = 0;
        erase_char = term_vi.c_cc[VERASE];
-       tcsetattr(0, TCSANOW, &term_vi);
+       tcsetattr_stdin_TCSANOW(&term_vi);
 }
 
 static void cookmode(void)
 {
        fflush(stdout);
-       tcsetattr(0, TCSANOW, &term_orig);
+       tcsetattr_stdin_TCSANOW(&term_orig);
 }
 
 //----- Come here when we get a window resize signal ---------
index 5b6a2dae1d2db00a9fd1441632840cbf4931a000..086a59f05e5224478bba9b83fa30fe95d117a80c 100644 (file)
@@ -1142,6 +1142,8 @@ extern void print_login_prompt(void) FAST_FUNC;
 /* NB: typically you want to pass fd 0, not 1. Think 'applet | grep something' */
 int get_terminal_width_height(int fd, unsigned *width, unsigned *height) FAST_FUNC;
 
+int tcsetattr_stdin_TCSANOW(const struct termios *tp) FAST_FUNC;
+
 /* NB: "unsigned request" is crucial! "int request" will break some arches! */
 int ioctl_or_perror(int fd, unsigned request, void *argp, const char *fmt,...) __attribute__ ((format (printf, 4, 5))) FAST_FUNC;
 int ioctl_or_perror_and_die(int fd, unsigned request, void *argp, const char *fmt,...) __attribute__ ((format (printf, 4, 5))) FAST_FUNC;
index ae2dd6a7a7be727250dd686a36242d9c7b03994c..3d171d71f1dc2f966bff35b4c0f65028a8d0a00c 100644 (file)
@@ -261,7 +261,7 @@ static void set_sane_term(void)
        tty.c_lflag =
                ISIG | ICANON | ECHO | ECHOE | ECHOK | ECHOCTL | ECHOKE | IEXTEN;
 
-       tcsetattr(STDIN_FILENO, TCSANOW, &tty);
+       tcsetattr_stdin_TCSANOW(&tty);
 }
 
 /* Open the new terminal device.
index fba82a07bf2a79175216a20a1e11182bb93c4476..c60ef3708986f0d7a9c519ff7480109d9f048141 100644 (file)
@@ -37,7 +37,7 @@ char* FAST_FUNC bb_askpass(int timeout, const char *prompt)
        tio = oldtio;
        tio.c_iflag &= ~(IUCLC|IXON|IXOFF|IXANY);
        tio.c_lflag &= ~(ECHO|ECHOE|ECHOK|ECHONL|TOSTOP);
-       tcsetattr(STDIN_FILENO, TCSANOW, &tio);
+       tcsetattr_stdin_TCSANOW(&tio);
 
        memset(&sa, 0, sizeof(sa));
        /* sa.sa_flags = 0; - no SA_RESTART! */
@@ -70,7 +70,7 @@ char* FAST_FUNC bb_askpass(int timeout, const char *prompt)
        }
        sigaction_set(SIGINT, &oldsa);
 
-       tcsetattr(STDIN_FILENO, TCSANOW, &oldtio);
+       tcsetattr_stdin_TCSANOW(&oldtio);
        bb_putchar('\n');
        fflush(stdout);
        return ret;
index ad56e740f7ee68ba5584c4e09096e6dcc3c509a5..74022b5437e35a174c1b885fa7f339193a3d2b4b 100644 (file)
@@ -13,7 +13,6 @@
 /* From <linux/kd.h> */
 enum { KDGKBTYPE = 0x4B33 };  /* get keyboard type */
 
-
 static int open_a_console(const char *fnam)
 {
        int fd;
@@ -37,7 +36,6 @@ static int open_a_console(const char *fnam)
  * We try several things because opening /dev/console will fail
  * if someone else used X (which does a chown on /dev/console).
  */
-
 int FAST_FUNC get_console_fd_or_die(void)
 {
        static const char *const console_names[] = {
index 1f21866ca65b28a34603d80a532febae8042332a..3953cc904b7f364375847e1b5907cd20de1c7811 100644 (file)
@@ -1436,7 +1436,7 @@ int FAST_FUNC read_line_input(const char *prompt, char *command, int maxsize, li
 #define _POSIX_VDISABLE '\0'
 #endif
        new_settings.c_cc[VINTR] = _POSIX_VDISABLE;
-       tcsetattr(STDIN_FILENO, TCSANOW, &new_settings);
+       tcsetattr_stdin_TCSANOW(&new_settings);
 
        /* Now initialize things */
        previous_SIGWINCH_handler = signal(SIGWINCH, win_changed);
@@ -1860,7 +1860,7 @@ int FAST_FUNC read_line_input(const char *prompt, char *command, int maxsize, li
 #endif
 
        /* restore initial_settings */
-       tcsetattr(STDIN_FILENO, TCSANOW, &initial_settings);
+       tcsetattr_stdin_TCSANOW(&initial_settings);
        /* restore SIGWINCH handler */
        signal(SIGWINCH, previous_SIGWINCH_handler);
        fflush(stdout);
index 8ef305ba08e2b1d4301ae92941bb29b23f2f6807..e2aee136c8631d731fdd298297e973e23ff076db 100644 (file)
@@ -289,3 +289,8 @@ int FAST_FUNC get_terminal_width_height(int fd, unsigned *width, unsigned *heigh
 
        return ret;
 }
+
+int FAST_FUNC tcsetattr_stdin_TCSANOW(const struct termios *tp)
+{
+       return tcsetattr(STDIN_FILENO, TCSANOW, tp);
+}
index 8b0e729fbb3a65252c49c36b64a28e8a8a273a0c..0f536888bdc7265355d3b52e2716407083afd576 100644 (file)
@@ -279,7 +279,7 @@ static void termios_init(struct termios *tp, int speed, struct options *op)
         */
 #ifdef __linux__
        /* flush input and output queues, important for modems! */
-       ioctl(0, TCFLSH, TCIOFLUSH);
+       ioctl(0, TCFLSH, TCIOFLUSH); /* tcflush(0, TCIOFLUSH)? - same */
 #endif
 
        tp->c_cflag = CS8 | HUPCL | CREAD | speed;
@@ -297,7 +297,7 @@ static void termios_init(struct termios *tp, int speed, struct options *op)
                tp->c_cflag |= CRTSCTS;
 #endif
 
-       ioctl(0, TCSETS, tp);
+       tcsetattr_stdin_TCSANOW(tp);
 
        debug("term_io 2\n");
 }
@@ -334,7 +334,7 @@ static void auto_baud(char *buf, unsigned size_buf, struct termios *tp)
        tp->c_iflag |= ISTRIP;          /* enable 8th-bit stripping */
        vmin = tp->c_cc[VMIN];
        tp->c_cc[VMIN] = 0;             /* don't block if queue empty */
-       ioctl(0, TCSETS, tp);
+       tcsetattr_stdin_TCSANOW(tp);
 
        /*
         * Wait for a while, then read everything the modem has said so far and
@@ -359,7 +359,7 @@ static void auto_baud(char *buf, unsigned size_buf, struct termios *tp)
        /* Restore terminal settings. Errors will be dealt with later on. */
        tp->c_iflag = iflag;
        tp->c_cc[VMIN] = vmin;
-       ioctl(0, TCSETS, tp);
+       tcsetattr_stdin_TCSANOW(tp);
 }
 
 /* do_prompt - show login prompt, optionally preceded by /etc/issue contents */
@@ -404,7 +404,7 @@ static char *get_logname(char *logname, unsigned size_logname,
 
        /* Flush pending input (esp. after parsing or switching the baud rate). */
        sleep(1);
-       ioctl(0, TCFLSH, TCIFLUSH);
+       ioctl(0, TCFLSH, TCIFLUSH); /* tcflush(0, TCIOFLUSH)? - same */
 
        /* Prompt for and read a login name. */
        logname[0] = '\0';
@@ -552,12 +552,13 @@ static void termios_final(struct options *op, struct termios *tp, struct chardat
        }
 #endif
        /* Optionally enable hardware flow control */
-#ifdef  CRTSCTS
+#ifdef CRTSCTS
        if (op->flags & F_RTSCTS)
                tp->c_cflag |= CRTSCTS;
 #endif
 
        /* Finally, make the new settings effective */
+       /* It's tcsetattr_stdin_TCSANOW() + error check */
        ioctl_or_perror_and_die(0, TCSETS, tp, "%s: TCSETS", op->tty);
 }
 
@@ -689,6 +690,7 @@ int getty_main(int argc UNUSED_PARAM, char **argv)
         * by patching the SunOS kernel variable "zsadtrlow" to a larger value;
         * 5 seconds seems to be a good value.
         */
+       /* tcgetattr() + error check */
        ioctl_or_perror_and_die(0, TCGETS, &termios, "%s: TCGETS", options.tty);
 
 #ifdef __linux__
@@ -753,7 +755,7 @@ int getty_main(int argc UNUSED_PARAM, char **argv)
                        baud_index = (baud_index + 1) % options.numspeed;
                        termios.c_cflag &= ~CBAUD;
                        termios.c_cflag |= options.speeds[baud_index];
-                       ioctl(0, TCSETS, &termios);
+                       tcsetattr_stdin_TCSANOW(&termios);
                }
        }
 
index 42ef447326baca7d951963e924f3ea3442e4e49a..0262da5bd0e1b918eb5e50588d396407b7e46e24 100644 (file)
@@ -87,7 +87,7 @@ int vlock_main(int argc UNUSED_PARAM, char **argv)
        term.c_iflag |= IGNBRK;
        term.c_lflag &= ~ISIG;
        term.c_lflag &= ~(ECHO | ECHOCTL);
-       tcsetattr(STDIN_FILENO, TCSANOW, &term);
+       tcsetattr_stdin_TCSANOW(&term);
 
        do {
                printf("Virtual console%s locked by %s.\n",
@@ -101,6 +101,6 @@ int vlock_main(int argc UNUSED_PARAM, char **argv)
        } while (1);
 
        ioctl(STDIN_FILENO, VT_SETMODE, &ovtm);
-       tcsetattr(STDIN_FILENO, TCSANOW, &oterm);
+       tcsetattr_stdin_TCSANOW(&oterm);
        fflush_stdout_and_exit(EXIT_SUCCESS);
 }
index 7f748641b7aec671415694aa35fb4d7a43968ec7..efc2bf60740f1ec3aebf2698f2491898c80ba246 100644 (file)
@@ -243,7 +243,7 @@ make_new_session(
        termbuf.c_iflag |= ICRNL;
        termbuf.c_iflag &= ~IXOFF;
        /*termbuf.c_lflag &= ~ICANON;*/
-       tcsetattr(0, TCSANOW, &termbuf);
+       tcsetattr_stdin_TCSANOW(&termbuf);
 
        /* Uses FILE-based I/O to stdout, but does fflush(stdout),
         * so should be safe with vfork.
index 908ae81519b8d59ee8e9b755e51c405c1d930630..1d22871f68486be760a0027905a26a2271d9ac89 100644 (file)
@@ -649,7 +649,7 @@ static void clearmems(void)
 
 static void reset_term(void)
 {
-       tcsetattr(0, TCSANOW, &initial_settings);
+       tcsetattr_stdin_TCSANOW(&initial_settings);
        if (ENABLE_FEATURE_CLEAN_UP) {
                clearmems();
 #if ENABLE_FEATURE_TOP_CPU_USAGE_PERCENTAGE
@@ -941,7 +941,7 @@ int top_main(int argc UNUSED_PARAM, char **argv)
        new_settings.c_lflag &= ~(ISIG | ICANON | ECHO | ECHONL);
 
        bb_signals(BB_FATAL_SIGS, sig_catcher);
-       tcsetattr(0, TCSANOW, (void *) &new_settings);
+       tcsetattr_stdin_TCSANOW(&new_settings);
 #endif /* FEATURE_USE_TERMIOS */
 
 #if ENABLE_FEATURE_TOP_CPU_USAGE_PERCENTAGE
index 2dbd437a359038b6e2ef2a45cd44c909b6128552..78a7c826b71c82c5080da22963c2e2896eb4f870 100644 (file)
@@ -293,7 +293,7 @@ static void die(const char *str) NORETURN;
 static void die(const char *str)
 {
        if (termios_set)
-               tcsetattr(0, TCSANOW, &sv_termios);
+               tcsetattr_stdin_TCSANOW(&sv_termios);
        bb_error_msg_and_die("%s", str);
 }
 
@@ -1254,7 +1254,7 @@ int fsck_minix_main(int argc UNUSED_PARAM, char **argv)
                tcgetattr(0, &sv_termios);
                tmp = sv_termios;
                tmp.c_lflag &= ~(ICANON | ECHO);
-               tcsetattr(0, TCSANOW, &tmp);
+               tcsetattr_stdin_TCSANOW(&tmp);
                termios_set = 1;
        }
 
@@ -1299,7 +1299,7 @@ int fsck_minix_main(int argc UNUSED_PARAM, char **argv)
                write_superblock();
 
        if (OPT_manual)
-               tcsetattr(0, TCSANOW, &sv_termios);
+               tcsetattr_stdin_TCSANOW(&sv_termios);
 
        if (changed)
                retcode += 3;