- 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 de97ba6..03ffd78 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);