From f8e18ccd76d0f9af8721f55c9b2b1a8e7582cf18 Mon Sep 17 00:00:00 2001
From: Mike Pennisi <mike@mikepennisi.com>
Date: Fri, 26 Jun 2015 12:34:04 -0400
Subject: [PATCH] Remove unnecessary "include" file
The `fnExists` function defines a generic way to determine if any number
of values are function instances. Because it is only used by a single
test, the additional complexity required by the generalized code (and
the organizational drawbacks to maintaining another "include" file) are
not justified. Remove the file and update the test to assert the
function's existence directly.
---
harness/fnExists.js | 7 -------
test/language/expressions/delete/11.4.1-5-3.js | 3 +--
2 files changed, 1 insertion(+), 9 deletions(-)
delete mode 100644 harness/fnExists.js
diff --git a/harness/fnExists.js b/harness/fnExists.js
deleted file mode 100644
index 1f283b45d5..0000000000
--- a/harness/fnExists.js
+++ /dev/null
@@ -1,7 +0,0 @@
-//-----------------------------------------------------------------------------
-function fnExists(/*arguments*/) {
- for (var i = 0; i < arguments.length; i++) {
- if (typeof (arguments[i]) !== "function") return false;
- }
- return true;
-}
diff --git a/test/language/expressions/delete/11.4.1-5-3.js b/test/language/expressions/delete/11.4.1-5-3.js
index 1ded937657..e4c05744d7 100644
--- a/test/language/expressions/delete/11.4.1-5-3.js
+++ b/test/language/expressions/delete/11.4.1-5-3.js
@@ -12,7 +12,6 @@ description: >
flags: [noStrict]
includes:
- runTestCase.js
- - fnExists.js
---*/
function testcase() {
@@ -20,7 +19,7 @@ function testcase() {
// Now, deleting 'foo' directly should fail;
var d = delete foo;
- if(d === false && fnExists(foo))
+ if(d === false && typeof foo === 'function')
return true;
}
runTestCase(testcase);
--
GitLab