We log entirely too frequently from Terminal::CursorLineFeed #14801

Open
opened 2026-01-31 04:19:51 +00:00 by claunia · 0 comments
Owner

Originally created by @zadjii-msft on GitHub (Aug 5, 2021).

This is a follow-up for an internal bug. There's two pieces here:

  • (A) Change the CATCH_LOG_RETURN_FALSE to a newly authored CATCH_LOG_RETURN_FALSE_EXPECTED. We're expecting this exception, so make it less noisy. This will fix MSFT:33310649
  • (B) Get to the bottom of why Terminal::CursorLineFeed is throwing so often. It's an E_ABORT from rectangle.h:840, though that line of code is nothing in main, so we might need to look at what it is in 1.9. Presumably something in _AdjustCursorPosition throws, though impossible to say what exactly.
Originally created by @zadjii-msft on GitHub (Aug 5, 2021). This is a follow-up for an internal bug. There's two pieces here: * [x] (**A**) Change the `CATCH_LOG_RETURN_FALSE` to a newly authored `CATCH_LOG_RETURN_FALSE_EXPECTED`. We're expecting this exception, so make it less noisy. This will fix MSFT:33310649 - Done in #10901 * [ ] (**B**) Get to the bottom of why `Terminal::CursorLineFeed` is throwing so often. It's an `E_ABORT` from `rectangle.h:840`, though that line of code is nothing in main, so we might need to look at what it is in 1.9. Presumably something in `_AdjustCursorPosition` throws, though impossible to say what exactly.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#14801