Regex class: fix build error and, modify default enable. #454

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
1 participant
@ghost

ghost commented Sep 6, 2012

  • fix build error.
  • implement method
    • ::new
    • #match
  • add enc/ascii.c file from oniguruma-5.9.2
  • but Regex literal doesn't yet support.

Sample code:

puts "### Sample.1 ###"
str = "this is regexp"
rp = Regexp.new("^this is regexp")
p str == str # => true
p rp.match(str) # => #<MatchData "this is regexp">
p Regexp.last_match[0] #=> "this is regexp"

puts ""
puts "### Sample.2 ###"
str = "THIS IS REGEXP"
t1 = Regexp.compile("^this is regexp", Regexp::IGNORECASE)
t = t1.match(str)
puts t # => THIS IS REGEXP

Kouki Ooyatsu added some commits Sep 4, 2012

Kouki Ooyatsu Regex class: fix build error and, modify default enable.
* fix build error.
* implement method
  - ::new
  - #match
* add enc/ascii.c file from oniguruma-5.9.2
3d9e36e
Kouki Ooyatsu Including src/ext/enc directory at build. 824e8eb
Contributor

mauceri commented Nov 17, 2012

Hi Kouki, it would be great your work is merged on the main branch!

Contributor

mauceri commented Nov 17, 2012

It is important I believe to merge this branch because one can keep

 #define DISABLE_REGEXP         /* regular expression classes */

in the settings. It is painful to merge by hand this branch in order to keep on track with the main branch.

@ghost

ghost commented Jan 10, 2013

Hi, mauceri.
Thank you for your comment.

It is important I believe to merge this branch because one can keep

 #define DISABLE_REGEXP         /* regular expression classes */
in the settings. It is painful to merge by hand this branch in order to keep on track with the main branch.

I think so too. but I will close this pull-req is so old.
please refer to #719. (thank you @mattn)

ghost closed this Jan 10, 2013

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment