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

Unified Diff: src/string.js

Issue 1559006: Remove trailing regexp from .js files. (Closed)
Patch Set: Created 10 years, 9 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 | « src/runtime.js ('k') | src/uri.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/string.js
diff --git a/src/string.js b/src/string.js
index 77c54c20823ae68f507920ec88231ef3b56aadde..302dcdeb2e91fcfd3b0083c0d5069ce0da407905 100644
--- a/src/string.js
+++ b/src/string.js
@@ -406,7 +406,7 @@ function addCaptureString(builder, matchInfo, index) {
};
// TODO(lrn): This array will survive indefinitely if replace is never
-// called again. However, it will be empty, since the contents are cleared
+// called again. However, it will be empty, since the contents are cleared
// in the finally block.
var reusableReplaceArray = $Array(16);
@@ -427,16 +427,16 @@ function StringReplaceRegExpWithFunction(subject, regexp, replace) {
try {
// Must handle exceptions thrown by the replace functions correctly,
// including unregistering global regexps.
- var res = %RegExpExecMultiple(regexp,
- subject,
- lastMatchInfo,
+ var res = %RegExpExecMultiple(regexp,
+ subject,
+ lastMatchInfo,
resultArray);
regexp.lastIndex = 0;
if (IS_NULL(res)) {
- // No matches at all.
- return subject;
- }
- var len = res.length;
+ // No matches at all.
+ return subject;
+ }
+ var len = res.length;
var i = 0;
if (NUMBER_OF_CAPTURES(lastMatchInfo) == 2) {
var match_start = 0;
@@ -457,8 +457,8 @@ function StringReplaceRegExpWithFunction(subject, regexp, replace) {
match_start += elem.length;
}
i++;
- }
- } else {
+ }
+ } else {
while (i < len) {
var elem = res[i];
if (!%_IsSmi(elem)) {
@@ -484,7 +484,7 @@ function StringReplaceRegExpWithFunction(subject, regexp, replace) {
} else { // Not a global regexp, no need to loop.
var matchInfo = DoRegExpExec(regexp, subject, 0);
if (IS_NULL(matchInfo)) return subject;
-
+
var result = new ReplaceResultBuilder(subject);
result.addSpecialSlice(0, matchInfo[CAPTURE0]);
var endOfMatch = matchInfo[CAPTURE1];
@@ -520,7 +520,7 @@ function ApplyReplacementFunction(replace, matchInfo, subject) {
// ECMA-262 section 15.5.4.12
-function StringSearch(re) {
+function StringSearch(re) {
var regexp = new $RegExp(re);
var s = TO_STRING_INLINE(this);
var match = DoRegExpExec(regexp, s, 0);
« no previous file with comments | « src/runtime.js ('k') | src/uri.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698