Aren/rulimine
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #178 from lightning-speed/patch-2

Browse Source 
Replaced Multiple strlen(buffer) call with one call
This commit is contained in:
mint 2022-06-19 20:29:59 +02:00 committed by GitHub
commit dc3de36c7e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
common/menu.c
View File

@ -412,6 +412,7 @@ refresh:
term_double_buffer_flush();
int c = getchar();
size_t buffer_len = strlen(buffer);
switch (c) {
case GETCHAR_CURSOR_DOWN:
cursor_offset = get_next_line(cursor_offset, buffer);
@ -425,7 +426,7 @@ refresh:
}
break;
case GETCHAR_CURSOR_RIGHT:
if (cursor_offset < strlen(buffer)) {
if (cursor_offset < buffer_len) {
cursor_offset++;
}
break;
@ -458,9 +459,9 @@ refresh:
editor_no_term_reset ? editor_no_term_reset = false : reset_term();
return NULL;
default:
if (strlen(buffer) < EDITOR_MAX_BUFFER_SIZE - 1) {
if (buffer_len < EDITOR_MAX_BUFFER_SIZE - 1) {
if (isprint(c) || c == '\n') {
for (size_t i = strlen(buffer); ; i--) {
for (size_t i = buffer_len; ; i--) {
buffer[i+1] = buffer[i];
if (i == cursor_offset)
break;