rass und radiotext

[liab]

Wenn Du dann zurückschaltest (mit "0") solange die Text-Box noch angezeigt wird, dann bist Du zwar auf einem anderen Sender, aber die Text-Box mit dem Radiotext wird "mitgenommen".

Das ist bei mir so, daß es nicht stört. Sobald der neue Sender ertönt, ist das RT Fenster weg. Vielleicht 1/2 bis max. 1 Sekunde Überlappung.

Ist aber leicht zu beseitigen, ca. Zeile 680 in der infoviewer.cpp um RC_0 erweitern:

• if (((CNeutrinoApp::getInstance()->getMode()) == NeutrinoMessages::mode_radio) && (( msg == CRCInput::RC_up ) | ( msg == CRCInput::RC_down ) | ( msg == CRCInput::RC_0 )))

[Mourice]

Das ist bei mir leider nicht so.

Die Text-Box bleibt solange, bis der "neue" Sender Radio-Text-Daten empfängt.

Im schlimmsten Fall also "für immer".

[liab]

Das ist bei mir leider nicht so.

Die Text-Box bleibt solange, bis der "neue" Sender Radio-Text-Daten empfängt.

Im schlimmsten Fall also "für immer".

Habe meinen zuvorstehenden Text geändert mit der möglichen Abhilfe. Geht bei mir jetzt perfekt.

[liab]

Habe noch eine merkwürdige Erscheinung gefunden. Tritt beim Image vom 17.09.09 und auch vom 23.09.09 auf, hat also mit neueren Änderungen nichts zu tun. Auftreten:

1. Box mit einem RT Sender runterfahren.
2. Box neu starten und warten bis der Sender RT anzeigt. Wenn man dann bei der FB die Null betätigt passiert scheinbar nichts, da es ja noch keinen Sender Lastkey gibt. Bei meinem Patch wird natürlich der RT sofort gelöscht.

Es passiert aber, ohne daß man es sofort bemerkt doch was. Der RT wird blockiert und wird erst wieder aktiv, wenn RC_left / right betätigt wird oder der Sender gewechselt wird. Ist zwar harmlos, aber für mich im Moment unerklärlich.

[liab]

Das ist bei mir leider nicht so.
Die Text-Box bleibt solange, bis der "neue" Sender Radio-Text-Daten empfängt.
Im schlimmsten Fall also "für immer".

Bei meinem vorhin erstellten Image mit der radiotext.cpp,v 1.5 und der infoviewer.cpp,v 1.275 ist es bei mir ohne meinen zuvor gezeigten Patch jetzt auch so!

rhabarber1848 bitte Zeile 681 der infoviewer.cpp ändern in

Code: Alles auswählen

				if (((CNeutrinoApp::getInstance()->getMode()) == NeutrinoMessages::mode_radio) && (( msg == CRCInput::RC_up ) | ( msg == CRCInput::RC_down ) | ( msg == CRCInput::RC_0 )))

Der Fehler mit dem bei einigen Sendern trotz gelben Icons erst nach etlichen Minuten kommenden Textes ist leider immer noch.

[rhabarber1848]

rhabarber1848 bitte Zeile 681 der infoviewer.cpp ändern in

Danke, erledigt: http://article.gmane.org/gmane.comp.vid ... x.scm/1156

[seife]

Code: Alles auswählen

				if (((CNeutrinoApp::getInstance()->getMode()) == NeutrinoMessages::mode_radio) && (( msg == CRCInput::RC_up ) | ( msg == CRCInput::RC_down ) | ( msg == CRCInput::RC_0 )))

Das ist falsch (| vs ||). Und hat wesentlich zuviele Klammern ;-)

[rhabarber1848]

Jetzt, wo durch aktivierten Radiotext das Umschalten nicht
mehr verlangsamt wird, würde ich gerne das Kompilieren von
Radiotext in cdk/configure als default aktivieren.
--disable-radiotext bleibt natürlich weiterhin möglich.

Was denkt ihr darüber?

[liab]

Radiotext in cdk/configure als default aktivieren.

Finde ich gut. RT erhöht das Radiovergnügen deutlich und sollte Standard sein.

[Gaucho316]

Bin auch dafür.

[GetAway]

Folgender Fehler tritt jetzt auf:
Direkt nach dem Booten, wenn noch nicht umgeschaltet wurde und
man bei eingeblendeter Infobar(TV) die 0 drückt, bleibt die Infobar eingefroren stehen.
Genauso, wenn man sie anschließend mit "?" aufruft und dann 0 drückt.
Kann das jemand bestätigen?

[Gaucho316]

Ja, kann ich bestätigen. Tritt also nur auf, wenn noch nichts in der Programm-History gespeichert ist.

[dbt]

Ja, ist voll daneben jetzt.

[GetAway]

Bitte testen, also im Radio-, TV-Mode und bei der Bildregie.

fix_button0_infoviewer.diff

[dbt]

Scheint zu gehen. Ich hab noch mal einen drauf gesetzt und versucht das MuteIcon zu handeln. Das geht soweit nur der letzte Kick fehlt, wenn man es bei laufendem Radiotext wegmacht. Es wird dann zwar wieder alles gut, dank des laufend aktualisiertem RT-Fensters, aber ich fürchte, das wird nicht besser werden.
fix_button0_muticon_2009-09-25-22-43.patch

