-
Notifications
You must be signed in to change notification settings - Fork 361
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.
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
update macros for codecvt and support C++26 support on clang19 #1100
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1100 +/- ##
===========================================
Coverage 100.00% 100.00%
===========================================
Files 17 17
Lines 4546 5034 +488
Branches 0 1029 +1029
===========================================
+ Hits 4546 5034 +488 ☔ View full report in Codecov by Sentry. |
Please see #1113 and specifically 9a2884a where I fix the build error on LLVM 19 with C++26 after rooting the actual template issue (see #1098 (comment)) |
* Modifies #1100 * That PR comes from https://github.com/phlptp/CLI11 and unfortunately I have no way of opening a PR to that repo, pointing to that branch... The commit you care about is 9a2884a @phlptp where I fix the build error on LLVM 19 with C++26 after rooting the actual template issue (see #1098 (comment)) It comes down to C++26 adding a tuple interface to std::complex https://en.cppreference.com/w/cpp/numeric/complex/tuple_size --------- Co-authored-by: Philip Top <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
check updated macros for C++26 and llvm 19 support
specifically codecvt from issue #1098