[PR #14869] [MERGED] Move to CppWinRT 2.0.230207.1 #30285

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/14869
Author: @DHowett
Created: 2/17/2023
Status: Merged
Merged: 2/18/2023
Merged by: @DHowett

Base: mainHead: dev/duhowett/2023-compiler/4-cppwinrt


📝 Commits (3)

  • 83cd359 Move to CppWinRT 2.0.230207.1
  • 1b00fe2 Fix ambiguous conversions of til::color to W::U::Color
  • 378bc61 Clean up one co_await of an std::chrono::duration

📊 Changes

9 files changed (+11 additions, -11 deletions)

View changed files

📝 dep/nuget/packages.config (+1 -1)
📝 scratch/ScratchIslandApp/SampleApp/packages.config (+1 -1)
📝 scratch/ScratchIslandApp/WindowExe/packages.config (+1 -1)
📝 src/cascadia/TerminalApp/Pane.cpp (+1 -1)
📝 src/cascadia/TerminalControl/ControlCore.cpp (+1 -1)
📝 src/cascadia/TerminalSettingsModel/TerminalSettings.cpp (+1 -1)
📝 src/cascadia/WindowsTerminal/AppHost.cpp (+1 -1)
📝 src/common.nugetversions.props (+1 -1)
📝 src/common.nugetversions.targets (+3 -3)

📄 Description

Interesting things we could do after this:

  • remove all InitializeComponent calls - they do it automatically
  • have some Clang support (!)
  • use std::optional<->IReference automatic binding
  • use std::format support (!) for json/uri/hostname/http stuff/all IStringables
  • potentially move to /await:strict for C++20 coroutines

I've also fixed up a couple ambiguities introduced by this change.


🔄 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/14869 **Author:** [@DHowett](https://github.com/DHowett) **Created:** 2/17/2023 **Status:** ✅ Merged **Merged:** 2/18/2023 **Merged by:** [@DHowett](https://github.com/DHowett) **Base:** `main` ← **Head:** `dev/duhowett/2023-compiler/4-cppwinrt` --- ### 📝 Commits (3) - [`83cd359`](https://github.com/microsoft/terminal/commit/83cd3593e66f4596008cd3b8a260519777a8c5da) Move to CppWinRT 2.0.230207.1 - [`1b00fe2`](https://github.com/microsoft/terminal/commit/1b00fe284b26d15a459c3046033ac3955a8eb7b1) Fix ambiguous conversions of til::color to W::U::Color - [`378bc61`](https://github.com/microsoft/terminal/commit/378bc6110a15d26006400dae8240e68ed38afa5e) Clean up one co_await of an std::chrono::duration ### 📊 Changes **9 files changed** (+11 additions, -11 deletions) <details> <summary>View changed files</summary> 📝 `dep/nuget/packages.config` (+1 -1) 📝 `scratch/ScratchIslandApp/SampleApp/packages.config` (+1 -1) 📝 `scratch/ScratchIslandApp/WindowExe/packages.config` (+1 -1) 📝 `src/cascadia/TerminalApp/Pane.cpp` (+1 -1) 📝 `src/cascadia/TerminalControl/ControlCore.cpp` (+1 -1) 📝 `src/cascadia/TerminalSettingsModel/TerminalSettings.cpp` (+1 -1) 📝 `src/cascadia/WindowsTerminal/AppHost.cpp` (+1 -1) 📝 `src/common.nugetversions.props` (+1 -1) 📝 `src/common.nugetversions.targets` (+3 -3) </details> ### 📄 Description Interesting things we could do after this: - remove all `InitializeComponent` calls - they do it automatically - have some Clang support (!) - use `std::optional`<->`IReference` automatic binding - use `std::format` support (!) for json/uri/hostname/http stuff/all `IStringable`s - potentially move to `/await:strict` for C++20 coroutines I've also fixed up a couple ambiguities introduced by this change. --- <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:39:49 +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#30285