Welches Device vom wem genutzt und auf welchem Kanal?

_Martin_ · 19. November 2014

Hi,
für ein neues Feature in meinem Skin habe ich eine kleine Sache von Louis übernommen, dabei geht es darum herauszufinden welches DVB-Device von wem verwendet wird und auf welchem Kanal dieser geschaltet ist. Dies ist aber anscheinend schwieriger als gedacht.

Das ganze sieht dann ungefähr so aus:

Code

// check device which currently displays live tv
    int deviceLiveTV = -1;
    cDevice *primaryDevice = cDevice::PrimaryDevice();
    if (primaryDevice) {
        if (!primaryDevice->Replaying() || primaryDevice->Transferring())
            deviceLiveTV = cDevice::ActualDevice()->DeviceNumber();
        else
            deviceLiveTV = primaryDevice->DeviceNumber();
    }


    // check currently recording devices
    bool *recDevices = new bool[numDevices];
    for( int i=0; i < numDevices; i++ )
        recDevices[i] = false;
    for (cTimer *timer = Timers.First(); timer; timer = Timers.Next(timer)) {
        if (!timer->Recording()) {
            continue;
        }
        if (cRecordControl *RecordControl = cRecordControls::GetRecordControl(timer)) {
            const cDevice *recDevice = RecordControl->Device();
            if (recDevice) {
                recDevices[recDevice->DeviceNumber()] = true;
            }
        }
    }
    int actualNumDevices = 0;
    for (int i = 0; i < numDevices; i++) {
        const cDevice *device = cDevice::GetDevice(i);
        if (!device || !device->NumProvidedSystems()) {
            continue;
        }
        actualNumDevices++;
        std::stringstream strDevice;


        strDevice << i << ": " << *(device->DeviceType()) << " - ";


        const cChannel *channel = device->GetCurrentlyTunedTransponder();

Alles anzeigen

Ist nicht komplett aber man sollte erkennen was getan wird.
Das erste Problem ist wie man herausfindet welches Device von wem verwendet wird. Hier sieht man das man das nicht einfach abfragen kann, sondern derzeit auf 2 Sachen geprüft wird, LiveTV und Recording. Was ist aber mit EPG-Scan, Streamdev, etc. ?

Das 2. Problem besteht beim herausfinden des Kanals mittels:

Code

const cChannel *channel = device->GetCurrentlyTunedTransponder();

Laut diesem Forumspost funktioniert dies nicht richtig. Auch ich kann bestätigen das ich mit diesem Aufruf und Streamdev-Client kein Kanal zurück bekomme (warum nicht?). Ich hatte dies dann wie vom Poster beschrieben geändert in

Code

const cChannel *channel = Channels.GetByNumber(device->CurrentChannel());

Damit erhalte ich auch mit streamdev-client den richtigen Kanal allerdings soll laut MegaV0lt hier immer der gleiche Kanal bei mehreren DVB-Devices erscheinen.

Leider habe ich derzeit auf meinem Entwicklungsrechner nicht die Möglichkeit alle Konstellationen durchzutesten da ich dort derzeit nur 1 streamdev-client habe.

Kann mir hier jemand evtl. helfen? Bekomme ich das überhaupt sauber gelöst? Beim ersten Problem sehe ich schwarz das 2. Problem mit dem Kanal sollte sich doch aber lösen lassen!?!

Grüße
Martin

mini73 · 19. November 2014

streamdev sollte dann wohl am besten GetCurrentlyTunedTransponder implementieren, sonst kommt da immer NULL zurück.
Das ist keine zwingend notwendige Funktion, jedenfalls benutzt der vdr das nirgendwo.

Lars.

_Martin_ · 19. November 2014

Das hatte ich mir auch schon gedacht das man das 2.Problem zumindest bei streamdev damit lösen kann, indem streamdev die Funktion implementiert, allerdings übersehe ich das nicht ob es damit wirklich getan ist und ob streamdev das überhaupt kann, vielleicht mache ich dazu mal ein Ticket bei redmine in streamdev auf, Danke!

mini73 · 19. November 2014

http://projects.vdr-developer.…/tree/client/device.c#n80

Vermutlich sowas ähnliches, bloß statt vergleichen dann einfach m_Channel zurückgeben. Aber ich rate auch nur.

Code

const cChannel *cStreamdevDevice::GetCurrentlyTunedTransponder(void) const
{
  if (m_ClientSocket->DataSocket(siLive) != NULL)
     return m_Channel;
  return NULL;
}

Und den Header natürlich anpassen, kannst ja mal ausprobieren.

Lars.

_Martin_ · 19. November 2014

Oh danke ich probier das aus und stell das dann in streamdev ein wenns klappt.

_Martin_ · 19. November 2014

Danke Lars das funktioniert, hier der diff wenn es jemand interessiert:

Diff

diff --git a/client/device.c b/client/device.c
index de0865f..ff64562 100644
--- a/client/device.c
+++ b/client/device.c
@@ -88,6 +88,14 @@ bool cStreamdevDevice::IsTunedToTransponder(const cChannel *Channel)
                        Channel->Transponder() == m_Channel->Transponder();
 }
 
