diff options
| author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2017-07-12 02:49:53 +0000 |
|---|---|---|
| committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2017-07-12 02:49:53 +0000 |
| commit | 89bf90a370992e3906bca56d90835682a895bba0 () | |
| tree | db25efefd03979afe2c264cab4d5e713aafdb230 /lib/optparse.rb | |
| parent | 561c253512eb83dba822d50ccc95a90187ab945e (diff) | |
optparse.rb: get rid of eval
* lib/optparse.rb: try Float() and Integer() instead of eval, which does too much things. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@59312 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
| -rw-r--r-- | lib/optparse.rb | 20 |
1 files changed, 13 insertions, 7 deletions
@@ -1842,7 +1842,7 @@ XXX # # Float number format, and converts to Float. # - float = "(?:#{decimal}(?:\\.(?:#{decimal})?)?|\\.#{decimal})(?:E[-+]?#{decimal})?" floatpat = %r"\A[-+]?#{float}\z"io accept(Float, floatpat) {|s,| s.to_f if s} @@ -1851,11 +1851,13 @@ XXX # for float format, and Rational for rational format. # real = "[-+]?(?:#{octal}|#{float})" - accept(Numeric, /\A(#{real})(?:\/(#{real}))?\z/io) {|s, d, n| if n Rational(d, n) - elsif s - eval(s) end } @@ -1889,10 +1891,14 @@ XXX # integer format, Float for float format. # DecimalNumeric = floatpat # decimal integer is allowed as float also. - accept(DecimalNumeric, floatpat) {|s,| begin - eval(s) - rescue SyntaxError raise OptionParser::InvalidArgument, s end if s } |