Fortschrittsbalken im Movieplayer

[Gaucho316]

Wenn man im Moviebrowser einen Film startet und dann im Movieplayer die OK-Taste drückt, wird ja der Moviebrowser aufgerufen. Verlässt man diesen nun mit HOME und hat den Fortschrittsbalken aktiviert, ist die Anzeige des Balkens im LCD völlig "zerstört". Hat jemand eine Idee, warum das so ist?

Ich weiß, dass diese Frage schon mehrmals gestellt wurde. Leider hatte bis jetzt keiner eine Lösung dafür. Vielleicht ja jetzt ...

[SnowHead]

Die zusätzliche Anzeige in der Kanalliste hat mit der LCD-Anzeige nach dem Movieplayer
absolut nichts zu tun. Sie ist lediglich ein zusätzliches Bildschirmelement innerhalb der
normalen Kanalliste.

[dbt]

Ich glaube er meint das anders, also nicht die Kanalliste, da er ja vom Moviebrowser schreibt, aber kann das deswegen auch nicht nachvollziehen.

ist die Anzeige des Balkens im LCD völlig "zerstört".

Hast du da mal ein Bild von oder kannst du mal genau sagen was du mit

...und hat den Fortschrittsbalken aktiviert

meinst? Und bitte genau unterscheiden zwischen Moviebrowser und Movieplayer!

[bellum]

Hast du da mal ein Bild von oder kannst du mal genau sagen was du mit

...und hat den Fortschrittsbalken aktiviert

meinst? Und bitte genau unterscheiden zwischen Moviebrowser und Movieplayer!

Es ist schon so wie Gaucho316 es schreibt, ich bin da auch schon drüber gestolpert.
Einfach mal über den Moviebrowser einen Film starten und während der Wiedergabe OK drücken. Dann kommt wieder der Moviebrowser. Wenn man den dann mit Home wieder verlässt, ist der Fortschrittsbalken des gerade laufenden Films "zerstört"...

Gruß bellum

[Gorcon]

Da wird das alte Logo unter dem Balken nicht gelöscht und liegt dann teilweise darunter.

Wenn man übrigens zu schnell nach dem OK Home drückt dann hängt sich die Box komplett auf.

[Gaucho316]

Würde vielleicht ein geschickt platziertes "CLCD::getInstance()->setMode(CLCD::MODE_TVRADIO)" am Ende der Funktion "exec(const char* path)" in der "moviebrowser.cpp" helfen?

Code: Alles auswählen

int CMovieBrowser::exec(const char* path)
{
	...
	saveSettings(&m_settings);	// might be better done in ~CMovieBrowser, but for any reason this does not work if MB is killed by neutrino shutdown

	CLCD::getInstance()->setMode(CLCD::MODE_TVRADIO);

	// make stale if we should reload the next time, but not if movie has to be played
	if(m_settings.reload == true && res == false)
	...
}

[Gaucho316]

Jetzt ist über ein Monat vergangen. Hat meinen Vorschlag inzwischen jemand ausprobiert?

[Gaucho316]

Und wieder ist knapp ein Monat ins Lang gegangen und ich frage mal wieder, ob jemand meinen Vorschlag getestet hat. Ich post dieses Mal auch ein diff. Also, wie sieht's aus?

Code: Alles auswählen

--- moviebrowser.cpp.ORIG	2009-01-24 17:07:25.000000000 +0100
+++ moviebrowser.cpp	2009-01-24 17:08:25.000000000 +0100
@@ -1157,6 +1157,8 @@
 
 	saveSettings(&m_settings);	// might be better done in ~CMovieBrowser, but for any reason this does not work if MB is killed by neutrino shutdown	
 
