summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenys Vlasenko2010-09-03 14:08:24 +0200
committerDenys Vlasenko2010-09-03 14:08:24 +0200
commitba0e103a662f640060041c5b56c2afdce09408b3 (patch)
tree513ca8b725d9f69c5d7cbd2c451e11983fa3e4bb
parenta46e16ef52952b1ed82ab1dbc4b9da2aeb40107e (diff)
downloadbusybox-ba0e103a662f640060041c5b56c2afdce09408b3.zip
busybox-ba0e103a662f640060041c5b56c2afdce09408b3.tar.gz
lineedit: de-indent large block in input_tab. No logic changes.
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
-rw-r--r--libbb/lineedit.c262
1 files changed, 131 insertions, 131 deletions
diff --git a/libbb/lineedit.c b/libbb/lineedit.c
index dda7024..ff5d284 100644
--- a/libbb/lineedit.c
+++ b/libbb/lineedit.c
@@ -924,7 +924,7 @@ static NOINLINE int build_match_prefix(char *matchBuf)
for (i = 0; int_buf[i]; i++) {
if (int_buf[i] == '(' || int_buf[i] == '{') {
remove_chunk(int_buf, 0, i + 1);
- i = -1; /* hack increment */
+ i = -1; /* back to square 1 */
}
}
@@ -1036,163 +1036,163 @@ static char *add_quote_for_spec_chars(char *found)
/* Do TAB completion */
static NOINLINE void input_tab(smallint *lastWasTab)
{
- if (!(state->flags & TAB_COMPLETION))
- return;
-
- if (!*lastWasTab) {
- char *chosen_match;
- size_t len_found;
-/* char matchBuf[MAX_LINELEN]; */
+ char *chosen_match;
+ size_t len_found;
+/* char matchBuf[MAX_LINELEN]; */
#define matchBuf (S.input_tab__matchBuf)
- /* Length of string used for matching */
- unsigned match_pfx_len = match_pfx_len;
- int find_type;
+ /* Length of string used for matching */
+ unsigned match_pfx_len = match_pfx_len;
+ int find_type;
#if ENABLE_UNICODE_SUPPORT
- /* cursor pos in command converted to multibyte form */
- int cursor_mb;
+ /* cursor pos in command converted to multibyte form */
+ int cursor_mb;
#endif
+ if (!(state->flags & TAB_COMPLETION))
+ return;
- *lastWasTab = 1;
+ if (*lastWasTab) {
+ /* The last char was a TAB too.
+ * Print a list of all the available choices.
+ */
+ if (num_matches > 0) {
+ /* cursor will be changed by goto_new_line() */
+ int sav_cursor = cursor;
+ goto_new_line();
+ showfiles();
+ redraw(0, command_len - sav_cursor);
+ }
+ return;
+ }
+
+ *lastWasTab = 1;
- /* Make a local copy of the string --
- * up to the position of the cursor */
+ /* Make a local copy of the string --
+ * up to the position of the cursor */
#if !ENABLE_UNICODE_SUPPORT
- save_string(matchBuf, cursor + 1);
+ save_string(matchBuf, cursor + 1);
#else
- {
- CHAR_T wc = command_ps[cursor];
- command_ps[cursor] = BB_NUL;
- save_string(matchBuf, MAX_LINELEN);
- command_ps[cursor] = wc;
- cursor_mb = strlen(matchBuf);
- }
+ {
+ CHAR_T wc = command_ps[cursor];
+ command_ps[cursor] = BB_NUL;
+ save_string(matchBuf, MAX_LINELEN);
+ command_ps[cursor] = wc;
+ cursor_mb = strlen(matchBuf);
+ }
#endif
- find_type = build_match_prefix(matchBuf);
+ find_type = build_match_prefix(matchBuf);
- /* Free up any memory already allocated */
- free_tab_completion_data();
+ /* Free up any memory already allocated */
+ free_tab_completion_data();
#if ENABLE_FEATURE_USERNAME_COMPLETION
- /* If the word starts with `~' and there is no slash in the word,
- * then try completing this word as a username. */
- if (state->flags & USERNAME_COMPLETION)
- if (matchBuf[0] == '~' && strchr(matchBuf, '/') == NULL)
- match_pfx_len = complete_username(matchBuf);
+ /* If the word starts with `~' and there is no slash in the word,
+ * then try completing this word as a username. */
+ if (state->flags & USERNAME_COMPLETION)
+ if (matchBuf[0] == '~' && strchr(matchBuf, '/') == NULL)
+ match_pfx_len = complete_username(matchBuf);
#endif
- /* Try to match a command in $PATH, or a directory, or a file */
- if (!matches)
- match_pfx_len = complete_cmd_dir_file(matchBuf, find_type);
- /* Remove duplicates */
- if (matches) {
- unsigned i;
- unsigned n = 0;
- qsort_string_vector(matches, num_matches);
- for (i = 0; i < num_matches - 1; ++i) {
- //if (matches[i] && matches[i+1]) { /* paranoia */
- if (strcmp(matches[i], matches[i+1]) == 0) {
- free(matches[i]);
- //matches[i] = NULL; /* paranoia */
- } else {
- matches[n++] = matches[i];
- }
- //}
- }
- matches[n++] = matches[i];
- num_matches = n;
+ /* Try to match a command in $PATH, or a directory, or a file */
+ if (!matches)
+ match_pfx_len = complete_cmd_dir_file(matchBuf, find_type);
+ /* Remove duplicates */
+ if (matches) {
+ unsigned i;
+ unsigned n = 0;
+ qsort_string_vector(matches, num_matches);
+ for (i = 0; i < num_matches - 1; ++i) {
+ //if (matches[i] && matches[i+1]) { /* paranoia */
+ if (strcmp(matches[i], matches[i+1]) == 0) {
+ free(matches[i]);
+ //matches[i] = NULL; /* paranoia */
+ } else {
+ matches[n++] = matches[i];
+ }
+ //}
}
- /* Did we find exactly one match? */
- if (num_matches != 1) { /* no */
- char *cp;
- beep();
- if (!matches)
- return; /* no matches at all */
- /* Find common prefix */
- chosen_match = xstrdup(matches[0]);
- for (cp = chosen_match; *cp; cp++) {
- unsigned n;
- for (n = 1; n < num_matches; n++) {
- if (matches[n][cp - chosen_match] != *cp) {
- goto stop;
- }
+ matches[n++] = matches[i];
+ num_matches = n;
+ }
+ /* Did we find exactly one match? */
+ if (num_matches != 1) { /* no */
+ char *cp;
+ beep();
+ if (!matches)
+ return; /* no matches at all */
+ /* Find common prefix */
+ chosen_match = xstrdup(matches[0]);
+ for (cp = chosen_match; *cp; cp++) {
+ unsigned n;
+ for (n = 1; n < num_matches; n++) {
+ if (matches[n][cp - chosen_match] != *cp) {
+ goto stop;
}
}
+ }
stop:
- if (cp == chosen_match) { /* have unique prefix? */
- free(chosen_match); /* no */
- return;
- }
- *cp = '\0';
- cp = add_quote_for_spec_chars(chosen_match);
- free(chosen_match);
- chosen_match = cp;
- len_found = strlen(chosen_match);
- } else { /* exactly one match */
- /* Next <tab> is not a double-tab */
- *lastWasTab = 0;
-
- chosen_match = add_quote_for_spec_chars(matches[0]);
- len_found = strlen(chosen_match);
- if (chosen_match[len_found-1] != '/') {
- chosen_match[len_found] = ' ';
- chosen_match[++len_found] = '\0';
- }
+ if (cp == chosen_match) { /* have unique prefix? */
+ free(chosen_match); /* no */
+ return;
}
+ *cp = '\0';
+ cp = add_quote_for_spec_chars(chosen_match);
+ free(chosen_match);
+ chosen_match = cp;
+ len_found = strlen(chosen_match);
+ } else { /* exactly one match */
+ /* Next <tab> is not a double-tab */
+ *lastWasTab = 0;
+
+ chosen_match = add_quote_for_spec_chars(matches[0]);
+ len_found = strlen(chosen_match);
+ if (chosen_match[len_found-1] != '/') {
+ chosen_match[len_found] = ' ';
+ chosen_match[++len_found] = '\0';
+ }
+ }
#if !ENABLE_UNICODE_SUPPORT
+ /* Have space to place the match? */
+ /* The result consists of three parts with these lengths: */
+ /* cursor + (len_found - match_pfx_len) + (command_len - cursor) */
+ /* it simplifies into: */
+ if ((int)(len_found - match_pfx_len + command_len) < S.maxsize) {
+ int pos;
+ /* save tail */
+ strcpy(matchBuf, &command_ps[cursor]);
+ /* add match and tail */
+ sprintf(&command_ps[cursor], "%s%s", chosen_match + match_pfx_len, matchBuf);
+ command_len = strlen(command_ps);
+ /* new pos */
+ pos = cursor + len_found - match_pfx_len;
+ /* write out the matched command */
+ redraw(cmdedit_y, command_len - pos);
+ }
+#else
+ {
+ char command[MAX_LINELEN];
+ int len = save_string(command, sizeof(command));
/* Have space to place the match? */
- /* The result consists of three parts with these lengths: */
- /* cursor + (len_found - match_pfx_len) + (command_len - cursor) */
- /* it simplifies into: */
- if ((int)(len_found - match_pfx_len + command_len) < S.maxsize) {
+ /* cursor_mb + (len_found - match_pfx_len) + (len - cursor_mb) */
+ if ((int)(len_found - match_pfx_len + len) < MAX_LINELEN) {
int pos;
/* save tail */
- strcpy(matchBuf, &command_ps[cursor]);
+ strcpy(matchBuf, &command[cursor_mb]);
+ /* where do we want to have cursor after all? */
+ strcpy(&command[cursor_mb], chosen_match + match_pfx_len);
+ len = load_string(command, S.maxsize);
/* add match and tail */
- sprintf(&command_ps[cursor], "%s%s", chosen_match + match_pfx_len, matchBuf);
- command_len = strlen(command_ps);
- /* new pos */
- pos = cursor + len_found - match_pfx_len;
+ sprintf(&command[cursor_mb], "%s%s", chosen_match + match_pfx_len, matchBuf);
+ command_len = load_string(command, S.maxsize);
/* write out the matched command */
- redraw(cmdedit_y, command_len - pos);
- }
-#else
- {
- char command[MAX_LINELEN];
- int len = save_string(command, sizeof(command));
- /* Have space to place the match? */
- /* cursor_mb + (len_found - match_pfx_len) + (len - cursor_mb) */
- if ((int)(len_found - match_pfx_len + len) < MAX_LINELEN) {
- int pos;
- /* save tail */
- strcpy(matchBuf, &command[cursor_mb]);
- /* where do we want to have cursor after all? */
- strcpy(&command[cursor_mb], chosen_match + match_pfx_len);
- len = load_string(command, S.maxsize);
- /* add match and tail */
- sprintf(&command[cursor_mb], "%s%s", chosen_match + match_pfx_len, matchBuf);
- command_len = load_string(command, S.maxsize);
- /* write out the matched command */
- /* paranoia: load_string can return 0 on conv error,
- * prevent passing pos = (0 - 12) to redraw */
- pos = command_len - len;
- redraw(cmdedit_y, pos >= 0 ? pos : 0);
- }
+ /* paranoia: load_string can return 0 on conv error,
+ * prevent passing pos = (0 - 12) to redraw */
+ pos = command_len - len;
+ redraw(cmdedit_y, pos >= 0 ? pos : 0);
}
+ }
#endif
- free(chosen_match);
+ free(chosen_match);
#undef matchBuf
- } else {
- /* Ok -- the last char was a TAB. Since they
- * just hit TAB again, print a list of all the
- * available choices... */
- if (num_matches > 0) {
- /* cursor will be changed by goto_new_line() */
- int sav_cursor = cursor;
- goto_new_line();
- showfiles();
- redraw(0, command_len - sav_cursor);
- }
- }
}
#endif /* FEATURE_TAB_COMPLETION */