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

Reply via email to