Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix cursor invalidation, again #17617

Merged
merged 1 commit into from
Jul 26, 2024

Conversation

lhecker
Copy link
Member

@lhecker lhecker commented Jul 25, 2024

Regressed in #15500, incorrectly fixed in #17332, exposed by #17583.
My ineptitude on full display. If this isn't the last cursor
invalidation bug I'm going to cry.

Closes #17615

Validation Steps Performed

  • cmd.exe
  • a directory with 6 files
  • 80x24 viewport
  • run cls
  • run dir twice

Copy link
Member

@DHowett DHowett left a comment

Choose a reason for hiding this comment

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

🥲

@lhecker lhecker merged commit 295cd17 into main Jul 26, 2024
21 of 25 checks passed
@lhecker lhecker deleted the dev/lhecker/17510-cursor-invalidation-attempt-4721 branch July 26, 2024 18:09
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.

Cursor droppings in WT cmd shell
3 participants