- shrink less.

01a: svn version
01b: merge past_eof and inp_stdin into flags.
01c: merge match_backwards into flags.
01d: add handling of <HOME> and <END> keys, remove set_tty_raw, use fflush_stdout_and_exit
01e: rewrite special-key handling to be smaller.
01f: changing the format saves 4 bytes (alot of places would benefit from this)
   text    data     bss     dec     hex filename
   9034       8     648    9690    25da less.o.01a
   9023       8     640    9671    25c7 less.o.01b
   9027       8     636    9671    25c7 less.o.01c
   9068       8     636    9712    25f0 less.o.01d
   8990       8     636    9634    25a2 less.o.01e
   8986       8     636    9630    259e less.o.01f

1 files changed, 49 insertions, 56 deletions

diff --git a/miscutils/less.c b/miscutils/less.c
index de97ba6b9..03ffd78ed 100644
--- a/miscutils/less.c
+++ b/miscutils/less.c
@@ -44,14 +44,18 @@
 #define REAL_KEY_LEFT 'D'
 #define REAL_PAGE_UP '5'
 #define REAL_PAGE_DOWN '6'
+#define REAL_KEY_HOME '7'
+#define REAL_KEY_END '8'
 
 /* These are the special codes assigned by this program to the special keys */
-#define PAGE_UP 20
-#define PAGE_DOWN 21
-#define KEY_UP 22
-#define KEY_DOWN 23
-#define KEY_RIGHT 24
-#define KEY_LEFT 25
+#define KEY_UP 20
+#define KEY_DOWN 21
+#define KEY_RIGHT 22
+#define KEY_LEFT 23
+#define PAGE_UP 24
+#define PAGE_DOWN 25
+#define KEY_HOME 26
+#define KEY_END 27
 
 /* The escape codes for highlighted and normal text */
 #define HIGHLIGHT "\033[7m"
@@ -73,7 +77,6 @@ static int current_file = 1;
 static int line_pos;
 static int num_flines;
 static int num_files = 1;
-static int past_eof;
 
 /* Command line options */
 static unsigned flags;
@@ -82,10 +85,11 @@ static unsigned flags;
 #define FLAG_m (1<<2)
 #define FLAG_N (1<<3)
 #define FLAG_TILDE (1<<4)
-
-/* This is needed so that program behaviour changes when input comes from
-   stdin */
-static int inp_stdin;
+/* hijack command line options variable for internal state vars */
+#define LESS_STATE_INP_STDIN (1<<5)
+#define LESS_STATE_PAST_EOF  (1<<6)
+#define LESS_STATE_MATCH_BACKWARDS (1<<7)
+/* INP_STDIN is used to change behaviour when input comes from stdin */
 
 #ifdef CONFIG_FEATURE_LESS_MARKS
 static int mark_lines[15][2];
@@ -97,7 +101,6 @@ static int match_found;
 static int *match_lines;
 static int match_pos;
 static int num_matches;
-static int match_backwards;
 static regex_t old_pattern;
 #endif
 
@@ -114,12 +117,6 @@ static void set_tty_cooked(void)
         tcsetattr(fileno(inp), TCSANOW, &term_orig);
 }
 
