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 gcc build and tidy issues after upgrade to v17 #12

Merged
merged 1 commit into from Jan 31, 2024

Conversation

kingsamchen
Copy link
Owner

  1. A move constructor with noexcept(false) but =default will still be treated as nothrow-move-constructible in gcc-11 (i.e. default gcc compiler on Ubuntu 22.04), so replace =default with explicit ctor body to address the issue.
  2. After upgrading clang-tools to v17 there are new tidy checks failed and got fixed
  3. Added/deleted headers as suggested by include-what-you-use

@kingsamchen kingsamchen merged commit f473cf8 into master Jan 31, 2024
@kingsamchen kingsamchen deleted the dev-fix-build branch January 31, 2024 09:58
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.

None yet

1 participant