Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

"require 'rational'" has been deprecated. So require rational only when the Rational module has not already been loaded #10

Merged
merged 1 commit into from

2 participants

@amatsuda

This eliminates a Ruby warning in recent versions of Ruby.

see: e027896

@amatsuda amatsuda "require 'rational'" has been deprecated. So require rational only wh…
…en the Rational module has not already been loaded

see: e0278964938ea346b2bf5237857e8656c2ca754d
a9f16a4
@philr philr merged commit a9f16a4 into from
@philr
Owner

Thank you for the pull request.

This has been merged as a9f16a4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Nov 11, 2013
  1. @amatsuda

    "require 'rational'" has been deprecated. So require rational only wh…

    amatsuda authored
    …en the Rational module has not already been loaded
    
    see: e0278964938ea346b2bf5237857e8656c2ca754d
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  lib/tzinfo/time_or_datetime.rb
View
2  lib/tzinfo/time_or_datetime.rb
@@ -22,7 +22,7 @@
require 'date'
-require 'rational'
+require 'rational' unless defined?(Rational)
require 'time'
module TZInfo
Something went wrong with that request. Please try again.