Always issue deprecation warning when calling Regexp.new with 3rd positional argument

jeremyevans · nobu · hsbt · commit 1f23e6e7f0ab · 2022-12-23T08:32:09.000+09:00
Previously, only certain values of the 3rd argument triggered a
deprecation warning.

First step for fix for bug #18797.  Support for the 3rd argument
will be removed after the release of Ruby 3.2.

Fix minor fallout discovered by the tests.

Co-authored-by: Nobuyoshi Nakada &lt;nobu@ruby-lang.org&gt;
diff --git a/test/psych/test_yaml.rb b/test/psych/test_yaml.rb
@@ -34,7 +34,7 @@ def test_multiline_regexp
 
     # [ruby-core:34969]
     def test_regexp_with_n
-        assert_cycle(Regexp.new('',0,'n'))
+        assert_cycle(Regexp.new('',Regexp::NOENCODING))
     end
 	#
 	# Tests modified from 00basic.t in Psych.pm

Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,7 @@ def test_multiline_regexp`
`34`	`34`
`35`	`35`	`# [ruby-core:34969]`
`36`	`36`	`def test_regexp_with_n`
`37`		`- assert_cycle(Regexp.new('',0,'n'))`
	`37`	`+ assert_cycle(Regexp.new('',Regexp::NOENCODING))`
`38`	`38`	`end`
`39`	`39`	`#`
`40`	`40`	`# Tests modified from 00basic.t in Psych.pm`