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

fix template default errors and other warnings #6

Merged
merged 1 commit into from
Jan 13, 2024

Conversation

nathanhhughes
Copy link
Collaborator

@Schmluk switched to use clang for some testing I'm doing downstream and found out that redeclaring template defaults is a compiler error (and found some other warnings). Might be worth considering a clang-based github action workflow at some point

@nathanhhughes nathanhhughes requested a review from Schmluk January 13, 2024 16:53
Copy link
Collaborator

@Schmluk Schmluk left a comment

Choose a reason for hiding this comment

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

LGTM!

Interesting that clang throws more warnings than GCC. For adding clang CI would you make the build fail on warnings? Or just to make sure it also compiles with clang? Feel free to add an additional check for clang compilation if you think that helps!

@nathanhhughes nathanhhughes merged commit 0a540ff into main Jan 13, 2024
2 checks passed
@nathanhhughes
Copy link
Collaborator Author

Just to make sure it compiles (though the extra warnings clang offers are nice / kinda useful). I'll make a note to add the clang workflow, but it'll likely be a little while before I actually get around to it

@nathanhhughes nathanhhughes deleted the fix/clang_compilation branch January 13, 2024 18:56
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.

2 participants