[PR #6226] [MERGED] Add stdexcept header explicitly to u8u16test tool #26601

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/6226
Author: @miniksa
Created: 5/27/2020
Status: Merged
Merged: 5/27/2020
Merged by: @miniksa

Base: masterHead: dev/miniksa/stl_changed


📝 Commits (1)

  • 5760e7e Add stdexcept header explicitly to u8u16test tool because it was implicitly included in STL 16.5 but no longer in STL 16.6.

📊 Changes

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

View changed files

📝 src/tools/U8U16Test/U8U16Test.hpp (+1 -0)

📄 Description

Summary of the Pull Request

Adds implicit stdexcept header include to u8u16test tool.

PR Checklist

  • Closes regression introduced when moving from VS 16.5 to VS 16.6 (which the CI did of its own accord)
  • I work here.
  • Built it.
  • No doc.
  • Am core contributor.

Detailed Description of the Pull Request / Additional comments

In VS 16.5, the header was pulled in by <string> or <string_view> or <array> or <algorithm> implicitly. In VS 16.6, that's gone. No one wrote it in the header because it was just automatically there in the past. Now I wrote it in the header.

Validation Steps Performed

  • Built it on my machine after upgrading to VS 16.6.0.
  • Built it in CI.

🔄 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/6226 **Author:** [@miniksa](https://github.com/miniksa) **Created:** 5/27/2020 **Status:** ✅ Merged **Merged:** 5/27/2020 **Merged by:** [@miniksa](https://github.com/miniksa) **Base:** `master` ← **Head:** `dev/miniksa/stl_changed` --- ### 📝 Commits (1) - [`5760e7e`](https://github.com/microsoft/terminal/commit/5760e7ed6067ea29383a86902ee823aae583b45b) Add stdexcept header explicitly to u8u16test tool because it was implicitly included in STL 16.5 but no longer in STL 16.6. ### 📊 Changes **1 file changed** (+1 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `src/tools/U8U16Test/U8U16Test.hpp` (+1 -0) </details> ### 📄 Description ## Summary of the Pull Request Adds implicit stdexcept header include to u8u16test tool. ## PR Checklist * [x] Closes regression introduced when moving from VS 16.5 to VS 16.6 (which the CI did of its own accord) * [x] I work here. * [x] Built it. * [x] No doc. * [x] Am core contributor. ## Detailed Description of the Pull Request / Additional comments In VS 16.5, the <stdexcept> header was pulled in by `<string>` or `<string_view>` or `<array>` or `<algorithm>` implicitly. In VS 16.6, that's gone. No one wrote it in the header because it was just automatically there in the past. Now I wrote it in the header. ## Validation Steps Performed * [x] Built it on my machine after upgrading to VS `16.6.0`. * [ ] Built it in CI. --- <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:17:05 +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#26601