[ruby-core:80422] [Ruby trunk Bug#13371][Rejected] Syntax Error with regex when parens are omitted

From: nobu@...
Date: 2017-03-28 05:50:25 UTC
List: ruby-core #80422
Issue #13371 has been updated by nobu (Nobuyoshi Nakada).

Status changed from Open to Rejected
Assignee deleted (matz (Yukihiro Matsumoto))

It's unable to tell that slash is a part of a regexp.


----------------------------------------
Bug #13371: Syntax Error with regex when parens are omitted
https://bugs.ruby-lang.org/issues/13371#change-63915

* Author: ndn (Nikola Nenkov)
* Status: Rejected
* Priority: Normal
* Assignee: 
* Target version: 
* ruby -v: 2.4.1
* Backport: 2.2: UNKNOWN, 2.3: UNKNOWN, 2.4: UNKNOWN
----------------------------------------
When you skip the parens of method invocations and pass a regex, you can get a `SyntaxError` if the regex starts with a space.

~~~ ruby
''.split(/ ./) # => []
''.split /./   # => []
''.split / ./  # !> SyntaxError: unexpected '.'
~~~

As per the syntax highlighting in my text editor and a discussion with another person, it seems that the opening slash is interpreted as division:

~~~ ruby
require 'ripper'

Ripper.lex("''.split /./")
  # =>
  # [[[1, 0], :on_tstring_beg, "'"],
  #  [[1, 1], :on_tstring_end, "'"],
  #  [[1, 2], :on_period, "."],
  #  [[1, 3], :on_ident, "split"],
  #  [[1, 8], :on_sp, " "],
  #  [[1, 9], :on_regexp_beg, "/"],
  #  #         ^^^
  #  [[1, 10], :on_tstring_content, "."],
  #  [[1, 11], :on_regexp_end, "/"]]

Ripper.lex("''.split / ./")
  # =>
  # [[[1, 0], :on_tstring_beg, "'"],
  #  [[1, 1], :on_tstring_end, "'"],
  #  [[1, 2], :on_period, "."],
  #  [[1, 3], :on_ident, "split"],
  #  [[1, 8], :on_sp, " "],
  #  [[1, 9], :on_op, "/"],
  #  #         ^^^
  #  [[1, 10], :on_sp, " "],
  #  [[1, 11], :on_period, "."],
  #  [[1, 12], :on_op, "/"]]
~~~


Reproducible with older versions as well (tried it until 2.1.2)



-- 
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>

In This Thread

Prev Next