OK, I've undone this line for now. Tomas
-----Original Message----- From: Curt Hagenlocher Sent: Friday, October 03, 2008 5:12 PM To: Tomas Matousek; IronRuby External Code Reviewers Cc: [email protected] Subject: RE: Code Review: NoSymbolId1 Changes look good, overall. I like the approach. In Tokenizer.cs, you replaced Symbols.Minus with the hardcoded "-" -- but Symbols.Minus has separately been redefined to "-". Best to continue to use the symbolic name, no? -----Original Message----- From: Tomas Matousek Sent: Friday, October 03, 2008 3:08 PM To: IronRuby External Code Reviewers Cc: [email protected] Subject: Code Review: NoSymbolId1 tfpt review "/shelveset:NoSymbolId1;REDMOND\tomat" Removes SymbolIds from IronRuby compiler and runtime. Still keeps SymbolId as a representation of Symbol Ruby class. The plan is to replace SymbolId completely by System.String with interning table on RubyContext (the integer values will be unique per script runtime). The table will hold on additional information about the strings (encoding, taint, instance variables, etc.). Tomas _______________________________________________ Ironruby-core mailing list [email protected] http://rubyforge.org/mailman/listinfo/ironruby-core
