Commit-Wünsche für das CVS II

[rhabarber1848]

movieinfo.cpp
remove_currently_unused_variables.diff

Ist bei diesem Patch jetzt nicht zuviel rausgeflogen? Jetzt wird doch nichts mehr gespeichert, weil write(fd, text, text_size) auskommentiert wurde.

So müsste es richtig sein: http://article.gmane.org/gmane.comp.vid ... x.scm/3436

[rhabarber1848]

Und schließlich habe ich einen Patch von Neutrino-HD portiert, der gestern eingecheckt wurde. Der sorgt laut Log-Eintrag dafür, dass Events weniger Speicher verbrauchen, da std::vector für components und ratings statt std::multiset bzw. std::set verwendet wird. Und ein geringerer Speicherverbrauch ist auf der d-box 2 immer sinnvoll.

Sectionsd: use vector for components and ratings to decrease event size
sectionsd_use-vector-for-components-and-ratings-to-decrease-event-size_2012-05-30_1948.diff

Gibt es noch andere Testergebnisse zu diesem Patch?

[GetAway]

movieinfo.cpp
remove_currently_unused_variables.diff

Ist bei diesem Patch jetzt nicht zuviel rausgeflogen? Jetzt wird doch nichts mehr gespeichert, weil write(fd, text, text_size) auskommentiert wurde.

So müsste es richtig sein: http://article.gmane.org/gmane.comp.vid ... x.scm/3436

Aua, dank dir!

[rhabarber1848]

Ist bei diesem Patch jetzt nicht zuviel rausgeflogen? Jetzt wird doch nichts mehr gespeichert, weil write(fd, text, text_size) auskommentiert wurde.

So müsste es richtig sein: http://article.gmane.org/gmane.comp.vid ... x.scm/3436

Aua, dank dir!

Kein Problem, ist mir auch durchgerutscht

[Gaucho316]

Eine Sache habe ich die Tage noch vor. Ich will den Notifier COnOffNotifier (setting_helpers.h/cpp) so aufbohren, dass er universeller einsetzbar ist und so ein paar andere Notifierklassen ersetzen kann.

Ich habe die Postings über die Überarbeitung der Klasse COnOffNotifier ausgelagert, da wohl noch Diskussionsbedarf besteht.

http://forum.tuxbox.org/forum/viewtopic.php?f=2&t=50110

[Gaucho316]

Hier ein Patch, der bei der Suche nach dem Videoformat-Fehler im Audioplayer abgefallen ist. Er macht nur den Code übersichtlicher. Sonst ändert sich nichts.

Neutrino: use constants instead of numbers to compare video format
Link entfernt, da Patch im CVS

[Gaucho316]

[zapit] cmd 13 refused in standby mode

Mal noch etwas ganz anderes. Dieser Log-Eintrag kommt dadurch, dass g_Zapit->getLastChannel(...) nach g_Zapit->setStandby(true) aufgerufen wird. Ich denke, dass die Reihenfolge vertauscht werden müsste. Bei "cmd 13" handelt es sich nämlich um CMD_GET_LAST_CHANNEL und das ist die Message, die an Zapit geschickt wird, wenn getLastChannel() aufgerufen wird.

Neutrino: pause zapit after getting last channel in Audioplayer, Esound, Movieplayer and UPnP Browser
Link entfernt, da Patch im CVS

[Mourice]

Hi,

Fehlermeldung ist jetzt weg.

Aber ob sich dadurch etwas verbessert hat ... subjektiv keine Änderungen bemerkbar.

MfG

[Gaucho316]

Wenn ich den Code richtig verstanden habe, wird durch das Vertauschen der Code-Reihenfolge die Möglichkeit geschaffen, einen unnötigen Umschaltvorgang zu vermeiden. Frag mich aber nicht, in welchen Situationen das zutrifft.

[rhabarber1848]

committed:

neutrino_use-constants-instead-of-numbers-to-compare-video-format_2012-06-13_1538.diff

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

neutrino_pause-zapit-after-getting-last-channel_2012-06-13_1822.diff

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

[Gaucho316]

