Besteht noch das Problem mit femon und ecm requests auf dem vorherigen Kanal?

Hi,
die Frage richtet sich vor allem an Copperhead da du glaub ich das Ticket mal aufgemacht hattest
Ich hatte mal mitbekommen das in Zusammenhang mit einem bestimmten Plugin (dies soll hier bitte nicht Thema werden, ansonsten bitte an die Mods die Post löschen und nicht den Thread ) und dem femon Code im Skin flatPlus beim umschalten weiterhin ecm vom vorherigen Kanal angefordert wurden und das dies wohl am femon-Code liegen soll.
Besteht dieses Problem immer noch und wurde das mal beim Femon-Maintainer angesprochen? Wenn mir jemand kurz beschreibt wie sich das Problem darstellt würde ich mir das auch nochmal anschauen. Irgendwie hatte Louis doch den Teil von femon dann wieder aus seinem Skin rausgeschmissen da es nicht funktioniert hatte.

Grüße
Martin

Ja das sehe ich auch so, will wie gesagt nicht das der Thread gelöscht wird. Ich warte mal noch auf die Antwort von Copperhead, dann können wir uns ja vielleicht zu 3. kurzschließen und ich schau mir das mal an.

Grüße
Martin

Letztendlich geht es ja um diese Funktion

void cFlatDisplayChannel::ChannelSwitch(const cDevice * device, int channelNumber, bool liveView)
{
    if( !Config.ChannelBitrateShow )
        return;


    bitrateVideo = bitrateAudio = bitrateDolby = 0.0;
    eTrackType track = cDevice::PrimaryDevice()->GetCurrentAudioTrack();
    const cChannel *channel = Channels.GetByNumber(cDevice::CurrentChannel());


    if (!liveView || !channelNumber || !channel || channel->Number() != channelNumber)
        return;


    if (m_Receiver) {
        m_Receiver->Deactivate();
        DELETENULL(m_Receiver);
    }
    if (channel) {
        m_Receiver = new cFemonReceiver(channel, IS_AUDIO_TRACK(track) ? int(track - ttAudioFirst) : 0, IS_DOLBY_TRACK(track) ? int(track - ttDolbyFirst) : 0);
        cDevice::ActualDevice()->AttachReceiver(m_Receiver);
    }
}

Diese wird aufgerufen wenn ein Kanal umgeschaltet wird um dann für den neuen Kanal den Receiver zu attachen um die Daten für die Bitraten zu erhalten. Hier wird natürlich der Receiver für den alten Kanal wieder deattached und dort läuft wohl was falsch!
Dies ist bis auf die ersten beiden Zeilen von femon 1:1 übernommen. In den ersten beiden Zeilen ist aber schon ein Fehler drin. Wenn ich im Plugin-Menü ChannelBitrateShow auf True stelle, umschalte, danach auf False stelle und wieder umschalte wird nie m_Receiver->Deactivate(); aufgerufen und so immer weiter für den alten Kanal die Daten an den Receiver gesendet.
Dies ist natürlich falsch und ist schnell geändert. Ich weiß aber nicht ob das der ganze Fehler ist, ich kann mir gerade aber schwer vorstellen das "Ihr" so getestet habt. Ich gehe davon aus das das Problem auch auftritt wenn ChannelBitrateShow einfach immer auf True steht und umgeschaltet wird. Dies kann natürlich noch mit

if (!liveView || !channelNumber || !channel || channel->Number() != channelNumber)
        return;

zusammenhängen aber dies ist so von femon und muss ich mir mal anschauen was dort "rausgefiltert" wird.

Grüße
Martin

Kannst du mir dann kurz beschreiben wie ich den Fehler erkennen kann damit ich das einmal durchtesten kann?

Naja aber dies

if (!liveView || !channelNumber || !channel || channel->Number() != channelNumber)
        return;

hattest du auch noch drinn, nicht das dort der Fehler liegt. Ich teste das einfach mal mit ein paar Debug Ausgaben und ich packe

if (m_Receiver) {
        m_Receiver->Deactivate();
        DELETENULL(m_Receiver);
    }

einfach nach ganz oben das wird auf jeden Fall immer der Receiver deattached!

Grüße
Martin

Ja ich habe da auch nicht viel drüber nachgedacht und das so übernommen
Ich teste das mal und geb dann bescheid!

Grüße
Martin

So im git gibt es eine neue Version. Ich konnte das ganze bei mir nachstellen und konnte auch sehen das von 2 Kanälen ECMs angefordert wurden. Mit der neuen Version ist das bei mir behoben.
ChalleSwitch sieht nun bei mir so aus

// Indicates a channel switch on the given DVB device.
// If ChannelNumber is 0, this is before the channel is being switched,
// otherwise ChannelNumber is the number of the channel that has been switched to.
// LiveView tells whether this channel switch is for live viewing.
void cFlatDisplayChannel::ChannelSwitch(const cDevice * device, int channelNumber, bool liveView)
{
    if( liveView == false) // ChannelSwitch from EPG-Scan, streamdev or other?
        return;
    if( channelNumber == 0 ) { // before the channel is being switched -> remove receiver
        if (m_Receiver) {
            //dsyslog("ChannelSwitch: Receiver REMOVE");
            m_Receiver->Deactivate();
            DELETENULL(m_Receiver);
        }
        return;
    }


    // at this point we finally switch to a new channel on liveView
    bitrateVideo = bitrateAudio = bitrateDolby = 0.0;
    eTrackType track = cDevice::PrimaryDevice()->GetCurrentAudioTrack();
    const cChannel *channel = Channels.GetByNumber(cDevice::CurrentChannel());


    if( channel && Config.ChannelBitrateShow && doOutput ) {
        if ( m_Receiver ) { // this should not be happen!!!
            //dsyslog("ChannelSwitch: Receiver is still Active -> remove it!");
            m_Receiver->Deactivate();
            DELETENULL(m_Receiver);
        }
        //dsyslog("ChannelSwitch: Receiver ATTACH %d - %s", cDevice::PrimaryDevice()->DeviceNumber(), channel->Name());


        m_Receiver = new cFemonReceiver(channel, IS_AUDIO_TRACK(track) ? int(track - ttAudioFirst) : 0, IS_DOLBY_TRACK(track) ? int(track - ttDolbyFirst) : 0);
        cDevice::ActualDevice()->AttachReceiver(m_Receiver);
    }
}

louis wenn du es wieder übernehmen möchtest guck es dir komplett an, z.B. Attache ich den FemonReceiver nicht mehr im Konstruktor von DisplayChannel sondern im Flush, also es hat sich nicht nur ChannelSwitch geändert.

Copperhead könntest du bitte die git Version einmal testen und bestätigen das das Problem behoben ist?

Grüße
Martin