[PR #587] [CLOSED] Fix VS 2019 (Toolset 142) build #24196

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/587
Author: @ArcEarth
Created: 5/8/2019
Status: Closed

Base: masterHead: dev/arc/fix_vc_142_vs2019_build


📝 Commits (1)

  • d1c3d9e Add the missing header to PCH

📊 Changes

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

View changed files

📝 src/inc/LibraryIncludes.h (+1 -0)

📄 Description

The code used std::function<...> without include <functional> header.
Add the missing header to PCH.

  • Note, <functional> was included by <string> in the STL come with VS 2017 (Toolset 141).
  • The STL team restructured the code, thus now we need to include it directly.

🔄 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/587 **Author:** [@ArcEarth](https://github.com/ArcEarth) **Created:** 5/8/2019 **Status:** ❌ Closed **Base:** `master` ← **Head:** `dev/arc/fix_vc_142_vs2019_build` --- ### 📝 Commits (1) - [`d1c3d9e`](https://github.com/microsoft/terminal/commit/d1c3d9eab5fb596f1736fd692f405cb6c6628444) Add the missing <functional> header to PCH ### 📊 Changes **1 file changed** (+1 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `src/inc/LibraryIncludes.h` (+1 -0) </details> ### 📄 Description The code used `std::function<...>` without include `<functional>` header. Add the missing <functional> header to PCH. * Note, `<functional>` was included by `<string>` in the STL come with VS 2017 (Toolset 141). * The STL team restructured the code, thus now we need to include it directly. --- <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:01:55 +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#24196