StringType::cmpStatic: fix accidental reference to std::right

Because of the `using namespace std` directive at the top of
StringType.cpp, within `StringType::cmpStatic`, `right` is
resolved to the function `std::right`, from the <ios> header,
included indirectly by <iostream>. `std::right`, a function,
is always non-null. The intention here was clearly to use
the parameter `right_data`, not `std::right`.

Fix that bug.

Author: tdp2110

typed_python/StringType.cpp

@@ -1444,13 +1444,13 @@ char StringType::cmpStatic(layout* left, layout* right) {
 }
 
 char StringType::cmpStatic(layout* left, uint8_t* right_data, int right_pointcount, int right_bytes_per_codepoint) {
-    if ( !left && !right ) {
+    if ( !left && !right_data ) {
         return 0;
     }
-    if ( !left && right ) {
+    if ( !left && right_data ) {
         return -1;
     }
-    if ( left && !right ) {
+    if ( left && !right_data ) {
         return 1;
     }