source: webkit/trunk/JavaScriptCore/runtime/JSNotAnObject.cpp@ 42478

Last change on this file since 42478 was 39809, checked in by Darin Adler, 16 years ago

2009-01-11 Darin Adler <Darin Adler>

Reviewed by Dan Bernstein.

Bug 23239: improve handling of unused arguments in JavaScriptCore
https://bugs.webkit.org/show_bug.cgi?id=23239

  • runtime/DatePrototype.cpp: Moved LocaleDateTimeFormat enum outside #if so we can use this on all platforms. Changed valueOf to share the same function with getTime, since the contents of the two are identical. Removed a FIXME since the idea isn't really specific enough or helpful enough to need to sit here in the source code. (JSC::formatLocaleDate): Changed the Mac version of this function to take the same arguments as the non-Mac version so the caller doesn't have to special-case the two platforms. Also made the formatString array be const; before the characters were, but the array was a modifiable global variable. (JSC::dateProtoFuncToLocaleString): Changed to call the new unified version of formatLocaleDate and remove the ifdef. (JSC::dateProtoFuncToLocaleDateString): Ditto. (JSC::dateProtoFuncToLocaleTimeString): Ditto.
  • runtime/JSNotAnObject.cpp: (JSC::JSNotAnObject::toObject): Use the new ASSERT_UNUSED instead of the old UNUSED_PARAM.
  • runtime/RegExp.cpp: (JSC::RegExp::RegExp): Changed to only use UNUSED_PARAM when the parameter is actually unused.
  • wtf/TCSystemAlloc.cpp: (TCMalloc_SystemRelease): Changed to only use UNUSED_PARAM when the parameter is actually unused. (TCMalloc_SystemCommit): Changed to omit the argument names instead of using UNUSED_PARAM.
File size: 4.0 KB
Line 
1/*
2 * Copyright (C) 2008 Apple Inc. All rights reserved.
3 *
4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions
6 * are met:
7 *
8 * 1. Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer.
10 * 2. Redistributions in binary form must reproduce the above copyright
11 * notice, this list of conditions and the following disclaimer in the
12 * documentation and/or other materials provided with the distribution.
13 * 3. Neither the name of Apple Computer, Inc. ("Apple") nor the names of
14 * its contributors may be used to endorse or promote products derived
15 * from this software without specific prior written permission.
16 *
17 * THIS SOFTWARE IS PROVIDED BY APPLE AND ITS CONTRIBUTORS "AS IS" AND ANY
18 * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
19 * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
20 * DISCLAIMED. IN NO EVENT SHALL APPLE OR ITS CONTRIBUTORS BE LIABLE FOR ANY
21 * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
22 * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
23 * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
24 * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
25 * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
26 * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
27 */
28
29
30#include "config.h"
31#include "JSNotAnObject.h"
32
33#include <wtf/UnusedParam.h>
34
35namespace JSC {
36
37ASSERT_CLASS_FITS_IN_CELL(JSNotAnObject);
38
39// JSValue methods
40JSValuePtr JSNotAnObject::toPrimitive(ExecState* exec, PreferredPrimitiveType) const
41{
42 ASSERT_UNUSED(exec, exec->hadException() && exec->exception() == m_exception);
43 return m_exception;
44}
45
46bool JSNotAnObject::getPrimitiveNumber(ExecState* exec, double&, JSValuePtr&)
47{
48 ASSERT_UNUSED(exec, exec->hadException() && exec->exception() == m_exception);
49 return false;
50}
51
52bool JSNotAnObject::toBoolean(ExecState* exec) const
53{
54 ASSERT_UNUSED(exec, exec->hadException() && exec->exception() == m_exception);
55 return false;
56}
57
58double JSNotAnObject::toNumber(ExecState* exec) const
59{
60 ASSERT_UNUSED(exec, exec->hadException() && exec->exception() == m_exception);
61 return NaN;
62}
63
64UString JSNotAnObject::toString(ExecState* exec) const
65{
66 ASSERT_UNUSED(exec, exec->hadException() && exec->exception() == m_exception);
67 return "";
68}
69
70JSObject* JSNotAnObject::toObject(ExecState* exec) const
71{
72 ASSERT_UNUSED(exec, exec->hadException() && exec->exception() == m_exception);
73 return m_exception;
74}
75
76// Marking
77void JSNotAnObject::mark()
78{
79 JSCell::mark();
80 if (!m_exception->marked())
81 m_exception->mark();
82}
83
84// JSObject methods
85bool JSNotAnObject::getOwnPropertySlot(ExecState* exec, const Identifier&, PropertySlot&)
86{
87 ASSERT_UNUSED(exec, exec->hadException() && exec->exception() == m_exception);
88 return false;
89}
90
91bool JSNotAnObject::getOwnPropertySlot(ExecState* exec, unsigned, PropertySlot&)
92{
93 ASSERT_UNUSED(exec, exec->hadException() && exec->exception() == m_exception);
94 return false;
95}
96
97void JSNotAnObject::put(ExecState* exec, const Identifier& , JSValuePtr, PutPropertySlot&)
98{
99 ASSERT_UNUSED(exec, exec->hadException() && exec->exception() == m_exception);
100}
101
102void JSNotAnObject::put(ExecState* exec, unsigned, JSValuePtr)
103{
104 ASSERT_UNUSED(exec, exec->hadException() && exec->exception() == m_exception);
105}
106
107bool JSNotAnObject::deleteProperty(ExecState* exec, const Identifier&)
108{
109 ASSERT_UNUSED(exec, exec->hadException() && exec->exception() == m_exception);
110 return false;
111}
112
113bool JSNotAnObject::deleteProperty(ExecState* exec, unsigned)
114{
115 ASSERT_UNUSED(exec, exec->hadException() && exec->exception() == m_exception);
116 return false;
117}
118
119void JSNotAnObject::getPropertyNames(ExecState* exec, PropertyNameArray&)
120{
121 ASSERT_UNUSED(exec, exec->hadException() && exec->exception() == m_exception);
122}
123
124} // namespace JSC
Note: See TracBrowser for help on using the repository browser.