Mir sind mal wieder ein paar Sachen aufgefallen.

Audio setup: fix segfault on TD
Link entfernt, da Patch im CVS

Extended settings: fix memleak on TD
Link entfernt, da Patch im CVS

Record setup: rename notifier to clarify its purpose
Link entfernt, da Patch im CVS

Scan setup: fix memleak
Link zum Patch entfernt

Themes: create color setup notifier in constructor
Link entfernt, da Patch im CVS

Video setup: remove unnecessary setting of video format
Link entfernt, da Patch im CVS

Zapit setup: use just one notifier
Link entfernt, da Patch im CVS

Edit: Den Patch fürs Scan-Setup habe ich aufgrund neuer Erkenntnisse meinerseits wieder entfernt.

[Gaucho316]

Ich habe mal zwei Patches für zapit aus dem Coolstream GIT portiert, die mir sinnvoll erschienen.

Zapit Bouquetmanager: add destructor
Link entfernt, da Patch im CVS

Zapit Bouquetmanager: fix invalid iterator advance
Link entfernt, da Patch im CVS

[Gaucho316]

Ich habe hier noch einen Patch für zapit. Dieses Mal ist es aber keine Portierung. Er fördert die Verständlichkeit des Codes. Ansonsten ändert sich nichts.

Zapit: use enums instead of numbers for service type
Link entfernt, da Patch im CVS

[rhabarber1848]

committed:

audio-setup_fix-segfault-on-td_2012-06-20_1925.diff

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

miscsettings-menu_fix-memleak-on-td_2012-06-20_1820.diff

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

record-setup_rename-notifier-to-clarify-its-purpose_2012-06-20_1822.diff

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

themes_create-color-setup-notifier-in-constructor_2012-06-20_1928.diff

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

video-setup_remove-unnecessary-setting-of-video-format_2012-06-20_1827.diff

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

zapit-setup_use-just-one-notifier_2012-06-20_1828.diff

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

zapit_bouquetmanager_add-destructor_2012-06-25_1821.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3482
http://article.gmane.org/gmane.comp.vid ... x.scm/3483

zapit_bouquetmanager_fix-invalid-iterator-advance_2012-06-25_1824.diff

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

zapit_use-enums-instead-of-numbers-for-service-type_2012-06-26_1025.diff

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

[rhabarber1848]

Sectionsd: use vector for components and ratings to decrease event size

sectionsd_use-vector-for-components-and-ratings-to-decrease-event-size_2012-05-30_1948.diff

63 Downloads ohne negative Rückmeldungen,
funktioniert seit Wochen problemlos mit positivem Erfolg,
committed: http://article.gmane.org/gmane.comp.vid ... x.scm/3487

[Gaucho316]

Immer mal wieder fällt mir auf, dass die LCD-Anzeige in Menüs nicht aktualisiert wird, wenn man einen Menüeintrag per Direkttaste aufruft. Sehen kann man das bspw. im Hauptmenü, wenn man das Skriptemenü per Direkttaste aufruft, oder auch im Servicemenü, wenn man "Ucodes überprüfen" per Direkttaste aufruft. Dann steht in der LCD-Anzeige immer noch der Text des vorher gewählten Eintrags. Der folgende Patch behebt dieses Fehlverhalten.

Menu widget: update selected item on pressing direct key
Link entfernt, da Patch im CVS

[GetAway]

sectionsd.cpp: fix Segmentation fault if max_events == 0, ported from Neutrino-HD

sectionsd_fix_segmentation_fault_2012_06_27_0904.diff

Seen in Neutrino-HD: use .empty() instead of .size()
use_empty_instead_of_size_2012_06_27_1522.diff

[Gaucho316]

Und noch drei Patches.

Bookmarkmanager: fix value of variable res
Link entfernt, da Patch im CVS

Network setup: use enums and remove unnecessary vars
Link entfernt, da Patch im CVS

Neutrino: remove unnecessary repaint calls
Link entfernt, da Patch im CVS

[rhabarber1848]

committed:

menue_update-selected-item-on-pressing-direct-key_2012-06-26_2230.diff

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

