[Ironruby-core] Code Review: RandomRubyFixes05

Curt Hagenlocher curth at microsoft.com
Fri Oct 24 11:23:24 EDT 2008


tfpt review "/shelveset:RandomRubyFixes05;REDMOND\curth"

Comment  :
  Factored class IOWrapper.cs out of Marshal and into the Ruby core so it can be accessed from multiple places
  Factored object creation out of Marshal and into RubyUtils so it can share code with Yaml
  Disabled name-mangling for symbol "initialize" so that CLR methods with the name Initialize don't cause trouble
  Implemented String.hex and String.oct
  Modified Socket and StringIO classes so they open in binary mode by default
  Extensive modifications to zlib:
    Fixed uncompressed blocks to inflate properly
    Reformatted to match standard
    Many small changes to cleanup code and improve performance
    Added support for reading from any object implementing "read" -- not just IO objects -- by using IOWrapper
  Added an overload to IO.seek that accepts BigInteger.  (This turned out not to be needed, but the code works.)
  Modified Yaml construction to call yaml_initialize if it exists

--
Curt Hagenlocher
curth at microsoft.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: RandomRubyFixes05.diff
Type: application/octet-stream
Size: 138733 bytes
Desc: RandomRubyFixes05.diff
URL: <http://rubyforge.org/pipermail/ironruby-core/attachments/20081024/f8a2f17c/attachment-0001.obj>


More information about the Ironruby-core mailing list