In the spec, why not use each_with_index instead of times? (http://github.com/jschementi/ironruby/commit/e8406f60c9426f4b675bf67180c24f2bb5b0f609#L1R99)
Other than that, looks good. JD From: Jimmy Schementi Sent: Saturday, April 11, 2009 4:42 PM To: ironruby-core@rubyforge.org Cc: IronRuby External Code Reviewers Subject: Code Review: Allows symbols to be in a YAML array http://github.com/jschementi/ironruby/commit/e8406f60c9426f4b675bf67180c24f2bb5b0f609 Allows symbols to be in a YAML array ("date: [:year, :month, :day]"). Fixes http://ironruby.codeplex.com/WorkItem/View.aspx?WorkItemId=375. (Shri: gitdiff wasn't working for me, is there any setup I need to do?)
_______________________________________________ Ironruby-core mailing list Ironruby-core@rubyforge.org http://rubyforge.org/mailman/listinfo/ironruby-core