Use cast_int when shrinking size_t to uint
Remove duplicate definition
Return the right type from vectormap::count()
Use libmesh_dbg_var() where appropriate
Avoid conversion warnings
Avoid conversion warnings
Fix conversion, missing-default, undefined param

More warnings fixes
Fix missing default warning
Fix conversion warnings

We probably ought to just go all self-documenting and define unsigned
char as side_index_t
Comments describing id types
Wconversion fixes

A couple Wunsafe-loop-optimizations fixes are in here too.