Issue #12700 has been updated by Shyouhei Urabe.


Martin Drst wrote:
> I'm confused. You only give arguments for rejection, but then reopen the issue.

My private opinion is I don't need this.
But I don't want to rule out my being wrong-headed.
The OP might have other use-case where this is useful.
So the reopen.
Everyone who need this feature are encouraged to involve this thread.

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

* Author: Allen Morris
* Status: Open
* 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:ruby-core-request / ruby-lang.org?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>