ETM-Schurig/powersync-core
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge PR #495: Fix cleaning up things when a setup fails in python plugins

Browse Source
This commit is contained in:
Jenkins nymea 2022-03-27 19:49:31 +02:00
commit 013c94a907
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
libnymea-core/integrations/pythonintegrationplugin.cpp
View File

@ -544,6 +544,7 @@ void PythonIntegrationPlugin::setupThing(ThingSetupInfo *info)
connect(info->thing(), &Thing::destroyed, this, [=](){
PyEval_RestoreThread(m_threadState);
m_things.remove(thing); // In case thingRemoved is never called (e.g. failed setup) it needs to be removed too
pyThing->thing = nullptr;
Py_DECREF(pyThing);
m_threadPool->setMaxThreadCount(m_threadPool->maxThreadCount() - 1);
@ -598,7 +599,7 @@ void PythonIntegrationPlugin::executeAction(ThingActionInfo *info)
void PythonIntegrationPlugin::thingRemoved(Thing *thing)
{
PyThing *pyThing = m_things.take(thing);
PyThing *pyThing = m_things.take(thing); // removing thing from myThings() before the thingRemoved call
callPluginFunction("thingRemoved", reinterpret_cast<PyObject*>(pyThing));
}