Patch submitted - [#19230] Time implementation

Hey Guys,

Josh C. put a bug in my ear (thanks Josh!), so I got off my duff
and
submitted a patch for the Time class that’s been sitting on my laptop
for
months. It should have implementations for most of the basic Time
functionality save for the strftime method. It’ll pass 22 of the 55
time
specs. It would pass almost all of them, but the specs depend on
ENV[“TZ”]
to set the system time zone, which I’m not sure is really possible on
Windows, and seems a little sketchy anyway (who wants specs monkeying
with
their system time?). I had more luck with the JRuby tests when I ran
them.

I hope it’s useful for somebody… Cheers!
Eric N.

Thanks for submitting this, Eric!

This reminds me that I need to add a simple implementation for ENV as
well …

Thanks,
-John

This forum is not affiliated to the Ruby language, Ruby on Rails framework, nor any Ruby applications discussed here.

| Privacy Policy | Terms of Service | Remote Ruby Jobs