Changeset 36742 in webkit for trunk/JavaScriptCore/VM/CodeGenerator.cpp

Timestamp:

Sep 21, 2008, 1:41:22 PM (17 years ago)

Author:

Darin Adler

Message:

2008-09-21 Darin Adler <Darin Adler>

• VM/CodeGenerator.cpp: (JSC::CodeGenerator::emitLoad): One more tweak: Wrote this in a slightly clearer style.

File:

• trunk/JavaScriptCore/VM/CodeGenerator.cpp (modified) (1 diff)

trunk/JavaScriptCore/VM/CodeGenerator.cpp

@@ -733 +733 @@
 RegisterID* CodeGenerator::emitLoad(RegisterID* dst, double number)
 {
-    pair<NumberMap::iterator, bool> addResult = m_numberMap.add(number, 0);
-    if (addResult.second)
-        addResult.first->second = jsNumber(globalExec(), number);
-    return emitLoad(dst, addResult.first->second);
+    JSValue*& valueInMap = m_numberMap.add(number, 0).first->second;
+    if (!valueInMap)
+        valueInMap = jsNumber(globalExec(), number);
+    return emitLoad(dst, valueInMap);
 }
 
 RegisterID* CodeGenerator::emitLoad(RegisterID* dst, const Identifier& identifier)
 {
-    pair<IdentifierStringMap::iterator, bool> addResult = m_stringMap.add(identifier.ustring().rep(), 0);
-    if (addResult.second)
-        addResult.first->second = jsOwnedString(globalExec(), identifier.ustring());
-    return emitLoad(dst, addResult.first->second);
+    JSString*& valueInMap = m_stringMap.add(identifier.ustring().rep(), 0).first->second;
+    if (!valueInMap)
+        valueInMap = jsOwnedString(globalExec(), identifier.ustring());
+    return emitLoad(dst, valueInMap);
 }