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

variant,tuple: fix/silence misc warnings #564

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

virt00l
Copy link
Contributor

@virt00l virt00l commented Feb 21, 2025

variant,tuple: fix/silence misc warnings

msvc:
https://learn.microsoft.com/en-us/cpp/error-messages/compiler-warnings/c5267
Similar to neighboring C4625

Fix for warnings (errors in /WX mode) like
.../variant.h(759): warning C5267: definition of implicit copy constructor for '...' is deprecated because it has a user-provided destructor
(ditto for tuple).

Also fix for -Wunused-parameter in variant.h (visit_throw_bad_variant_access)

@virt00l virt00l force-pushed the variant-silence-msvc-c5267 branch from 69457ac to abe6ba5 Compare February 21, 2025 18:16
msvc:
https://learn.microsoft.com/en-us/cpp/error-messages/compiler-warnings/c5267
Similar to neighboring C4625

Fix for warnings (errors in /WX mode) like
.../variant.h(759): warning C5267: definition of implicit copy constructor for '...' is deprecated because it has a user-provided destructor
(ditto for tuple).

Also fix for -Wunused-parameter in variant.h (`visit_throw_bad_variant_access`)
@virt00l virt00l force-pushed the variant-silence-msvc-c5267 branch from abe6ba5 to 91dff36 Compare February 25, 2025 12:42
@virt00l virt00l changed the title variant.h: silce msvc-specific C5267 variant,tuple: fix/silence misc warnings Feb 25, 2025
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.

1 participant