diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2007-06-01 01:31:56 +0400 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2007-06-01 01:31:56 +0400 |
commit | c2f011aa031b8598422953d5e9a93ca008ad0867 (patch) | |
tree | 0f4c197180854003f7e169adbb0d0c056cc6f8b0 /util-linux | |
parent | 22a9a3c6f874b4741f9e4193e2b9b54c9eb39bdc (diff) |
more: stop using bss
# make && make bloatcheck
function old new delta
gotsig 86 107 +21
more_main 777 781 +4
cin_fileno 4 - -4
set_tty_to_initial_mode 25 - -25
new_settings 120 60 -60
initial_settings 120 60 -60
------------------------------------------------------------------------------
(add/remove: 0/2 grow/shrink: 2/2 up/down: 25/-149) Total: -124 bytes
Diffstat (limited to 'util-linux')
-rw-r--r-- | util-linux/more.c | 40 |
1 files changed, 27 insertions, 13 deletions
diff --git a/util-linux/more.c b/util-linux/more.c index 30ef896bc..4083d8eea 100644 --- a/util-linux/more.c +++ b/util-linux/more.c @@ -15,26 +15,39 @@ */ #include "libbb.h" +#if ENABLE_FEATURE_USE_TERMIOS +#include <termios.h> +#endif /* FEATURE_USE_TERMIOS */ #if ENABLE_FEATURE_USE_TERMIOS -static int cin_fileno; -#include <termios.h> -#define setTermSettings(fd, argp) tcsetattr(fd, TCSANOW, argp) -#define getTermSettings(fd, argp) tcgetattr(fd, argp); -static struct termios initial_settings, new_settings; +struct globals { + int cin_fileno; + struct termios initial_settings; + struct termios new_settings; +}; +#define G (*(struct globals*)bb_common_bufsiz1) +//#define G (*ptr_to_globals) +#define initial_settings (G.initial_settings) +#define new_settings (G.new_settings ) +#define cin_fileno (G.cin_fileno ) +#define INIT_G() ((void)0) +//#define INIT_G() PTR_TO_GLOBALS = xzalloc(sizeof(G)) -static void set_tty_to_initial_mode(void) -{ - setTermSettings(cin_fileno, &initial_settings); -} +#define setTermSettings(fd, argp) tcsetattr(fd, TCSANOW, argp) +#define getTermSettings(fd, argp) tcgetattr(fd, argp) static void gotsig(int sig) { putchar('\n'); + setTermSettings(cin_fileno, &initial_settings); exit(EXIT_FAILURE); } + +#else /* !FEATURE_USE_TERMIOS */ +#define INIT_G() ((void)0) +#define setTermSettings(fd, argp) ((void)0) #endif /* FEATURE_USE_TERMIOS */ @@ -50,6 +63,8 @@ int more_main(int argc, char **argv) int terminal_width; int terminal_height; + INIT_G(); + argv++; /* Another popular pager, most, detects when stdout * is not a tty and turns into cat. This makes sense. */ @@ -68,7 +83,6 @@ int more_main(int argc, char **argv) new_settings.c_cc[VMIN] = 1; new_settings.c_cc[VTIME] = 0; setTermSettings(cin_fileno, &new_settings); - atexit(set_tty_to_initial_mode); signal(SIGINT, gotsig); signal(SIGQUIT, gotsig); signal(SIGTERM, gotsig); @@ -95,7 +109,6 @@ int more_main(int argc, char **argv) lines = 0; page_height = terminal_height; while ((c = getc(file)) != EOF) { - if ((please_display_more_prompt & 3) == 3) { len = printf("--More-- "); if (/*file != stdin &&*/ st.st_size > 0) { @@ -129,8 +142,8 @@ int more_main(int argc, char **argv) /* * There are two input streams to worry about here: * - * c : the character we are reading from the file being "mored" - * input : a character received from the keyboard + * c : the character we are reading from the file being "mored" + * input: a character received from the keyboard * * If we hit a newline in the _file_ stream, we want to test and * see if any characters have been hit in the _input_ stream. This @@ -169,5 +182,6 @@ int more_main(int argc, char **argv) fflush(stdout); } while (*argv && *++argv); end: + setTermSettings(cin_fileno, &initial_settings); return 0; } |