You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It seems that we missed a lot of the deprecation warnings because they were turned off. Now that we have settled on minimum compiler support in #2005, it seems appropriate to turn these on and not ignore them. If we run into something specific that is deprecated in modern GCC/Clang but has no replacement for the minimum compiler version we support, we can handle those case-by-case.
The build for this PR will intentionally fail until many of @aroffringa's PRs are merged. Once those are merged I will rebase and we can see if we missed anything.
Good idea. I see in your changes the warnings were disabled in CI, but not on a normal build (those related to a normal build were all -Wno-error options). If I do a local build with mine and your changes combined I don't see warnings.
I can't approve, but the changes look fine to me once the warning fixes are merged.
I see in your changes the warnings were disabled in CI, but not on a normal build (those related to a normal build were all -Wno-error options).
The GNU_PLUS_PLUS_11_DEPRECATIONS part was enabling -Wno-error=deprecated-declarations for local builds as well. If I try build now locally it doesn't get very far and runs into std::binary_function and std::auto_ptr.
If I do a local build with mine and your changes combined I don't see warnings.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
It seems that we missed a lot of the deprecation warnings because they were turned off. Now that we have settled on minimum compiler support in #2005, it seems appropriate to turn these on and not ignore them. If we run into something specific that is deprecated in modern GCC/Clang but has no replacement for the minimum compiler version we support, we can handle those case-by-case.
The build for this PR will intentionally fail until many of @aroffringa's PRs are merged. Once those are merged I will rebase and we can see if we missed anything.