Commit-Wünsche für das CVS II

[Gaucho316]

Und eine weitere kleine Änderung für themes.cpp. Ich finde es passender, wenn vor den Theme-Einträgen das OK-Button-Symbol angezeigt wird.

Link entfernt, da Patch im CVS

[rhabarber1848]

themes_show-ok-button-on-selected-item_2012-03-15_2011.diff

committed: http://article.gmane.org/gmane.comp.vid ... x.scm/3294

[Gaucho316]

Ich habe noch ein paar Stellen in Menüs rausgesucht, von denen ich denke, dass das OK-Button-Symbol besser passt. Dort wird nämlich über den MenuForwarder eine Aktion direkt ausgeführt, ohne dass sich etwas öffnet, wie man es bei dem Pfeil-Symbol erwarten würde.

Link entfernt, da Patch im CVS

[rhabarber1848]

neutrino_show-ok-button-on-some-menuforwarders_2012-03-18_2022.diff

committed:
http://article.gmane.org/gmane.comp.vid ... x.scm/3315
http://article.gmane.org/gmane.comp.vid ... x.scm/3316

[Gaucho316]

Ich habe mal wieder drei Patches zum Einchecken. Alle drei sind getestet und funktionieren.

Mir ist aufgefallen, dass an einigen wenigen Stellen in Menu Widgets am Ende noch ein Menu Separator rangehängt wird. Ich habe das mal rausgesucht und die Menu Separators entfernt.

Link entfernt, da Patch im CVS

Dabei ist mir wiederum ein Speicherleck im Audioplayer aufgefallen. Dort wird mit new ein Objekt vom Typ CMenuSelectorTarget angelegt und nie wieder gelöscht.

Link entfernt, da Patch im CVS

Und dabei habe ich die Klasse CMenuSelectorTarget überhaupt erst entdeckt, die sich wunderbar für das Startpositionsauswahlmenü des Moviebrowsers eignet. Dann kann nämlich der wenig praktikable Code entfallen, der über getSelectedLine das gewählte Element herausfindet und bei dem man immer darauf achten muss, dass man die Intro Items beachtet und nicht mit in die Berechnung einbezieht.

Link entfernt, da Patch im CVS

P.S.: Bitte das hier auch gleich einchecken:
http://forum.tuxbox.org/forum/viewtopic ... 05#p382505


Edit: Ich habe den Patch für das Startpositionsauswahlmenü des Moviebrowsers eben noch um eine Kleinigkeit ergänzt. Früher gab es mal eine Trennlinie zwischen den Standard- und den Benutzerbookmarks, die aber enfernt werden musste, da sie die Berechnung durcheinander gebracht hat. Nun kann die aber wieder rein, da sie den neuen Code nicht beeinflusst.

[rhabarber1848]

committed:

neutrino_remove-menu-separators-from-the-end-of-menu-widgets_2012-03-31_1832.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3321
http://article.gmane.org/gmane.comp.vid ... x.scm/3322

audioplayer_fix-memleak_2012-03-31_1926.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3323

moviebrowser_rework-start-pos-selection-menu_2012-04-01_1636.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3324
http://article.gmane.org/gmane.comp.vid ... x.scm/3325

[Gaucho316]

Wie immer danke fürs Einchecken.

Und noch einer. Das war ein ganz schönes Stück Arbeit. Ausgehend vom Moviebrowser ist mir aufgefallen, dass im Menü "Film-informationen" die Umlaute und das ß nicht richtig dargestellt und bearbeitet werden können. Der folgende Patch löst das Problem und sorgt außerdem dafür, dass man nun auch Umlaute und das ß in der Bouquet-Verwaltung, für Bookmarks im Movieplayer/-browser, in der EPG-Suche, für Erinnerungs-Timer und für die Beschriftung der Benutzermenüs verwenden kann. Der Patch ist getestet und ich konnte bis jetzt keine Nebenwirkungen erkennen.

Link entfernt, da Patch im CVS

Edit: Ich habe den Patch eben aktualisiert. An der Funktionalität ändert sich nichts. Ich habe nur ein paar Bezeichner umbenannt.

[Gaucho316]

Und noch zwei kleine Patches.

Im Bookmarkmanager (Hauptmenü -> Movieplayer -> Bookmarks) wird in der rechten oberen Ecke das Hilfe-Icon angezeigt, obwohl überhaupt keine Hilfe existiert. Ich habe das Icon deshalb auskommentiert.