sectionsd_fix_segmentation_fault_2012_06_27_0904.diff

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

use_empty_instead_of_size_2012_06_27_1522.diff[/url]

http://article.gmane.org/gmane.comp.vid ... x.scm/3498
http://article.gmane.org/gmane.comp.vid ... x.scm/3499
http://article.gmane.org/gmane.comp.vid ... x.scm/3500
http://article.gmane.org/gmane.comp.vid ... x.scm/3501
http://article.gmane.org/gmane.comp.vid ... x.scm/3502
http://article.gmane.org/gmane.comp.vid ... x.scm/3503
http://article.gmane.org/gmane.comp.vid ... x.scm/3504
http://article.gmane.org/gmane.comp.vid ... x.scm/3505
http://article.gmane.org/gmane.comp.vid ... x.scm/3506
http://article.gmane.org/gmane.comp.vid ... x.scm/3507
http://article.gmane.org/gmane.comp.vid ... x.scm/3508

bookmarkmanager_fix-value-of-variable-res_2012-06-27_1757.diff

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

network-setup_use-enums-and-remove-unnecessary-vars_2012-06-27_1802.diff

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

neutrino_remove-unnecessary-repaint-calls_2012-06-27_1811.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3511
http://article.gmane.org/gmane.comp.vid ... x.scm/3512
http://article.gmane.org/gmane.comp.vid ... x.scm/3513

[rhabarber1848]

Wie wäre es mit CVS commit-Rechten für Gaucho316?

Ach, lass mal. So geht's doch auch. In nächster Zeit wird keine so geballte Ladung mehr kommen. Ich muss ab Donnerstag wieder arbeiten.

Nach dem Urlaub ist vor dem Urlaub

[Gaucho316]

Im Benutzermenü gibt es noch ein paar Speicherlecks, die aber vermutlich in der Praxis nicht auftreten werden. Dazu müsste man in ein Benutzermenü dasselbe Element mehrfach hinzufügen. Der folgende Patch löst das Problem aber trotzdem, denn sicher ist sicher.

Usermenu: fix memleaks
Link entfernt, da Patch im CVS

Wie wäre es mit CVS commit-Rechten für Gaucho316?

Ach, lass mal. So geht's doch auch. In nächster Zeit wird keine so geballte Ladung mehr kommen. Ich muss ab Donnerstag wieder arbeiten.

Nach dem Urlaub ist vor dem Urlaub

Alle Jahre wieder ...

[GetAway]

Zapping-History Bouquet wurde in Programmverlauf umbenannt, dies sollte
man unter Einstellungen->Tasten nachholen. Desweiteren würde zur Logik
des Programmverlaufs passen, den Startkanal da mit zu rein zunehmen.

Das war wohl bei Zapping-History unlogisch.

[Gaucho316]

Desweiteren würde zur Logik des Programmverlaufs passen, den Startkanal da mit zu rein zunehmen.

Ich nutze den Programmverlauf normalerweise nicht, habe ihn aber mal zum Testen aktiviert. Der Startkanal landet bei mir im Programmverlauf, wenn ich die Box starte und dann umschalte. Bei dir nicht?

Zapping-History Bouquet wurde in Programmverlauf umbenannt, dies sollte
man unter Einstellungen->Tasten nachholen.

update channel history locales
Link entfernt, da Patch im CVS

[dietmarw]

SATFINDER ZEIGT PROVIDER UND NICHT SAT NAMEN.....WIESO ????

[Gaucho316]

Gestern wurde im Coolstream GIT ein Bug in Neutrino-HD beseitigt, der in Neutrino nicht auftritt. Allerdings ist mir durch diesen Commit aufgefallen, dass die Klassen CBEBouquetWidget (bouqueteditor_bouquets.h), CBEChannelWidget (bouqueteditor_channels.h), CListBox (listbox.h) und CDirMenu (moviebrowser.h) unnötigerweise von CMenuWidget statt einfach von CMenuTarget abgeleitet werden.

Neutrino: no need to inherit CBEBouquetWidget, CBEChannelWidget, CListBox and CDirMenu from CMenuWidget
Link entfernt, da Patch im CVS