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

Reply via email to