Comment 3 for bug 1258667

Clang/LLVM 3.5 has just been released so reran this. Since the last time there has of course been plenty of code changes, but we've also tweaked the build system and warnings used. Now we set the flag -Weverything when building with Clang which means it report anything it is able to find in the code (including experimental checks). The code base seem to be in good shape though, there are some warnings which are code-related but the vast majority deals with documentation and comments.

Most of the documentation should be easy to fix, though it would be nice if someone could doublecheck some of them to verify Clang is checking for the comment style we wish to use. I'm particularly thinking of comments like
//< Default width for ware's menu icons
where Clang doesn't like the '<' character. Which to some extent is understandable, though I'm not familiar with Doxygen so I don't know whether it was added for a reason and used when Doxygen is collection the documentation from source files (or something else for that matter).

PS. Removed milestone for the time being. Personally I'd say becoming warning free should be a goal for the release, but as new warnings are added and compilers are released I realize this is an ongoing process.