Trying this. Seemingly now all symbol(String) in RubySymbol will be 8…

…859_1 or a raw string. So I made toString() decode to actual charset based on encoding. This may need another field if we discover toString() on RubySymbol is called all the time but let's roll with this made on-demand for now.
jruby · Dec 14, 2017 · 4a07463 · 4a07463
1 parent 6d1577b
commit 4a07463
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/core/src/main/java/org/jruby/RubySymbol.java b/core/src/main/java/org/jruby/RubySymbol.java
@@ -151,12 +151,14 @@ public final String asJavaString() {
      * Return a raw (ISO-8859_1) string for use with our method tables etc.
      */
     public String getRawString() {
-        return StringSupport.bytelistAsBinaryString(symbolBytes);
+        return symbol;
     }
 
     @Override
     public final String toString() {
-        return symbol;
+        // FIXME: This is not efficient but in theory this is just for display purposes
+        // which generally are for errors?
+        return StringSupport.byteListAsString(symbolBytes);
     }
 
     final ByteList getBytes() {