-/* Set terminal input to raw mode  (taken from vi.c) */
-static void set_tty_raw(void)
-{
-        tcsetattr(fileno(inp), TCSANOW, &term_vi);
-}
-
 /* Exit the program gracefully */
 static void tless_exit(int code)
 {
@@ -129,7 +126,7 @@ static void tless_exit(int code)
                  only termios.h? */
 
         putchar('\n');
-        exit(code);
+        fflush_stdout_and_exit(code);
 }
 
 /* Grab a character from input without requiring the return key. If the
@@ -138,28 +135,23 @@ static void tless_exit(int code)
 static int tless_getch(void)
 {
         int input;
-
-        set_tty_raw();
+        /* Set terminal input to raw mode  (taken from vi.c) */
+        tcsetattr(fileno(inp), TCSANOW, &term_vi);
 
         input = getc(inp);
         /* Detect escape sequences (i.e. arrow keys) and handle
            them accordingly */
 
         if (input == '\033' && getc(inp) == '[') {
+                unsigned int i;
                 input = getc(inp);
                 set_tty_cooked();
-                if (input == REAL_KEY_UP)
-                        return KEY_UP;
-                else if (input == REAL_KEY_DOWN)
-                        return KEY_DOWN;
-                else if (input == REAL_KEY_RIGHT)
-                        return KEY_RIGHT;
-                else if (input == REAL_KEY_LEFT)
-                        return KEY_LEFT;
-                else if (input == REAL_PAGE_UP)
-                        return PAGE_UP;
-                else if (input == REAL_PAGE_DOWN)
-                        return PAGE_DOWN;
+
+                i = input - REAL_KEY_UP;
+                if (i < 4)
+                        return 20 + i;
+                else if ((i = input - REAL_PAGE_UP) < 4)
+                        return 24 + i;
         }
         /* The input is a normal ASCII value */
         else {
@@ -200,7 +192,7 @@ static void data_readlines(void)
         char current_line[256];
         FILE *fp;
 
-        fp = (inp_stdin) ? stdin : xfopen(filename, "r");
+        fp = (flags & LESS_STATE_INP_STDIN) ? stdin : xfopen(filename, "r");
         flines = NULL;
         for (i = 0; (feof(fp)==0) && (i <= MAXLINES); i++) {
                 strcpy(current_line, "");
@@ -215,12 +207,12 @@ static void data_readlines(void)
         /* Reset variables for a new file */
 
         line_pos = 0;
-        past_eof = 0;
+        flags &= ~LESS_STATE_PAST_EOF;
 
         fclose(fp);
 
         if (inp == NULL)
-                inp = (inp_stdin) ? xfopen(CURRENT_TTY, "r") : stdin;
+                inp = (flags & LESS_STATE_INP_STDIN) ? xfopen(CURRENT_TTY, "r") : stdin;
 
         if (flags & FLAG_N)
                 add_linenumbers();
@@ -240,7 +232,7 @@ static void m_status_print(void)
 {
         int percentage;
 
-        if (!past_eof) {
+        if (!(flags & LESS_STATE_PAST_EOF)) {
                 if (!line_pos) {
                         if (num_files > 1)
                                 printf("%s%s %s%i%s%i%s%i-%i/%i ", HIGHLIGHT,
@@ -372,7 +364,7 @@ static void buffer_down(int nlines)
 {
         int i;
 
-        if (!past_eof) {
+        if (!(flags & LESS_STATE_PAST_EOF)) {
                 if (line_pos + (height - 3) + nlines < num_flines) {
                         line_pos += nlines;
                         for (i = 0; i < (height - 1); i++) {
@@ -403,7 +395,7 @@ static void buffer_up(int nlines)
         int i;
         int tilde_line;
 
-        if (!past_eof) {
+        if (!(flags & LESS_STATE_PAST_EOF)) {
                 if (line_pos - nlines >= 0) {
                         line_pos -= nlines;
                         for (i = 0; i < (height - 1); i++) {
@@ -431,7 +423,7 @@ static void buffer_up(int nlines)
                 /* Going backwards nlines lines has taken us to a point where
                    nothing is past the EOF, so we revert to normal. */
                 if (line_pos < num_flines - height + 3) {
-                        past_eof = 0;
+                        flags &= ~LESS_STATE_PAST_EOF;
                         buffer_up(nlines);
                 }
                 else {
@@ -453,7 +445,7 @@ static void buffer_up(int nlines)
 static void buffer_line(int linenum)
 {
         int i;
-        past_eof = 0;
+        flags &= ~LESS_STATE_PAST_EOF;
 
         if (linenum < 0 || linenum > num_flines) {
                 clear_line();
@@ -477,7 +469,7 @@ static void buffer_line(int linenum)
                 }
                 line_pos = linenum;
                 /* Set past_eof so buffer_down and buffer_up act differently */
-                past_eof = 1;
+                flags |= LESS_STATE_PAST_EOF;
                 buffer_print();
         }
 }
@@ -513,7 +505,7 @@ static void examine_file(void)
         current_file = num_files + 1;
         num_files++;
 
-        inp_stdin = 0;
+        flags &= ~LESS_STATE_INP_STDIN;
         reinitialise();
 }
 
@@ -666,13 +658,14 @@ static void regex_process(void)
         regex_t pattern;
         /* Get the uncompiled regular expression from the user */
         clear_line();
-        putchar((match_backwards) ? '?' : '/');
+        putchar((flags & LESS_STATE_MATCH_BACKWARDS) ? '?' : '/');
         uncomp_regex[0] = 0;
         fgets(uncomp_regex, sizeof(uncomp_regex), inp);
 
         if (strlen(uncomp_regex) == 1) {
                 if (num_matches)
-                        goto_match(match_backwards ? match_pos - 1 : match_pos + 1);
+                        goto_match((flags & LESS_STATE_MATCH_BACKWARDS)
+                                                ? match_pos - 1 : match_pos + 1);
                 else
                         buffer_print();
                 return;
@@ -710,7 +703,7 @@ static void regex_process(void)
 
         num_matches = j;
         if ((match_lines[0] != -1) && (num_flines > height - 2)) {
-                if (match_backwards) {
+                if (flags & LESS_STATE_MATCH_BACKWARDS) {
                         for (i = 0; i < num_matches; i++) {
                                 if (match_lines[i] > line_pos) {
                                         match_pos = i - 1;
@@ -776,11 +769,11 @@ static void number_process(int first_digit)
                         goto_match(match_pos + num);
                         break;
                 case '/':
-                        match_backwards = 0;
+                        flags &= ~LESS_STATE_MATCH_BACKWARDS;
                         regex_process();
                         break;
                 case '?':
-                        match_backwards = 1;
+                        flags |= LESS_STATE_MATCH_BACKWARDS;
                         regex_process();
                         break;
 #endif
@@ -878,7 +871,7 @@ static void save_input_to_file(void)
                 buffer_print();
         }
         else
-                printf("%sNo log file%s", HIGHLIGHT, NORMAL);
+                printf("%s%s%s", HIGHLIGHT, "No log file", NORMAL);
 }
 
 #ifdef CONFIG_FEATURE_LESS_MARKS
@@ -1029,10 +1022,10 @@ static void keypress_process(int keypress)
                         buffer_up((height - 1) / 2);
                         buffer_print();
                         break;
-                case 'g': case 'p': case '<': case '%':
+                case KEY_HOME: case 'g': case 'p': case '<': case '%':
                         buffer_line(0);
                         break;
-                case 'G': case '>':
+                case KEY_END: case 'G': case '>':
                         buffer_line(num_flines - height + 2);
                         break;
                 case 'q': case 'Q':
@@ -1055,7 +1048,7 @@ static void keypress_process(int keypress)
                         full_repaint();
                         break;
                 case 's':
-                        if (inp_stdin)
+                        if (flags & LESS_STATE_INP_STDIN)
                                 save_input_to_file();
                         break;
                 case 'E':
@@ -1069,7 +1062,7 @@ static void keypress_process(int keypress)
 #endif
 #ifdef CONFIG_FEATURE_LESS_REGEXP
                 case '/':
-                        match_backwards = 0;
+                        flags &= ~LESS_STATE_MATCH_BACKWARDS;
                         regex_process();
                         break;
                 case 'n':
@@ -1079,7 +1072,7 @@ static void keypress_process(int keypress)
                         goto_match(match_pos - 1);
                         break;
                 case '?':
-                        match_backwards = 1;
+                        flags |= LESS_STATE_MATCH_BACKWARDS;
                         regex_process();
                         break;
 #endif
@@ -1124,14 +1117,14 @@ int less_main(int argc, char **argv) {
 
         if (!num_files) {
                 if (ttyname(STDIN_FILENO) == NULL)
-                        inp_stdin = 1;
+                        flags |= LESS_STATE_INP_STDIN;
                 else {
                         bb_error_msg("missing filename");
                         bb_show_usage();
                 }
         }
 
-        strcpy(filename, (inp_stdin) ? bb_msg_standard_input : files[0]);
+        strcpy(filename, (flags & LESS_STATE_INP_STDIN) ? bb_msg_standard_input : files[0]);
         get_terminal_width_height(0, &width, &height);
         data_readlines();
         tcgetattr(fileno(inp), &term_orig);