Skip to content

Add source level deprecation warning disabling to certain tests/utils#2645

Merged
senhuang42 merged 1 commit intofacebook:devfrom
senhuang42:deprecate_macro_tests
May 13, 2021
Merged

Add source level deprecation warning disabling to certain tests/utils#2645
senhuang42 merged 1 commit intofacebook:devfrom
senhuang42:deprecate_macro_tests

Conversation

@senhuang42
Copy link

As title. This adds the macro to fuzzer, zstreamtest, fullbench, and Pzstd.

Also adds thread_dep target to fuzzer in meson build, and -Wno-deprecated-declarations to the pzstd target in meson. For some reason, including the deprecate macro isn't sufficient in Pzstd.h

I think this is all of them.

@Cyan4973
Copy link
Contributor

For some reason, including the deprecate macro isn't sufficient in Pzstd.h

That might be because zstd.h is included from Options.h too ?

@senhuang42 senhuang42 force-pushed the deprecate_macro_tests branch from 9f54baf to 24cddbb Compare May 13, 2021 17:25
@senhuang42 senhuang42 force-pushed the deprecate_macro_tests branch from 24cddbb to 71ffd88 Compare May 13, 2021 17:40
@senhuang42
Copy link
Author

I have verified that this now builds locally. Still, there is the remaining question of what to do with all the warnings -Wdocumentation -Wcomment.

@senhuang42 senhuang42 merged commit 40def70 into facebook:dev May 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants