[Ironruby-core] Code Review - Time#strftime ignore invalid directives on format string

[Tomas Matousek]

You can remove compatibility check (including the one that's there). IronRuby only supports MRI 1.9.

Tomas

-----Original Message-----
From: ironruby-core-bounces at rubyforge.org [mailto:ironruby-core-bounces at rubyforge.org] On Behalf Of Enrico Sada
Sent: Tuesday, August 02, 2011 9:45 AM
To: ironruby-core at rubyforge.org
Subject: [Ironruby-core] Code Review - Time#strftime ignore invalid directives on format string

i asked a pull request ( https://github.com/IronLanguages/main/pull/28
) for fix Time#strftime behaviour on invalid directives on format string, ex:

ruby 1.8:
Time.now.strftime '%$' => '$'
ruby 1.9:
Time.now.strftime '%$' => '%$'

this make green a mspec test on core/time/strftime_spec.rb

I have some question:
1) i added a check for ruby compatibility >= ruby 1.9, is needed?
2) i removed 'fails:' from
ironruby-tags-19/core/time/strftime_tags.txt is correct?
3) usually code review is in mailing list or github pull request? (so i dont need to write two times the same questions) _______________________________________________
Ironruby-core mailing list
Ironruby-core at rubyforge.org
http://rubyforge.org/mailman/listinfo/ironruby-core