Commit-Wünsche für das CVS II

[Gaucho316]

Es geht nicht um die Aufnahme-Einstellungen sondern um die Movieplayer-Einstellungen.

[Gaucho316]

Dann habe ich noch einen Patch für die Movieplayer-Einstellungen. Ich finde das Menü ziemlich überfrachtet. Deshalb habe ich die Streamingserver-Einstellungen in ein Untermenü verfrachtet.

...

Ich habe hier noch eine andere Variante, die mir persönlich besser gefällt. Sucht euch eine aus.

Movieplayer setup: add submenu for streaming server settings
movieplayer-setup_add-submenu-for-streaming-server-settings_2012-09-02_1719.diff

In die 2. Variante des Patches hat sich leider ein grober Fehler eingeschlichen. Hat man den Streamingserver deaktiviert, wurde auch das neue Streamingserver-Einstellungsmenü deaktiviert und man konnte ihn nie wieder aktivieren. Das war noch ein Überbleibsel aus der ersten Variante.

Movieplayer setup: add submenu for streaming server settings
Link entfernt, da Patch im CVS

[Gaucho316]

Durch den vorherigen Patch ist mir aufgefallen, dass das Deaktivieren des Streamingservers (VLC) in den Movieplayer-Einstellungen überhaupt keine Auswirkungen hat. Es werden nur die Menüpunkte im Einstellungsmenü deaktiviert. Sie bleiben aber aktiv. Deshalb habe ich eine Hinweisbox eingebaut, die beim Aufruf der Menüpunkte "Datei via VLC", "DVD" und "(S)VCD" erscheint, wenn der Streamingserver deaktiviert ist. Anschließend wird der Movieplayer wieder beendet.

Movieplayer: show hint if streaming server disabled
Link zum Patch entfernt


Edit: Ich habe da noch eine andere Idee im Kopf. Man könnte die Menüpunkte, mit denen man den Movieplayer im VLC-Modus aufruft, auch deaktivieren, wenn man den Streamingserver deaktiviert hat. Ich muss aber erst einmal sehen, ob das überhaupt so funktioniert, wie ich mir das denke.

Edit: Ich habe meine Idee nun mal umgesetzt. Funktionieren tut's. Man kann sich dann den vorherigen Patch sparen.

Movieplayer menu: disable streaming server entries if streaming server disabled
Link entfernt, da Patch im CVS

[Gaucho316]

Wenn ihr den vorherigen Patch für das Movieplayer-Menü mal getestet habt, könnte euch aufgefallen sein, dass beim Deaktivieren und Aktivieren des Streamingservers über das Einstellungsmenü im Movieplayer-Menü ganz kurz die VLC-Menüpunkte auftauchen und dann aber sofort wieder übermalt werden. Da das nicht besonders schön ist, habe ich folgenden Patch erstellt.

Er löst auch gleichzeitig ein Problem im Moviebrowser, das ich entdeckt habe. Wählt man einen der unteren Einträge in Einstellungen -> Film Informationen -> Markierungen und ändert den Namen, wird fälschlicherweise ein Menüeintrag am unteren Bildschirmrand gezeichnet.

CMenuWidget: don't paint hidden menu items on setActive changes
Link entfernt, da Patch im CVS

[Mourice]

Hi,

die "neuen" Patches von @Gaucho316 (Streamingserver Variante 2 und folgende) laufen bei mir ohne Auffälligkeiten.

Ich lass es aber noch von "Anderen" testen.

Aber im Prinzip ... man könnte es einchecken ...

MfG

Edit:
Auch das Einrücken von Untermenüs zu den jeweiligen Überschriften ist "sauber" ...

[rhabarber1848]

committed:

neutrino_use-addintroitems-in-menus_2012-09-01_2250.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3567
http://article.gmane.org/gmane.comp.vid ... x.scm/3568

movieplayer-setup_add-submenu-for-streaming-server-settings_2012-09-04_1959.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3569
http://article.gmane.org/gmane.comp.vid ... x.scm/3570
http://article.gmane.org/gmane.comp.vid ... x.scm/3571

movieplayer-menu_disable-streaming-server-entries-if-streaming-server-disabled_2012-09-05_1920.diff

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

menue_dont-paint-hidden-menu-items-on-setactive-changes_2012-09-06_1842.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3573
PS:

menue_fix-icon-offset-calculation_2012-08-29_1957.diff

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

[Gaucho316]

Und weiter geht's.

In einigen Menüklassen lassen sich die Konstruktoren vereinfachen, wenn man die Standardwerte sinnvoller setzt.

