From 2a57608f673698ddbde6704ef1f01f2fd443710b Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Mon, 1 Apr 2019 16:15:51 +0200 Subject: [PATCH] vi: code shrink function old new delta get_one_char 108 103 -5 edit_file 651 644 -7 do_cmd 4696 4688 -8 ------------------------------------------------------------------------------ (add/remove: 0/0 grow/shrink: 0/3 up/down: 0/-20) Total: -20 bytes Signed-off-by: Denys Vlasenko --- editors/vi.c | 58 +++++++++++++++++++++++++--------------------------- 1 file changed, 28 insertions(+), 30 deletions(-) diff --git a/editors/vi.c b/editors/vi.c index 5c585a390..af23ae7d2 100644 --- a/editors/vi.c +++ b/editors/vi.c @@ -1029,43 +1029,41 @@ static int readit(void) // read (maybe cursor) key from stdin return c; } +#if ENABLE_FEATURE_VI_DOT_CMD static int get_one_char(void) { int c; -#if ENABLE_FEATURE_VI_DOT_CMD if (!adding2q) { // we are not adding to the q. - // but, we may be reading from a q - if (ioq == 0) { - // there is no current q, read from STDIN - c = readit(); // get the users input - } else { - // there is a queue to get chars from first + // but, we may be reading from a saved q. + // (checking "ioq" for NULL is wrong, it's not reset to NULL + // when done - "ioq_start" is reset instead). + if (ioq_start != NULL) { + // there is a queue to get chars from. // careful with correct sign expansion! c = (unsigned char)*ioq++; - if (c == '\0') { - // the end of the q, read from STDIN - free(ioq_start); - ioq_start = ioq = 0; - c = readit(); // get the users input - } + if (c != '\0') + return c; + // the end of the q + free(ioq_start); + ioq_start = NULL; + // read from STDIN: } + return readit(); + } + // we are adding STDIN chars to q. + c = readit(); + if (lmc_len >= MAX_INPUT_LEN - 1) { + status_line_bold("last_modifying_cmd overrun"); } else { - // adding STDIN chars to q - c = readit(); // get the users input - if (lmc_len >= MAX_INPUT_LEN - 1) { - status_line_bold("last_modifying_cmd overrun"); - } else { - // add new char to q - last_modifying_cmd[lmc_len++] = c; - } + last_modifying_cmd[lmc_len++] = c; } -#else - c = readit(); // get the users input -#endif /* FEATURE_VI_DOT_CMD */ return c; } +#else +# define get_one_char() readit() +#endif // Get input line (uses "status line" area) static char *get_input_line(const char *prompt) @@ -1781,7 +1779,7 @@ static void start_new_cmd_q(char c) // get buffer for new cmd // if there is a current cmd count put it in the buffer first if (cmdcnt > 0) { - lmc_len = sprintf(last_modifying_cmd, "%d%c", cmdcnt, c); + lmc_len = sprintf(last_modifying_cmd, "%u%c", cmdcnt, c); } else { // just save char c onto queue last_modifying_cmd[0] = c; lmc_len = 1; @@ -3415,9 +3413,8 @@ static void do_cmd(int c) case '.': // .- repeat the last modifying command // Stuff the last_modifying_cmd back into stdin // and let it be re-executed. - if (lmc_len > 0) { - last_modifying_cmd[lmc_len] = 0; - ioq = ioq_start = xstrdup(last_modifying_cmd); + if (lmc_len != 0) { + ioq = ioq_start = xstrndup(last_modifying_cmd, lmc_len); } break; #endif @@ -4221,7 +4218,7 @@ static void edit_file(char *fn) c = '\0'; #if ENABLE_FEATURE_VI_DOT_CMD free(ioq_start); - ioq = ioq_start = NULL; + ioq_start = NULL; lmc_len = 0; adding2q = 0; #endif @@ -4273,7 +4270,8 @@ static void edit_file(char *fn) #if ENABLE_FEATURE_VI_DOT_CMD // These are commands that change text[]. // Remember the input for the "." command - if (!adding2q && ioq_start == NULL + if (!adding2q + && ioq_start == NULL && cmd_mode == 0 // command mode && c > '\0' // exclude NUL and non-ASCII chars && c < 0x7f // (Unicode and such) -- 2.25.1