Skip to content

Comments

Fix auto complete and clear terminal#11

Open
xmow49 wants to merge 2 commits intodimmykar:developfrom
xmow49:develop
Open

Fix auto complete and clear terminal#11
xmow49 wants to merge 2 commits intodimmykar:developfrom
xmow49:develop

Conversation

@xmow49
Copy link
Contributor

@xmow49 xmow49 commented Jul 23, 2025

Hello,

I found a bug when using the auto-complete feature:
If no result is found, the auto-complete returns directly:

    if (cmplt_tkn_arr[0] == NULL) {
        return microrlERRCPLT;
    }

So after the return, the command no longer works because the command string is still separated by 0.

I added the whitespace conversion again before the return.
Now, if no result is found, the command is not edited.

I also fixed the clear function with an argument to specify if we want to clear the current command.

Thanks

commit:

  • fix: auto-complete restore whitespaces on error
  • fix: terminal clear with a reset argument

@xmow49
Copy link
Contributor Author

xmow49 commented Sep 1, 2025

Hi,
Just following up, have you had a chance to review this PR?

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant