I'll reply to both patches with this mail. We actually can't accept patches to anything in the DLR and Ruby parser and runtime. So really the patches have to be to the libraries, or the infrastructure. We also need to back out some of your commits. I can ask the DLR to make these changes for you though.
Sorry about this. We can try to make it clearer if you have a suggestion on how to do it. -----Original Message----- From: Seo Sanghyeon <[EMAIL PROTECTED]> Sent: August 14, 2008 1:58 PM To: ironruby-core@rubyforge.org <ironruby-core@rubyforge.org> Subject: [Ironruby-core] Code Review: warning2 Attached patch fixes 5 more unused local variable warnings. It should be uncontroversial. If some unused variable assignment is there for debugging or other purposes, I think it is advisable to write a comment saying it is so. If this is the case, please tell me. All warnings were found by Mono C# compiler. -- Seo Sanghyeon _______________________________________________ Ironruby-core mailing list Ironruby-core@rubyforge.org http://rubyforge.org/mailman/listinfo/ironruby-core