GSoC/GCI Archive
Google Code-in 2012 BRL-CAD

Compile with -Wstrict-prototypes, fix warnings that ensue

completed by: javamonn

mentors: Sean

We hold BRL-CAD to a rather high standard of code quality.  We compile with nearly every useful warning that GCC is capable of producing and consider all warnings as errors that stop compilation.  This is done because many/most warnings are "code smells" that are eventually usually costly to ignore over time.

This task involves adding the -Wstrict-prototypes warning to our compilation and fixing the issues that result.  You can add the flag by editing misc/CMake/BRLCAD_CompilerFlags.cmake

Make sure you compile cleanly before beginning.  Make sure you're using a fresh SVN checkout.  See http://brlcad.org/wiki/Compiling for help.

Protip: run this to save all errors to a file so you can work on them in bulk: make -k 2>&1 | tee build.log

Note that the majority of errors will be functions that take no arguments that weren't declared/defined as (void).  Be careful to not add a 'void' where it's actually an old k&r empty argument declaration, though.  Submit you work as a single patch file, see http://brlcad.org/wiki/Patches

There are about 99 unique places in the code that need fixing.