Build Errors and Warnings Should be Addressed

[borupdaniel]

There are a ton of warnings and errors during the build from pyflakes, pep8, astyle (?), and gcc/clang. It would improve our ability to find real errors if we addressed things like:

• missing whitespace
• trailing whitespace
• unused variables
• incorrect indentation
• unused imports
• multiple imports (redefinition of unused import)
• comparison of different types

I think @aganders3 addressed some of this by removing aestyle as a requirement last year, but there may still be a lot of warnings generated during compilation. The output is still quite long, anyhow.