Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MSVC C++20 Support #126

Merged
merged 20 commits into from
Nov 13, 2024
Merged

MSVC C++20 Support #126

Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
7ab7a4f
Updating cmake for fixes.
sean-parent Nov 6, 2024
900f620
Update CMake configuration and improve code compatibility with C++20.…
sean-parent Nov 7, 2024
5dd244f
git clang-format
sean-parent Nov 7, 2024
b670fc3
Complete docs for macro_utilities.hpp
sean-parent Nov 7, 2024
58c457b
Fixing sscanf_s issue.
sean-parent Nov 7, 2024
d68084d
Adding C++20 support for macos
sean-parent Nov 7, 2024
2e78a81
Windows doesn't support __STDC_LIB_EXT1__ macro... sigh.
sean-parent Nov 7, 2024
622afeb
Fixing another warning in CI
sean-parent Nov 7, 2024
2e8b826
Removing variable only used in debug.
sean-parent Nov 7, 2024
0f69ed8
Hack to fix C++17 build.
sean-parent Nov 8, 2024
c93d480
Left in debugging code!
sean-parent Nov 8, 2024
32d3c7b
Addressing PR comments
sean-parent Nov 12, 2024
8888b16
More PR feedback
sean-parent Nov 12, 2024
57e3f22
Fixing Issue#119 and more PR cleanup.
sean-parent Nov 12, 2024
01ff792
Fixing name lookup issue with operator<< and array_t.
sean-parent Nov 13, 2024
7a61a75
More PR comments addressed.
sean-parent Nov 13, 2024
1cb0b49
Two step removal.
sean-parent Nov 13, 2024
7d3c408
Don't sink vscode settings.
sean-parent Nov 13, 2024
a46be16
Using generate in eve.cpp and some boost removal.
sean-parent Nov 13, 2024
ef7ca40
Update adobe/macro_utilities.hpp
sean-parent Nov 13, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 20 additions & 8 deletions .github/matrix.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,40 +5,52 @@
"compiler": "gcc",
"version": "11",
"os": "ubuntu-22.04",
"cxxstd" : "17"
"cxxstd": "17"
},
{
"name": "Linux GCC 11 C++20",
"compiler": "gcc",
"version": "11",
"os": "ubuntu-22.04",
"cxxstd" : "20"
"cxxstd": "20"
},
{
"name": "Linux Clang 14 C++17",
"compiler": "clang",
"version": "14",
"os": "ubuntu-22.04",
"cxxstd" : "17"
"cxxstd": "17"
},
{
"name": "Linux Clang 14 C++20",
"compiler": "clang",
"version": "14",
"os": "ubuntu-22.04",
"cxxstd" : "20"
"cxxstd": "20"
},
{
"name": "macOS apple-clang 14.3.1",
"name": "macOS apple-clang 14.3.1 C++17",
"compiler": "apple-clang",
"version": "14.3.1",
"os": "macos-13",
"cxxstd" : "17"
"cxxstd": "17"
},
{
"name": "Windows VS2022",
"name": "macOS apple-clang 14.3.1 C++20",
"compiler": "apple-clang",
"version": "14.3.1",
"os": "macos-13",
"cxxstd": "20"
},
{
"name": "Windows VS2022 C++17",
"os": "windows-2022",
"cxxstd": "17"
},
{
"name": "Windows VS2022 C++20",
"os": "windows-2022",
"cxxstd" : "17"
"cxxstd": "20"
}
]
}
4 changes: 2 additions & 2 deletions .github/workflows/adobe_source_libraries.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,15 +109,15 @@ jobs:
shell: bash
run: |
mkdir ../build
cmake -S. -B../build -GNinja -DCMAKE_BUILD_TYPE=Release -DCMAKE_CXX_STANDARD=17
cmake -S. -B../build -GNinja -DCMAKE_BUILD_TYPE=Release -DCMAKE_CXX_STANDARD=${{matrix.config.cxxstd}}

- name: Configure (Windows)
if: ${{ startsWith(matrix.config.os, 'windows') }}
shell: cmd
run: |
call "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Auxiliary\Build\vcvarsall.bat" x64
mkdir ..\build
cmake -S. -B../build -GNinja -DCMAKE_BUILD_TYPE=Release -DCMAKE_CXX_STANDARD=17 -DCMAKE_TOOLCHAIN_FILE=C:/vcpkg/scripts/buildsystems/vcpkg.cmake -DBOOST_TEST_DYN_LINK=TRUE
cmake -S. -B../build -GNinja -DCMAKE_BUILD_TYPE=Release -DCMAKE_CXX_STANDARD=${{matrix.config.cxxstd}} -DCMAKE_TOOLCHAIN_FILE=C:/vcpkg/scripts/buildsystems/vcpkg.cmake -DBOOST_TEST_DYN_LINK=TRUE