+#if APIVERSNUM >= 10713
+const cChannel *cStreamdevDevice::GetCurrentlyTunedTransponder(void) const {
+    if (m_ClientSocket->DataSocket(siLive) != NULL)
+        return m_Channel;
+    return NULL;
+}
+#endif
+
 bool cStreamdevDevice::ProvidesChannel(const cChannel *Channel, int Priority, 
                bool *NeedsDetachReceivers) const {
        if (m_Disabled || Channel == m_DenyChannel)
diff --git a/client/device.h b/client/device.h
index 1edf956..36646f6 100644
--- a/client/device.h
+++ b/client/device.h
@@ -66,6 +66,9 @@ public:
 #else
        virtual bool IsTunedToTransponder(const cChannel *Channel);
 #endif
+#if APIVERSNUM >= 10713
+    virtual const cChannel *GetCurrentlyTunedTransponder(void) const;
+#endif
        virtual cString DeviceName(void) const;
        virtual cString DeviceType(void) const;
        virtual int SignalStrength(void) const;

Alles anzeigen

Poste das aber noch in redmine.

Grüße
Martin

mini73 · 19. November 2014

Die ifdef Geschichte brauchst du nicht zwingend. Sollte das jemand mit einem älteren vdr übersetzen, gibt es keinen Fehler. Die Funktion kann dann eben nur nicht über cDevice genutzt werden, aber schon über cStreamdevDevice.

ifdefs braucht man nur an den Stellen, wo man sowas aufruft, was erst später hinzugekommen ist.

Schön, dass es funktioniert!

Lars

_Martin_ · 19. November 2014

Aber die Funktion ist doch erst mit VDR Version 1.7.13 hinzugekommen oder verstehe ich dich falsch?

Grüße
Martin

mini73 · 19. November 2014

Das ist richtig. Aber ob sie ihren Ursprung in cDevice oder in cStreamdevDevice hat, spielt für den Compiler keinen Unterschied.
Alte vdr-Version (aber auch neue) rufen diese Funktion sowieso nicht auf, die ist nur für Plugins da. Wenn ein Plugin gegen einen älteren vdr kompiliert ist, weiß er nichts von dieser Funktion, versucht also auch nicht, sie aufzurufen.

In anderen Sprachen muss man virtuelle Funktionen explizit mit override überladen, in C++ ist das nicht so. Ob nun cDevice die erste Klasse in der Hierarchie ist, die diese Funktion als virtual definiert oder eine abgeleitete, ist egal. Probier's einfach, dann wirst du es sehen. ifdefs sind schon häßlich zu lesen, wenn man sie weglassen kann, ist es nur vorteilhaft.

Lars.

_Martin_ · 19. November 2014

Hmm das wusste ich nicht und klingt interessant. Danke für die Info!

mini73 · 19. November 2014

Hab leider gerade nicht viel Zeit, sonst würde ich das ausführlicher erklären. Kann ich nachher evtl. mal versuchen...

Lars.

Welches Device vom wem genutzt und auf welchem Kanal?

Jetzt mitmachen!

Teilen

Benutzer online in diesem Thema