diff options
author | Simon Hausmann <simon.hausmann@digia.com> | 2013-05-27 14:19:35 +0200 |
---|---|---|
committer | Lars Knoll <lars.knoll@digia.com> | 2013-05-27 20:35:25 +0200 |
commit | 6b5c964caaa1076c988e41c011bf766801570fdd (patch) | |
tree | 37ebe0bb891569070aa6f72e959d9bb8e0f31fc7 /src/qml/qml/v8/qv8valuetypewrapper.cpp | |
parent | 8a822866b68f5f425b2e1f5c793fe36acf291f4d (diff) |
Fix file and line number reporting of QML/JS errors
* Replace QUrl with QString in V4 for the source location (we don't need URL parsing)
* Replace line number and file in QV4::Exception with the information we retrieve from
the stack trace of the exception.
Fixes about five tst_qqmlecmascript tests that relied on correct file/line number information
Change-Id: I2a3daa72be6c5587fd965211ea8f6fb77142e7ee
Reviewed-by: Lars Knoll <lars.knoll@digia.com>
Diffstat (limited to 'src/qml/qml/v8/qv8valuetypewrapper.cpp')
-rw-r--r-- | src/qml/qml/v8/qv8valuetypewrapper.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/qml/qml/v8/qv8valuetypewrapper.cpp b/src/qml/qml/v8/qv8valuetypewrapper.cpp index 3218da4e38..c51be06d1f 100644 --- a/src/qml/qml/v8/qv8valuetypewrapper.cpp +++ b/src/qml/qml/v8/qv8valuetypewrapper.cpp @@ -397,7 +397,7 @@ v8::Handle<v8::Value> QV8ValueTypeWrapper::Setter(v8::Handle<v8::String> propert QV4::ExecutionEngine::StackFrame frame = v4->currentStackFrame(); newBinding = new QQmlBinding(function->v4Value(), reference->object, context, - frame.source.url(), qmlSourceCoordinate(frame.line), qmlSourceCoordinate(frame.column)); + frame.source, qmlSourceCoordinate(frame.line), qmlSourceCoordinate(frame.column)); newBinding->setTarget(reference->object, cacheData, context); newBinding->setEvaluateFlags(newBinding->evaluateFlags() | QQmlBinding::RequiresThisObject); |