[macruby-changes] [3738] MacRuby/branches/icu/re.cpp

source_changes at macosforge.org source_changes at macosforge.org
Thu Mar 11 14:07:20 PST 2010


Revision: 3738
          http://trac.macosforge.org/projects/ruby/changeset/3738
Author:   lsansonetti at apple.com
Date:     2010-03-11 14:07:20 -0800 (Thu, 11 Mar 2010)
Log Message:
-----------
revert latest change

Modified Paths:
--------------
    MacRuby/branches/icu/re.cpp

Modified: MacRuby/branches/icu/re.cpp
===================================================================
--- MacRuby/branches/icu/re.cpp	2010-03-11 22:01:01 UTC (rev 3737)
+++ MacRuby/branches/icu/re.cpp	2010-03-11 22:07:20 UTC (rev 3738)
@@ -99,7 +99,7 @@
 
     rb_str_get_uchars(str, &chars, &chars_len, &need_free);
 
-    UnicodeString *unistr = new UnicodeString(false, (const UChar *)chars, chars_len);
+    UnicodeString *unistr = new UnicodeString((const UChar *)chars, chars_len);
 
     if (need_free) {
 	free(chars);
@@ -132,8 +132,9 @@
 
     sanitize_regexp_string(unistr);
 
+    UParseError pe;
     UErrorCode status = U_ZERO_ERROR;
-    RegexPattern *pattern = RegexPattern::compile(*unistr, option, status);
+    RegexPattern *pattern = RegexPattern::compile(*unistr, option, pe, status);
 
     if (pattern == NULL) {
 	delete unistr;
@@ -149,6 +150,7 @@
     regexp_finalize(regexp);
     regexp->pattern = pattern;
     regexp->unistr = unistr;
+
     return true;
 }
 
@@ -561,11 +563,11 @@
     if (NIL_P(match) || FL_TEST(match, MATCH_BUSY)) {
 	// Creating a new Match object.
 	match = (VALUE)match_alloc(rb_cMatch, 0);
+	rb_backref_set(match);
 	res = (rb_match_result_t *)xmalloc(sizeof(rb_match_result_t)
 		* res_count);
 	GC_WB(&RMATCH(match)->results, res);
 	GC_WB(&RMATCH(match)->str, rb_str_new(NULL, 0));
-	rb_backref_set(match);
     }
     else {
 	// Reusing the previous Match object.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.macosforge.org/pipermail/macruby-changes/attachments/20100311/d73fce94/attachment-0001.html>


More information about the macruby-changes mailing list