Remove String->const char* conversion operator in favor of c_str method

Remove String->const char* conversion operator in favor of c_str method

This was dumb to begin with (I reserve the right to be mean to my
younger self) and really only caused a number of evil cases of
ambiguious overloading whenever a String and std::string are used
together.