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 handling of surrogate pairs in length calculation #4462

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

totkeks
Copy link

@totkeks totkeks commented Feb 7, 2025

PR Summary

Adds new conditions to handle surrogate pairs when calculating the buffer length of a string.

Fixes #3857.

PR Checklist

  • PR has a meaningful title
    • Use the present tense and imperative mood when describing your changes
  • Summarized changes
  • Make sure you've added one or more new tests
  • Make sure you've tested these changes in terminals that PowerShell is commonly used in (i.e. conhost.exe, Windows Terminal, Visual Studio Code Integrated Terminal, etc.)
  • User-facing changes
    • Not Applicable
Microsoft Reviewers: Open in CodeFlow

@totkeks
Copy link
Author

totkeks commented Feb 8, 2025

@microsoft-github-policy-service agree

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.

Using an emoji as the prompt text parse error replacement causes glitchy text to remain in the text buffer
1 participant