Kleiner fix für dem movieplayer

[Heinrich Hansen]

Wenn man eine Datei anwählt, die der VLC nicht abspielen kann, bleibt der movieplayer in einer Endlosschleife (while(true){}) hängen, in der immer wieder versucht wird, den nicht vorhandenen Stream zu öffnen.

Die Fernbedienung wird dabei auch nicht abgefragt und so bleibt nichts außer Reset.

Einfache Abhilfe:
in apps/tuxbox/neutrino/src/gui/movieplayer.cpp in Zeile 652 nach der inneren while(true) Schleife eine Abfrage der Fernbedienung einfügen:

Code: Alles auswählen

while(true)
{
    ...
    while(true)
    {
        ....
    }
    if(g_playstate == CMoviePlayerGui::STOPPED)
    {
        close(skt);
        pthread_exit(NULL);
    }
}

Jetzt läßt sich das ewige Warten mit einem Druck auf die "Home" Taste beenden.

Vielleicht mag das ja mal jemand ins CVS mit aufnehmen.

Gruß Heini

[eule]

Danke....

[Kickersfan]

Was benutzt ihr eigentlich für ein Neutrino ? Ich wollte den Fix bei mir auch einbauen aber habe nix mit movieplayer.cpp gefunden,Ich benutz das Yadi 2.1.0.1.Gibts da doch so große Unterschiede.Habe gedacht dass neutrino gleich neutrino ist und nur die gui anders ist.
Gruß Andre

[hannebamb(el)]

jo, Neutrino ist Neutrino

das was Hansen da gepostet hat ist Source Code ==> den muß man erst compilieren, damit da ein ausführbares Programm draus wird.
Im einem IMage wirst du da so nichts finden

[Kickersfan]

Ok danke,hab gedacht das kann ich malk so auf die schnelle einbinden .Aber des is dann doch zu hoch für mich
Gruß Andre

[Sedonion]

Bitte baut das ins CVS ein, würde auch mir den einen oder anderen Neustart der Box erleichtern bzw. verhindern.

Wäre echt klasse.

[DieMade]

Bitte ein diff erstellen oder zumindest 3 Codezeilen davor und dahinter erkennbar in den Codeschnipsel einfügen.

[mogway]

Bitte ein diff erstellen oder zumindest 3 Codezeilen davor und dahinter erkennbar in den Codeschnipsel einfügen.

Bin schon am testen

Wird die Tage in CVS kommen, wenn es läuft.

Gruß
mogway

[Heinrich Hansen]

Ich hoffe mal, dass ich das richtige Format erwischt habe.

Es sind noch ein paar zusätzliche kosmetische Änderungen drin, wenn nicht gewünscht, bitte weglassen.

Gruß Heini ;-)

Code: Alles auswählen

Index: movieplayer.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/gui/movieplayer.cpp,v
retrieving revision 1.105
diff -r1.105 movieplayer.cpp
473c473
< 	printf("[movieplayer.cpp] URL(enc) : %s?sout=%s\n",baseurl.c_str(), tmp);
---
> //	printf("[movieplayer.cpp] URL(enc) : %s?sout=%s\n",baseurl.c_str(), tmp);
553a554
> 		 !strcasecmp(sMRL.substr(sMRL.length()-4).c_str(), "bin") ||
590,591c591
< 	printf ("[movieplayer.cpp] Server: %s\n", server);
< 	printf ("[movieplayer.cpp] Port: %d\n", port);
---
> 	printf ("[movieplayer.cpp] streamingserver is %s:%d\n", server, port);
599c599
< 		printf ("[movieplayer.cpp] Trying to connect socket\n");
---
> 		printf ("[movieplayer.cpp] connecting to VLC...\n");
607c607
< 		printf ("[movieplayer.cpp] Socket OK\n");
---
> 		printf ("[movieplayer.cpp] connection established.\n");
619c619
< 		printf ("[movieplayer.cpp] GET Sent\n");
---
> 		printf ("[movieplayer.cpp] opening stream...\n");
649a650,654
> 		if(g_playstate == CMoviePlayerGui::STOPPED)
> 		{
> 			close(skt);
> 			pthread_exit (NULL);
> 		}
652c657
< 	printf ("[movieplayer.cpp] Now VLC is sending. Read sockets created\n");
---
> 	printf ("[movieplayer.cpp] VLC is now sending.\n");
655c660
< 	printf ("[movieplayer.cpp] Buffering approx. 3 seconds\n");
---
> 	printf ("[movieplayer.cpp] buffering approx. 3 seconds\n");
690c695
< 				printf("[movieplayer.cpp] Searching for vpid and apid\n");
---
> 				printf("[movieplayer.cpp] searching for vpid and apid\n"...);

[the_Desruptor]

Bitte baut das ins CVS ein, würde auch mir den einen oder anderen Neustart der Box erleichtern bzw. verhindern.

Wäre echt klasse.

und ich müsste nicht letzte nacht vom schlafzimmer 3 mal ein stockwerk runterlaufen, ins wohnzimmer, um die d-box neu zu starten.

[Sat_Man]

Ich hoffe mal, dass ich das richtige Format erwischt habe.

Allgemein üblich ist diff -u in Verbindung mit dem CVS also z.B. so:

Code: Alles auswählen

cvs diff -u movieplayer.cpp > movieplayer.diff