Commit-Wünsche für das CVS II

[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]

Patch 0008
0008-yweb-No-caching-allowed-for-timer-list.patch

[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]

Patch 0011
0011-yweb-file-not-needed-anymore.patch

Patch 0012
0012-yweb-fix-some-html-and-js-format-errors.patch

[Gaucho316]

Neutrino infoviewer: remove offset for green button
Link entfernt, da Patch in Git

[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]

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]

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]

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]

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.

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]

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]

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]

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]

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]

Vor Montag wird das bei mir nichts. Du kannst dich also daran austoben.

[Gaucho316]

Patch 0014
0014-menu.cpp-add-pulldown-list-functionality-for-CMenuOp.patch

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]

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]

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.

[Gaucho316]

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

Ahh, kommt mir irgendwie bekannt vor.

Stammt der ursprünglich von dir? Das war mir nicht mehr bewusst.

[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]

@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]

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]

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]

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]

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.