I tried building with -Wall -Wextra and gcc found a few issues, like variables being used without initialization in some code paths. These code paths probably aren't being hit too often -- otherwise we would have heard about crashes, I'm sure, but it's probably a good idea to fix them up. I created a
pull request that fixes the issues, but it could certainly use a review to make sure I fixed them correctly : ) Maybe we could add -Wall -Wextra to the flags in the standard build? It does catch some important issues before they become a problem.