Skip to content
This repository has been archived by the owner on Jan 3, 2020. It is now read-only.

Add more Regexp, MatchData, and String APIs #117

Merged
merged 1 commit into from
Jun 14, 2019
Merged

Conversation

lopopolo
Copy link
Member

This commit is in support of GH-85.

Add the following APIs:

  • Regexp::escape
  • Regexp::union
  • Regexp#===
  • Regexp#names
  • Regexp#named_captures
  • Regexp#options
  • MatchData#length
  • MatchData#named_captures
  • String#scan
  • String#-@

Added the constants Regexp::IGNORECASE, Regexp::EXTENDED, and
Regexp::MULTILINE.

Regexp#to_s and Regexp#inspect were erroneously implemented with the
same function. Regexp#to_s now supports roundtripping through
Regexp::compile.

Imported escape functionality from Rust regex crate.

Fix null pointer exception in String#=~ when Regexp does not match.

Reimplemented the following String APIs to work with Regexp patterns:

  • String#split (TODO: does not handle blocks)
  • String#sub
  • String#gsub
  • String#gsub!

This commit is in support of GH-85.

Add the following APIs:

- Regexp::escape
- Regexp::union
- Regexp#===
- Regexp#names
- Regexp#named_captures
- Regexp#options
- MatchData#length
- MatchData#named_captures
- String#scan
- String#-@

Added the constants Regexp::IGNORECASE, Regexp::EXTENDED, and
Regexp::MULTILINE.

Regexp#to_s and Regexp#inspect were erroneously implemented with the
same function. Regexp#to_s now supports roundtripping through
Regexp::compile.

Imported escape functionality from Rust regex crate.

Fix null pointer exception in String#=~ when Regexp does not match.

Reimplemented the following String APIs to work with Regexp patterns:

- String#split (TODO: does not handle blocks)
- String#sub
- String#gsub
- String#gsub!
@lopopolo lopopolo merged commit d6a4e5f into master Jun 14, 2019
@lopopolo lopopolo deleted the more-regexp-apis branch June 14, 2019 09:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant