Removed use of strlen in favor of storing a m_length variable so rbegin() can be used to assign a null terminator etc
Made StringCompare functionality inline to remove duplicate definitions in usage
Fixed StringCompare where one or both strings are null by adding ret = rhs-lhs as a fallback when either are null