Skip to content

allow sol to compile with C++20#1840

Merged
white238 merged 1 commit intodevelopfrom
task/white238/sol_result_fix
Apr 9, 2026
Merged

allow sol to compile with C++20#1840
white238 merged 1 commit intodevelopfrom
task/white238/sol_result_fix

Conversation

@white238
Copy link
Copy Markdown
Member

@white238 white238 commented Apr 8, 2026

Fixes a C++20 compilation found in the Smith project on macos.

In file included from smith/axom/src/axom/inlet/LuaReader.cpp:25:
smith/axom/src/thirdparty/axom/sol.hpp:14498:71: error: no template named 'result_of_t' in namespace 'std'
 14498 |                 template <typename R, typename... Args, typename F, typename = std::result_of_t<meta::unqualified_t<F>(Args...)>>
       |                                                                                ~~~~~^
smith/axom/src/thirdparty/axom/sol.hpp:14522:61: error: no template named 'result_of_t' in namespace 'std'
 14522 |                 template <typename... Args, typename F, typename R = std::result_of_t<F&(Args...)>>
       |                                                                      ~~~~~^
smith/axom/src/thirdparty/axom/sol.hpp:20655:86: error: no template named 'result_of_t' in namespace 'std'
 20655 |                 template <typename R, typename... Args, typename Fx, typename Key, typename = std::result_of_t<Fx(Args...)>>
       |                                                                                               ~~~~~^
3 errors generated.

@btalamini

Copy link
Copy Markdown
Contributor

@Arlie-Capps Arlie-Capps left a comment

Choose a reason for hiding this comment

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

Thanks, Chris.

@white238 white238 merged commit 3f8ecd7 into develop Apr 9, 2026
15 checks passed
@white238 white238 deleted the task/white238/sol_result_fix branch April 9, 2026 04:47
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.

4 participants