Changeset 34499 in webkit for trunk/JavaScriptCore/VM/CodeBlock.cpp

Timestamp:

Jun 11, 2008, 3:01:40 PM (17 years ago)

Author:

Darin Adler

Message:

2008-06-11 Darin Adler <Darin Adler>

Reviewed by Maciej.

• a little bit of cleanup and prep for some upcoming optimizations

• JavaScriptCore.exp: Re-sorted this file (with sort command line tool).
• VM/CodeBlock.cpp: (KJS::CodeBlock::dump): Fixed printf to avoid warnings -- to use %lu we need to make sure the type is unsigned long.
• kjs/object.cpp: (KJS::Error::create): Eliminated unused error names array, and also put the strings into the code since there was already a switch statment. This also avoids having to contemplate a hypothetical access past the end of the array.
• kjs/object.h: Got rid of errorNames.
• kjs/property_slot.cpp: Deleted unused ungettableGetter.
• kjs/property_slot.h: Ditto.
• wtf/AlwaysInline.h: Added LIKELY alongside UNLIKELY.

File:

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

trunk/JavaScriptCore/VM/CodeBlock.cpp

@@ -155 +155 @@
             ++instructionCount;
 
-    printf("%lu instructions; %lu bytes at %p; %d locals (%d parameters); %d temporaries\n\n", instructionCount, instructions.size() * sizeof(Instruction), this, numLocals, numParameters, numTemporaries);
+    printf("%lu instructions; %lu bytes at %p; %d locals (%d parameters); %d temporaries\n\n",
+        static_cast<unsigned long>(instructionCount),
+        static_cast<unsigned long>(instructions.size() * sizeof(Instruction)),
+        this, numLocals, numParameters, numTemporaries);
     
     for (Vector<Instruction>::const_iterator it = begin; it != end; ++it)