Neutrino: simplify some menu class constructors
Link entfernt, da Patch im CVS


Dann habe ich einige fehlende Menü-Untertitel ergänzt, damit es einheitlich ist:
Einstellungen -> OSD -> Sprache
Einstellungen -> OSD -> Farben -> Theme
Service -> Software-Aktualisierung
Service -> Software-Aktualisierung -> *
Und im Moviebrowser habe ich das komplette Einstellungemenü mit Untertiteln versehen.

Neutrino: add subheads to some menus
Link entfernt, da Patch im CVS


Dabei ist mir aufgefallen, dass der Hintergrund von Menü-Untertiteln mit zu langen Texten über das Fenster hinaus gezeichnet wird.

Menu separator: fix subhead painting
Link entfernt, da Patch im CVS


Außerdem habe ich im Menü "Internetradios einlesen" des Audioplayers, in der EPG-Suche und in der Timerliste beim Anlegen und Bearbeiten eines Timers den Zurück-Button gegen den Abbrechen-Button getauscht. Der passt meiner Meinung nach dort besser.

Neutrino: use cancel button in some menus
Link entfernt, da Patch im CVS


Schließlich habe ich die fehlenden "Intro Items" im Mountchooser und im EPG Plus in den Menüs, die man über die rote und blaue Taste erreicht, ergänzt.

Neutrino: add intro items to some menus
Link entfernt, da Patch im CVS


Edit: Ich habe den Patch "Menu separator: fix subhead painting" noch ein klein wenig optimiert.

Edit: Ich habe noch eine Stelle gefunden, an der Menü-Untertitel fehlen, und deshalb den Patch "Neutrino: add subheads to some menus" auch noch einmal ergänzt.

[Gaucho316]

Satbaby hat einen weiteren möglichen Segfault in sectionsd.cpp gefixt.

http://git.coolstreamtech.de/?p=cst-pub ... 0a9704b3a6

Eventuell wäre dieser Fix auch noch etwas.
http://git.coolstreamtech.de/?p=cst-pub ... cb6b68d03b

Den ersten Patch habe ich mal übernommen und noch ein klein wenig erweitert, um zu verhindern, dass Neutrino nie herunterfährt, wenn beim Speichern der EPG-Daten ein Fehler auftritt. Ob der zweite Patch auch wichtig ist, kann ich nicht einschätzen.

sectionsd: fix possible segfault and never ending shutdown
Link entfernt, da Patch im CVS

[rhabarber1848]

committed:

neutrino_simplify-some-menu-class-constructors_2012-09-14_1918.diff

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

neutrino_add-subheads-to-some-menus_2012-09-15_2145.diff

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

menue_fix-subhead-painting_2012-09-15_2029.diff

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

neutrino_use-cancel-button-in-some-menus_2012-09-15_1750.diff

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

neutrino_add-intro-items-to-some-menus_2012-09-15_1817.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3591
http://article.gmane.org/gmane.comp.vid ... x.scm/3592

sectionsd_fix-possible-segfault-and-never-ending-shutdown_2012-09-16_2011.diff

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

[Gaucho316]

Im Coolstream GIT wurde ein Schreibfehler in der XML-Datei mit den Aufnahme-Informationen korrigiert. Ich habe das mal übernommen, gebe aber zu Bedenken, dass dadurch der Eintrag <qualitiy>...</qualitiy> in vorhandenen XML-Dateien nicht mehr eingelesen wird. Da man diesen Wert aber so oder so von Hand eingeben muss, finde ich das nicht schlimm.

Movieinfo: fix typo in xml-entry "quality"
Link zum Patch entfernt

[seife]

Damit macht man aber die Arbeit zunichte, wenn das jemand von hand eingegeben hat.

Aus Kompatibilitätsgründen sollte man also beide Varianten einlesen, aber nur noch die richtige rausschreiben.

...das ist so ein Beispiel einer Änderung, wegen der dann alle immer sagen "Settingsübernahme ist böse", obwohl man bei defensiver Programmierung damit keine Probleme hat

[Gaucho316]

Ok, hast mich überzeugt. Der zusätzliche Code ist ja auch recht klein.

Movieinfo: fix typo in xml-entry "quality"
Link entfernt, da Patch im CVS

Edit: Ich habe vorsichtshalber die Einlesereihenfolge von <quality>...</quality> und <qualitiy>...</qualitiy> getauscht.

[rhabarber1848]

movieinfo_fix-typo-in-xml-entry-quality_2012-09-24_1959.diff

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

