Changes look good.

-----Original Message-----
From: Tomas Matousek
Sent: Sunday, November 16, 2008 1:11 PM
To: IronRuby External Code Reviewers
Cc: [email protected]
Subject: Code Review: VisibilityAndOtherFixes2

  tfpt review "/shelveset:VisibilityAndOtherFixes2;REDMOND\tomat"
  Comment  :
  Implements
      - private method visibility
      - DATA constant
      - popen and duplex IO streams (will need some refactoring of RubyIO 
object)

      Fixes:
      - warnings (verbosity levels, Kernel#warn)
      - Kernel#gets, print, puts (no dynamic call to IO#puts)
      - method reflection: Kernel#methods, Kernel#singleton_methods (methods 
were missing in some cases)
      - $~ variable scoping (should be an inner-most non-block scope)
      - inclusion of module already included in a super class was handled 
incorrectly

Tomas
_______________________________________________
Ironruby-core mailing list
[email protected]
http://rubyforge.org/mailman/listinfo/ironruby-core

Reply via email to