Bug in der Filminfovorschau des Moviebrowsers

[Gaucho316]

Ich habe noch einen Fehler gefunden. In der Filminfovorschau (der untere Teil des Moviebrowser-Fensters) wird das letzte Zeichen abgeschnitten. Kann das jemand bestätigen?

[rhabarber1848]

Ein Bildschirmphoto sagt mehr als 1000 Worte

[GetAway]

Standardmäßig ist beim Aufruf des MB der 1. Film markiert.
Bei diesem muss man aber 2x die OK-Taste zum Starten drücken,
nach dem Scrollen auf einen anderen Film ist dieser mit nur einem
Tastendruck startbar.

[Gaucho316]

Kann es sein, dass die Erzeugung von "m_pcInfo" in der "moviebrowser.cpp" das Problem verursacht?

Code: Alles auswählen

m_pcInfo = new CTextBox(" ",
						NULL,
						CTextBox::SCROLL,
						&m_cBoxFrameInfo);

Der erste Parameter des Konstruktors von "CTextBox" muss nämlich laut "textbox.h" ein Zeiger sein. Hier wird aber direkt ein Wert zugewiesen.

[GetAway]

Ich habe noch einen Fehler gefunden. In der Filminfovorschau (der untere Teil des Moviebrowser-Fensters) wird das letzte Zeichen abgeschnitten. Kann das jemand bestätigen?

Bestätigt. Es fehlt immer das letzte Zeichen. Es fehlt auch beim Aufruf der FilmInfo, während
des laufenden Films (2x "?") das allerletzte Zeichen. Der Dateiname sollte .ts lauten, angezeigt wird nur .t

Standardmäßig ist beim Aufruf des MB der 1. Film markiert.
Bei diesem muss man aber 2x die OK-Taste zum Starten drücken,
nach dem Scrollen auf einen anderen Film ist dieser mit nur einem
Tastendruck startbar.

@seife, könntest Du dieses mit Deinen RC_Input lösen?

[seife]

@seife, könntest Du dieses mit Deinen RC_Input lösen?

Ich hab's nicht kaputt gemacht...
Movieplayer steht ganz unten auf der Liste der anzuschauenden dinge. Das ist einfach zu spassfrei.

[Gaucho316]

Ich habe noch einen Fehler gefunden. In der Filminfovorschau (der untere Teil des Moviebrowser-Fensters) wird das letzte Zeichen abgeschnitten. Kann das jemand bestätigen?

Bestätigt. Es fehlt immer das letzte Zeichen. Es fehlt auch beim Aufruf der FilmInfo, während
des laufenden Films (2x "?") das allerletzte Zeichen. Der Dateiname sollte .ts lauten, angezeigt wird nur .t

Folgender Workaround ist mir eingefallen. Der ist zwar nicht wirklich schön, sollte aber funktionieren, falls niemandem etwas anderes einfällt. Wie immer ist dieser Code von mir ungetestet.

Code: Alles auswählen

--- tuxbox-cvs/apps/tuxbox/neutrino/src/gui/moviebrowser.cpp	2008-12-29 18:17:16.000000000 +0100
+++ tuxbox-src/apps/tuxbox/neutrino/src/gui/moviebrowser.cpp	2008-12-30 15:30:36.000000000 +0100
@@ -1349,7 +1349,8 @@
 	}
 	else
 	{
-		m_pcInfo->setText(&m_movieSelectionHandler->epgInfo2);
+		std::string text = m_movieSelectionHandler->epgInfo2 + " ";
+		m_pcInfo->setText(&text);
 	}
 }
 


--- tuxbox-cvs/apps/tuxbox/neutrino/src/gui/movieinfo.cpp	2008-05-01 02:08:24.000000000 +0200
+++ tuxbox-src/apps/tuxbox/neutrino/src/gui/movieinfo.cpp	2008-12-30 15:30:37.000000000 +0100
@@ -530,6 +530,7 @@
         print_buffer += g_Locale->getText(LOCALE_MOVIEBROWSER_INFO_PATH);
           print_buffer += ": ";
         print_buffer += movie_info.file.Name;
+        print_buffer += " ";
  
      ShowMsg2UTF( 	movie_info.epgTitle.empty()? movie_info.file.getFileName().c_str() : movie_info.epgTitle.c_str() , 
 					print_buffer.c_str(), 

[Gaucho316]

Der Fehler liegt wahrscheinlich in textbox.h und/oder textbox.cpp (apps/tuxbox/neutrino/src/gui/widget). Die Klasse CTextBox wurde damals von Günther im Zusammenhang mit der Entwicklung des Moviebrowsers erstellt und soweit ich das sehe auch nur dort genutzt. Vielleicht gibt's ja in der Klasse einen grundsätzlichen Fehler.

[ChakaZulu]

hi,

ich habe mal kurz in den Code geschaut...

Line 443:

Code: Alles auswählen

if( m_cText[pos] == '\n' ||
	loop == false)
{
	// current line ends with an carriage return, make new line
	aktLine.erase(aktLine.size() - 1,1);

Das erase sollte eigentlich nur gemacht werden, wenn man tatsächlich einen Zeilenumbruch hat, weil in Zeile 412 loop auf false gesetzt wird, wenn es überhaupt keinen line-break gibt. Allerdings steht da auch ein Kommentar über abgeschnittene Zeichen...
Die Bedingung "|| loop == false)"
bezieht sich wohl auf die Prüfung direkt darüber für den Auto-Linebreak, trifft aber auch auf den Fall "gar kein linebreak nötig" zu.

Passiert das ganze denn nur, wenn die Zeichen gerade noch auf den Bildschirm passen sollten (wie der Kommentar suggeriert) oder auch bei kurzen Zeichenketten?

ciao,

ChakaZulu

[seife]

Nein, das triggerte immer dann, wenn kein newline am Ende war. Ich hab's grad mal gefixt. Der Andere Kommentar ist vermutlich für den Fall, wen ein Wort gar nicht auf eine Zeile passt. Oder so. Hab ich nicht probiert