Commit-Wünsche für das CVS II

[Gaucho316]

Inzwischen gibt es ja in vielen Menüs Unterüberschriften. Auf der LCD-Anzeige sieht man aber immer nur die Hauptüberschrift eines Menüs. Das ist bspw. in den Einstellungen ziemlich unnütz. Der folgende Patch sorgt nun dafür, dass die Unterüberschrift statt der Hauptüberschrift für die LCD-Anzeige genommen wird. Gibt es keine Unterüberschrift, wird natürlich wie bisher die Hauptüberschrift angezeigt.

Menu widget: use subhead as title on LCD
Link entfernt, da Patch im CVS


Außerdem ist mir im Moviebrowser aufgefallen, dass nach dem ersten Start des Moviebrowsers nach dem Start von Neutrino die Movieinfos auf der LCD-Anzeige fehlen. Der folgende Patch behebt das Problem.

Moviebrowser: show movie infos on LCD even after first start
Link entfernt, da Patch im CVS

[Gaucho316]

2 Portierungen aus dem Coolstream GIT:

Channel list: don't paint channel list if event list returns menu_return::RETURN_EXIT_ALL
Link entfernt, da Patch im CVS

Bouquet list: add bouquet id safe-checks
Link entfernt, da Patch im CVS

[GetAway]

Fix: Show global progressbar even if same value after hide and paint();
fix_globalstatus_progresswindow.cpp_2012_10_23_2158.diff

[Gaucho316]

Und noch eine Portierung aus dem Coolstream GIT für die Kanalliste:

Channel list: use CRCInput::getNumericValue() instead of switch()
Link entfernt, da Patch im CVS

[Gaucho316]

Ich habe mich heute gefragt, warum bei der Ankündigung eines Umschalttimers EPG-Infos angezeigt werden und bei der Ankündigung eines Aufnahmetimers nicht. Ich habe deshalb den Code einfach kopiert, so dass nun auch bei der Ankündigung eines Aufnahmetimers EPG-Infos angezeigt werden.

Edit: Ich habe noch eine einfachere Variante im Kopf. Die gibt's dann morgen.

Edit: Ich habe das nun etwas umgebaut und spare mir das Holen der gesamten Timerliste, indem ich die Daten aus dem gesendeten Timerevent nutze. Den Code habe ich auch für die Ankündigung eines Umschalttimers übernommen, so dass nun auch dort nicht mehr unnötigerweise die gesamte Timerliste geholt werden muss.

Edit: Ich habe den Patch nochmals aktualisiert. Eine Codezeile, die ich zur Sicherheit eingebaut hatte, habe ich wieder entfernt, da sie unnütz war.

Edit: Leider musste ich noch eine Änderung vornehmen. Wenn zapit im Standby ist (z.B. im Movieplayer), wird getChannelName() nicht unterstützt und liefert einen undefinierten Rückgabewert. Deshalb habe ich vor die Ausführung von getChannelName() eine Abfrage, ob zapit im Standby ist, eingebaut.

Neutrino: show EPG infos on announce recording and rework announce zapping
Link entfernt, da Patch im CVS


Nebenbei ist mir aufgefallen, dass CMD_GETTIMER und CMD_GETTIMERLIST in timerd.cpp leicht unterschiedlich sind. Ich habe den Code angeglichen.

timerd: bring CMD_GETTIMER in line with CMD_GETTIMERLIST
Link entfernt, da Patch im CVS

[Gaucho316]

In der EPG-Suche fehlt ein Notifier. Wenn man nämlich die Option "[2] Suche in" ändert, wird die Option "[3]" nicht angepasst.

Event list: add notifier to search menu
Link entfernt, da Patch im CVS

[rhabarber1848]

committed:

neutrino_add-possibility-to-cancel-sleep-timer_2012-10-20_2030.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3607
http://article.gmane.org/gmane.comp.vid ... x.scm/3608

menue_use-subhead-as-title-on-lcd_2012-10-22_1809.diff

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