[seife]

Das bisher vorhandene Mute-Icon-handling ist doch ein einziger Pfusch.
Man sollte das in der framebuffer-Klasse machen, die weiss dann auch, dass sie dort nicht hinmalen darf und gut ist.

Also das tut jetzt nicht weh, da noch was draufzusetzen, aber eine Lösung ist das so wie vorgeschlagen nicht

[GetAway]

Das läßt sich doch jetzt an verschiedenen Stellen einsetzen, funktioniert jedenfalls schon ganz gut.

[dbt]

Sagte ich ja, dass es nicht besser werden wird, und Gefallen tut mir das auch nicht wirklich und nicht nur das Teil. In neutrino.cpp müsste mal noch einiges ausgemistet werden. Da fiele das auch drunter. Da wäre auch Volumebar, die ganzen settings, einige Klassen sind da auch noch untergemischt, die man auch einzeln unter bringen könnte, videosettings fallen mir auf Anhiep ein etc...Ich wollte bei den Menüs etwas mehr aufteilen, aber das ist alles so verwachsen, dass es keinen Spass macht. Wer machts freiwillig?

Edit: Den Fix hab ich erst mal eingecheckt, weils einfach vorher nicht brauchbar war.

[liab]

Folgender Fehler tritt jetzt auf:
Direkt nach dem Booten, wenn noch nicht umgeschaltet wurde und
man bei eingeblendeter Infobar(TV) die 0 drückt, bleibt die Infobar eingefroren stehen.
Genauso, wenn man sie anschließend mit "?" aufruft und dann 0 drückt.
Kann das jemand bestätigen?

Hatte ich schon am Do Sep 24, 2009 10:31 pm gemeldet. Fehler ist auch schon bei Images, die eine Woche alt sind.

[liab]

Ich meine soviel Aufmerksamkeit hat das Mute Icon nicht verdient. Man hat sich an die Stelle des Icons gewöhnt. Ich lebe sehr gut mit der Lösung, die ich vor ein paar Tagen hier vorschlug. Habe das RT Fenster soweit runter gesetzt, daß es nicht mit Mute crasht. Optisch ist das ok.

[dbt]

Es ist ja nicht nur wegen dem RT-Fenster, welches das Icon zerlegt. Das zieht sich durch die ganze GUI, und das ist nunmal unschön.

[rhabarber1848]

Radiotext in cdk/configure als default aktivieren.

Finde ich gut.

Erledigt: http://cvs.tuxbox-cvs.sourceforge.net/t ... 5&r2=1.246

[liab]

Log leider nicht vorhanden. Ich habe bei mir das Problem, dass da wo die Box steht, kein PC in der nähe ist ... da wo ein PC in der nähe ist, habe ich keinen Kabelanschluss für die Box.

Ich mache das so, braucht nur Ethernet.

1. /etc/init.d/start_neutrino in /etc/init.d/1start_neutrino umbenennen.
2. Box starten. Sie bleibt dann stehen, weil sie start_neutrino nicht findet.
3. Jetzt mit telnet die Kontrolle über die Box übernehmen und ./etc/init.d/1start_neutrino eingeben. Die Box bootet dann weiter und man hat in telnet das Log.

Ich habe das natürlich mit einem Skript vollautomatisiert.

[Gaucho316]

Wenn ich jetzt den Radio-Mode aufrufe, erscheint im Radio-Text (in diesem Fall Kabel UM WDR4) in der ersten Zeile
"Radiotext" und dahinter (abgeschnitten durch "Radiotext") das zuletzt aufgerufene Plugin (also "Radiotext box/plugins/01_.....").
Das bleibt so die ganze Zeit. Auch wenn der Radio-Text vom Sender geändert wird.

Ohne wirklich Ahnung von dem Radiotext-Code zu haben, poste ich mal einen möglichen Lösungsvorschlag, den ich allerdings nicht ausprobiert habe. Würde es vielleicht reichen, in "radiotext.cpp" in den Konstruktor von "CRadioText" einfach folgendes einzufügen?

Code: Alles auswählen

CRadioText::CRadioText(void)
{
	...
	strcpy(RDS_PTYN, "");
	...
}

[rhabarber1848]

strcpy(RDS_PTYN, "");

Klingt nicht schlecht, sollte aber auch in CRadioText::setPid drin
sein, damit nach einen Kanalwechsel die Variable ebenfalls geleert
wird. Dabei habe ich noch etwas entdeckt:

Code: Alles auswählen

for (int i=0; i<5; i++) strcpy(g_Radiotext->RT_Text[i], "");

g_Radiotext wird doch in neutrino.cpp erstellt:

Code: Alles auswählen

#ifdef ENABLE_RADIOTEXT
         g_Radiotext     = NULL;
#endif

Innerhalb von driver/radiotext.cpp sollte RT_Text reichen, oder?
Deshalb schlage ich folgenden Patch zum Testen vor:
EDIT: Patch ist im CVS