Enable deprecation warnings#2036
Conversation
7683c15 to
ef7e1af
Compare
|
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. |
The
Yay! That means you found everything. |
This reverts commit 4e82005.
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.