moviebrowser_show-movie-infos-on-lcd-even-after-first-start_2012-10-22_1815.diff

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

channellist_dont-paint-channel-list-if-event-list-returns-exit-all_2012-10-23_2126.diff

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

bouquetlist_add-bouquet-id-safe-checks_2012-10-23_2137.diff

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

fix_globalstatus_progresswindow.cpp_2012_10_23_2158.diff

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

channellist_use-getnumericvalue-instead-of-switch_2012-10-24_1815.diff

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

neutrino_show-epg-infos-on-announce-recording_2012-10-27_2150.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3615
http://article.gmane.org/gmane.comp.vid ... x.scm/3616
http://article.gmane.org/gmane.comp.vid ... x.scm/3617

timerd_bring-cmd-gettimer-in-line-with-cmd-gettimerlist_2012-10-26_1824.diff

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

eventlist_add-notifier-to-search-menu_2012-11-01_1629.diff

Danke für diesen Patch! http://article.gmane.org/gmane.comp.vid ... x.scm/3619

[Gaucho316]

Ich habe heute Abend noch etwas aus dem Coolstream GIT portiert. Nun kann man die weiteren Ausstrahlungen einer Sendung im EPG-Info-Fenster direkt über die blaue Taste programmieren. Vielleicht kann das auch noch schnell ins CVS.

Edit: Ich habe eine Kleinigkeit ergänzt, die im Originalcode fehlt. In epgview.cpp habe ich die Auswertung des Rückgabewertes von eventList->exec(...) hinzugefügt.

EPG Viewer: show more screenings in extra window
Link entfernt, da Patch im CVS

[rhabarber1848]

epgview_show-more-screenings-in-extra-window_2012-11-02_1649.diff

nach positivem test committed:
http://article.gmane.org/gmane.comp.vid ... x.scm/3621
http://article.gmane.org/gmane.comp.vid ... x.scm/3622
http://article.gmane.org/gmane.comp.vid ... x.scm/3623

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

Gibt es zu diesem Patch noch Veränderungen oder kann er committed werden?

[Gaucho316]

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

Gibt es zu diesem Patch noch Veränderungen oder kann er committed werden?

Laut seife ist diese Änderung wohl richtig. Er würde es aber mit valgrind nachprüfen. Da steige ich aber aus. Valgrind auf der d-box soll ja ein ziemlicher Krampf sein, wenn ich mich richtig erinnere. Ich habe auch noch nie damit gearbeitet. Wir werden wohl einfach darauf vertrauen müssen, dass das stimmt, was ich da sehe.

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.

Vielleicht kann hier auch delete ganz raus. Möglicherweise ist aber auch der komplette Codeteil überflüssig, weil CRCInput::RC_ignore nie an der Stelle ankommt?
Hier mal der angesprochene Codeauszug:

Code: Alles auswählen

if (msg == CRCInput::RC_ignore)
{
    delete (unsigned char*) data;
    return messages_return::handled;
}

[Gaucho316]

Ich habe im EPG Viewer überflüssig gewordenen Code entdeckt.

EPG Viewer: remove superfluous code
Link entfernt, da Patch in Git

[Gaucho316]

Nachdem ich die Ergebnisliste der EPG-Suche geschlossen habe, würde ich gerne die ursprüngliche EPG-Liste angezeigt bekommen. Der folgende Patch sorgt dafür.

Edit: Ich habe noch eine Ergänzung eingebaut, so dass auch in der EPG-Suche menu_return::RETURN_EXIT_ALL vernünftig durchgereicht wird.

Event list: show event list after closing search results
Link entfernt, da Patch in Git

[Gaucho316]

String input: center numeric pad horizontally, ported from Coolstream GIT
Link entfernt, da Patch in Git

Neutrino: add some icon defines
Link entfernt, da Patch in Git

Locale: fix typo
Link entfernt, da Patch in Git

[Mourice]

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

Gibt es zu diesem Patch noch Veränderungen oder kann er committed werden?

