Skip to content

Fix completion index assertion error#2531

Merged
kcgen merged 1 commit intodosbox-staging:mainfrom
MeAreJeenius:completion-assert-fix
May 25, 2023
Merged

Fix completion index assertion error#2531
kcgen merged 1 commit intodosbox-staging:mainfrom
MeAreJeenius:completion-assert-fix

Conversation

@MeAreJeenius
Copy link
Copy Markdown
Contributor

Fixes #2530
Just an ill-formed assertion, nothing wrong with the actual function.

@MeAreJeenius MeAreJeenius self-assigned this May 25, 2023
@MeAreJeenius MeAreJeenius added the regression We broke something 😊 label May 25, 2023
@kcgen kcgen self-requested a review May 25, 2023 04:47
Copy link
Copy Markdown
Member

@kcgen kcgen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @MeAreJeenius 🙂

@kcgen kcgen merged commit a3da2ac into dosbox-staging:main May 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

regression We broke something 😊

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

Assertion catches issue when backspacing after Tab-expanded name

2 participants