Some newlines appear to be broken #5764

Closed
opened 2026-01-31 00:21:05 +00:00 by claunia · 5 comments
Owner

Originally created by @DHowett-MSFT on GitHub (Jan 3, 2020).

As an example, the one you see when PowerShell starts up:

bad

image

good

image

Originally created by @DHowett-MSFT on GitHub (Jan 3, 2020). As an example, the one you see when PowerShell starts up: ## bad ![image](https://user-images.githubusercontent.com/14316954/71752084-96b66600-2e32-11ea-931d-319e06661b2f.png) ## good ![image](https://user-images.githubusercontent.com/14316954/71752088-9b7b1a00-2e32-11ea-89fd-f15f05707e88.png)
Author
Owner

@DHowett-MSFT commented on GitHub (Jan 3, 2020):

Looks like only WT...

@DHowett-MSFT commented on GitHub (Jan 3, 2020): Looks like only WT...
Author
Owner

@zadjii-msft commented on GitHub (Jan 4, 2020):

Is this a recent regression? I would have thought it might be #3271 but that hasn't merged yet.

@zadjii-msft commented on GitHub (Jan 4, 2020): Is this a recent regression? I would have thought it might be #3271 but that hasn't merged yet.
Author
Owner

@DHowett-MSFT commented on GitHub (Jan 4, 2020):

Very. The current selfhost build from 12/31 does not exhibit this issue. Maybe related to the audit mode refactoring?

@DHowett-MSFT commented on GitHub (Jan 4, 2020): Very. The current selfhost build from 12/31 does not exhibit this issue. Maybe related to the audit mode refactoring?
Author
Owner

@j4james commented on GitHub (Jan 5, 2020):

I believe the problem is in the TerminalDispatch::CursorPosition method here:
d711d731d7/src/cascadia/TerminalCore/TerminalDispatch.cpp (L40-L44)

If the SizeTToShort calls succeed, then the FAILED tests are going to be false, and the RETURN_BOOL_IF_FALSE will force the method to abort.

@j4james commented on GitHub (Jan 5, 2020): I believe the problem is in the `TerminalDispatch::CursorPosition` method here: https://github.com/microsoft/terminal/blob/d711d731d7d00259b2e0fe5847928c9f1c1121ed/src/cascadia/TerminalCore/TerminalDispatch.cpp#L40-L44 If the `SizeTToShort` calls succeed, then the `FAILED` tests are going to be false, and the `RETURN_BOOL_IF_FALSE` will force the method to abort.
Author
Owner

@DHowett-MSFT commented on GitHub (Jan 5, 2020):

Oof. Thanks.

@DHowett-MSFT commented on GitHub (Jan 5, 2020): Oof. Thanks.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#5764