Hi,
Für Klassen, die von cEpgHandler abgeleitet sind, hatten wir ja diskutiert dass es am Besten/Einfachsten ist in Initialize() mit new ... eine Instanz zu erzeugen, und diese Instanz nie wieder zu löschen.
Gilt das so auch analog für Klassen, die von cStatus abgeleitet sind?
In "epg.h" ist ja dokumentiert, dass VDR die von cEpgHandler abgeleiteten Klassen am Programmende löscht. Gilt das auch für Klassen, die von cStatus abgeleitet sind? In "status.h" habe ich dazu nichts gefunden.
~ Markus
Siehe PLUGINS.html#Status monitor.
A Plugin can implement any number of cStatus derived objects, and once the plugin has been started it may create and delete them as necessary.
Führt das Anlegen/Löschen dieser Instanzen zum Zugriff auf eine globale Tabelle? Brauchen wir dafür ein Locking?
Solange das im Main-Thread passiert gibt es kein Problem. Ich sehe jetzt auch nicht, wieso ein Plugin das aus einem separaten Thread heraus machen sollte.
> Ich sehe jetzt auch nicht, wieso ein Plugin das aus einem separaten Thread heraus machen sollte.
Ich denke auch nicht, dass das in der Praxis ein Problem ist. Sollte halt dokumentiert sein.
Dann wohl doch lieber so:
--- status.c 2025/03/02 21:02:12 5.4
+++ status.c 2025/03/03 13:22:45
@@ -8,19 +8,26 @@
*/
#include "status.h"
+#include "thread.h"
// --- cStatus ---------------------------------------------------------------
cList<cStatus> cStatus::statusMonitors;
+static cMutex Mutex;
+
cStatus::cStatus(void)
{
+ Mutex.Lock();
statusMonitors.Add(this);
+ Mutex.Unlock();
}
cStatus::~cStatus()
{
+ Mutex.Lock();
statusMonitors.Del(this, false);
+ Mutex.Unlock();
}
void cStatus::MsgChannelChange(const cChannel *Channel)Sieht jetzt gar nicht soo aufwändig aus.
Könnte man auch für cEpgHandler analog machen.
Mach ich.
Don’t have an account yet? Register yourself now and be a part of our community!