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.

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 meta_info.cpp #99

Merged
merged 1 commit into from
Jul 8, 2022
Merged

Update meta_info.cpp #99

merged 1 commit into from
Jul 8, 2022

Conversation

trodge
Copy link
Contributor

@trodge trodge commented Jul 7, 2022

No description provided.

@TheJJ
Copy link
Member

TheJJ commented Jul 7, 2022

thanks!
can you briefly explain why the <utility> is needed here please?
I wonder why it's not sufficient that plenty of std headers are included through lang_error.h where we also include error.h and location.h.
If it's needed for your build we'll of course add it.

@tusharpm
Copy link
Member

tusharpm commented Jul 7, 2022

The function templates std::move and std::as_const are defined in <utility>. It seems that the CI build included this header indirectly. It would be great to know which standard library attained such clear separation in its headers, and add it to CI if feasible.

@heinezen heinezen merged commit 7b5517f into SFTtech:master Jul 8, 2022
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