Reviewers: rossberg,

Message:
This still doesn't cover all of the builtin JS files. The following are still left for various reasons and I will update the issue tracker accordingly. Also I will only land this after we made our branch. The following is the list of JS
files still left sloppy:

- src/debug-debugger.js
- src/liveedit-debugger.js
- src/messages.js
- src/mirror-debugger.js
- src/runtime.js
- src/string.js
- src/v8natives.js

Description:
Move more builtin files to strict mode.

R=rossb...@chromium.org
BUG=v8:1955

Please review this at https://codereview.chromium.org/279543002/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files (+16, -28 lines):
  M src/apinatives.js
  M src/array-iterator.js
  M src/date.js
  M src/i18n.js
  M src/json.js
  M src/math.js
  M src/regexp.js
  M src/uri.js


Index: src/apinatives.js
diff --git a/src/apinatives.js b/src/apinatives.js
index 0579caf544b9065f061aabb2d01e4c9b4dbb8b77..76e6292af6751350866bf46b0d493485ce334892 100644
--- a/src/apinatives.js
+++ b/src/apinatives.js
@@ -2,6 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.

+"use strict";
+
 // This file contains infrastructure used by the API.  See
 // v8natives.js for an explanation of these files are processed and
 // loaded.
Index: src/array-iterator.js
diff --git a/src/array-iterator.js b/src/array-iterator.js
index 10116b1d10b502c2079ce5e320db21b06ef60858..b50c8b436a499729d364ebcd145e83fc27bed0e6 100644
--- a/src/array-iterator.js
+++ b/src/array-iterator.js
@@ -1,29 +1,6 @@
 // Copyright 2013 the V8 project authors. All rights reserved.
-// Redistribution and use in source and binary forms, with or without
-// modification, are permitted provided that the following conditions are
-// met:
-//
-//     * Redistributions of source code must retain the above copyright
-//       notice, this list of conditions and the following disclaimer.
-//     * Redistributions in binary form must reproduce the above
-//       copyright notice, this list of conditions and the following
-//       disclaimer in the documentation and/or other materials provided
-//       with the distribution.
-//     * Neither the name of Google Inc. nor the names of its
-//       contributors may be used to endorse or promote products derived
-//       from this software without specific prior written permission.
-//
-// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
-// 'AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
-// LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
-// A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
-// OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
-// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
-// LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
-// DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
-// THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
-// (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
-// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.

 'use strict';

Index: src/date.js
diff --git a/src/date.js b/src/date.js
index 2a445979e6ee867e13bf44e3b1fa0da52ebcad51..c58903cc29c23f653f99e0baf4e61ab129ccee3f 100644
--- a/src/date.js
+++ b/src/date.js
@@ -2,6 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.

+"use strict";
+
// This file relies on the fact that the following declarations have been made
 // in v8natives.js:
 // var $isFinite = GlobalIsFinite;
Index: src/i18n.js
diff --git a/src/i18n.js b/src/i18n.js
index 4fcb02b4408dff4282cfcf109760194e47110798..076845bb111b5ee9742f00f7df2909cbcd2a40bc 100644
--- a/src/i18n.js
+++ b/src/i18n.js
@@ -1,7 +1,8 @@
 // Copyright 2013 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.
-// limitations under the License.
+
+"use strict";

 // ECMAScript 402 API implementation.

@@ -11,8 +12,6 @@
  */
$Object.defineProperty(global, "Intl", { enumerable: false, value: (function() {

-'use strict';
-
 var Intl = {};

 var undefined = global.undefined;
Index: src/json.js
diff --git a/src/json.js b/src/json.js
index 93e38b0dba7860f98a0d41098bbc98be6fd88dcc..f767f4a195e2256861a02a68faac9b2c6fdc98e9 100644
--- a/src/json.js
+++ b/src/json.js
@@ -2,6 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.

+"use strict";
+
// This file relies on the fact that the following declarations have been made
 // in runtime.js:
 // var $Array = global.Array;
Index: src/math.js
diff --git a/src/math.js b/src/math.js
index f8738b5f8580bee815b307dddd3965056e741143..fe10de48a7a2239b7b998e1a6d8c80da13830d74 100644
--- a/src/math.js
+++ b/src/math.js
@@ -2,6 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.

+"use strict";
+
// This file relies on the fact that the following declarations have been made
 // in runtime.js:
 // var $Object = global.Object;
Index: src/regexp.js
diff --git a/src/regexp.js b/src/regexp.js
index 6a0e2b5d92c00782d29ef1a78b42d382b10ae55c..a636605961dc1b7bdc84ac0107ce0dd4df819cbc 100644
--- a/src/regexp.js
+++ b/src/regexp.js
@@ -2,6 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.

+"use strict";
+
// This file relies on the fact that the following declaration has been made
 // in runtime.js:
 // var $Object = global.Object;
Index: src/uri.js
diff --git a/src/uri.js b/src/uri.js
index 0e50f0b7001e4e8c9ee68275607d1aedcfffb067..123503b83bab4e4ef1b8ed33a72badba221e9017 100644
--- a/src/uri.js
+++ b/src/uri.js
@@ -2,6 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.

+"use strict";
+
// This file relies on the fact that the following declaration has been made
 // in runtime.js:
 // var $Array = global.Array;


--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to