Remove CONSOLE_API_MSG::UpdateUserBufferPointers hack #13782

Closed
opened 2026-01-31 03:51:56 +00:00 by claunia · 1 comment
Owner

Originally created by @DHowett on GitHub (May 11, 2021).

Originally assigned to: @lhecker on GitHub.

I didn't want to write a copy constructor for CONSOLE_API_MSG while fixing #9692, because we were taking the fix up through the OS to conhost and due to the development cycle it needed to be minimal and cheaply reviewable.

@lhecker wrote a copy constructor for CONSOLE_API_MSG that leverages the legality of memcpy()ing a union to avoid us having to name that union.

Originally created by @DHowett on GitHub (May 11, 2021). Originally assigned to: @lhecker on GitHub. I didn't want to write a copy constructor for CONSOLE_API_MSG while fixing #9692, because we were taking the fix up through the OS to conhost and due to the development cycle it needed to be minimal and cheaply reviewable. @lhecker wrote a copy constructor for CONSOLE_API_MSG that leverages the legality of memcpy()ing a union to avoid us having to _name_ that union.
Author
Owner

@ghost commented on GitHub (Jul 14, 2021):

:tada:This issue was addressed in #10326, which has now been successfully released as Windows Terminal Preview v1.10.1933.0.🎉

Handy links:

@ghost commented on GitHub (Jul 14, 2021): :tada:This issue was addressed in #10326, which has now been successfully released as `Windows Terminal Preview v1.10.1933.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.10.1933.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#13782