[PR #211] [MERGED] Ide: Fix compilation error and warnings related to MSVC 22 #233

Closed
opened 2026-05-05 03:43:31 -06:00 by gitea-mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ultimatepp/ultimatepp/pull/211
Author: @klugier
Created: 10/22/2024
Status: Merged
Merged: 10/22/2024
Merged by: @mirek-fidler

Base: masterHead: klugier/msvc22-ide-compilation-fixes


📝 Commits (2)

  • ca7b2a5 Ide: Fix compilation error and warnings related to MSVC 22
  • 1eabed6 Ide: instant setup now detects llvm include and bin directory for MSVC to compile TheIDE out of the box.

📊 Changes

6 files changed (+20 additions, -12 deletions)

View changed files

📝 uppsrc/ide/InstantSetup.cpp (+10 -2)
📝 uppsrc/ide/LayDes/LayDes.h (+3 -3)
📝 uppsrc/ide/LayDes/laydes.cpp (+3 -3)
📝 uppsrc/ide/LayDes/laywin.cpp (+2 -2)
📝 uppsrc/ide/clang/Visitor.cpp (+1 -1)
📝 uppsrc/ide/ide.cpp (+1 -1)

📄 Description

TheIDE doesn't compile with the latest stable MSVC. This PR address this problem. Also, I decieded to change instant setup for MSVC to configure llvm dir by default. With that change MSVC should compile TheIDE out of the box, without the need for manual configuration.


🔄 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/ultimatepp/ultimatepp/pull/211 **Author:** [@klugier](https://github.com/klugier) **Created:** 10/22/2024 **Status:** ✅ Merged **Merged:** 10/22/2024 **Merged by:** [@mirek-fidler](https://github.com/mirek-fidler) **Base:** `master` ← **Head:** `klugier/msvc22-ide-compilation-fixes` --- ### 📝 Commits (2) - [`ca7b2a5`](https://github.com/ultimatepp/ultimatepp/commit/ca7b2a55e8067285893cffbb0d73c66ddcd8a3dc) Ide: Fix compilation error and warnings related to MSVC 22 - [`1eabed6`](https://github.com/ultimatepp/ultimatepp/commit/1eabed64b66dbf3a1cb4d697e6378c05034f3154) Ide: instant setup now detects llvm include and bin directory for MSVC to compile TheIDE out of the box. ### 📊 Changes **6 files changed** (+20 additions, -12 deletions) <details> <summary>View changed files</summary> 📝 `uppsrc/ide/InstantSetup.cpp` (+10 -2) 📝 `uppsrc/ide/LayDes/LayDes.h` (+3 -3) 📝 `uppsrc/ide/LayDes/laydes.cpp` (+3 -3) 📝 `uppsrc/ide/LayDes/laywin.cpp` (+2 -2) 📝 `uppsrc/ide/clang/Visitor.cpp` (+1 -1) 📝 `uppsrc/ide/ide.cpp` (+1 -1) </details> ### 📄 Description TheIDE doesn't compile with the latest stable MSVC. This PR address this problem. Also, I decieded to change instant setup for MSVC to configure llvm dir by default. With that change MSVC should compile TheIDE out of the box, without the need for manual configuration. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
gitea-mirror 2026-05-05 03:43:31 -06:00
Sign in to join this conversation.
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/ultimatepp#233
No description provided.