Changeset 89228 in webkit for trunk/Source/JavaScriptCore/interpreter/Interpreter.cpp

Timestamp:

Jun 19, 2011, 8:50:36 PM (14 years ago)

Author:

[email protected]

Message:

2011-06-19 Oliver Hunt <​[email protected]>

Reviewed by Sam Weinig.

Correct logic for putting errors on the correct line when handling JSONP
​https://bugs.webkit.org/show_bug.cgi?id=62962

Add test case this time

• fast/js/parser-syntax-check-expected.txt:
• fast/js/script-tests/parser-syntax-check.js:

2011-06-19 Oliver Hunt <​[email protected]>

Reviewed by Sam Weinig.

Correct logic for putting errors on the correct line when handling JSONP
​https://bugs.webkit.org/show_bug.cgi?id=62962

Minor fix for the minor fix. *sigh*

• interpreter/Interpreter.cpp: (JSC::Interpreter::execute):

File:

• trunk/Source/JavaScriptCore/interpreter/Interpreter.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/interpreter/Interpreter.cpp

@@ -772 +772 @@
                         PropertySlot slot(globalObject);
                         if (!globalObject->getPropertySlot(callFrame, JSONPPath[i].m_pathEntryName, slot)) {
-                            if (i)
+                            if (entry)
                                 return throwError(callFrame, createUndefinedVariableError(globalObject->globalExec(), JSONPPath[i].m_pathEntryName));
                             goto failedJSONP;