So müsste es richtig sein: http://article.gmane.org/gmane.comp.vid ... x.scm/3436Gaucho316 hat geschrieben:Ist bei diesem Patch jetzt nicht zuviel rausgeflogen? Jetzt wird doch nichts mehr gespeichert, weil write(fd, text, text_size) auskommentiert wurde.GetAway hat geschrieben:movieinfo.cpp
remove_currently_unused_variables.diff
Commit-Wünsche für das CVS II
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
Gibt es noch andere Testergebnisse zu diesem Patch?Gaucho316 hat geschrieben: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
-
- Contributor
- Beiträge: 1509
- Registriert: Donnerstag 27. Dezember 2007, 12:59
Re: Commit-Wünsche für das CVS II
Aua, dank dir!rhabarber1848 hat geschrieben:So müsste es richtig sein: http://article.gmane.org/gmane.comp.vid ... x.scm/3436Gaucho316 hat geschrieben:Ist bei diesem Patch jetzt nicht zuviel rausgeflogen? Jetzt wird doch nichts mehr gespeichert, weil write(fd, text, text_size) auskommentiert wurde.GetAway hat geschrieben:movieinfo.cpp
remove_currently_unused_variables.diff
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
Kein Problem, ist mir auch durchgerutschtGetAway hat geschrieben:Aua, dank dir!rhabarber1848 hat geschrieben:So müsste es richtig sein: http://article.gmane.org/gmane.comp.vid ... x.scm/3436Gaucho316 hat geschrieben:Ist bei diesem Patch jetzt nicht zuviel rausgeflogen? Jetzt wird doch nichts mehr gespeichert, weil write(fd, text, text_size) auskommentiert wurde.
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Ich habe die Postings über die Überarbeitung der Klasse COnOffNotifier ausgelagert, da wohl noch Diskussionsbedarf besteht.Gaucho316 hat geschrieben: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.
http://forum.tuxbox.org/forum/viewtopic.php?f=2&t=50110
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
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
Neutrino: use constants instead of numbers to compare video format
Link entfernt, da Patch im CVS
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
Neutrino: pause zapit after getting last channel in Audioplayer, Esound, Movieplayer and UPnP BrowserGaucho316 hat geschrieben: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.Mourice hat geschrieben:[zapit] cmd 13 refused in standby mode
Link entfernt, da Patch im CVS
Re: Commit-Wünsche für das CVS II
Hi,
Fehlermeldung ist jetzt weg.
Aber ob sich dadurch etwas verbessert hat ... subjektiv keine Änderungen bemerkbar.
MfG
Fehlermeldung ist jetzt weg.
Aber ob sich dadurch etwas verbessert hat ... subjektiv keine Änderungen bemerkbar.
MfG
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
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.
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
committed:
http://article.gmane.org/gmane.comp.vid ... x.scm/3437Gaucho316 hat geschrieben:neutrino_use-constants-instead-of-numbers-to-compare-video-format_2012-06-13_1538.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3438Gaucho316 hat geschrieben:neutrino_pause-zapit-after-getting-last-channel_2012-06-13_1822.diff
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
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.
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.
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
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
Zapit Bouquetmanager: add destructor
Link entfernt, da Patch im CVS
Zapit Bouquetmanager: fix invalid iterator advance
Link entfernt, da Patch im CVS
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
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
Zapit: use enums instead of numbers for service type
Link entfernt, da Patch im CVS
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
committed:
http://article.gmane.org/gmane.comp.vid ... x.scm/3483
http://article.gmane.org/gmane.comp.vid ... x.scm/3476Gaucho316 hat geschrieben:audio-setup_fix-segfault-on-td_2012-06-20_1925.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3477Gaucho316 hat geschrieben:miscsettings-menu_fix-memleak-on-td_2012-06-20_1820.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3478Gaucho316 hat geschrieben:record-setup_rename-notifier-to-clarify-its-purpose_2012-06-20_1822.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3479Gaucho316 hat geschrieben:themes_create-color-setup-notifier-in-constructor_2012-06-20_1928.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3480Gaucho316 hat geschrieben:video-setup_remove-unnecessary-setting-of-video-format_2012-06-20_1827.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3481Gaucho316 hat geschrieben:zapit-setup_use-just-one-notifier_2012-06-20_1828.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3482Gaucho316 hat geschrieben:zapit_bouquetmanager_add-destructor_2012-06-25_1821.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3483
http://article.gmane.org/gmane.comp.vid ... x.scm/3484Gaucho316 hat geschrieben:zapit_bouquetmanager_fix-invalid-iterator-advance_2012-06-25_1824.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3485Gaucho316 hat geschrieben:zapit_use-enums-instead-of-numbers-for-service-type_2012-06-26_1025.diff
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
63 Downloads ohne negative Rückmeldungen,Gaucho316 hat geschrieben: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
funktioniert seit Wochen problemlos mit positivem Erfolg,
committed: http://article.gmane.org/gmane.comp.vid ... x.scm/3487
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
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
Menu widget: update selected item on pressing direct key
Link entfernt, da Patch im CVS
-
- Contributor
- Beiträge: 1509
- Registriert: Donnerstag 27. Dezember 2007, 12:59
Re: Commit-Wünsche für das CVS II
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
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
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
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
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
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
committed:
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
http://article.gmane.org/gmane.comp.vid ... x.scm/3512
http://article.gmane.org/gmane.comp.vid ... x.scm/3513
http://article.gmane.org/gmane.comp.vid ... x.scm/3496Gaucho316 hat geschrieben:menue_update-selected-item-on-pressing-direct-key_2012-06-26_2230.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3497GetAway hat geschrieben:sectionsd_fix_segmentation_fault_2012_06_27_0904.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3498GetAway hat geschrieben:use_empty_instead_of_size_2012_06_27_1522.diff[/url]
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
http://article.gmane.org/gmane.comp.vid ... x.scm/3509Gaucho316 hat geschrieben:bookmarkmanager_fix-value-of-variable-res_2012-06-27_1757.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3510Gaucho316 hat geschrieben:network-setup_use-enums-and-remove-unnecessary-vars_2012-06-27_1802.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3511Gaucho316 hat geschrieben:neutrino_remove-unnecessary-repaint-calls_2012-06-27_1811.diff
http://article.gmane.org/gmane.comp.vid ... x.scm/3512
http://article.gmane.org/gmane.comp.vid ... x.scm/3513
-
- CDK-Experte
- Beiträge: 4335
- Registriert: Donnerstag 3. April 2008, 14:05
Re: Commit-Wünsche für das CVS II
Nach dem Urlaub ist vor dem UrlaubGaucho316 hat geschrieben: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.rhabarber1848 hat geschrieben:Wie wäre es mit CVS commit-Rechten für Gaucho316?
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
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
Usermenu: fix memleaks
Link entfernt, da Patch im CVS
Alle Jahre wieder ...rhabarber1848 hat geschrieben:Nach dem Urlaub ist vor dem UrlaubGaucho316 hat geschrieben: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.rhabarber1848 hat geschrieben:Wie wäre es mit CVS commit-Rechten für Gaucho316?
-
- Contributor
- Beiträge: 1509
- Registriert: Donnerstag 27. Dezember 2007, 12:59
Re: Commit-Wünsche für das CVS II
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.
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.
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
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?GetAway hat geschrieben:Desweiteren würde zur Logik des Programmverlaufs passen, den Startkanal da mit zu rein zunehmen.
update channel history localesGetAway hat geschrieben:Zapping-History Bouquet wurde in Programmverlauf umbenannt, dies sollte
man unter Einstellungen->Tasten nachholen.
Link entfernt, da Patch im CVS
-
- Contributor
- Beiträge: 1833
- Registriert: Mittwoch 10. April 2002, 15:39
Re: Commit-Wünsche für das CVS II
Download Bereiche für DBox2, TD und Spark Distributionen
http://dietmarw.polsum.net
http://dietmarw.trale.de (r.i.p.)
http://dietmarw.polsum.net
http://dietmarw.trale.de (r.i.p.)
-
- Contributor
- Beiträge: 1688
- Registriert: Donnerstag 17. Februar 2005, 20:24
Re: Commit-Wünsche für das CVS II
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
Neutrino: no need to inherit CBEBouquetWidget, CBEChannelWidget, CListBox and CDirMenu from CMenuWidget
Link entfernt, da Patch im CVS