diff options
author | Lars Knoll <lars.knoll@theqtcompany.com> | 2014-11-07 18:51:19 +0100 |
---|---|---|
committer | Simon Hausmann <simon.hausmann@digia.com> | 2014-11-12 12:13:26 +0100 |
commit | 31084c37f60a54d0d1ab2e07a79e070268540498 (patch) | |
tree | 598bf007edb899f6c0965e82b904bf7ba557d362 /src/qml/jsruntime/qv4value.cpp | |
parent | 73a8f5f4845d34d74470b5a524d655be9eaf6dfc (diff) |
Convert methods in RuntimeHelpers to take an engine pointer
This is safer and cleaner than to use a context pointer.
Change-Id: Id5ef4e6667571897cd029125a0bdc18ce299da6d
Reviewed-by: Simon Hausmann <simon.hausmann@digia.com>
Diffstat (limited to 'src/qml/jsruntime/qv4value.cpp')
-rw-r--r-- | src/qml/jsruntime/qv4value.cpp | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/src/qml/jsruntime/qv4value.cpp b/src/qml/jsruntime/qv4value.cpp index e4f84b22fb..004f7cb6a3 100644 --- a/src/qml/jsruntime/qv4value.cpp +++ b/src/qml/jsruntime/qv4value.cpp @@ -269,21 +269,26 @@ double Primitive::toInteger(double number) #ifndef V4_BOOTSTRAP String *Value::toString(ExecutionEngine *e) const { - return toString(e->currentContext()); + if (isString()) + return stringValue(); + return RuntimeHelpers::convertToString(e, ValueRef::fromRawValue(this))->getPointer(); } String *Value::toString(ExecutionContext *ctx) const { - if (isString()) - return stringValue(); - return RuntimeHelpers::convertToString(ctx, ValueRef::fromRawValue(this))->getPointer(); + return toString(ctx->engine()); } -Object *Value::toObject(ExecutionContext *ctx) const +Object *Value::toObject(ExecutionEngine *e) const { if (isObject()) return objectValue(); - return RuntimeHelpers::convertToObject(ctx, ValueRef::fromRawValue(this))->getPointer(); + return RuntimeHelpers::convertToObject(e, ValueRef::fromRawValue(this))->getPointer(); +} + +Object *Value::toObject(ExecutionContext *ctx) const +{ + return toObject(ctx->engine()); } #endif // V4_BOOTSTRAP |