Laut seife ist diese Änderung wohl richtig. Er würde es aber mit valgrind nachprüfen. Da steige ich aber aus. Valgrind auf der d-box soll ja ein ziemlicher Krampf sein, wenn ich mich richtig erinnere. Ich habe auch noch nie damit gearbeitet. Wir werden wohl einfach darauf vertrauen müssen, dass das stimmt, was ich da sehe.

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.

Vielleicht kann hier auch delete ganz raus. Möglicherweise ist aber auch der komplette Codeteil überflüssig, weil CRCInput::RC_ignore nie an der Stelle ankommt?
Hier mal der angesprochene Codeauszug:

Code: Alles auswählen

if (msg == CRCInput::RC_ignore)
{
    delete (unsigned char*) data;
    return messages_return::handled;
}

Hi,
die Änderung in der neutrino.cpp (Zeile 2613) macht bei mir keine erkennbare Probleme.
Ob allerdings der komplette Codeteil überflüssig ist, kann ich nicht beurteilen.
Werde aber mal testen, was die Box macht, wenn der ganze "Teil" nicht mehr vorhanden ist.

MfG

[Gaucho316]

Ich habe mal wieder überflüssigen Code entfernt. Die Anregung dazu habe ich aus dem Coolstream GIT.

CListFrame: remove superfluous code
Link entfernt, da Patch in Git

CTextBox: remove superfluous code
Link entfernt, da Patch in Git

die Änderung in der neutrino.cpp (Zeile 2613) macht bei mir keine erkennbare Probleme.
Ob allerdings der komplette Codeteil überflüssig ist, kann ich nicht beurteilen.

Ich hoffe ja noch auf eine Antwort von seife.

[Gaucho316]

Ich habe den Code, der für die Anzeige der weiteren Termine im EPG-Info-Fenster zuständig ist, ein klein wenig überarbeitet.

EPG Viewer: small rework of follow screenings code
Link entfernt, da Patch in Git

[GetAway]

Ja, Check ein @Gaucho316

[Gaucho316]

Ja, Check ein @Gaucho316

Ich lasse lieber einchecken.


CRemoteControl: enable AC3 auto-switch even if EPG unavailable
Link entfernt, da Patch in Git

[Gaucho316]

Und noch eine Kleinigkeit. Es ist möglich, dass im EPG-Info-Fenster bei den Namen der Tonspuren nur Leerzeichen und Kommas stehen, wenn die Tonspurnamen leer sind.

EPG Viewer: don't show empty audio track names
Link entfernt, da Patch in Git

[Gaucho316]

CFlashUpdate: fix memleaks
Link entfernt, da Patch in Git

[Gaucho316]

EPG Plus: fix memleaks
Link entfernt, da Patch in Git

Scan setup: fix memleak
Link entfernt, da Patch in Git

[Gaucho316]

Ich habe mal das Timer-Icon durchgängig in die Bearbeitungsmenüs der Timerliste eingebaut. Wenn wir schon ein eigenes Icon haben, können wir es ja auch benutzen.

Timer list: use timer icon in menus
Link entfernt, da Patch in Git


Ich habe noch ein paar andere Stellen rausgesucht, an denen meiner Meinung nach andere Icons besser passen als immer nur das Settings-Icon.

Neutrino: use appropriate icons in some menus
Link entfernt, da Patch in Git

[Gaucho316]

Hier zur Abwechslung mal ein Patch, den ich (zumindest teilweise) aus dem Neutrino-MP GIT portiert habe.

String input: avoid uninitialized access
Link zum Patch entfernt

[seife]

Der ist nur dazu da, um Valgrind zu beruhigen. Für die dbox würde ich den weglassen, oder wenn, dann nur ein

dispval[size] = 0;
+dispval[0] = 0;

das reicht schon zu, um die Funktion sicher zu gewährleisten (aber nicht, um Valgrind zum Schweigen zu bringen

[Gaucho316]

Einverstanden. Hier also nun die abgespeckte Variante des Patches.

String input: avoid uninitialized access
Link entfernt, da Patch in Git