Commit-Wünsche für das CVS II

Kreuzuebersetzer, Diskussion über Änderungen im Tuxbox-CDK und Tuxbox-CVS
seife
Developer
Beiträge: 4189
Registriert: Sonntag 2. November 2003, 12:36

Re: Commit-Wünsche für das CVS II

Beitrag von seife »

...und einigermassen moderne compiler selbst die temporäre Kopie im post-inkrement wegoptimieren, wenn der returnwert nicht benutzt wird. Was innerhalb der Schleifen meist der Fall ist. Deswegen "humbug".

Allerdings: bei der dbox wird ja kein moderner Compiler benutzt, insofern kann es schon sinnvoll sein :-) Und es überall einheitlich zu machen ist sicher auch ein Vorteil. Insofern ist "humbug" wohl ein etwas zu starker Ausdruck.

Nur weil etwas in neutrino-HD drin ist, bezeugt noch lange nicht dessen Qualität :-)
Z.B. wurden da mal blind alle memcpy() in memmove() geändert, was auf Maschinen wie der dbox vermutlich die performance spürbar beeinflussen dürfte.
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: Commit-Wünsche für das CVS II

Beitrag von GetAway »

Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: Commit-Wünsche für das CVS II

Beitrag von Gaucho316 »

Neutrino pictureviewer: free formathandlers in destructor
Link entfernt, da Patch in Git

Neutrino pictureviewer: fix resource leaks
Link entfernt, da Patch in Git
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: Commit-Wünsche für das CVS II

Beitrag von GetAway »

Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: Commit-Wünsche für das CVS II

Beitrag von Gaucho316 »

Neutrino infoviewer: remove offset for green button
Link entfernt, da Patch in Git
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: Commit-Wünsche für das CVS II

Beitrag von GetAway »

Patch 0014
Patch entfernt. Bitte weiter unten von Gaucho316 neu runterladen

Patch 0015
0015-scan-setup-use-pulldown-list-to-select-providers-sat.patch

Patch 0016
0016-settings.cpp-hardcoded-sat-provider-not-longer-needed.patch
Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: Commit-Wünsche für das CVS II

Beitrag von Gaucho316 »

GetAway hat geschrieben:Patch 0014
0014-menu.cpp-add-pulldown-list-functionality-for-CMenuOp.patch
Einspruch, der macht etwas kaputt, was ich gerade erst korrigiert hatte. Ich überarbeite den Patch die Tage mal ein wenig.
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: Commit-Wünsche für das CVS II

Beitrag von GetAway »

Jetzt sag auch was nicht mehr geht ;)
Ich habe bei meinen Tests nichts gefunden, das nicht mehr geht. Aber...
anderer User, anderes Verhalten. :)

Da könnte man noch einiges schöner machen. Mir ging es aber erstmal nur um die Funktion.
Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: Commit-Wünsche für das CVS II

Beitrag von Gaucho316 »

Die Höhenberechnung geht dann wieder nicht mehr. (siehe d111c4e6c462f3032d8137fa42bdb37c44572fe6)

Und außerdem sind da viele Änderungen drin, die mit der eigentlichen "pulldown list functionality" überhaupt nichts zu tun haben. Diese Änderungen sollte man in einen eigenen Patch auslagern. Die neuen Methoden
  • paintItemBackground()
  • prepareItem()
  • paintItemCaption()
können nämlich auch in allen anderen Klassen verwendet werden.
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: Commit-Wünsche für das CVS II

Beitrag von GetAway »

Gaucho316 hat geschrieben:Die Höhenberechnung geht dann wieder nicht mehr. (siehe d111c4e6c462f3032d8137fa42bdb37c44572fe6)
Die mögliche Gesamthöhe von 100% gefällt mir auch nicht. Manchmal meine ich es
könnten auch mehr als 100% sein, da oben und unten schon mal etwas rausfällt,
obwohl der Bildbereich stimmt.

Gaucho316 hat geschrieben: Und außerdem sind da viele Änderungen drin, die mit der eigentlichen "pulldown list functionality" überhaupt nichts zu tun haben.
So bin ich halt. Werde mich bessern.
Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: Commit-Wünsche für das CVS II

Beitrag von Gaucho316 »

