Fix detecting of constants with Unicode

Yorick Peterse · Yorick Peterse · commit af157db29474 · 2015-04-29T22:57:28.000+02:00
There's no need to check the characters that follow the initial one, instead we can assert that a constant simply starts with a capital latin letter. Fixes #3382
diff --git a/vm/symbol_table.cpp b/vm/symbol_table.cpp
@@ -16,13 +16,8 @@ namespace rubinius {
   SymbolTable::Kind SymbolTable::detect_kind(const char* str, size_t size) {
     const char one = str[0];
 
-    // A constant begins with an uppercase letter.
-    if(one >= 'A' && one <= 'Z') {
-      // Make sure that the rest of it is only alphanumerics
-      for(size_t i = 1; i < size; i++) {
-        if((isalnum(str[i]) || str[i] == '_') == false)
-          return SymbolTable::Normal;
-      }
+    // Constants start with A-Z
+    if(isupper(one)) {
       return SymbolTable::Constant;
     }
 
diff --git a/vm/test/test_symbol_table.hpp b/vm/test/test_symbol_table.hpp
@@ -22,6 +22,22 @@ class TestSymbolTable : public CxxTest::TestSuite, public VMTest {
     destroy();
   }
 
+  void test_detect_kind_with_constant() {
+    const char *input = "Pettson";
+
+    SymbolTable::Kind kind = symbols->detect_kind(input, strlen(input));
+
+    TS_ASSERT_EQUALS(kind, SymbolTable::Constant);
+  }
+
+  void test_detect_kind_with_unicode_constant() {
+    const char *input = "Pettsonλ";
+
+    SymbolTable::Kind kind = symbols->detect_kind(input, strlen(input));
+
+    TS_ASSERT_EQUALS(kind, SymbolTable::Constant);
+  }
+
   void test_lookup_with_c_str() {
     Object* sym = symbols->lookup(state, std::string("unique"));
     TS_ASSERT(sym->symbol_p());
