Project

General

Profile

Actions

Bug #19471

closed

Regexp::compile does not handle :timeout argument

Added by os (Shigeki OHARA) almost 2 years ago. Updated over 1 year ago.

Status:
Closed
Assignee:
-
Target version:
-
ruby -v:
ruby 3.2.1 (2023-02-08 revision 31819e82c8) [x86_64-freebsd13.1]
[ruby-core:112652]

Description

I think Regexp::compile is just an alias for Regexp::new, so I assume it handles the :timeout argument.
However, Regexp::compile does not seem to handle the :timeout argument properly.

$ irb
irb(main):001:0> RUBY_VERSION
=> "3.2.1"
irb(main):002:0> re_new = Regexp.new('hoge', timeout: 1.0)
=> /hoge/
irb(main):003:0> re_new.timeout
=> 1.0
irb(main):004:0> re_compile = Regexp.compile('hoge', timeout: 1.0)
=> /hoge/i
irb(main):005:0> re_compile.timeout
=> nil
irb(main):006:0>

Actions #1

Updated by nobu (Nobuyoshi Nakada) almost 2 years ago

  • Backport changed from 2.7: UNKNOWN, 3.0: UNKNOWN, 3.1: UNKNOWN, 3.2: UNKNOWN to 2.7: DONTNEED, 3.0: DONTNEED, 3.1: DONTNEED, 3.2: REQUIRED
Actions #2

Updated by nobu (Nobuyoshi Nakada) almost 2 years ago

  • Status changed from Open to Closed

Applied in changeset git|680bd9027f8cb7977bbc216609db2f4e3cf199a8.


[Bug #19471] Regexp.compile should handle keyword arguments

As well as Regexp.new, it should pass keyword arguments to the
Regexp#initialize method.

Updated by naruse (Yui NARUSE) over 1 year ago

  • Backport changed from 2.7: DONTNEED, 3.0: DONTNEED, 3.1: DONTNEED, 3.2: REQUIRED to 2.7: DONTNEED, 3.0: DONTNEED, 3.1: DONTNEED, 3.2: DONE

ruby_3_2 fb4ffce0dd8cae3b7d0141d3b1ea3f0ab710a45a merged revision(s) 680bd9027f8cb7977bbc216609db2f4e3cf199a8.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0