Link entfernt, da Patch im CVS

In der EPG-Suche finde ich es schön, wenn der Titel des gewählten Events direkt als Suchwort übernommen wird.

Link zum Patch entfernt

[rhabarber1848]

neutrino_fix-input-of-german-umlauts-and-special-chars_2012-04-07_1248.diff

committed:
http://article.gmane.org/gmane.comp.vid ... x.scm/3331
http://article.gmane.org/gmane.comp.vid ... x.scm/3332
http://article.gmane.org/gmane.comp.vid ... x.scm/3333
http://article.gmane.org/gmane.comp.vid ... x.scm/3334

bookmarkmanager_comment-out-help-icon_2012-04-07_1551.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3335

[rhabarber1848]

In der EPG-Suche finde ich es schön, wenn der Titel des gewählten Events direkt als Suchwort übernommen wird.

eventlist_use-selected-event-title-as-default-search-word_2012-04-07_1627.diff

Habe ich noch nicht in Aktion testen können, kommt aber noch

PS: Ich würde "m_search_keyword.empty()" rauslassen. Wenn von der Eventliste ausgehend
das Suchmenü geöffnet wird, wird der markierte Sendungstitel übertragen. Wenn das Suchfenster
geschlossen, in der Eventliste eine andere Sendung markiert und erneut das Suchmenü geöffnet
wird, wird der bisherige Sendungsname nicht durch die neu markierte Sendung überschrieben.
Imho ist das nicht intuitiv.

[Gaucho316]

Ich würde "m_search_keyword.empty()" rauslassen.

Hatte ich auch zuerst so gehabt. Das Problem ist dann aber, wenn man etwas von Hand eingegeben hat und die Suche wieder öffnet, wird einem das gnadenlos überschrieben. Das wollte ich verhindern.

[rhabarber1848]

Ich würde "m_search_keyword.empty()" rauslassen.

Hatte ich auch zuerst so gehabt. Das Problem ist dann aber, wenn man etwas von Hand eingegeben hat und die Suche wieder öffnet, wird einem das gnadenlos überschrieben. Das wollte ich verhindern.

Stimmt. Alternativvorschlag: Ein hotkey im ansonsten leeren Eingabefeld, dass den
ausgewählten Sendungstitel übernimmt und die bisherigen Eingaben überschreibt?

[Gaucho316]

Halte ich für zu aufwendig. Ich habe mir da mal etwas einfacheres überlegt. Jetzt wird vor dem Aufruf des Suchmenüs das letzte automatisch gesetzte Suchwort mit dem tatsächlichen verglichen. Sind sie gleich, wurde es folglich nicht geändert und kann wieder automatisch gesetzt werden. Wurde es geändert, bleibt es dagegen so, wie es ist.

Link zum Patch entfernt

[Gaucho316]

Einen Fehler habe ich in der Klasse CStringInput noch gefunden. Wenn man einen String (vom Typ std::string) bearbeitet, der länger als das Eingabefeld ist, und das Bearbeiten mit der Home-Taste abbricht, wird der String trotzdem auf die Länge des Eingabefeldes gestutzt. Dieses Fehlverhalten kann man sehr gut bei der Bearbeitung der Film-Informationen im Moviebrowser beobachten.

Link entfernt, da Patch im CVS

Außerdem sollte der alte Wert des bearbeiteten Strings (vom Typ char*) wiederhergestellt werden, wenn das Bearbeiten "von außen" abgebrochen wird.

Link entfernt, da Patch im CVS

[Gaucho316]

Jetzt wird vor dem Aufruf des Suchmenüs das letzte automatisch gesetzte Suchwort mit dem tatsächlichen verglichen. Sind sie gleich, wurde es folglich nicht geändert und kann wieder automatisch gesetzt werden. Wurde es geändert, bleibt es dagegen so, wie es ist.

Ich habe nun doch wieder den Test auf m_search_keyword.empty() aus der ersten Version eingebaut. Dann wird zusätzlich das Suchwort vorausgefüllt, wenn es leer ist. Das sollte es nun aber endgültig sein.

Link entfernt, da Patch im CVS

[Gaucho316]

Außerdem sollte der alte Wert des bearbeiteten Strings (vom Typ char*) wiederhergestellt werden, wenn das Bearbeiten "von außen" abgebrochen wird.

