You could be a bit more descriptive with your titles so they actually make sense in review. If you are going through the code just to find the small naming failures and other odd code then you are in for some fun. Would make more sense to spend the effort toward improving the almost decade old documentation and actually document some of the code and functionality that has yet to even receive documentation. :)

Sure, you're right! But it's more fun to have a compilation with no error, no warning and no message.
If not, it means that something you have done has impacted the code elsewhere (assuming that you have previously considered errors in your changes).

Warngins =/= bad code, depending on which compiler you use and how your build environment is configured you may get more or less. Historically there were almost ten times the warnings, I pushed last year to have most of them cleared out so the actual issues would become more apparent later on, you can go back to the commits about the "Pesky warnings".

Very few issues actually stem from compile warnings now, most of what I encounter is generally to do with code that was either rushed or not well planned, little things here and there, again just read some recent commits. What functionality there should be vs. what there actually is, that's pretty much the phase the project is in currently.

Feel free to join the irc channel on freenode #opensim-dev if you want to discuss more about the current status of the project and where help and testing may be needed :)