Die eigentliche Änderung "pulldown list functionality" versteckt sich im Patch in den Zeilen 170 bis 212 (+ ein paar wenigen Änderungen in anderen Zeilen), wenn ich das richtig sehe.
Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: Commit-Wünsche für das CVS II

Beitrag von Gaucho316 »

GetAway hat geschrieben:Patch 0015
0015-scan-setup-use-pulldown-list-to-select-providers-sat.patch
Ist die "pulldown list functionality" überhaupt nötig? Man kann doch dasselbe mit einer Kombination aus MenuForwarder und MenuWidget erreichen, oder?
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: Commit-Wünsche für das CVS II

Beitrag von GetAway »

Das habe ich nicht ausprobiert, ich denke aber das dies die Lesbarkeit des Codes nicht gerade
erhöht, wenn es denn funktionieren sollte. Ein Beispiel im Code habe ich auch nicht gefunden.
Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: Commit-Wünsche für das CVS II

Beitrag von Gaucho316 »

Hast Recht, ich habe nochmal genau drüber nachgedacht. Das wäre sehr umständlich.

Es wäre aber schön, wenn du erst einmal nur den "pulldown list functionality"-Teil ohne die zusätzlichen Änderungen als Patch umsetzt. Den Code kann man übrigens auch im CMenuOptionChooser einbauen (wie in Neutrino-HD). Wenn du dazu keine Lust hast, kann ich das auch machen. Den Rest können wir dann auch später noch einbauen und dann direkt an allen Stellen in menue.h/.cpp nutzen, an denen es möglich ist.
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: Commit-Wünsche für das CVS II

Beitrag von GetAway »

Vor Montag wird das bei mir nichts. Du kannst dich also daran austoben.
Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: Commit-Wünsche für das CVS II

Beitrag von Gaucho316 »

GetAway hat geschrieben:Patch 0014
0014-menu.cpp-add-pulldown-list-functionality-for-CMenuOp.patch
Gaucho316 hat geschrieben:Es wäre aber schön, wenn du erst einmal nur den "pulldown list functionality"-Teil ohne die zusätzlichen Änderungen als Patch umsetzt. Den Code kann man übrigens auch im CMenuOptionChooser einbauen (wie in Neutrino-HD). Wenn du dazu keine Lust hast, kann ich das auch machen. Den Rest können wir dann auch später noch einbauen und dann direkt an allen Stellen in menue.h/.cpp nutzen, an denen es möglich ist.
Hier nun der versprochene Patch, der den genannten ersetzt. Der Rest der erst einmal rausgenommenen Änderungen kommt dann demnächst von mir als weiterer Patch.

Neutrino: add pulldown list functionality to CMenuOption*Chooser
Link entfernt, da Patch in Git

Neutrino moviebrowser: use pulldown list to change genre
Link entfernt, da Patch in Git
Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: Commit-Wünsche für das CVS II

Beitrag von Gaucho316 »

Neutrino moviebrowser: reduce console spam
Link entfernt, da Patch in Giturl]


Ich finde, dass die Pulldown-Liste ohne Icon im Header besser aussieht, da das Einstellungen-Icon nicht immer passt. Dazu musste ich zuvor einen Absturz beseitigen, der auftritt, wenn man ein Menü ohne Icon definiert. Außerdem habe ich die Breite der Pulldown-Liste der Breite des dazugehörigen Menüeintrags angepasst.

Neutrino CMenuWidget: fix segfault if head iconname is empty
Link entfernt, da Patch in Git

Neutrino CMenuOption*Chooser: remove icon from pulldown list and use width of corresponding menu option chooser
Link entfernt, da Patch in Git


Und hier noch der von mir erweiterte Rest von GetAways Patch, der redundanten Code in den Klassen der Menüelemente verhindert.

Neutrino: move redundant code from menu item classes into base class
Link entfernt, da Patch in Git
dbt
Administrator
Beiträge: 2675
Registriert: Donnerstag 28. September 2006, 18:18

Re: Commit-Wünsche für das CVS II

Beitrag von dbt »

Gaucho316 hat geschrieben:

Und hier noch der von mir erweiterte Rest von GetAways Patch, der redundanten Code in den Klassen der Menüelemente verhindert.

