commit bc92e28b30784657b1b0aa288c034bae54d0afca
parent c5b0e1958ab458e3c297431db8314b154668143c
Author: Benno Schulenberg <bensberg@telfort.nl>
Date: Wed, 3 Jun 2020 18:38:11 +0200
tweaks: elide an unused parameter, and rename the other and a variable
Also, regroup a few lines.
Diffstat:
1 file changed, 13 insertions(+), 11 deletions(-)
diff --git a/src/winio.c b/src/winio.c
@@ -811,17 +811,19 @@ int convert_sequence(const int *seq, size_t length, int *consumed)
/* Interpret the escape sequence in the keystroke buffer, the first
* character of which is kbinput. Assume that the keystroke buffer
* isn't empty, and that the initial escape has already been read in. */
-int parse_escape_sequence(WINDOW *win, int kbinput)
+int parse_escape_sequence(int firstbyte)
{
- int retval, *sequence, length, consumed;
+ int *sequence, length, consumed, keycode;
- /* Put back the non-escape code, then grab at most six integers
- * (the longest possible escape sequence) from the keybuffer and
- * translate the sequence into its corresponding keycode. */
- put_back(kbinput);
+ /* Put the first character of the sequence back into the keybuffer. */
+ put_back(firstbyte);
+
+ /* Grab at most six integers (the longest possible escape sequence)
+ * from the keybuffer. */
length = (key_buffer_len < 6 ? key_buffer_len : 6);
sequence = get_input(NULL, length);
- retval = convert_sequence(sequence, length, &consumed);
+
+ keycode = convert_sequence(sequence, length, &consumed);
/* If not all grabbed integers were consumed, put the leftovers back. */
for (int i = length - 1; i >= consumed; i--)
@@ -829,7 +831,7 @@ int parse_escape_sequence(WINDOW *win, int kbinput)
free(sequence);
- return retval;
+ return keycode;
}
/* Extract a single keystroke from the input stream. Translate escape
@@ -892,7 +894,7 @@ int parse_kbinput(WINDOW *win)
} else
/* One escape followed by a non-escape, and there
* are more codes waiting: escape sequence mode. */
- retval = parse_escape_sequence(win, keycode);
+ retval = parse_escape_sequence(keycode);
escapes = 0;
break;
case 2:
@@ -983,7 +985,7 @@ int parse_kbinput(WINDOW *win)
} else {
/* Two escapes followed by a non-escape, and there are more
* codes waiting: combined meta and escape sequence mode. */
- retval = parse_escape_sequence(win, keycode);
+ retval = parse_escape_sequence(keycode);
meta_key = TRUE;
escapes = 0;
}
@@ -1003,7 +1005,7 @@ int parse_kbinput(WINDOW *win)
* escape sequence mode. First interpret the escape
* sequence, then the result as a control sequence. */
retval = get_control_kbinput(
- parse_escape_sequence(win, keycode));
+ parse_escape_sequence(keycode));
escapes = 0;
break;
}