Merge pull request #3408 from rubinius/regex-new-2959

Regex new 2959
rubinius · May 24, 2015 · ffa628e · ffa628e
2 parents 73e862e + a60dfb3
commit ffa628e
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/kernel/common/regexp.rb b/kernel/common/regexp.rb
@@ -128,6 +128,8 @@ def initialize(pattern, opts=nil, lang=nil)
     if pattern.kind_of?(Regexp)
       opts = pattern.options
       pattern = pattern.source
+    elsif pattern.kind_of?(Fixnum) or pattern.kind_of?(Float)
+      raise TypeError, "can't convert Fixnum into String"
     elsif opts.kind_of?(Fixnum)
       opts = opts & (OPTION_MASK | KCODE_MASK) if opts > 0
     elsif opts

diff --git a/spec/ruby/core/regexp/new_spec.rb b/spec/ruby/core/regexp/new_spec.rb
@@ -16,3 +16,15 @@
   it_behaves_like :regexp_new_regexp_ascii, :new
   it_behaves_like :regexp_new_regexp_ascii_8bit, :new
 end
+
+describe "Regexp.new given a Fixnum" do
+  it "raises a TypeError" do
+    lambda { Regexp.new(1) }.should raise_error(TypeError)
+  end
+end
+
+describe "Regexp.new given a Float" do
+  it "raises a TypeError" do
+    lambda { Regexp.new(1.0) }.should raise_error(TypeError)
+  end
+end