Neutrino: move redundant code from menu item classes into base class
0021-Neutrino-move-redundant-code-from-menu-item-classes.patch
Ahh, kommt mir irgendwie bekannt vor. :wink:
Gaucho316
Contributor
Beiträge: 1688
Registriert: Donnerstag 17. Februar 2005, 20:24

Re: Commit-Wünsche für das CVS II

Beitrag von Gaucho316 »

Neutrino CMenuWidget: consider scrollbar width when calculating start pos
Link entfernt, da Patch in Git

dbt hat geschrieben:Ahh, kommt mir irgendwie bekannt vor. :wink:
Stammt der ursprünglich von dir? Das war mir nicht mehr bewusst.
dbt
Administrator
Beiträge: 2675
Registriert: Donnerstag 28. September 2006, 18:18

Re: Commit-Wünsche für das CVS II

Beitrag von dbt »

Ich hatte das mal so ähnlich bei Neutrino-HD gemacht und bin mir fast sicher, dass da noch einiges geht. Ich hatte z.B. damals noch nicht wirklich genau drauf geschaut, ob da nicht einige vererbte Members mehrfach unnötig in den Subklassen definiert waren. Das würde unter Umständen zu seltsamen Verhalten bei einigen Aktionen führen, aber da ist soweit ich weis nichts auffällig gewesen. Aber da waren schon heftige Redundanzen drin, insofern ist das schon gut, dass das den Weg hierher gefunden hat.
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: Commit-Wünsche für das CVS II

Beitrag von GetAway »

@Gaucho316

Danke fürs aufbröseln des großen Patches.

@dbt

Da ich beim portieren zwischendurch mal 2-3 Sachen zusammenwerfe, geht der
eigentliche Autor schon mal unter. :) Es sind aber auch Sachen dabei, z.B. die
Pulldownliste, da läßt sich kein Autor ermitteln, weil die Sachen schon im initial-Git
von Neutrino-HD drin waren.

Patch 0023
0023-settings-remove-useDefaults-init-all-variables-to-default.patch
dbt
Administrator
Beiträge: 2675
Registriert: Donnerstag 28. September 2006, 18:18

Re: Commit-Wünsche für das CVS II

Beitrag von dbt »

Es ist auch nicht wirklich so mal schnell was gepickt. Da finde ich Eure Gedult schon bemerkenswert. Da müsste man sich den Neutrino Subtree via git rausfiltern, um eine passendes Gegenstück zu haben, damit die Baumstruktur wenigstens halbwegs zusammen passt. Und selbst dann ist das meist wohl auch nicht ohne Konflikte möglich.
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: Commit-Wünsche für das CVS II

Beitrag von GetAway »

dbt hat geschrieben:Da müsste man sich den Neutrino Subtree via git rausfiltern, um eine passendes Gegenstück zu haben, damit die Baumstruktur wenigstens halbwegs zusammen passt. Und selbst dann ist das meist wohl auch nicht ohne Konflikte möglich.
Vielleicht meldet sich rhabarber1848 irgendwann mal wieder, weil ich kanns nicht. :-?
rhabarber1848
CDK-Experte
Beiträge: 4335
Registriert: Donnerstag 3. April 2008, 13:05

Re: Commit-Wünsche für das CVS II

Beitrag von rhabarber1848 »

GetAway hat geschrieben:Vielleicht meldet sich rhabarber1848 irgendwann mal wieder, weil ich kanns nicht. :-?
Ich bin noch da ;) und lese regelmäßig mit, nur leider habe ich aufgrund
anderer privater IT-Projekte neben meinem real life derzeit keine Zeit
mehr für Tuxbox. Das wird sich wieder ändern, aber sicher nicht bis
Jahresmitte 2013.
GetAway
Contributor
Beiträge: 1509
Registriert: Donnerstag 27. Dezember 2007, 12:59

Re: Commit-Wünsche für das CVS II

Beitrag von GetAway »

Schön zu hören. Bis bald, u-boot wartet schon. Ich hatte mal versucht damit anzufangen,
ist mir aber zu kompliziert. Wahrscheinlich sind schon wieder zuviele Änderungen drin, das
erhöht den Aufwand immens.

Zurück zu „Tuxbox-SD Buildsystem“