Changeset 38767 in webkit for trunk/JavaScriptCore/runtime

Timestamp:

Nov 25, 2008, 4:09:30 PM (17 years ago)

Author:

[email protected]

Message:

2008-11-25 Cameron Zwarich <​[email protected]>

Reviewed by Maciej Stachowiak.

Remove the unused member function JSFunction::getParameterName().

• runtime/JSFunction.cpp:
• runtime/JSFunction.h:

Location:

trunk/JavaScriptCore/runtime

Files:

• JSFunction.cpp (modified) (1 diff)
• JSFunction.h (modified) (1 diff)

trunk/JavaScriptCore/runtime/JSFunction.cpp

@@ -148 +148 @@
 }
 
-/* Returns the parameter name corresponding to the given index. eg:
- * function f1(x, y, z): getParameterName(0) --> x
- *
- * If a name appears more than once, only the last index at which
- * it appears associates with it. eg:
- * function f2(x, x): getParameterName(0) --> null
- */
-const Identifier& JSFunction::getParameterName(int index)
-{
-    const Identifier* parameters = m_body->parameters();
-
-    if (static_cast<size_t>(index) >= m_body->parameterCount())
-        return m_scopeChain.globalObject()->globalData()->propertyNames->nullIdentifier;
-  
-    const Identifier& name = parameters[index];
-
-    // Are there any subsequent parameters with the same name?
-    size_t size = m_body->parameterCount();
-    for (size_t i = index + 1; i < size; ++i) {
-        if (parameters[i] == name)
-            return m_scopeChain.globalObject()->globalData()->propertyNames->nullIdentifier;
-    }
-
-    return name;
-}
-
 // ECMA 13.2.2 [[Construct]]
 ConstructType JSFunction::getConstructData(ConstructData& constructData)

trunk/JavaScriptCore/runtime/JSFunction.h

@@ -55 +55 @@
         JSValue* call(ExecState*, JSValue* thisValue, const ArgList&);
 
-        // Note: Returns a null identifier for any parameters that will never get set
-        // due to a later parameter with the same name.
-        const Identifier& getParameterName(int index);
-
         void setScope(const ScopeChain& scopeChain) { m_scopeChain = scopeChain; }
         ScopeChain& scope() { return m_scopeChain; }