Revision: 9094
Author: [email protected]
Date: Thu Sep 1 04:28:10 2011
Log: Added check for trailing whitespaces and corrected existing
violations.
Review URL: http://codereview.chromium.org/7826007
http://code.google.com/p/v8/source/detail?r=9094
Modified:
/branches/bleeding_edge/benchmarks/crypto.js
/branches/bleeding_edge/benchmarks/earley-boyer.js
/branches/bleeding_edge/benchmarks/regexp.js
/branches/bleeding_edge/build/all.gyp
/branches/bleeding_edge/include/v8.h
/branches/bleeding_edge/src/d8.js
/branches/bleeding_edge/src/macros.py
/branches/bleeding_edge/test/es5conform/es5conform.status
/branches/bleeding_edge/test/mjsunit/array-constructor.js
/branches/bleeding_edge/test/mjsunit/array-iteration.js
/branches/bleeding_edge/test/mjsunit/array-sort.js
/branches/bleeding_edge/test/mjsunit/bugs/618.js
/branches/bleeding_edge/test/mjsunit/bugs/bug-618.js
/branches/bleeding_edge/test/mjsunit/compiler/global-accessors.js
/branches/bleeding_edge/test/mjsunit/const-redecl.js
/branches/bleeding_edge/test/mjsunit/date-parse.js
/branches/bleeding_edge/test/mjsunit/debug-compile-event.js
/branches/bleeding_edge/test/mjsunit/debug-evaluate-recursive.js
/branches/bleeding_edge/test/mjsunit/debug-handle.js
/branches/bleeding_edge/test/mjsunit/debug-listbreakpoints.js
/branches/bleeding_edge/test/mjsunit/debug-references.js
/branches/bleeding_edge/test/mjsunit/debug-return-value.js
/branches/bleeding_edge/test/mjsunit/debug-stepin-call-function-stub.js
/branches/bleeding_edge/test/mjsunit/debug-stepin-constructor.js
/branches/bleeding_edge/test/mjsunit/delete-in-with.js
/branches/bleeding_edge/test/mjsunit/function-source.js
/branches/bleeding_edge/test/mjsunit/get-own-property-descriptor.js
/branches/bleeding_edge/test/mjsunit/global-deleted-property-keyed.js
/branches/bleeding_edge/test/mjsunit/html-string-funcs.js
/branches/bleeding_edge/test/mjsunit/in.js
/branches/bleeding_edge/test/mjsunit/instanceof.js
/branches/bleeding_edge/test/mjsunit/keyed-storage-extend.js
/branches/bleeding_edge/test/mjsunit/mirror-array.js
/branches/bleeding_edge/test/mjsunit/mirror-function.js
/branches/bleeding_edge/test/mjsunit/mirror-script.js
/branches/bleeding_edge/test/mjsunit/mirror-unresolved-function.js
/branches/bleeding_edge/test/mjsunit/no-semicolon.js
/branches/bleeding_edge/test/mjsunit/object-define-properties.js
/branches/bleeding_edge/test/mjsunit/object-literal-conversions.js
/branches/bleeding_edge/test/mjsunit/object-literal-overwrite.js
/branches/bleeding_edge/test/mjsunit/object-prevent-extensions.js
/branches/bleeding_edge/test/mjsunit/regress/regress-1081309.js
/branches/bleeding_edge/test/mjsunit/regress/regress-1092.js
/branches/bleeding_edge/test/mjsunit/regress/regress-1110.js
/branches/bleeding_edge/test/mjsunit/regress/regress-1213575.js
/branches/bleeding_edge/test/mjsunit/regress/regress-1919169.js
/branches/bleeding_edge/test/mjsunit/regress/regress-20070207.js
/branches/bleeding_edge/test/mjsunit/regress/regress-269.js
/branches/bleeding_edge/test/mjsunit/regress/regress-619.js
/branches/bleeding_edge/test/mjsunit/regress/regress-678525.js
/branches/bleeding_edge/test/mjsunit/regress/regress-696.js
/branches/bleeding_edge/test/mjsunit/regress/regress-720.js
/branches/bleeding_edge/test/mjsunit/regress/regress-747.js
/branches/bleeding_edge/test/mjsunit/regress/regress-760-1.js
/branches/bleeding_edge/test/mjsunit/regress/regress-760-2.js
/branches/bleeding_edge/test/mjsunit/regress/regress-798.js
/branches/bleeding_edge/test/mjsunit/regress/regress-918.js
/branches/bleeding_edge/test/mjsunit/regress/regress-925537.js
/branches/bleeding_edge/test/mjsunit/regress/regress-937896.js
/branches/bleeding_edge/test/mjsunit/setter-on-constructor-prototype.js
/branches/bleeding_edge/test/mjsunit/string-compare-alignment.js
/branches/bleeding_edge/test/mjsunit/string-indexof-1.js
/branches/bleeding_edge/test/mjsunit/string-indexof-2.js
/branches/bleeding_edge/test/mjsunit/string-slices.js
/branches/bleeding_edge/test/mjsunit/string-split.js
/branches/bleeding_edge/test/mjsunit/substr.js
/branches/bleeding_edge/test/mjsunit/this-property-assignment.js
/branches/bleeding_edge/test/mjsunit/try.js
/branches/bleeding_edge/test/mjsunit/unicode-test.js
/branches/bleeding_edge/test/mjsunit/value-wrapper.js
/branches/bleeding_edge/test/mozilla/mozilla.status
/branches/bleeding_edge/tools/gdb-v8-support.py
/branches/bleeding_edge/tools/presubmit.py
/branches/bleeding_edge/tools/process-heap-prof.py
=======================================
--- /branches/bleeding_edge/benchmarks/crypto.js Tue Dec 7 03:01:02 2010
+++ /branches/bleeding_edge/benchmarks/crypto.js Thu Sep 1 04:28:10 2011
@@ -1406,7 +1406,7 @@
// Mix in the current time (w/milliseconds) into the pool
function rng_seed_time() {
- // Use pre-computed date to avoid making the benchmark
+ // Use pre-computed date to avoid making the benchmark
// results dependent on the current date.
rng_seed_int(1122926989487);
}
=======================================
--- /branches/bleeding_edge/benchmarks/earley-boyer.js Tue Dec 7 03:01:02
2010
+++ /branches/bleeding_edge/benchmarks/earley-boyer.js Thu Sep 1 04:28:10
2011
@@ -134,7 +134,7 @@
/*** META ((export #t)) */
function sc_any2String(o) {
return jsstring2string(sc_toDisplayString(o));
-}
+}
/*** META ((export #t)
(peephole (infix 2 2 "==="))
@@ -923,7 +923,7 @@
tmp.cdr = l2;
return l1;
}
-
+
/*** META ((export #t)) */
function sc_appendBang() {
var res = null;
@@ -1163,7 +1163,7 @@
"us": "\037",
"sp": "\040",
"del": "\177"};
-
+
sc_Char.prototype.toString = function() {
return this.val;
};
@@ -1533,7 +1533,7 @@
}
return l1_orig;
}
-
+
/*** META ((export #t)) */
function sc_forEach(proc, l1) {
if (l1 === undefined)
@@ -1871,7 +1871,7 @@
evalStr += ", arguments[" + i + "]";
evalStr +=")";
return eval(evalStr);
-}
+}
// ======================== RegExp ====================
/*** META ((export #t)) */
@@ -1883,9 +1883,9 @@
function sc_pregexpMatch(re, s) {
var reg = (re instanceof RegExp) ? re : sc_pregexp(re);
var tmp = reg.exec(sc_string2jsstring(s));
-
+
if (tmp == null) return false;
-
+
var res = null;
for (var i = tmp.length-1; i >= 0; i--) {
if (tmp[i] !== null) {
@@ -1896,7 +1896,7 @@
}
return res;
}
-
+
/*** META ((export #t)) */
function sc_pregexpReplace(re, s1, s2) {
var reg;
@@ -1914,7 +1914,7 @@
return jss1.replace(reg, jss2);
}
-
+
/*** META ((export pregexp-replace*)) */
function sc_pregexpReplaceAll(re, s1, s2) {
var reg;
@@ -1945,7 +1945,7 @@
return sc_vector2list(tmp);
}
-
+
/*
===========================================================================
*/
/* Other library stuff */
@@ -2136,7 +2136,7 @@
sc_ErrorInputPort.prototype.isCharReady = function() {
return false;
};
-
+
/* .............. String port ..........................*/
@@ -2200,7 +2200,7 @@
};
sc_Tokenizer.prototype.nextToken = function() {
var port = this.port;
-
+
function isNumberChar(c) {
return (c >= "0" && c <= "9");
};
@@ -2280,7 +2280,7 @@
else
return new sc_Token(12/*NUMBER*/, res - 0);
};
-
+
function skipWhitespaceAndComments() {
var done = false;
while (!done) {
@@ -2299,7 +2299,7 @@
}
}
};
-
+
function readDot() {
if (isWhitespace(port.peekChar()))
return new sc_Token(10/*DOT*/);
@@ -2429,7 +2429,7 @@
while (true) {
var token = tokenizer.peekToken();
-
+
switch (token.type) {
case 2/*CLOSE_PAR*/:
case 4/*CLOSE_BRACE*/:
@@ -2491,7 +2491,7 @@
else
throw "bad reference: " + nb;
};
-
+
var tokenizer = this.tokenizer;
var token = tokenizer.readToken();
@@ -2499,7 +2499,7 @@
// handle error
if (token.type === 13/*ERROR*/)
throw token.val;
-
+
switch (token.type) {
case 1/*OPEN_PAR*/:
case 3/*OPEN_BRACE*/:
@@ -2550,7 +2550,7 @@
port = SC_DEFAULT_IN; // THREAD: shared var...
var t = port.peekChar();
return t === SC_EOF_OBJECT? t: new sc_Char(t);
-}
+}
/*** META ((export #t)
(type bool))
*/
@@ -2722,7 +2722,7 @@
function sc_getOutputString(sp) {
return sc_jsstring2string(sp.res);
}
-
+
function sc_ErrorOutputPort() {
}
@@ -2852,7 +2852,7 @@
p = SC_DEFAULT_OUT;
p.appendJSString("\n");
}
-
+
/* ------------------ write-char
---------------------------------------------------*/
/*** META ((export #t)) */
@@ -2927,7 +2927,7 @@
}
var res = "";
-
+
if (this[symb] !== undefined) { // implies > 0
this[symb + "use"] = true;
if (inList)
@@ -2939,10 +2939,10 @@
if (!inList)
res += "(";
-
+
// print car
res += sc_genToWriteCircleString(this.car, symb);
-
+
if (sc_isPair(this.cdr)) {
res += " " + this.cdr.sc_toWriteCircleString(symb, true);
} else if (this.cdr !== null) {
@@ -3072,7 +3072,7 @@
p.appendJSString(arguments[j].toString(2));
i += 2; j++;
break;
-
+
case 37:
case 110:
// %, n
@@ -3186,7 +3186,7 @@
function sc_number2symbol(x, radix) {
return sc_SYMBOL_PREFIX + sc_number2jsstring(x, radix);
}
-
+
/*** META ((export number->string integer->string)) */
var sc_number2string = sc_number2jsstring;
=======================================
--- /branches/bleeding_edge/benchmarks/regexp.js Tue Dec 7 03:01:02 2010
+++ /branches/bleeding_edge/benchmarks/regexp.js Thu Sep 1 04:28:10 2011
@@ -33,7 +33,7 @@
// the popularity of the pages where it occurs and the number of times
// it is executed while loading each page. Furthermore the literal
// letters in the data are encoded using ROT13 in a way that does not
-// affect how the regexps match their input. Finally the strings are
+// affect how the regexps match their input. Finally the strings are
// scrambled to exercise the regexp engine on different input strings.
@@ -47,7 +47,7 @@
regExpBenchmark = new RegExpBenchmark();
RegExpRun(); // run once to get system initialized
}
-
+
function RegExpRun() {
regExpBenchmark.run();
}
@@ -1759,6 +1759,6 @@
runBlock11();
}
}
-
+
this.run = run;
}
=======================================
--- /branches/bleeding_edge/build/all.gyp Wed Aug 31 07:15:42 2011
+++ /branches/bleeding_edge/build/all.gyp Thu Sep 1 04:28:10 2011
@@ -1,4 +1,4 @@
-# Copyright (c) 2011 The Chromium Authors. All rights reserved.
+# Copyright 2011 the V8 project authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
=======================================
--- /branches/bleeding_edge/include/v8.h Tue Aug 23 04:11:12 2011
+++ /branches/bleeding_edge/include/v8.h Thu Sep 1 04:28:10 2011
@@ -1656,7 +1656,7 @@
V8EXPORT bool IsCallable();
/**
- * Call an Object as a function if a callback is set by the
+ * Call an Object as a function if a callback is set by the
* ObjectTemplate::SetCallAsFunctionHandler method.
*/
V8EXPORT Local<Value> CallAsFunction(Handle<Object> recv,
@@ -3562,7 +3562,7 @@
* // V8 Now no longer locked.
* \endcode
*
- *
+ *
*/
class V8EXPORT Unlocker {
public:
=======================================
--- /branches/bleeding_edge/src/d8.js Thu Aug 18 22:24:39 2011
+++ /branches/bleeding_edge/src/d8.js Thu Sep 1 04:28:10 2011
@@ -1786,7 +1786,7 @@
function decodeLolListResponse(body, title) {
-
+
var result;
var total_count = body.count;
var total_size = body.size;
=======================================
--- /branches/bleeding_edge/src/macros.py Mon Jul 25 08:01:45 2011
+++ /branches/bleeding_edge/src/macros.py Thu Sep 1 04:28:10 2011
@@ -170,7 +170,7 @@
const CAPTURE0 = 3;
const CAPTURE1 = 4;
-# PropertyDescriptor return value indices - must match
+# PropertyDescriptor return value indices - must match
# PropertyDescriptorIndices in runtime.cc.
const IS_ACCESSOR_INDEX = 0;
const VALUE_INDEX = 1;
=======================================
--- /branches/bleeding_edge/test/es5conform/es5conform.status Wed Aug 3
02:53:14 2011
+++ /branches/bleeding_edge/test/es5conform/es5conform.status Thu Sep 1
04:28:10 2011
@@ -75,11 +75,11 @@
# NOT IMPLEMENTED: RegExp.prototype.multiline
chapter15/15.2/15.2.3/15.2.3.3/15.2.3.3-4-215: UNIMPLEMENTED
-# All of the tests below marked SUBSETFAIL (in 15.2.3.4) fail because
+# All of the tests below marked SUBSETFAIL (in 15.2.3.4) fail because
# the tests assumes that objects can not have more properties
-# than those described in the spec - but according to spec they can
+# than those described in the spec - but according to spec they can
# have additional properties.
-# All compareArray calls in these tests could be exchanged with a
+# All compareArray calls in these tests could be exchanged with a
# isSubsetOfArray call (I will upload a patch to the es5conform site).
# SUBSETFAIL
=======================================
--- /branches/bleeding_edge/test/mjsunit/array-constructor.js Fri Mar 18
12:41:05 2011
+++ /branches/bleeding_edge/test/mjsunit/array-constructor.js Thu Sep 1
04:28:10 2011
@@ -73,7 +73,7 @@
a = new Array(0, 1, 2, 3, 4, 5, 6, 7, 8);
assertArrayEquals([0, 1, 2, 3, 4, 5, 6, 7, 8], a);
a = new Array(0, 1, 2, 3, 4, 5, 6, 7, 8, 9);
- assertArrayEquals([0, 1, 2, 3, 4, 5, 6, 7, 8, 9], a);
+ assertArrayEquals([0, 1, 2, 3, 4, 5, 6, 7, 8, 9], a);
}
@@ -91,9 +91,9 @@
var a = eval('[' + str + ']');
var b = eval('new Array(' + str + ')')
var c = eval('Array(' + str + ')')
- assertEquals(n, a.length);
- assertArrayEquals(a, b);
- assertArrayEquals(a, c);
+ assertEquals(n, a.length);
+ assertArrayEquals(a, b);
+ assertArrayEquals(a, c);
}
=======================================
--- /branches/bleeding_edge/test/mjsunit/array-iteration.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/array-iteration.js Thu Sep 1
04:28:10 2011
@@ -134,7 +134,7 @@
a = [0,1];
assertFalse(a.every(function(n, index, array) { array[index] = n + 1;
return n == 1;}));
assertArrayEquals([1,1], a);
-
+
// Only loop through initial part of array eventhough elements are
// added.
a = [1,1];
@@ -156,23 +156,23 @@
//
(function() {
var a = [0,1,2,3,4];
-
+
// Simple use.
var result = [1,2,3,4,5];
assertArrayEquals(result, a.map(function(n) { return n + 1; }));
assertEquals(a, a);
-
+
// Use specified object as this object when calling the function.
var o = { delta: 42 }
result = [42,43,44,45,46];
assertArrayEquals(result, a.map(function(n) { return this.delta + n; },
o));
-
+
// Modify original array.
a = [0,1,2,3,4];
result = [1,2,3,4,5];
assertArrayEquals(result, a.map(function(n, index, array) { array[index]
= n + 1; return n + 1;}));
assertArrayEquals(result, a);
-
+
// Only loop through initial part of array eventhough elements are
// added.
a = [0,1,2,3,4];
@@ -197,7 +197,7 @@
// Simple use.
assertTrue(a.some(function(n) { return n == 3}));
assertFalse(a.some(function(n) { return n == 5}));
-
+
// Use specified object as this object when calling the function.
var o = { element: 42 };
a = [1,42,3];
=======================================
--- /branches/bleeding_edge/test/mjsunit/array-sort.js Wed Jun 29 01:35:10
2011
+++ /branches/bleeding_edge/test/mjsunit/array-sort.js Thu Sep 1 04:28:10
2011
@@ -392,7 +392,7 @@
// Test that sort calls compare function with global object as receiver,
// and with only elements of the array as arguments.
-function o(v) {
+function o(v) {
return {__proto__: o.prototype, val: v};
}
var arr = [o(1), o(2), o(4), o(8), o(16), o(32), o(64), o(128), o(256),
o(-0)];
=======================================
--- /branches/bleeding_edge/test/mjsunit/bugs/618.js Tue Dec 7 03:01:02
2010
+++ /branches/bleeding_edge/test/mjsunit/bugs/618.js Thu Sep 1 04:28:10
2011
@@ -32,14 +32,14 @@
var c1 = new C1();
assertEquals(23, c1.x);
assertEquals("undefined", typeof c1.y);
-
+
// Add setter somewhere on the prototype chain after having constructed the
// first instance.
C1.prototype = { set x(value) { this.y = 23; } };
var c1 = new C1();
assertEquals("undefined", typeof c1.x);
assertEquals(23, c1.y);
-
+
// Simple class using inline constructor.
function C2() {
this.x = 23;
=======================================
--- /branches/bleeding_edge/test/mjsunit/bugs/bug-618.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/bugs/bug-618.js Thu Sep 1
04:28:10 2011
@@ -33,11 +33,11 @@
this.x = 23;
}
-// If a setter is added to the prototype chain of a simple constructor
setting
-// one of the properties assigned in the constructor then this setter is
+// If a setter is added to the prototype chain of a simple constructor
setting
+// one of the properties assigned in the constructor then this setter is
// ignored when constructing new objects from the constructor.
-// This only happens if the setter is added _after_ an instance has been
+// This only happens if the setter is added _after_ an instance has been
// created.
assertEquals(23, new C().x);
=======================================
--- /branches/bleeding_edge/test/mjsunit/compiler/global-accessors.js Fri
Apr 1 04:54:04 2011
+++ /branches/bleeding_edge/test/mjsunit/compiler/global-accessors.js Thu
Sep 1 04:28:10 2011
@@ -26,7 +26,7 @@
// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
// This test tests that no bailouts are missing by not hitting asserts in
debug
-// mode.
+// mode.
test_count_operation()
test_compound_assignment()
=======================================
--- /branches/bleeding_edge/test/mjsunit/const-redecl.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/const-redecl.js Thu Sep 1
04:28:10 2011
@@ -55,7 +55,7 @@
function TestGlobal(s,e) {
// Collect the global properties before the call.
var properties = [];
- for (var key in this) properties.push(key);
+ for (var key in this) properties.push(key);
// Compute the result.
var result;
try {
@@ -113,7 +113,7 @@
// Eval second definition.
TestAll("TypeError", def0 + '; eval("' + def1 + '")');
// Eval both definitions separately.
- TestAll("TypeError", 'eval("' + def0 +'"); eval("' + def1 + '")');
+ TestAll("TypeError", 'eval("' + def0 +'"); eval("' + def1 + '")');
}
=======================================
--- /branches/bleeding_edge/test/mjsunit/date-parse.js Fri Jul 1 04:41:45
2011
+++ /branches/bleeding_edge/test/mjsunit/date-parse.js Thu Sep 1 04:28:10
2011
@@ -286,7 +286,7 @@
// Negative tests.
var testCasesNegative = [
'May 25 2008 1:30 (PM)) UTC', // Bad unmatched ')' after number.
- 'May 25 2008 1:30( )AM (PM)', //
+ 'May 25 2008 1:30( )AM (PM)', //
'May 25 2008 AAA (GMT)']; // Unknown word after number.
testCasesNegative.forEach(function (s) {
=======================================
--- /branches/bleeding_edge/test/mjsunit/debug-compile-event.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/debug-compile-event.js Thu Sep 1
04:28:10 2011
@@ -81,7 +81,7 @@
assertTrue('context' in msg.body.script);
// Check that we pick script name from //@ sourceURL, iff present
- assertEquals(current_source.indexOf('sourceURL') >= 0 ?
+ assertEquals(current_source.indexOf('sourceURL') >= 0 ?
'myscript.js' : undefined,
event_data.script().name());
}
=======================================
--- /branches/bleeding_edge/test/mjsunit/debug-evaluate-recursive.js Tue
Dec 7 03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/debug-evaluate-recursive.js Thu
Sep 1 04:28:10 2011
@@ -110,7 +110,7 @@
if (event == Debug.DebugEvent.Break)
{
break_count++;
-
+
// Call functions with break using the FrameMirror directly.
if (break_count == 1) {
// First break event evaluates with break enabled.
=======================================
--- /branches/bleeding_edge/test/mjsunit/debug-handle.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/debug-handle.js Thu Sep 1
04:28:10 2011
@@ -72,7 +72,7 @@
// The base part of all lookup requests.
var base_request = '"seq":0,"type":"request","command":"lookup"'
-
+
// Generate request with the supplied arguments.
var request;
if (arguments) {
@@ -214,7 +214,7 @@
'Handle not in the request: ' + handle);
count++;
}
- assertEquals(count, obj.properties.length,
+ assertEquals(count, obj.properties.length,
'Unexpected number of resolved objects');
=======================================
--- /branches/bleeding_edge/test/mjsunit/debug-listbreakpoints.js Fri Jan
7 03:59:41 2011
+++ /branches/bleeding_edge/test/mjsunit/debug-listbreakpoints.js Thu Sep
1 04:28:10 2011
@@ -39,7 +39,7 @@
// below. The test checks for these line numbers.
function g() { // line 40
- var x = 5;
+ var x = 5;
var y = 6;
var z = 7;
};
=======================================
--- /branches/bleeding_edge/test/mjsunit/debug-references.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/debug-references.js Thu Sep 1
04:28:10 2011
@@ -52,7 +52,7 @@
} else {
request = '{' + base_request + '}'
}
-
+
// Process the request and check expectation.
var response = safeEval(dcp.processDebugJSONRequest(request));
if (success) {
@@ -88,7 +88,7 @@
var response = safeEval(dcp.processDebugJSONRequest(evaluate_point));
assertTrue(response.success, "Evaluation of Point failed");
var handle = response.body.handle;
-
+
// Test some legal references requests.
testRequest(dcp, '{"handle":' + handle + ',"type":"referencedBy"}',
true);
testRequest(dcp, '{"handle":' + handle + ',"type":"constructedBy"}',
=======================================
--- /branches/bleeding_edge/test/mjsunit/debug-return-value.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/debug-return-value.js Thu Sep 1
04:28:10 2011
@@ -103,12 +103,12 @@
// Position at the end of the function.
assertEquals(debugger_source_position + 50,
exec_state.frame(0).sourcePosition());
-
+
// Just about to return from the function.
assertTrue(exec_state.frame(0).isAtReturn())
assertEquals(expected_return_value,
exec_state.frame(0).returnValue().value());
-
+
// Check the same using the JSON commands.
var dcp = exec_state.debugCommandProcessor(false);
var request = '{"seq":0,"type":"request","command":"backtrace"}';
@@ -118,7 +118,7 @@
assertTrue(frames[0].atReturn);
assertEquals(expected_return_value,
response.lookup(frames[0].returnValue.ref).value);
-
+
listener_complete = true;
}
}
@@ -132,7 +132,7 @@
// Four steps from the debugger statement in this function will position
us at
// the function return.
-// 0 1 2 3 4 5
+// 0 1 2 3 4 5
// 0123456789012345678901234567890123456789012345678901
function f(x) {debugger; if (x) { return 1; } else { return 2; } };
=======================================
--- /branches/bleeding_edge/test/mjsunit/debug-stepin-call-function-stub.js
Tue Dec 7 03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/debug-stepin-call-function-stub.js
Thu Sep 1 04:28:10 2011
@@ -62,7 +62,7 @@
Debug.setListener(listener);
-function g() {
+function g() {
return "s"; // expected line
}
@@ -71,7 +71,7 @@
var s = 1 +f(10);
}
-function g2() {
+function g2() {
return "s2"; // expected line
}
=======================================
--- /branches/bleeding_edge/test/mjsunit/debug-stepin-constructor.js Tue
Dec 7 03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/debug-stepin-constructor.js Thu
Sep 1 04:28:10 2011
@@ -38,7 +38,7 @@
if (exec_state.frameCount() > 1) {
exec_state.prepareStep(Debug.StepAction.StepIn);
}
-
+
// Test that there is a script.
assertTrue(typeof(event_data.func().script()) == 'object');
}
=======================================
--- /branches/bleeding_edge/test/mjsunit/delete-in-with.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/delete-in-with.js Thu Sep 1
04:28:10 2011
@@ -29,6 +29,6 @@
// objects from within 'with' statements.
(function(){
var tmp = { x: 12 };
- with (tmp) { assertTrue(delete x); }
+ with (tmp) { assertTrue(delete x); }
assertFalse("x" in tmp);
})();
=======================================
--- /branches/bleeding_edge/test/mjsunit/function-source.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/function-source.js Thu Sep 1
04:28:10 2011
@@ -36,7 +36,7 @@
}
h();
}
-
+
function g() {
function h() {
assertEquals(Debug.scriptSource(f), Debug.scriptSource(h));
=======================================
--- /branches/bleeding_edge/test/mjsunit/get-own-property-descriptor.js Thu
Feb 3 02:19:41 2011
+++ /branches/bleeding_edge/test/mjsunit/get-own-property-descriptor.js Thu
Sep 1 04:28:10 2011
@@ -27,7 +27,7 @@
// This file only tests very simple descriptors that always have
// configurable, enumerable, and writable set to true.
-// A range of more elaborate tests are performed in
+// A range of more elaborate tests are performed in
// object-define-property.js
function get() { return x; }
=======================================
--- /branches/bleeding_edge/test/mjsunit/global-deleted-property-keyed.js
Tue Dec 7 03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/global-deleted-property-keyed.js
Thu Sep 1 04:28:10 2011
@@ -33,6 +33,6 @@
var name = "fisk";
natives[name] = name;
function foo() { natives[name] + 12; }
-for(var i = 0; i < 3; i++) foo();
+for(var i = 0; i < 3; i++) foo();
delete natives[name];
for(var i = 0; i < 3; i++) foo();
=======================================
--- /branches/bleeding_edge/test/mjsunit/html-string-funcs.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/html-string-funcs.js Thu Sep 1
04:28:10 2011
@@ -29,7 +29,7 @@
// HTML.
function CheckSimple(f, tag) {
assertEquals('<' + tag + '>foo</' + tag + '>',
- "foo"[f]().toLowerCase());
+ "foo"[f]().toLowerCase());
};
var simple = { big: 'big', blink: 'blink', bold: 'b',
fixed: 'tt', italics: 'i', small: 'small',
=======================================
--- /branches/bleeding_edge/test/mjsunit/in.js Tue Dec 7 03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/in.js Thu Sep 1 04:28:10 2011
@@ -86,7 +86,7 @@
assertFalse(0 in a);
assertTrue(1 in a);
assertFalse(2 in a);
-assertFalse('0' in a);
+assertFalse('0' in a);
assertTrue('1' in a);
assertFalse('2' in a);
assertTrue('toString' in a, "toString");
=======================================
--- /branches/bleeding_edge/test/mjsunit/instanceof.js Tue Dec 7 03:01:02
2010
+++ /branches/bleeding_edge/test/mjsunit/instanceof.js Thu Sep 1 04:28:10
2011
@@ -60,10 +60,10 @@
function TestExceptions() {
function F() { }
- var items = [ 1, new Number(42),
- true,
+ var items = [ 1, new Number(42),
+ true,
'string', new String('hest'),
- {}, [],
+ {}, [],
F, new F(),
Object, String ];
=======================================
--- /branches/bleeding_edge/test/mjsunit/keyed-storage-extend.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/keyed-storage-extend.js Thu Sep 1
04:28:10 2011
@@ -37,7 +37,7 @@
}
function GrowKeyed(o) {
- var names = ['a','b','c','d','e','f'];
+ var names = ['a','b','c','d','e','f'];
var i = 0;
o[names[i++]] = i;
o[names[i++]] = i;
=======================================
--- /branches/bleeding_edge/test/mjsunit/mirror-array.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/mirror-array.js Thu Sep 1
04:28:10 2011
@@ -64,7 +64,7 @@
assertTrue(mirror.protoObject() instanceof debug.Mirror, 'Unexpected
mirror hierachy');
assertTrue(mirror.prototypeObject() instanceof debug.Mirror, 'Unexpected
mirror hierachy');
assertEquals(mirror.length(), a.length, "Length mismatch");
-
+
var indexedProperties = mirror.indexedPropertiesFromRange();
assertEquals(indexedProperties.length, a.length);
for (var i = 0; i < indexedProperties.length; i++) {
@@ -110,7 +110,7 @@
var found = false;
for (var j = 0; j < fromJSON.properties.length; j++) {
if (names[i] == fromJSON.properties[j].name) {
- found = true;
+ found = true;
}
}
assertTrue(found, names[i])
=======================================
--- /branches/bleeding_edge/test/mjsunit/mirror-function.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/mirror-function.js Thu Sep 1
04:28:10 2011
@@ -65,7 +65,7 @@
assertTrue(mirror.constructorFunction() instanceof debug.ObjectMirror);
assertTrue(mirror.protoObject() instanceof debug.Mirror);
assertTrue(mirror.prototypeObject() instanceof debug.Mirror);
-
+
// Test text representation
assertEquals(f.toString(), mirror.toText());
=======================================
--- /branches/bleeding_edge/test/mjsunit/mirror-script.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/mirror-script.js Thu Sep 1
04:28:10 2011
@@ -62,7 +62,7 @@
if (eval_from_line) {
assertEquals(eval_from_line, mirror.evalFromLocation().line);
}
-
+
// Parse JSON representation and check.
var fromJSON = JSON.parse(json);
assertEquals('script', fromJSON.type);
=======================================
--- /branches/bleeding_edge/test/mjsunit/mirror-unresolved-function.js Tue
Dec 7 03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/mirror-unresolved-function.js Thu
Sep 1 04:28:10 2011
@@ -64,7 +64,7 @@
assertEquals('undefined', mirror.constructorFunction().type());
assertEquals('undefined', mirror.protoObject().type());
assertEquals('undefined', mirror.prototypeObject().type());
-
+
// Parse JSON representation of unresolved functions and check.
var fromJSON = eval('(' + json + ')');
assertEquals('function', fromJSON.type, 'Unexpected mirror type in JSON');
=======================================
--- /branches/bleeding_edge/test/mjsunit/no-semicolon.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/no-semicolon.js Thu Sep 1
04:28:10 2011
@@ -30,7 +30,7 @@
function f() { return }
-function g() {
+function g() {
return
4;
}
=======================================
--- /branches/bleeding_edge/test/mjsunit/object-define-properties.js Tue
Dec 7 03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/object-define-properties.js Thu
Sep 1 04:28:10 2011
@@ -26,7 +26,7 @@
// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
// Tests the Object.defineProperties method - ES 15.2.3.7
-// Note that the internal DefineOwnProperty method is tested through
+// Note that the internal DefineOwnProperty method is tested through
// object-define-property.js, this file only contains tests specific for
// Object.defineProperties. Also note that object-create.js contains
// a range of indirect tests on this method since Object.create uses
=======================================
--- /branches/bleeding_edge/test/mjsunit/object-literal-conversions.js Fri
Mar 18 12:41:05 2011
+++ /branches/bleeding_edge/test/mjsunit/object-literal-conversions.js Thu
Sep 1 04:28:10 2011
@@ -43,4 +43,4 @@
assertEquals(7, test5[13]);
assertEquals(7, test6[17.31]);
-
+
=======================================
--- /branches/bleeding_edge/test/mjsunit/object-literal-overwrite.js Fri
Mar 18 12:41:05 2011
+++ /branches/bleeding_edge/test/mjsunit/object-literal-overwrite.js Thu
Sep 1 04:28:10 2011
@@ -79,7 +79,7 @@
// Test for the classic code generator.
function fun(x) {
- var inner = { j: function(x) { return x; }, j: 7 };
+ var inner = { j: function(x) { return x; }, j: 7 };
return inner.j;
}
=======================================
--- /branches/bleeding_edge/test/mjsunit/object-prevent-extensions.js Sun
Mar 27 23:11:08 2011
+++ /branches/bleeding_edge/test/mjsunit/object-prevent-extensions.js Thu
Sep 1 04:28:10 2011
@@ -33,7 +33,7 @@
assertTrue(Object.isExtensible(obj1));
Object.preventExtensions(obj1);
-// Make sure the is_extensible flag is set.
+// Make sure the is_extensible flag is set.
assertFalse(Object.isExtensible(obj1));
obj1.x = 42;
assertEquals(undefined, obj1.x);
=======================================
--- /branches/bleeding_edge/test/mjsunit/regress/regress-1081309.js Tue
Dec 7 03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/regress/regress-1081309.js Thu
Sep 1 04:28:10 2011
@@ -67,7 +67,7 @@
// The expected backtrace is
// 1: g
// 0: [anonymous]
-
+
// Get the debug command processor.
var dcp = exec_state.debugCommandProcessor(false);
=======================================
--- /branches/bleeding_edge/test/mjsunit/regress/regress-1092.js Thu Feb 3
11:29:10 2011
+++ /branches/bleeding_edge/test/mjsunit/regress/regress-1092.js Thu Sep 1
04:28:10 2011
@@ -25,7 +25,7 @@
// (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-// Test that CodeGenerator::EmitKeyedPropertyAssignment for the start
+// Test that CodeGenerator::EmitKeyedPropertyAssignment for the start
// of an initialization block doesn't normalize the properties of the
// JSGlobalProxy.
this.w = 0;
=======================================
--- /branches/bleeding_edge/test/mjsunit/regress/regress-1110.js Tue Feb 8
10:46:13 2011
+++ /branches/bleeding_edge/test/mjsunit/regress/regress-1110.js Thu Sep 1
04:28:10 2011
@@ -29,7 +29,7 @@
try {
function Crash() { continue;if (Crash) {
- } }
+ } }
Crash();
assertTrue(false);
} catch (e) {
=======================================
--- /branches/bleeding_edge/test/mjsunit/regress/regress-1213575.js Tue
Dec 7 03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/regress/regress-1213575.js Thu
Sep 1 04:28:10 2011
@@ -33,7 +33,7 @@
var caught = false;
try {
- eval('const x');
+ eval('const x');
} catch(e) {
assertTrue(e instanceof TypeError);
caught = true;
=======================================
--- /branches/bleeding_edge/test/mjsunit/regress/regress-1919169.js Tue
Dec 7 03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/regress/regress-1919169.js Thu
Sep 1 04:28:10 2011
@@ -30,7 +30,7 @@
var s2 = "s2";
for (var i = 0; i < 2; i++) {
// Crashes in round i==1 with IllegalAccess in %StringAdd(x,y)
- var res = 1 + s2;
+ var res = 1 + s2;
s2 = 2;
}
}
=======================================
--- /branches/bleeding_edge/test/mjsunit/regress/regress-20070207.js Tue
Dec 7 03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/regress/regress-20070207.js Thu
Sep 1 04:28:10 2011
@@ -26,7 +26,7 @@
// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
// The following regression test illustrates a problem in using the
-// value of setting a property in the arguments object.
+// value of setting a property in the arguments object.
function f(s) {
arguments.length;
=======================================
--- /branches/bleeding_edge/test/mjsunit/regress/regress-269.js Fri Mar 18
12:41:05 2011
+++ /branches/bleeding_edge/test/mjsunit/regress/regress-269.js Thu Sep 1
04:28:10 2011
@@ -40,7 +40,7 @@
function g() {
}
-
+
function f() {
debugger;
g.apply(null, ['']);
=======================================
--- /branches/bleeding_edge/test/mjsunit/regress/regress-619.js Tue Dec 7
03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/regress/regress-619.js Thu Sep 1
04:28:10 2011
@@ -25,7 +25,7 @@
// (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-// Tests that Object.defineProperty works correctly on array indices.
+// Tests that Object.defineProperty works correctly on array indices.
// Please see http://code.google.com/p/v8/issues/detail?id=619 for details.
var obj = {};
=======================================
--- /branches/bleeding_edge/test/mjsunit/regress/regress-678525.js Tue Dec
7 03:01:02 2010
+++ /branches/bleeding_edge/test/mjsunit/regress/regress-678525.js Thu Sep
1 04:28:10 2011
@@ -36,16 +36,16 @@
assertEquals(56, '\8'.charCodeAt(0));
assertEquals('\010', '\10');
-assertEquals('\011', '\11');
+assertEquals('\011', '\11');
assertEquals('\012', '\12');
assertEquals('\013', '\13');
assertEquals('\014', '\14');
assertEquals('\015', '\15');
assertEquals('\016', '\16');
assertEquals('\017', '\17');
-
+
assertEquals('\020', '\20');
-assertEquals('\021', '\21');
+assertEquals('\021', '\21');
assertEquals('\022', '\22');
assertEquals('\023', '\23');
assertEquals('\024', '\24');
@@ -56,4 +56,4 @@
assertEquals(73, '\111'.charCodeAt(0));
assertEquals(105, '\151'.charCodeAt(0));
-
+
=======================================
***Additional files exist in this changeset.***
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