Ich denke, dass man das auch für die Klasse CExtendedInput machen sollte.

Link entfernt, da Patch im CVS

[Gaucho316]

Einen hab ich noch. In den Klassen CMenuOptionNumberChooser, CMenuOptionChooser und CMenuOptionStringChooser ist die Berechnung der Startposition des Optionswerts falsch. In der Klasse CMenuForwarder wird das richtig gemacht. Deshalb habe ich das von da übernommen. Aufgefallen ist mir das bei der Bearbeitung des Genres in den Film-Informationen des Moviebrowsers. Geht man dort alle Optionen durch, bleibt irgendwann links ein Zeichenrest übrig.

Link entfernt, da Patch im CVS

[Gaucho316]

15 Downloads und keine negative Meldung. Ich würde sagen, der Patch für die LCD-Anzeige kann auch ins CVS.

http://forum.tuxbox.org/forum/viewtopic ... 93#p382793

[rhabarber1848]

15 Downloads und keine negative Meldung.

Einer davon war ich Ich konnte den Patch und die anderen aber noch nicht
testen, die commits mache ich nach erfolgreichen Test am Wochenende

[rhabarber1848]

committed:

stringinput_only-change-value-string-if-ok-pressed_2012-04-11_0822.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3366

stringinput_restore-old-value-if-cancelled_2012-04-11_1507.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3367

stringinput_ext_restore-old-value-if-cancelled_2012-04-13_1420.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3368

eventlist_use-selected-event-title-as-default-search-word_2012-04-12_2308.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3369

menue_fix-start-pos-calculation-of-option-value_2012-04-14_1900.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3370

[Gaucho316]

Durch einen Thread im DBox2World-Forum bin ich darauf aufmerksam geworden, dass in der EPG-Liste bei Sortierung nach dem Titel Einträge mit demselben Titel nach der Event-ID sortiert werden. Das ist natürlich völlig sinnfrei. Ich habe das mal schnell so umgebaut, dass dann nach der Startzeit sortiert wird.

Link entfernt, da Patch im CVS

[rhabarber1848]

eventlist_sort-events-with-same-title-by-starttime_2012-04-26_0857.diff

committed to CVS: http://article.gmane.org/gmane.comp.vid ... x.scm/3374

[Gaucho316]

Da ich auch immer ein Auge auf die Commits im Coolstream GIT werfe, habe ich mal zwei Patches übernommen, die auch für Neutrino wichtig sind, weil sie Speicherlecks schließen.

CLocaleManager: delete char array in destructor properly
Link entfernt, da Patch im CVS

CPersonalizeGui: fix memleak in addSeparator()
Link entfernt, da Patch im CVS

[Gaucho316]

Der zweite Patch von gestern war nur ein Teil eines Patches aus dem Coolstream GIT. Hier nun der Rest, der dafür sorgt, dass auch die Personalisierungseinstellungen für das Haupt-, das Einstellungs- und das Servicemenü mit einem Druck auf die d-box-Taste komplett (d.h. inklusive aller vorher geöffneten Menüs) verlassen werden.

CPersonalizeGui: close all menus on dbox key
Link entfernt, da Patch im CVS


Und dann habe ich noch einen Patch für die Personalisierung, der diesmal aber von mir ist. Wenn man im Personalisierungsmenü etwas geändert hat, wird man ja beim Verlassen des Menüs gefragt, ob man die Änderungen übernehmen möchte. Wählt man "Nein", wird man dann gefragt, ob man die Änderungen verwerfen möchte. Wählt man hier "Ja", werden die Änderungen allerdings nicht, wie erwartet, zurückgesetzt. Außerdem landet man dann wieder im Personalisierungsmenü, was für mich keinen Sinn ergibt. Man hat ja schließlich vorher die Home-, Links- oder d-box-Taste gedrückt bzw. "Zurück" im Menü gewählt.

CPersonalizeGui: fix settings restore
Link entfernt, da Patch im CVS

[Gaucho316]

Und noch ein Patch aus dem Coolstream GIT, der ein Speicherleck in der Klasse CKeybindSetup schließt. Das ist mir wohl damals durch die Lappen gegangen, als ich alle Menüklassen durchgegangen bin.

CKeybindSetup: fix memleak
Link entfernt, da Patch im CVS