+	CLCD::getInstance()->setMode(CLCD::MODE_TVRADIO);
+
 	// make stale if we should reload the next time, but not if movie has to be played
 	if(m_settings.reload == true && res == false)
 	{

[dbt]

Ich habs mal auf gut dünken reingenommen. Kann also jeder mal testen. wenns nicht geht kann man's ja wieder reverten

[GetAway]

Funktioniert so leider nicht. Weder aus dem Filmarchiv heraus, noch
beim TS abspielen in Verbindung mit dem Filebrowser.

[Gaucho316]

Schade. Dann muss ich wohl noch mal drübergucken. Vielleicht fällt mir ja noch etwas anderes ein. Es ist auch mehr ein Rumstochern meinerseits.

[Gaucho316]

Die obige Änderung in "moviebrowser.cpp" kann im CVS wieder rückgängig gemacht werden. Der dort aufgerufene Befehl war an der Stelle überflüssig und konnte gar nichts bringen. Ich hatte den Code in der "movieplayer.cpp" leider nicht richtig verstanden.

Dieser Patch funktioniert aber vermutlich und wird hoffentlich möglichst bald von jemandem getestet und ins CVS eingecheckt.
Code entfernt

[GetAway]

Hi,

ich glaube zwar nicht das es das ist, aber es tritt auf jedenfall noch an
einer 3. Stelle auf. Und zwar beim setzen eines Bookmark mit der blauen Taste.
Dort wird nach verlassen eigentlich nur noch zusätzlich das EPG des aktuell
geschauten Senders angezeigt.

[Gaucho316]

Ich habe den Patch aufgrund deiner Feststellung mit der EPG-Anzeige etwas erweitert. Ich habe auch gleich einen weiteren Bugfix, der in die selbe Richtung geht, aus einem anderen Foren-Thread (http://tuxbox-forum.dreambox-fan.de/for ... 06#p364406) mit eingebaut.
Code entfernt

... aber es tritt auf jedenfall noch an einer 3. Stelle auf. Und zwar beim setzen eines Bookmark mit der blauen Taste.

Das behebt mein Patch (vermutlich) noch nicht. Ich habe noch nicht die richtge Stelle gefunden.

[seife]

Das einzig sinnvolle ist, CLCD::MODE_MOVIE zu benutzen, das ich extra deswegen eingebaut habe. Dann pfuscht keiner dazwischen und das temporäre umstellen der Konfiguration kann man sich auch sparen. Wie das funktioniert, habe ich im MP2 gezeigt, da gibt es auch all die Probleme nicht.

Ich weigere mich aber weiterhin standhaft, den movieplayer zu fixen ;-)

[GetAway]

Das einzig sinnvolle ist, CLCD::MODE_MOVIE zu benutzen, das ich extra deswegen eingebaut habe.

Ein reiner Tausch geht aber auch nicht, habe ich schon an den 3 stellen getestet.
Ich will ja nicht behaupten das dies am LCD Code liegt, der ja stark überarbeitet wurde,
es sieht aber danach aus.

[Gaucho316]

Ich weigere mich aber weiterhin standhaft, den movieplayer zu fixen ;-)

Deswegen versuche ich es ja mit mehr oder weniger (wohl eher weniger) großem Erfolg.

[Gaucho316]

So, meine Lösung sieht jetzt folgendermaßen aus. Damit sollte auch die zerstörte Fortschrittsanzeige nach dem Setzen eines Bookmarks gefixt sein. Es wäre schön, wenn sich da mal jemand ansehen und testen könnte. Falls es funktioniert, bitte beim Einchecken ins CVS nicht vergessen, die obige Änderung in der "moviebrowser.cpp" wieder zu entfernen.

Code: Alles auswählen

--- tuxbox-cvs/apps/tuxbox/neutrino/src/gui/movieplayer.cpp	2009-01-16 17:19:35.000000000 +0100
+++ tuxbox-src/apps/tuxbox/neutrino/src/gui/movieplayer.cpp	2009-01-30 21:01:21.000000000 +0100
@@ -3430,6 +3430,7 @@
 				break;
 			}
 
+			g_settings.lcd_setting[SNeutrinoSettings::LCD_SHOW_VOLUME] = g_lcdSetting;
 			CLCD::getInstance()->setMode (CLCD::MODE_TVRADIO);
 			update_lcd = true;
 		}
@@ -3540,6 +3541,7 @@
             APIDSelector.exec(NULL, ""); // otherwise use Dialog
             delete APIDChanger;
 			g_showaudioselectdialog = false;
+			update_lcd = true;
 
 			if(g_show_movieviewer)
 				showMovieViewer();
@@ -3726,8 +3728,7 @@
 						}
 					}
 				}
-				break; // do not process the old stuff
-	#endif /* MOVIEBROWSER */
+#else
 				if(bookmarkmanager->getBookmarkCount() < bookmarkmanager->getMaxBookmarkCount())
 				{
 					char timerstring[200];
@@ -3744,6 +3745,11 @@
 					fprintf(stderr, "too many bookmarks\n");
 					DisplayErrorMessage(g_Locale->getText(LOCALE_MOVIEPLAYER_TOOMANYBOOKMARKS)); // UTF-8
 				}
+#endif /* MOVIEBROWSER */
+
+				g_settings.lcd_setting[SNeutrinoSettings::LCD_SHOW_VOLUME] = g_lcdSetting;
+				CLCD::getInstance()->setMode(CLCD::MODE_TVRADIO);
+				update_lcd = true;
 
 				break;
 
@@ -4139,6 +4145,7 @@
 					break;
 			}
 
+			g_settings.lcd_setting[SNeutrinoSettings::LCD_SHOW_VOLUME] = g_lcdSetting;
 			CLCD::getInstance ()->setMode (CLCD::MODE_TVRADIO);
 		}
 

[dbt]

So mein Rechner scheint wieder zu laufen...den nutzlosen Patch hab' ich erst mal wieder raus. Testen kann ich deine Änderungen momentan leider noch nicht, aber wenn bestätigt wird, dass es geht, dann lässt sich das schnell übernehmen.

[Gaucho316]

