Skip to content

Update MSVC compiler options and CMake configuration for 1.5.1#457

Closed
j2doll wants to merge 7 commits into
QtExcel:masterfrom
j2doll:master
Closed

Update MSVC compiler options and CMake configuration for 1.5.1#457
j2doll wants to merge 7 commits into
QtExcel:masterfrom
j2doll:master

Conversation

@j2doll

@j2doll j2doll commented Jun 16, 2026

Copy link
Copy Markdown
Member

This pull request updates build configuration and compatibility for the QXlsx project, with a focus on improving C++ standard handling, supporting MSVC builds, and updating versioning. The changes enhance cross-platform compatibility and modernize the build process.

Build system and compatibility improvements:

  • Updated the project version from 1.5.0 to 1.5.1 in QXlsx/CMakeLists.txt to reflect the latest changes.
  • Added logic in QXlsx/QXlsx.pri to automatically set the C++ standard: uses C++11 for Qt 5 and C++17 for Qt 6, improving compatibility and modernization.
  • Added a conditional in QXlsx/CMakeLists.txt to force the /permissive flag for MSVC builds, ensuring compatibility with certain C++ standards and suppressing related build errors.
  • Added a macro in QXlsx/header/xlsxglobal.h to define stdext for MSVC compilers, improving compatibility with Microsoft's STL extensions.

Code and configuration cleanup:

  • Removed unused hard-coded QXlsx version string from ExcelViewer/mainwindow.cpp and cleaned up redundant header/source references in QXlsx/QXlsx.pro. [1] [2]

j2doll and others added 7 commits June 17, 2026 02:54
- Disable strict MSVC compliance mode to prevent build errors  caused by standard library ('stdext') changes in newer MSVC compilers with older Qt headers.
Replace MSVC compliance mode disabling with a definition to allow compiler and STL version mismatch.
Updated CMakeLists.txt to modify compiler options for MSVC.
Removed MSVC specific flags for permissive mode.
@j2doll j2doll closed this Jun 16, 2026
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant