YAML changes look good too. -- Oleg
-----Original Message----- From: Curt Hagenlocher Sent: Friday, October 24, 2008 8:23 AM To: IronRuby External Code Reviewers; Oleg Tkachenko Cc: ironruby-core@rubyforge.org Subject: Code Review: RandomRubyFixes05 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 [EMAIL PROTECTED] _______________________________________________ Ironruby-core mailing list Ironruby-core@rubyforge.org http://rubyforge.org/mailman/listinfo/ironruby-core