[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc

zherczeg at webkit.org zherczeg at webkit.org
Wed Dec 22 14:51:27 UTC 2010


The following commit has been merged in the debian/experimental branch:
commit ad008de5365c4462274e243f47cf7bfcddf73712
Author: zherczeg at webkit.org <zherczeg at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Fri Oct 22 10:54:23 2010 +0000

    2010-10-22  Zoltan Herczeg  <zherczeg at webkit.org>
    
            Reviewed by Csaba Osztrogonác.
    
            JSC interpreter regressions after r69940
            https://bugs.webkit.org/show_bug.cgi?id=47839
    
            Wrong "if": It should test whether the result exists,
            and not the opposite. It is an interpreter bug, hence
            the bots does not capture it.
    
            * interpreter/Interpreter.cpp:
            (JSC::Interpreter::resolveBase):
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@70299 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/JavaScriptCore/ChangeLog b/JavaScriptCore/ChangeLog
index 0d870a0..a3a33b5 100644
--- a/JavaScriptCore/ChangeLog
+++ b/JavaScriptCore/ChangeLog
@@ -1,3 +1,17 @@
+2010-10-22  Zoltan Herczeg  <zherczeg at webkit.org>
+
+        Reviewed by Csaba Osztrogonác.
+
+        JSC interpreter regressions after r69940
+        https://bugs.webkit.org/show_bug.cgi?id=47839
+
+        Wrong "if": It should test whether the result exists,
+        and not the opposite. It is an interpreter bug, hence
+        the bots does not capture it.
+
+        * interpreter/Interpreter.cpp:
+        (JSC::Interpreter::resolveBase):
+
 2010-10-21  Adam Barth  <abarth at webkit.org>
 
         Reviewed by David Levin.
diff --git a/JavaScriptCore/interpreter/Interpreter.cpp b/JavaScriptCore/interpreter/Interpreter.cpp
index 2877565..1cadef8 100644
--- a/JavaScriptCore/interpreter/Interpreter.cpp
+++ b/JavaScriptCore/interpreter/Interpreter.cpp
@@ -283,7 +283,7 @@ NEVER_INLINE void Interpreter::resolveBase(CallFrame* callFrame, Instruction* vP
     bool isStrictPut = vPC[3].u.operand;
     Identifier ident = callFrame->codeBlock()->identifier(property);
     JSValue result = JSC::resolveBase(callFrame, ident, callFrame->scopeChain(), isStrictPut);
-    if (!result) {
+    if (result) {
         callFrame->r(dst) = result;
         ASSERT(callFrame->r(dst).jsValue());
     } else

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list