ORC-1639: [C++] Simplify flags in CMake and fix corresponding compiling errors - Githubissues

apache / orc

Apache ORC - the smallest, fastest columnar storage for Hadoop workloads

https://orc.apache.org/

Apache License 2.0

670 stars 477 forks source link

ORC-1639: [C++] Simplify flags in CMake and fix corresponding compiling errors #1828

Closed wgtmac closed 4 months ago

wgtmac commented 4 months ago

What changes were proposed in this pull request?

Remove unnecessary compiler flags and replace -Weverything with -Wall.
Fix warnings and errors that emerge after the flag change.

Why are the changes needed?

The C++ library has added -Weverything which results in a bunch of -Wno-xxx flags. We should reduce them as many as possible.

How was this patch tested?

Tested it locally with clang13, clang17 and gcc13.
Make sure all CIs are passed.

Was this patch authored or co-authored using generative AI tooling?

No.

dongjoon-hyun commented 4 months ago

Merged to branch-2.0. Thank you!