Falsche Darstellung in der Movieinfo

[Gaucho316]

Ich habe beobachtet, dass der Text in der Movieinfo manchmal zentriert dargestellt wird. Könnte es sein, dass folgender Fix das Problem löst? CENTER und NO_AUTO_LINEBREAK haben nämlich im Moment denselben Wert. Das kann ja eigentlich nicht richtig sein.

Code: Alles auswählen

--- tuxbox-cvs/apps/tuxbox/neutrino/src/gui/widget/msgbox.h	2009-01-10 19:26:07.000000000 +0100
+++ tuxbox-src/apps/tuxbox/neutrino/src/gui/msgbox.h	2009-04-13 18:37:16.000000000 +0200
@@ -85,7 +85,7 @@
 			FOOT		= 0x10,
 			BORDER		= 0x20,
 			CENTER		= 0x40,
-			NO_AUTO_LINEBREAK= 0x40
+			NO_AUTO_LINEBREAK= 0x80
 		}mode;
 
 	private:

[Gaucho316]

Dieser Patch löst das Problem nicht. Allerdings scheint der obige Fix trotzdem richtig zu sein.

Ich werde mal meine Beobachtungen etwas genauer beschreiben. Wenn man im Moviebrowser oder Movieviewer per ?-Taste die Movieinfo aufruft, wird bei mir der Text manchmal zentriert und manchmal auch rechts abgeschnitten dargestellt. Schließe ich das Fenster und rufe es wieder auf, wird es beim 2. oder 3. Mal dann richtig dargestellt. Kann das jemand bestätigen? Ich vermute den Fehler in "textbox.cpp", habe ihn aber noch nicht gefunden.

[rhabarber1848]

- NO_AUTO_LINEBREAK= 0x40
+ NO_AUTO_LINEBREAK= 0x80

committed to CVS

[Gaucho316]

Ich habe den Fehler gefunden. seife ist Schuld. Er hat am 29.03. ein paar Shadow Warnings in den Dateien "msgbox.cpp" und "textbox.cpp" gefixt.
http://cvs.tuxbox.org/lists/tuxbox-cvs- ... 00243.html
Leider hat er beim Umbenennen von "mode" in "_mode" ein paar Stellen vergessen, so dass nun die falsche Variable (und zwar die früher gleichnamige Membervariable) verwendet wird, die niemals initialisiert wurde und deshalb irgendeinen Wert enthält, was meine Beobachtungen erklärt, dass es mal geht und mal nicht. Der folgende Patch löst das Problem. Ich wundere mich nur, dass das sonst noch keinem aufgefallen ist. Nutzt niemand die Movieinfo?

Link entfernt

P.S.: Der Patch für "textbox.cpp" ist nur kosmetischer Natur, da der von mir geänderte Code sowieso nie ausgeführt wird, da er von "#if 0 ... #endif" umgeben ist.

[rhabarber1848]

Link entfernt

Patch ist im CVS
Ja, ich benutze Movieinfo nie

[Gaucho316]

Danke.

[seife]

mir war es tatsächlich schon aufgefallen (nämlich dass im MP2 die movieinfo nicht mehr umgebrochen wurde), ich kam allerdings noch nicht dazu, es genauer zu untersuchen.

Aber gut, wenn dadurch auch andere im neutrino-source trainiert werden ...