[PR #5292] [MERGED] Control: make sure we actually have a dispatcher in 100ms #26237

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/5292
Author: @DHowett-MSFT
Created: 4/9/2020
Status: Merged
Merged: 4/9/2020
Merged by: @DHowett-MSFT

Base: masterHead: dev/duhowett/f_f_s


📝 Commits (1)

  • 37da7df Make sure we actually have a dispatcher in 100ms

📊 Changes

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

View changed files

📝 src/cascadia/TerminalControl/TermControl.cpp (+2 -1)

📄 Description

This fixes a silly regression introduced by 1299a839b. We would crash
when the connection updated the cursor as it was being torn down.

It would have been good for me to test this better. Sorry.


🔄 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/5292 **Author:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Created:** 4/9/2020 **Status:** ✅ Merged **Merged:** 4/9/2020 **Merged by:** [@DHowett-MSFT](https://github.com/DHowett-MSFT) **Base:** `master` ← **Head:** `dev/duhowett/f_f_s` --- ### 📝 Commits (1) - [`37da7df`](https://github.com/microsoft/terminal/commit/37da7dfc6515b2dddd9e8ae35b0cb79bc50a2c7a) Make sure we actually have a dispatcher in 100ms ### 📊 Changes **1 file changed** (+2 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `src/cascadia/TerminalControl/TermControl.cpp` (+2 -1) </details> ### 📄 Description This fixes a silly regression introduced by 1299a839b. We would crash when the connection updated the cursor as it was being torn down. It would have been good for me to test this better. Sorry. --- <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:14:51 +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#26237