Enable -Werror for Windows & cleanup warnings

ethereum / c-kzg-4844

A minimal implementation of the Polynomial Commitments API for EIP-4844 and EIP-7594, written in C.

Apache License 2.0

112 stars 105 forks source link

Closed jtraglia closed 1 month ago

jtraglia commented 1 month ago

For some reason, the Windows compiler shows more accurate warnings than Unix.

It accurately made these observations:

-Wc++-compat and -Wredundant-move are only for C++ code.
-Wlarger-than=512 should (and does) happen all the time.
- Lots of our stack variables are large than 512 bytes.
-Wlong-long might also happen all the time.

jtraglia commented 1 month ago

This should be non-controversial. Going to merge.