search

lbryio / lbrycrd

The blockchain that provides the digital content namespace for the LBRY protocol
https://lbry.com
MIT License
2.57k stars 178 forks source link

Clean up compiler warnings #129

Closed shyba closed 5 years ago

shyba commented 6 years ago

During compilation I'm getting a lot of warnings and most of them aren't relevant. We should at some point clean them up so its easier to read. Another idea was to add a linter, but I'm undecided on which one, so I will leave that open and let us start by cleaning the noise instead.

BrannonKing commented 6 years ago

Waiting on #174

BrannonKing commented 5 years ago

I feel like this should be the standard: -Wall -Wextra -Wno-implicit-fallthrough -Wno-unused-parameter Right now, that much works on the post-merged code.