Ich habe den Patch eben noch einmal überarbeitet. Dabei ist eine überflüssige Änderung meinerseits im VLC-Streaming-Teil wieder rausgeflogen, da hier das Problem mit der zerstörten LCD-Anzeige nach eigenen Beobachtungen nicht auftritt. Weiterhin habe ich den setMode-Befehl, der das LCD-Problem im Bookmarkmenü lösen soll, etwas weiter "nach innen" (d.h. in eine if-then-else-Anweisung hinein) im Code verschoben, so dass er nur noch aufgerufen wird, wenn es sein muss. Falls das nicht reichen sollte, kann ich diesen Teil auch wieder "nach außen" schieben. Ich bitte auch dieses Mal wieder um Rückmeldung, ob es geht.

Code: Alles auswählen

--- tuxbox-cvs/apps/tuxbox/neutrino/src/gui/movieplayer.cpp	2009-01-16 17:19:35.000000000 +0100
+++ tuxbox-src/apps/tuxbox/neutrino/src/gui/movieplayer.cpp	2009-02-01 14:49:37.000000000 +0100
@@ -3430,6 +3430,7 @@
 				break;
 			}
 
+			g_settings.lcd_setting[SNeutrinoSettings::LCD_SHOW_VOLUME] = g_lcdSetting;
 			CLCD::getInstance()->setMode (CLCD::MODE_TVRADIO);
 			update_lcd = true;
 		}
@@ -3540,6 +3541,7 @@
             APIDSelector.exec(NULL, ""); // otherwise use Dialog
             delete APIDChanger;
 			g_showaudioselectdialog = false;
+			update_lcd = true;
 
 			if(g_show_movieviewer)
 				showMovieViewer();
@@ -3724,10 +3726,13 @@
 							cMovieInfo.saveMovieInfo(*p_movie_info);/* save immediately in xml file */
 							cSelectedMenuBookStart[5].selected = false;// clear for next bookmark menu
 						}
+
+						g_settings.lcd_setting[SNeutrinoSettings::LCD_SHOW_VOLUME] = g_lcdSetting;
+						CLCD::getInstance()->setMode(CLCD::MODE_TVRADIO);
+						update_lcd = true;
 					}
 				}
-				break; // do not process the old stuff
-	#endif /* MOVIEBROWSER */
+#else
 				if(bookmarkmanager->getBookmarkCount() < bookmarkmanager->getMaxBookmarkCount())
 				{
 					char timerstring[200];
@@ -3744,6 +3749,7 @@
 					fprintf(stderr, "too many bookmarks\n");
 					DisplayErrorMessage(g_Locale->getText(LOCALE_MOVIEPLAYER_TOOMANYBOOKMARKS)); // UTF-8
 				}
+#endif /* MOVIEBROWSER */
 
 				break;
 

[GetAway]

@Gaucho316

Deine Mühen in allen Ehren, aber ohne eigene Entwicklungsumgebung wirst Du
nur schwer Leute finden die Deinen Code auf dauer testen werden.

[Gaucho316]

Ich mache ja sowieso nur kleine Bugfixes, die sich relativ schnell integrieren lassen, und das auch nur relativ selten. Es wäre schön, wenn sich dann eben ab und zu jemand finden würde, der das kurz testet. Für mehr fehlt mir auch das Wissen und die Zeit. Ich kann es natürlich auch ganz sein lassen.

[dbt]

Ich mache ja sowieso nur kleine Bugfixes, die sich relativ schnell integrieren lassen, und das auch nur relativ selten. Es wäre schön, wenn sich dann eben ab und zu jemand finden würde, der das kurz testet. Für mehr fehlt mir auch das Wissen und die Zeit. Ich kann es natürlich auch ganz sein lassen.

Das ist eigentlich "relativ" schnell eingerichtet. yBulid und yjogol's Kubuntu-VM-Image tut einem einiges abnehmen. Wenn du einmal an einem verwendbarem CDK "geleckt" hast, machst du es nichts mehr anders.

[Gaucho316]

Ich habe jetzt einmal im TuxboxWiki nach yBuild gesucht. Dabei bin ich auf den Link zur vorgefertigten VM gestoßen. Ich wußte bis zu deinem Posting gar nichts davon, dass es so etwas schon fertig konfiguriert gibt. Das ist irgendwie völlig an mir vorbeigegangen. Mal sehen, ob ich damit zurande komme.

[Tommy]

Die Kubuntu ist mittlerweile etwas angestaubt und auch riesengroß. Ich habe es geschaft, die komplette Umgebung auf einen Eisfair aufzusetzen. Das soll nur heißen, das es alle benötigten Tools entweder unter Packeis oder zum einfachen selbstkompilieren gibt. Evtl. kann ja mal jemand so einen fertigen Eisfair in einer VM aufsetzen. Das sollte sich unterhalb von 100MB bewegen und eine GUI bringt ja yBuild nun schon mit.