DaddyFrosty/yoga
1
0
Fork 0
Code Issues 96 Pull Requests 31 Actions Packages Projects Releases 33 Wiki Activity

Enable -Wextra in C++ builds #1294

Closed
NickGerleman wants to merge 2 commits from export-D45772846 into main
pull from: export-D45772846
merge into: DaddyFrosty:main
Conversation 7 Commits 2 Files Changed 16 +212 -189

2 Commits

Author SHA1 Message Date
Nick Gerleman
8bbd7c1d88 Enable -Wextra in C++ builds (#1294) 
Summary:
Pull Request resolved: https://github.com/facebook/yoga/pull/1294

X-link: https://github.com/facebook/react-native/pull/37383

Add -Wextra to the build, and fixup some more instances of -Wunused-parameter that it sufaces which were not automatically fixable.

Reviewed By: javache

Differential Revision: D45772846

fbshipit-source-id: 680498f66f2104c29a71366bf30dba8895d9baf4
 2023-05-11 09:39:41 -07:00
Nick Gerleman
86d3ac031d Apply fixes from clag-tidy misc-unused-parameters 
Summary: Gets the project clean of `-Wunused-parameter`, part of `-Wextra`. Enforced as part of the build now.

Differential Revision: https://internalfb.com/D45772554

fbshipit-source-id: e7ec6f4047b3994c3615a4557ffc4f3677f9c1e6
 2023-05-11 09:38:59 -07:00