Internals: Remove dead code, fix or sign off coverage

Remove/fix/signoff uncontroversial code coverage holes. Also added a
couple TODOs that should be investigated at some point

src/V3String.h

@@ -58,22 +58,6 @@ typename std::enable_if<std::is_integral<T>::value, std::string>::type cvtToHex(
     return os.str();
 }
 
-inline uint32_t cvtToHash(const void* vp) {
-    // We can shove a 64 bit pointer into a 32 bit bucket
-    // On 32-bit systems, lower is always 0, but who cares?
-    union {
-        const void* up;
-        struct {
-            uint32_t upper;
-            uint32_t lower;
-        } l;
-    } u;
-    u.l.upper = 0;
-    u.l.lower = 0;
-    u.up = vp;
-    return u.l.upper ^ u.l.lower;
-}
-
 inline string ucfirst(const string& text) {
     string result = text;
     result[0] = std::toupper(result[0]);