Changeset 17372 in webkit for trunk/JavaScriptCore/kjs/property_map.cpp

Timestamp:

Oct 27, 2006, 9:48:28 AM (19 years ago)

Author:

darin

Message:

Reviewed by Geoff.

• made changes so the code compiles with the highest warning level under MSVC (disabling some warnings, making some code fixes)

• API/JSCallbackConstructor.cpp: (KJS::JSCallbackConstructor::construct):
• API/JSCallbackFunction.cpp: (KJS::JSCallbackFunction::callAsFunction):
• API/JSCallbackObject.cpp: (KJS::JSCallbackObject::init): (KJS::JSCallbackObject::construct): (KJS::JSCallbackObject::callAsFunction):
• API/JSObjectRef.cpp: (JSPropertyNameArrayGetNameAtIndex):
• API/JSStringRef.cpp: (JSStringCreateWithCharacters):
• JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCore.vcproj:
• bindings/c/c_utility.cpp: (KJS::Bindings::convertUTF8ToUTF16): (KJS::Bindings::coerceValueToNPVariantStringType): (KJS::Bindings::convertValueToNPVariant):
• kjs/DateMath.h: (KJS::GregorianDateTime::GregorianDateTime):
• kjs/ExecState.h: (KJS::ExecState::hadException):
• kjs/JSImmediate.h: (KJS::JSImmediate::fromDouble): (KJS::JSImmediate::toDouble): (KJS::JSImmediate::NanAsBits): (KJS::JSImmediate::oneAsBits):
• kjs/Parser.h:
• kjs/PropertyNameArray.h: (KJS::PropertyNameArray::size):
• kjs/array_object.cpp: (ArrayObjectImp::callAsFunction):
• kjs/bool_object.cpp: (BooleanObjectImp::callAsFunction):
• kjs/collector.cpp: (KJS::Collector::allocate): (KJS::Collector::markCurrentThreadConservatively): (KJS::Collector::collect):
• kjs/completion.h: (KJS::Completion::isValueCompletion):
• kjs/date_object.cpp: (KJS::findMonth):
• kjs/debugger.cpp: (Debugger::sourceParsed): (Debugger::sourceUnused): (Debugger::exception): (Debugger::atStatement): (Debugger::callEvent): (Debugger::returnEvent):
• kjs/dtoa.cpp:
• kjs/error_object.cpp: (ErrorObjectImp::callAsFunction): (NativeErrorImp::callAsFunction):
• kjs/function.cpp: (KJS::FunctionImp::processVarDecls): (KJS::GlobalFuncImp::callAsFunction):
• kjs/function_object.cpp: (FunctionPrototype::callAsFunction):
• kjs/grammar.y:
• kjs/identifier.cpp: (KJS::CStringTranslator::translate): (KJS::Identifier::add):
• kjs/internal.h:
• kjs/lexer.cpp: (Lexer::lex): (Lexer::isIdentStart): (Lexer::isIdentPart): (isDecimalDigit): (Lexer::isHexDigit): (Lexer::isOctalDigit): (Lexer::matchPunctuator): (Lexer::singleEscape): (Lexer::convertOctal): (Lexer::convertHex): (Lexer::convertUnicode): (Lexer::record8):
• kjs/lexer.h:
• kjs/math_object.cpp: (MathFuncImp::callAsFunction):
• kjs/number_object.cpp: (integer_part_noexp): (intPow10): (NumberProtoFunc::callAsFunction): (NumberObjectImp::callAsFunction):
• kjs/object.cpp: (KJS::JSObject::deleteProperty): (KJS::JSObject::callAsFunction): (KJS::JSObject::toBoolean): (KJS::JSObject::toObject):
• kjs/object.h: (KJS::JSObject::getPropertySlot):
• kjs/property_map.cpp: (KJS::isValid): (KJS::PropertyMap::put): (KJS::PropertyMap::insert): (KJS::PropertyMap::containsGettersOrSetters):
• kjs/property_map.h: (KJS::PropertyMap::hasGetterSetterProperties):
• kjs/property_slot.h:
• kjs/string_object.cpp: (StringInstance::getPropertyNames): (StringObjectImp::callAsFunction): (StringObjectFuncImp::callAsFunction):
• kjs/ustring.cpp: (KJS::UString::Rep::computeHash): (KJS::UString::UString): (KJS::UString::from): (KJS::UString::append): (KJS::UString::ascii): (KJS::UString::operator=): (KJS::UString::find): (KJS::UString::rfind):
• kjs/ustring.h: (KJS::UChar::high): (KJS::UChar::low): (KJS::UCharReference::low): (KJS::UCharReference::high):
• kjs/value.cpp: (KJS::JSValue::toUInt16):
• kjs/value.h:
• pcre/pcre_compile.c: (get_othercase_range):
• pcre/pcre_exec.c: (match):
• pcre/pcre_internal.h:
• wtf/HashFunctions.h: (WTF::intHash): (WTF::PtrHash::hash):
• wtf/MathExtras.h: (isnan): (lround): (lroundf):
• wtf/StringExtras.h: (strncasecmp):
• wtf/unicode/icu/UnicodeIcu.h: (WTF::Unicode::isPrintableChar):

File:

• trunk/JavaScriptCore/kjs/property_map.cpp (modified) (5 diffs)

trunk/JavaScriptCore/kjs/property_map.cpp

@@ -106 +106 @@
 static inline bool isValid(UString::Rep* key)
 {
-    return reinterpret_cast<uintptr_t>(key) & ~0x1;
+    return !!(reinterpret_cast<uintptr_t>(key) & ~0x1);
 }
 
@@ -322 +322 @@
             _singleEntry.key = rep;
             _singleEntry.value = value;
-            _singleEntry.attributes = attributes;
+            _singleEntry.attributes = static_cast<short>(attributes);
             checkConsistency();
             return;
@@ -375 +375 @@
     entries[i].key = rep;
     entries[i].value = value;
-    entries[i].attributes = attributes;
+    entries[i].attributes = static_cast<short>(attributes);
     entries[i].index = ++_table->lastIndexUsed;
     ++_table->keyCount;
@@ -407 +407 @@
     entries[i].key = key;
     entries[i].value = value;
-    entries[i].attributes = attributes;
+    entries[i].attributes = static_cast<short>(attributes);
     entries[i].index = index;
 }
@@ -571 +571 @@
     if (!_table) {
 #if USE_SINGLE_ENTRY
-        return _singleEntry.attributes & GetterSetter;
-#endif
+        return !!(_singleEntry.attributes & GetterSetter);
+#else
         return false;
+#endif
     }