[ruby-core:77199] [Ruby trunk Feature#12700][Rejected] regexg heredoc support

From: matz@...
Date: 2016-09-07 08:47:25 UTC
List: ruby-core #77199
Issue #12700 has been updated by Yukihiro Matsumoto.

Status changed from Open to Rejected

Use `%r`.

Matz.


----------------------------------------
Feature #12700: regexg heredoc support
https://bugs.ruby-lang.org/issues/12700#change-60416

* Author: Allen Morris
* Status: Rejected
* Priority: Normal
* Assignee: 
----------------------------------------
There is support for ', ", and ` heredocs, but there is no support for /.

Example code with new feature:

~~~ ruby
first = 'first'

orig = /#{first}
match\s # match
this    # match this
/x

right = <</REGEXP/x
#{first}
match\s # match
this    # match this
REGEXP

raise unless orig == right
~~~

There is no straight forward way to replace a regexp heredoc as a double quote heredoc requires that '\\s' be escaped.
As shown in the code below you can't use the string heredoc to directly replace a regexp heredoc because of this need for extra escaping.

~~~ ruby
first = 'first'

orig = /#{first}
match\s # match
this    # match this
/x

wrong = Regexp.new(<<REGEXP, Regexp::EXTENDED)
#{first}
match\s # match
this    # match this
REGEXP

right = Regexp.new(<<REGEXP, Regexp::EXTENDED)
#{first}
match\\s # match
this    # match this
REGEXP

raise unless orig != wrong
raise unless orig == right
~~~








---Files--------------------------------
regex_heredoc_patch (2.99 KB)


-- 
https://bugs.ruby-lang.org/

Unsubscribe: <mailto:[email protected]?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>

In This Thread

Prev Next