- name: Build (Unix)
if: ${{ startsWith(matrix.config.os, 'ubuntu') || startsWith(matrix.config.os, 'macos') }}
Expand Down
1 change: 1 addition & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,4 @@ gh-pages/

documentation/doxyfile.bak
build_asl/
.vs/
20 changes: 0 additions & 20 deletions .vscode/c_cpp_properties.json
View file
Open in desktop

This file was deleted.

254 changes: 118 additions & 136 deletions .vscode/settings.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,136 +1,118 @@
{
"files.associations": {
"*.cpp": "cpp",
"__bit_reference": "cpp",
"__bits": "cpp",
"__config": "cpp",
"__debug": "cpp",
"__errc": "cpp",
"__hash_table": "cpp",
"__locale": "cpp",
"__mutex_base": "cpp",
"__node_handle": "cpp",
"__split_buffer": "cpp",
"__threading_support": "cpp",
"__tree": "cpp",
"__tuple": "cpp",
"__verbose_abort": "cpp",
"array": "cpp",
"atomic": "cpp",
"bit": "cpp",
"bitset": "cpp",
"cctype": "cpp",
"clocale": "cpp",
"cmath": "cpp",
"complex": "cpp",
"condition_variable": "cpp",
"cstdarg": "cpp",
"cstddef": "cpp",
"cstdint": "cpp",
"cstdio": "cpp",
"cstdlib": "cpp",
"cstring": "cpp",
"ctime": "cpp",
"cwchar": "cpp",
"cwctype": "cpp",
"deque": "cpp",
"exception": "cpp",
"fstream": "cpp",
"future": "cpp",
"initializer_list": "cpp",
"iomanip": "cpp",
"ios": "cpp",
"iosfwd": "cpp",
"iostream": "cpp",
"istream": "cpp",
"limits": "cpp",
"list": "cpp",
"locale": "cpp",
"map": "cpp",
"memory": "cpp",
"mutex": "cpp",
"new": "cpp",
"optional": "cpp",
"ostream": "cpp",
"queue": "cpp",
"ratio": "cpp",
"set": "cpp",
"sstream": "cpp",
"stack": "cpp",
"stdexcept": "cpp",
"streambuf": "cpp",
"string": "cpp",
"string_view": "cpp",
"system_error": "cpp",
"thread": "cpp",
"tuple": "cpp",
"type_traits": "cpp",
"typeinfo": "cpp",
"unordered_map": "cpp",
"variant": "cpp",
"vector": "cpp",
"__nullptr": "cpp",
"__string": "cpp",
"chrono": "cpp",
"compare": "cpp",
"concepts": "cpp",
"numeric": "cpp",
"random": "cpp",
"algorithm": "cpp",
"filesystem": "cpp",
"cfenv": "cpp",
"csignal": "cpp",
"strstream": "cpp",
"typeindex": "cpp",
"hash_map": "cpp",
"any": "cpp",
"barrier": "cpp",
"charconv": "cpp",
"cinttypes": "cpp",
"codecvt": "cpp",
"coroutine": "cpp",
"csetjmp": "cpp",
"execution": "cpp",
"format": "cpp",
"forward_list": "cpp",
"functional": "cpp",
"iterator": "cpp",
"latch": "cpp",
"memory_resource": "cpp",
"numbers": "cpp",
"ranges": "cpp",
"regex": "cpp",
"scoped_allocator": "cpp",
"semaphore": "cpp",
"shared_mutex": "cpp",
"source_location": "cpp",
"span": "cpp",
"stop_token": "cpp",
"syncstream": "cpp",
"unordered_set": "cpp",
"utility": "cpp",
"valarray": "cpp",
"xfacet": "cpp",
"xhash": "cpp",
"xiosbase": "cpp",
"xlocale": "cpp",
"xlocbuf": "cpp",
"xlocinfo": "cpp",
"xlocmes": "cpp",
"xlocmon": "cpp",
"xlocnum": "cpp",
"xloctime": "cpp",
"xmemory": "cpp",
"xstddef": "cpp",
"xstring": "cpp",
"xtr1common": "cpp",
"xtree": "cpp",
"xutility": "cpp",
"*.ipp": "cpp",
"*.tcc": "cpp",
"rope": "cpp",
"slist": "cpp",
"print": "cpp"
},
"C_Cpp.default.compilerPath": "/usr/bin/clang++"
}
{
"files.associations": {
"*.cpp": "cpp",
sean-parent marked this conversation as resolved.
Show resolved Hide resolved
"algorithm": "cpp",
"any": "cpp",
"array": "cpp",
"atomic": "cpp",
"barrier": "cpp",
"bit": "cpp",
"bitset": "cpp",
"cctype": "cpp",
"cfenv": "cpp",
"charconv": "cpp",
"chrono": "cpp",
"cinttypes": "cpp",
"clocale": "cpp",
"cmath": "cpp",
"codecvt": "cpp",
"compare": "cpp",
"complex": "cpp",
"concepts": "cpp",
"condition_variable": "cpp",
"coroutine": "cpp",
"csignal": "cpp",
"cstdarg": "cpp",
"cstddef": "cpp",
"cstdint": "cpp",
"cstdio": "cpp",
"cstdlib": "cpp",
"cstring": "cpp",
"ctime": "cpp",
"cwchar": "cpp",
"cwctype": "cpp",
"deque": "cpp",
"exception": "cpp",
"execution": "cpp",
"expected": "cpp",
"resumable": "cpp",
"filesystem": "cpp",
"format": "cpp",
"forward_list": "cpp",
"fstream": "cpp",
"functional": "cpp",
"future": "cpp",
"initializer_list": "cpp",
"iomanip": "cpp",
"ios": "cpp",
"iosfwd": "cpp",
"iostream": "cpp",
"istream": "cpp",
"iterator": "cpp",
"latch": "cpp",
"limits": "cpp",
"list": "cpp",
"locale": "cpp",
"map": "cpp",
"mdspan": "cpp",
"memory": "cpp",
"memory_resource": "cpp",
"mutex": "cpp",
"new": "cpp",
"numbers": "cpp",
"numeric": "cpp",
"optional": "cpp",
"ostream": "cpp",
"print": "cpp",
"queue": "cpp",
"random": "cpp",
"ranges": "cpp",
"ratio": "cpp",
"regex": "cpp",
"scoped_allocator": "cpp",
"semaphore": "cpp",
"set": "cpp",
"shared_mutex": "cpp",
"source_location": "cpp",
"span": "cpp",
"spanstream": "cpp",
"sstream": "cpp",
"stack": "cpp",
"stacktrace": "cpp",
"stdexcept": "cpp",
"stdfloat": "cpp",
"stop_token": "cpp",
"streambuf": "cpp",
"string": "cpp",
"strstream": "cpp",
"syncstream": "cpp",
"system_error": "cpp",
"thread": "cpp",
"tuple": "cpp",
"type_traits": "cpp",
"typeindex": "cpp",
"typeinfo": "cpp",
"unordered_map": "cpp",
"unordered_set": "cpp",
"utility": "cpp",
"valarray": "cpp",
"variant": "cpp",
"vector": "cpp",
"xfacet": "cpp",
"xhash": "cpp",
"xiosbase": "cpp",
"xlocale": "cpp",
"xlocbuf": "cpp",
"xlocinfo": "cpp",
"xlocmes": "cpp",
"xlocmon": "cpp",
"xlocnum": "cpp",
"xloctime": "cpp",
"xmemory": "cpp",
"xstring": "cpp",
"xtr1common": "cpp",
"xtree": "cpp",
"xutility": "cpp",
"*.ipp": "cpp"
}
}
5 changes: 2 additions & 3 deletions CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
cmake_minimum_required(VERSION 3.0)
cmake_minimum_required(VERSION 3.29)
include(FindGit)
include(CMakeParseArguments)
enable_testing()
Expand All @@ -7,8 +7,7 @@ project(adobe_source_libraries CXX)

set(CMAKE_CXX_EXTENSIONS OFF)
if(NOT DEFINED CMAKE_CXX_STANDARD)
set(CMAKE_CXX_STANDARD 17)
set(CMAKE_XCODE_ATTRIBUTE_CLANG_CXX_LANGUAGE_STANDARD "c++17")
set(CMAKE_CXX_STANDARD 20)
endif(NOT DEFINED CMAKE_CXX_STANDARD)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to assume that dropping the Xcode setting is intentional and correct.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. It isn't needed.


set(CMAKE_XCODE_ATTRIBUTE_CLANG_CXX_LIBRARY "libc++")
Expand Down
Loading
Loading