alecs/plasma-framework
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Qt ownership of the service object: they are owned by the caller, and so must be deleted by them

Browse Source 
svn path=/trunk/KDE/kdebase/runtime/; revision=1160005
This commit is contained in:
Aaron J. Seigo 2010-08-06 19:34:59 +00:00
parent 7500244520
commit 0c929d525d
3 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
scriptengines/javascript/dataengine/javascriptdataengine.cpp
View File

@ -228,7 +228,7 @@ QScriptValue JavaScriptDataEngine::serviceCtor(QScriptContext *context, QScriptE
JavaScriptService *service = new JavaScriptService(serviceName, iFace);
if (service->wasFound()) {
QScriptValue v = engine->newQObject(service, QScriptEngine::ScriptOwnership);
QScriptValue v = engine->newQObject(service, QScriptEngine::QtOwnership);
service->setScriptValue(v);
return v;
}
@ -301,6 +301,7 @@ Plasma::Service *JavaScriptDataEngine::serviceForSource(const QString &source)
if (service->destination().isEmpty()) {
service->setDestination(source);
}
return service;
} else {
delete rv.toQObject();

5
scriptengines/javascript/dataengine/javascriptservice.cpp
View File

@ -31,6 +31,11 @@ JavaScriptService::JavaScriptService(const QString &serviceName, JavaScriptDataE
setName(serviceName);
}
JavaScriptService::~JavaScriptService()
{
// kDebug();
}
void JavaScriptService::setScriptValue(QScriptValue &v)
{
m_scriptValue = v;

1
scriptengines/javascript/dataengine/javascriptservice.h
View File

@ -38,6 +38,7 @@ class JavaScriptService : public Plasma::Service
public:
JavaScriptService(const QString &serviceName, JavaScriptDataEngine *engine);
~JavaScriptService();
bool wasFound() const;
void setScriptValue(QScriptValue &v);