Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(138)

Unified Diff: patches/rematch.patch

Issue 7886048: Fixing an oversite in my first fix. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/deps/third_party/icu46/
Patch Set: Created 9 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | source/i18n/rematch.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: patches/rematch.patch
===================================================================
--- patches/rematch.patch (revision 101167)
+++ patches/rematch.patch (working copy)
@@ -10,7 +10,7 @@
#ifdef REGEX_SMART_BACKTRACKING
int32_t originalInputIdx = fp->fInputIdx;
-@@ -5607,23 +5608,29 @@
+@@ -5607,23 +5608,31 @@
foldOffset = foldLength = 0;
while (patternChars < patternEnd && success) {
@@ -46,6 +46,8 @@
+ }
}
}
++ } else {
++ c_is_valid = FALSE;
}
- if (fp->fInputIdx <= fActiveLimit) {
@@ -53,7 +55,7 @@
if (U_IS_BMP(c)) {
success = (*patternChars == c);
patternChars += 1;
-@@ -6070,4 +6077,3 @@
+@@ -6070,4 +6079,3 @@
U_NAMESPACE_END
#endif // !UCONFIG_NO_REGULAR_EXPRESSIONS
« no previous file with comments | « no previous file | source/i18n/rematch.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698