[PR #11486] [MERGED] Remove debug logging for PSReadline on newlines #28604

Open
opened 2026-01-31 09:29:36 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/11486
Author: @lhecker
Created: 10/12/2021
Status: Merged
Merged: 10/18/2021
Merged by: @carlos-zamora

Base: mainHead: dev/lhecker/debug-log-pwsh


📝 Commits (1)

  • c1e0b43 Remove debug logging for PSReadline on newlines

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 src/host/getset.cpp (+1 -1)

📄 Description

Up until this commit PSReadline caused OutputDebugString to be called
with a complex log message to on every newline. At the time of writing,
Visual Studio's Output window is fairly slow and after this change newlines
feel a fair bit snappier when running under Visual Studio's debugger.

Validation Steps Performed

  • pwsh.exe continues to work correctly ✔️

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/11486 **Author:** [@lhecker](https://github.com/lhecker) **Created:** 10/12/2021 **Status:** ✅ Merged **Merged:** 10/18/2021 **Merged by:** [@carlos-zamora](https://github.com/carlos-zamora) **Base:** `main` ← **Head:** `dev/lhecker/debug-log-pwsh` --- ### 📝 Commits (1) - [`c1e0b43`](https://github.com/microsoft/terminal/commit/c1e0b439a044a16114b908c474c88d9d58e9b9f2) Remove debug logging for PSReadline on newlines ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `src/host/getset.cpp` (+1 -1) </details> ### 📄 Description Up until this commit PSReadline caused OutputDebugString to be called with a complex log message to on every newline. At the time of writing, Visual Studio's Output window is fairly slow and after this change newlines feel a fair bit snappier when running under Visual Studio's debugger. ## Validation Steps Performed * pwsh.exe continues to work correctly ✔️ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:29:36 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#28604