[Gaucho316]

Heute ist mir mal wieder aufgefallen, dass im Movieinfo-Fenster (erreichbar über die ?-Taste im Movieplayer und Moviebrowser) der Dateiname oft abgeschnitten dargestellt wird. Der folgende Patch hilft dagegen, indem nun auch ein automatischer Umbruch nach Slashes und Unterstrichen möglich ist.

CTextBox: break lines also after slashes and underscores
Link entfernt, da Patch im CVS


Dabei habe ich bemerkt, dass in der Klasse CTextBox die XML-Sonderzeichen unnötigerweise dekodiert werden. Meiner Meinung nach können die hier gar nicht mehr vorkommen, weil die Sonderzeichen schon vorher an anderer Stelle dekodiert werden.

CTextBox: remove unnecessary special char decoding
Link entfernt, da Patch im CVS


Und weil ich schon mal dabei bin, habe ich außerdem noch zwei Patches für die Klasse CTextBox aus dem Coolstream GIT übernommen.

CTextBox: prevent possible division by zero
Link entfernt, da Patch im CVS

CTextBox: fix string access outside of its size
Link entfernt, da Patch im CVS

[Gaucho316]

Eine andere Sache, die mir immer mal wieder auffällt, ist, dass im Fenster mit den Sendungsinformationen (zweimal die ?-Taste im TV-Modus drücken) manchmal der Info1-Text (der kursiv geschriebene Text) fehlt. Das kommt dadurch, dass im Info2-Text nach dem Info1-Text gesucht wird. Wird der Info1-Text gefunden, wird er nicht angezeigt. Das habe ich diese Woche wieder bei der Serie "Alcatraz" bemerkt, weil dort die Folgentitel nach den gesuchten Personen benannt sind und diese auch im Info2-Text vorkommen. Auf den Sky Filmkanälen, bei denen im Info1-Text das Genre gesendet wird, kann man das auch gut beobachten. Ich finde dieses Verhalten nicht besonders schön. Deshalb habe ich den entsprechenden Code so abgeändert, dass der Info1-Text nur dann nicht mehr angezeigt wird, wenn er am Anfang des Info2-Textes gefunden wird. So wird es ein paar Zeilen höher im Code schließlich auch gemacht.

EPG Data Viewer: only hide info1 if info2 begins with info1
Link entfernt, da Patch im CVS

[Mourice]

Eine andere Sache, die mir immer mal wieder auffällt, ist, dass im Fenster mit den Sendungsinformationen (zweimal die ?-Taste im TV-Modus drücken) manchmal der Info1-Text (der kursiv geschriebene Text) fehlt. Das kommt dadurch, dass im Info2-Text nach dem Info1-Text gesucht wird. Wird der Info1-Text gefunden, wird er nicht angezeigt. Das habe ich diese Woche wieder bei der Serie "Alcatraz" bemerkt, weil dort die Folgentitel nach den gesuchten Personen benannt sind und diese auch im Info2-Text vorkommen. Auf den Sky Filmkanälen, bei denen im Info1-Text das Genre gesendet wird, kann man das auch gut beobachten. Ich finde dieses Verhalten nicht besonders schön. Deshalb habe ich den entsprechenden Code so abgeändert, dass der Info1-Text nur dann nicht mehr angezeigt wird, wenn er am Anfang des Info2-Textes gefunden wird. So wird es ein paar Zeilen höher im Code schließlich auch gemacht.

EPG Data Viewer: only hide info1 if info2 begins with info1
epgview_only-hide-info1-if-info2-begins-with-info1_2012-10-04_1903.diff

Hi,

funktioniert ...

Werde mir unverschämterweise herausnehmen, dieses auch in meinem Image zu benutzen ...

MfG

[Gaucho316]

Im Movieinfo-Fenster fehlt die Anzeige des Genres. Ich habe den entsprechenden Code einfach mal aus dem EPG-Viewer übernommen. Vielleicht gibt es ja noch eine einfachere Möglichkeit, aber wenn niemandem etwas besseres einfällt, könnte meine Variante ins CVS.

Movieinfo: show genre
Link zum Patch entfernt

[GetAway]

nfs.cpp: disable mount option if mountpoint is mounted. Ported from N-HD

disable mount option if mountpoint is mounted_2012_10_11_1818.diff

[Gaucho316]

Im Movieinfo-Fenster fehlt die Anzeige des Genres. Ich habe den entsprechenden Code einfach mal aus dem EPG-Viewer übernommen. Vielleicht gibt es ja noch eine einfachere Möglichkeit, aber wenn niemandem etwas besseres einfällt, könnte meine Variante ins CVS.

