Merge pull request #927 from armink/fix_shell

[Finsh] Fix an auto complete history bug.
This commit is contained in:
Bernard Xiong 2017-11-01 13:44:56 +08:00 committed by GitHub
commit 832d8b4e04
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions

View File

@ -372,7 +372,7 @@ static void shell_push_history(struct finsh_shell *shell)
if (shell->history_count >= FINSH_HISTORY_LINES) if (shell->history_count >= FINSH_HISTORY_LINES)
{ {
/* if current cmd is same as last cmd, don't push */ /* if current cmd is same as last cmd, don't push */
if (memcmp(&shell->cmd_history[FINSH_HISTORY_LINES - 1], shell->line, shell->line_position)) if (memcmp(&shell->cmd_history[FINSH_HISTORY_LINES - 1], shell->line, FINSH_CMD_SIZE))
{ {
/* move history */ /* move history */
int index; int index;
@ -391,7 +391,7 @@ static void shell_push_history(struct finsh_shell *shell)
else else
{ {
/* if current cmd is same as last cmd, don't push */ /* if current cmd is same as last cmd, don't push */
if (shell->history_count == 0 || memcmp(&shell->cmd_history[shell->history_count - 1], shell->line, shell->line_position)) if (shell->history_count == 0 || memcmp(&shell->cmd_history[shell->history_count - 1], shell->line, FINSH_CMD_SIZE))
{ {
shell->current_history = shell->history_count; shell->current_history = shell->history_count;
memset(&shell->cmd_history[shell->history_count][0], 0, FINSH_CMD_SIZE); memset(&shell->cmd_history[shell->history_count][0], 0, FINSH_CMD_SIZE);