From fff120dbaadaa09443f8ea5362e16bef6a22b36d Mon Sep 17 00:00:00 2001 From: "ager@chromium.org" Date: Tue, 9 Sep 2008 08:40:25 +0000 Subject: [PATCH] Use RUNTIME_ASSERT instead of CHECK in runtime.cc. Remove unused function from array.js. Review URL: http://codereview.chromium.org/1659 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@230 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- src/array.js | 9 --------- src/runtime.cc | 2 +- 2 files changed, 1 insertion(+), 10 deletions(-) diff --git a/src/array.js b/src/array.js index 6f229c0..fc44759 100644 --- a/src/array.js +++ b/src/array.js @@ -31,15 +31,6 @@ // ------------------------------------------------------------------- -// Determines if the array contains the element. -function Contains(array, element) { - var length = array.length; - for (var i = 0; i < length; i++) { - if (array[i] === element) return true; - } - return false; -}; - // Global list of arrays visited during toString, toLocaleString and // join invocations. diff --git a/src/runtime.cc b/src/runtime.cc index 0f7797c..e44fe19 100644 --- a/src/runtime.cc +++ b/src/runtime.cc @@ -3450,7 +3450,7 @@ static Object* Runtime_PushIfAbsent(Arguments args) { ASSERT(args.length() == 2); CONVERT_CHECKED(JSArray, array, args[0]); CONVERT_CHECKED(JSArray, element, args[1]); - CHECK(array->HasFastElements()); + RUNTIME_ASSERT(array->HasFastElements()); int length = Smi::cast(array->length())->value(); FixedArray* elements = FixedArray::cast(array->elements()); for (int i = 0; i < length; i++) { -- 2.7.4