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

COOKED_READ: Fix tab not erasing the prompt #16718

Merged
merged 1 commit into from Feb 21, 2024
Merged

Conversation

lhecker
Copy link
Member

@lhecker lhecker commented Feb 15, 2024

Write "<command that doesn't exist> foo" in cmd.exe, move yours cursor
past the and press tab. The "foo" will still be there but
will be inaccessible. This commit fixes the issue. As far as I can
tell, this never worked in any conhost version ever.

Closes #16704

@microsoft-github-policy-service microsoft-github-policy-service bot added Issue-Bug It either shouldn't be doing this or needs an investigation. Area-CookedRead The cmd.exe COOKED_READ handling Priority-3 A description (P3) Product-Conhost For issues in the Console codebase labels Feb 15, 2024
@DHowett DHowett added this to To Cherry Pick in 1.19 Servicing Pipeline via automation Feb 15, 2024
@DHowett DHowett added this to To Cherry Pick in 1.20 Servicing Pipeline via automation Feb 15, 2024
@DHowett DHowett added the Needs-Second It's a PR that needs another sign-off label Feb 21, 2024
@DHowett DHowett enabled auto-merge (squash) February 21, 2024 17:52
@DHowett DHowett merged commit 5e9f223 into main Feb 21, 2024
20 checks passed
@DHowett DHowett deleted the dev/lhecker/16704-cooked branch February 21, 2024 17:53
@microsoft-github-policy-service microsoft-github-policy-service bot removed the Needs-Second It's a PR that needs another sign-off label Feb 21, 2024
@DHowett DHowett moved this from To Cherry Pick to Cherry Picked in 1.20 Servicing Pipeline Feb 21, 2024
DHowett pushed a commit that referenced this pull request Feb 21, 2024
Write "<command that doesn't exist> foo" in cmd.exe, move yours cursor
past the <command> and press tab. The "foo" will still be there but
will be inaccessible. This commit fixes the issue. As far as I can
tell, this never worked in any conhost version ever.

Closes #16704

(cherry picked from commit 5e9f223)
Service-Card-Id: 91851715
Service-Version: 1.20
@DHowett DHowett moved this from To Cherry Pick to Cherry Picked in 1.19 Servicing Pipeline Feb 21, 2024
DHowett pushed a commit that referenced this pull request Feb 21, 2024
Write "<command that doesn't exist> foo" in cmd.exe, move yours cursor
past the <command> and press tab. The "foo" will still be there but
will be inaccessible. This commit fixes the issue. As far as I can
tell, this never worked in any conhost version ever.

Closes #16704

(cherry picked from commit 5e9f223)
Service-Card-Id: 91851714
Service-Version: 1.19
@DHowett DHowett moved this from Cherry Picked to Shipped in 1.20 Servicing Pipeline Mar 8, 2024
@DHowett DHowett moved this from Cherry Picked to Shipped in 1.19 Servicing Pipeline Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-CookedRead The cmd.exe COOKED_READ handling Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-3 A description (P3) Product-Conhost For issues in the Console codebase
Development

Successfully merging this pull request may close these issues.

COOKED_READ: cmd.exe tab fails to erase trailing characters
3 participants