Feature #8343: StringScanner#[] should accept names of named captures

• Status: Closed
• Priority: Normal
• Assignee: naruse (Yui NARUSE)
• Target version: -

[rkh (Konstantin Haase)]

Example:

s = StringScanner.new("Fri Dec 12 1975 14:39")
s.scan(/(?<wday>\w+) (?<month>\w+) (?<day>\d+) /)

# this currently works
s[0]                               # -> "Fri Dec 12 "
s[1]                               # -> "Fri"
s[2]                               # -> "Dec"
s[3]                               # -> "12"

# this currently does not work
s[:wday]                           # -> "Fri"
s[:month]                          # -> "Dec"
s[:day]                            # -> "12"

I attached a patch including tests for MRI, I don't know if Rubinius has a different implementation, I guess JRuby has.
I can look into this if the feature gets accepted.

[rkh (Konstantin Haase)]

On a related note: should the patch be submitted in a second issue on the ruby-trunk project?

[naruse (Yui NARUSE)]

• Status changed from Open to Assigned
• Assignee set to naruse (Yui NARUSE)

rkh (Konstantin Haase) wrote:

On a related note: should the patch be submitted in a second issue on the ruby-trunk project?

Here is also correct place now.
I'll merge this if there are no objection.

[naruse (Yui NARUSE)]

• Status changed from Assigned to Closed
• Project changed from CommonRuby to Ruby master

Merged at r40881!