Ich habe jetzt doch noch eine andere Variante des Patches, bei der das Kopieren der Locale-Arrays nach movieinfo.cpp entfällt. Mir gefällt diese Version besser.

Movieinfo: show genre
Link entfernt, da Patch im CVS

[GetAway]

Tuxboxapi: tell sectionsd after registering neutrino, where we are.
TODO: find a way to do it similar with sectionsdcontrol

tell_sectionsd_where_we_are!_tuxboxapi_2012_10_15_2243.diff

[rhabarber1848]

commited:

textbox_break-lines-also-after-slashes-and-underscores_2012-10-02_1934.diff

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

textbox_remove-unnecessary-special-char-decoding_2012-10-02_2002.diff

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

textbox_prevent-possible-division-by-zero_2012-10-02_1932.diff

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

textbox_fix-string-access-outside-of-its-size_2012-10-02_1933.diff

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

movieinfo_show-genre_2012-10-11_1922.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3601
http://article.gmane.org/gmane.comp.vid ... x.scm/3602

epgview_only-hide-info1-if-info2-begins-with-info1_2012-10-04_1903.diff

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

[rhabarber1848]

committed:

disable mount option if mountpoint is mounted_2012_10_11_1818.diff

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

tell_sectionsd_where_we_are!_tuxboxapi_2012_10_15_2243.diff

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

[Gaucho316]

Der folgende Patch schließt drei Speicherlecks in Neutrino im Zusammenhang mit delete. Ich habe ihn aus dem Coolstream GIT übernommen.

Neutrino: fix memleaks
Link entfernt, da Patch in Git


Die Idee für den nächsten Patch stammt auch aus dem Coolstream GIT. Ich habe es aber codetechnisch etwas anders gelöst. Hat man einen Sleeptimer gesetzt, wird man eine Minute vor Ausführung nicht mehr nur wie bisher gewarnt, sondern man hat die Möglichkeit, den Vorgang abzubrechen. Das Fenster wird für 30 Sekunden eingeblendet. Zusätzlich habe ich das Abbruch-Fenster bei Shutdowntimern auch so umgestellt, dass es für 30 Sekunden angezeigt wird. Bisher war das dort auf 5 Sekunden eingestellt, hat aber gar nicht funktioniert, weil ein Funktionsparameter gefehlt hat, so dass das Fenster nie ausgeblendet wurde. 5 Sekunden waren mir übrigens etwas zu kurz.

Neutrino: add possibility to cancel sleep timer
Link entfernt, da Patch im CVS

[seife]

Der folgende Patch schließt drei Speicherlecks in Neutrino im Zusammenhang mit delete. Ich habe ihn aus dem Coolstream GIT übernommen.

Neutrino: fix memleaks
neutrino_fix-memleaks_2012-10-19_2056.diff

Das muss nicht richtig sein, weil der rcinput-Code und der infoviewer-Code (der da leider auch mit involviert ist...) unterschiedlich sind. Beim neutrino habe ich das vor langem schon mal gefixt, im neutrino-HD ist das immer noch kaputt.
Du musst wirklich genau schauen wo die daten mit welchem Typ alloziert wurden um dann zu wissen, wie sie gelöscht werden können.

Generell sollte im neutrino jetzt eigentlich immer delete[] richtig sein, aber man sollte es nochmals prüfen.

[Gaucho316]

Der folgende Patch schließt drei Speicherlecks in Neutrino im Zusammenhang mit delete. Ich habe ihn aus dem Coolstream GIT übernommen.

Das muss nicht richtig sein, weil der rcinput-Code und der infoviewer-Code (der da leider auch mit involviert ist...) unterschiedlich sind.

Richtig blicke ich da auch nicht durch. Ich habe versucht zu verstehen, was da passiert. In rcinput.cpp in den Zeilen 1478 bis 1482 wird mit RC_WithData gearbeitet. Dabei wird

Code: Alles auswählen

*data = (neutrino_msg_data_t) p

gemacht. Und p wird vorher in Zeile 1208 mit

Code: Alles auswählen

p = new unsigned char[ emsg.dataSize + 1 ];

alloziert. Deshalb dachte ich, dass delete[] richtig ist.

Edit: In Zeile 2613 von neutrino.cpp habe ich übrigens auch überlegt, ob da nicht delete[] statt delete hingehört. Sicher bin ich mir aber nicht.