Commit-Wünsche für das CVS II

[Gaucho316]

Jetzt habe ich doch glatt selbst einen ziemlich offensichtlichen Button gefunden, bei dem der Text mit Standardschriftgrößenfaktor abgeschnitten wird. Es handelt sich um den Button "Weitere Termine" im EPG-Infofenster. Ich habe dem mal mehr Platz gegönnt, da ja zwischen ihm und den beiden Icons 16:9 und DD noch ungenutzter Platz ist.

Neutrino EPG Viewer: increase max width of more screenings button
Link entfernt, da Patch in Git


Und dann habe ich noch 'nen frischen Patch aus Neutrino-HD portiert.

CTextBox::scrollPageUp/Down: Suppress refresh when there is nothing to scroll
Link entfernt, da Patch in Git

[GetAway]

[PATCH] filebrowser: simplify code using font pointers and ...
... make size, time and mode fields scale with fontsize
Ported from Neutrino-mp GIT
0001-filebrowser-simplify-code-using-font-pointers-and.patch

überflüssiges Leerzeichen in deutsch.locale entfernt
0002-remove-unnecessary-blank.patch

[Gaucho316]

Und noch ein portierter Patch:

Neutrino infoviewer: make oldinfo a member of the class
Link entfernt, da Patch in Git

[PATCH] filebrowser: simplify code using font pointers and ...
... make size, time and mode fields scale with fontsize
Ported from Neutrino-mp GIT
0001-filebrowser-simplify-code-using-font-pointers-and.patch

Ehre, wem Ehre gebührt! Du solltest als Autor im Patch nicht dich sondern seife angeben. Du selbst kannst dich dann im "Signed-off-by" verewigen, wenn ich das Prinzip richtig verstanden habe. Und der, der das dann eincheckt, macht halt noch ein "Signed-off-by" dran.

[GetAway]

Ich habe es dann mal geändert.

Wie gibt man denn in der Kommandozeile an, dass der Autor ein anderer ist?
Das ist doch mit git config --global user.name und .email festgelegt.

[Gaucho316]

Das macht man mit dem Parameter:

Code: Alles auswählen

--author="Name <name@domain>"

[dbt]

Das macht man mit dem Parameter:

Code: Alles auswählen

--author="Name <name@domain>"

Jo, ist auch hier kurz mit Beispiel: Development:Git_Patch_erstellen#git_apply

[GetAway]

Hier auch nochmal ein weiterer portierter Patch. http://git.c00lstreamtech.de/?p=cst-pub ... 9350929787

* yWeb: Update timer sync for tvinfo.de

0001-yWeb-Update-timer-sync-for-tvinfo.de.patch

[Gaucho316]

http://forum.tuxbox.org/forum/viewtopic ... 91#p385591 (1 Patch)
http://forum.tuxbox.org/forum/viewtopic ... 99#p385599 (3 Patches)
http://forum.tuxbox.org/forum/viewtopic ... 04#p385604 (3 Patches)

Und die 4 noch offenen Patches aus diesem Thread bitte nicht vergessen.

= 11 Patches

[Gaucho316]

Nun sind's 13 Patches.

Neutrino bookmark manager: remove duplicate code and use h_max()
Link entfernt, da Patch in Git

Neutrino timer list: use h_max() for height calculation
Link entfernt, da Patch in Git

[GetAway]

Patch 14
0014-configfile-do-a-partial-sync-after-saving.patch

BUG: Beim Testen des Patches ist mir aufgefallen, dass immer wieder in den Flash geschrieben wird,
wenn man die Einstellungen speichert, ohne das Änderungen vorgenommen wurden. Das sollte sich mal
jemand anschauen.

Code: Alles auswählen

[ConfigFile] saving /var/tuxbox/config/controld.conf

Patch 15
0015-libconfigfile-update-config-file-atomically-when-sav.patch

[Gaucho316]

Durch die letzten Änderungen im Fontrenderer werden Zeilenumbrüche am Ende der Einträge in den Sprachdateien durch ein Absatzzeichen ersetzt. Wenn ich das richtig sehe, hat seife das eingebaut, um genau solche Einträge zu finden. Aufgefallen ist mir das in der URL in der Hilfe vom Movieplayer. Ich habe deshalb das \n am Ende der Zeilen in den Sprachdateien entfernt. Ich hoffe, dass das umproblematisch ist.

Neutrino locale: remove newline from the end of lines
Link entfernt, da Patch in Git

Edit: Ich habe den Patch erst einmal wieder entfernt. Ich muss mir das nochmal genauer ansehen. Ich glaube, dass es reicht, den überflüssigen Zeilenumbruch nur in der Movieplayer-Hilfe zu entfernen. An allen anderen Stellen wird das von den entsprechenden Klassen erledigt.

Edit: Und da ist der Patch wieder. Die Zeilenumbrüche müssen tatsächlich nur in der Movieplayer-Hilfe entfernt werden.


Außerdem habe ich alle Stellen rausgesucht, an denen man w_max(), h_max(), getScreenStartX() und getScreenStartY() nutzen kann, um den Code zu vereinfachen. Auch hier hoffe ich, dass ich nichts kaputt gemacht habe. Mir ist erst einmal nichts negatives aufgefallen.

Neutrino: use {w,h}_max() and getScreenStart{X,Y}() if possible
Link entfernt, da Patch in Git


Somit sind wir nun bei 17 Patches.

[GetAway]

Patch 14
BUG: Beim Testen des Patches ist mir aufgefallen, dass immer wieder in den Flash geschrieben wird,
wenn man die Einstellungen speichert, ohne das Änderungen vorgenommen wurden. Das sollte sich mal
jemand anschauen.

Fix dazu

Patch 18
0018-zapit-check-ModifiedFlag-before-save-controld.conf.patch

Patch 19
0019-neutrino-port-save-bigFonts-code-in-epgview.patch

[dbt]

Edit: Erlerdigt http://article.gmane.org/gmane.comp.vid ... ox13.scm/4

[GetAway]

Danke Thilo fürs einchecken

Patch 0001: Mir gefällt die Zentrierung der Kanalliste nicht so ganz.
Beim jetzigen Code liegt die DetailsItemsLine genau am linken Bildschirmrand, rechts sind ~12pixel Luft.
Der neue Code berechnet zur Fensterbreite der Kanalliste die DetailsItemLine dazu, so dass das ganze ein
wenig nach rechts verschoben wird und die DetailsItemLine nicht am äusseren Bildschirmrand anliegt.

Meinungen?
0001-neutrino-calculate-centering-of-channellist-including-itemline.patch

[Gaucho316]

Im Movieinfo-Fenster kann man g_settings.bigFonts auch wunderbar nutzen.

Neutrino: use big font setting also in movieinfo window
Link entfernt, da Patch in Git

Patch 0001: Mir gefällt die Zentrierung der Kanalliste nicht so ganz.
Beim jetzigen Code liegt die DetailsItemsLine genau am linken Bildschirmrand, rechts sind ~12pixel Luft.
Der neue Code berechnet zur Fensterbreite der Kanalliste die DetailsItemLine dazu, so dass das ganze ein
wenig nach rechts verschoben wird und die DetailsItemLine nicht am äusseren Bildschirmrand anliegt.

Ich habe nichts dagegen. Sieht wirklich besser aus. Meinetwegen kann das eingecheckt werden.

Edit: Man kann das übrigens auch ein ganz klein wenig einfacher schreiben:

Code: Alles auswählen

x = getScreenStartX(full_width + ConnectLineBox_Width);

[GetAway]

Edit: Man kann das übrigens auch ein ganz klein wenig einfacher schreiben:

Code: Alles auswählen

x = getScreenStartX(full_width + ConnectLineBox_Width);

Funktioniert bei mir nicht.

Patch 0003: Bitte mal testen.
0003-neutrino-epgview-osd-fix-dates-tab-for-screening.patch

[Gaucho316]

Geistige Umnachtung meinerseits. Man muss die Werte natürlich subtrahieren und nicht addieren. Ich hatte es nicht probiert, sondern nur darüber nachgedacht. War halt schon spät gestern.

Code: Alles auswählen

x = getScreenStartX(full_width - ConnectLineBox_Width);

Patch 0003: Bitte mal testen.
0003-neutrino-epgview-osd-fix-dates-tab-for-screening.patch

Ein paar Kleinigkeiten würde ich gerne ändern. Man muss ja nicht jeden Pups eins zu eins aus Neutrino-HD übernehmen.

[GetAway]

Gerne.

Patch 0004
0004-neutrino-show-epginfo-from-followlist-also.patch

[Gaucho316]

Patch 0001: Mir gefällt die Zentrierung der Kanalliste nicht so ganz.
0001-neutrino-calculate-centering-of-channellist-including-itemline.patch

Ich habe das mal so geändert, wie von mir vorgeschlagen:

Code: Alles auswählen

x = getScreenStartX(full_width - ConnectLineBox_Width);

neutrino: calculate centering of channellist including itemline
Link entfernt, da Patch in Git

Patch 0003: Bitte mal testen.
0003-neutrino-epgview-osd-fix-dates-tab-for-screening.patch

Ich habe die Änderungen mal ein klein wenig ausgedünnt und zwei Fehler korrigiert, die auch noch in Neutrino-HD stecken. Ich habe mich nämlich gefragt, warum "Mo." so nah an der folgenden Zahl klebt und warum der Abstand des Monats zur Uhrzeit so groß ist. Bei der Berechnung der maximalen Breiten wurde vergessen, dass der Punkt auch berücksichtigt werden muss und dass die Spracheinträge in UTF-8 kodiert sind.

neutrino: epgview: osd fix dates, tab for screening
Link entfernt, da Patch in Git

Patch 0004
0004-neutrino-show-epginfo-from-followlist-also.patch

Auch hier musste ich etwas korrigieren. Lässt man sich nämlich die EPG-Infos zu einem weiteren Termin anzeigen und kehrt dann zu den ursprünglichen EPG-Infos zurück, sieht man dort statt der ursprünglichen EPG-Infos die des zuvor angezeigten weiteren Termins. Am schnellsten sieht man das an der angezeigten Uhrzeit und dem Datum. In Neutrino-HD ist das korrekt und deshalb habe ich den zusätzlich nötigen Code auch übernommen. Allerdings sollte sich mal jemand die Aufrufe von show(...) innerhalb von epgview.cpp in Neutrino-HD ansehen. Ich denke, da fehlt an ein paar Stellen der Parameter call_fromfollowlist. In diesem Patch ist das für Neutrino berücksichtigt.

neutrino: show epginfo from followlist also
Link entfernt, da Patch in Git

[GetAway]

@Gaucho316

Ich habe deine Patche mal eben eingespielt. Die Formatierung der Termindaten sehen jetzt
besser aus. An UTF8 hatte ich in diesem Fall gar nicht mehr gedacht. Der Coolstream-GIT
Code ist auch nicht der schönste, und das der Sectionsd nicht immer UTF8 liefert macht
die Sache auch nicht einfacher. Schönen Dank.

@dbt

Das kann jetzt rein, wenn bei dir zeitlich Luft ist.

[dbt]

http://sourceforge.net/p/tuxbox-cvs/app ... log/?path=

[Gaucho316]

Ich habe doch noch eine Buttonleiste gefunden, in der die Beschriftungen abgeschnitten werden. Eine Umsortierung der Buttons und Kürzung einer Beschriftung hilft dagegen.

Neutrino EPG Plus: rearrange button bar
Link entfernt, da Patch in Git

Edit: Ich habe die Reihenfolge der ersten beiden Buttons vertauscht. Gefällt mir irgendwie besser.

[GetAway]

@Gauch316

Kompilierst du eigentlich noch mit gcc3? Mit gcc4 kommen viele Warnings.
Ich habe hier mal einen ersten Patch erstellt, der eine Teil davon fixt. Schau
mal bitte drauf.

0001-fix-gcc4-warnings-break-strict-aliasing-rules.patch

[Gaucho316]

Kompilierst du eigentlich noch mit gcc3? Mit gcc4 kommen viele Warnings.
Ich habe hier mal einen ersten Patch erstellt, der eine Teil davon fixt. Schau
mal bitte drauf.

0001-fix-gcc4-warnings-break-strict-aliasing-rules.patch

Ja, ich nutze noch den 3er. In deinem Patch fehlt leider eine nötige Änderung in movieplayer_setup.cpp, die in einem #ifndef ENABLE_MOVIEPLAYER2 steckt. Ich vermute mal, dass du den MP2 nutzt und es dir deshalb nicht aufgefallen ist. Ich habe den Teil mal ergänzt.

fix gcc4 warnings break-strict-aliasing rules
Link entfernt, da Patch in Git


Und dann habe ich noch einen Patch für den Big-Font-Code, der im Movieinfo-Fenster genutzt wird. Der sorgt dafür, dass nicht unnötig Font::setSize() aufgerufen wird, wenn man die Anzeige mit großen Fonts gar nicht aktiviert hat.

Neutrino: rework big font code in ShowMsg2UTF()
Link entfernt, da Patch in Git

[GetAway]

Kompilierst du eigentlich noch mit gcc3? Mit gcc4 kommen viele Warnings.
Ich habe hier mal einen ersten Patch erstellt, der eine Teil davon fixt. Schau
mal bitte drauf.

0001-fix-gcc4-warnings-break-strict-aliasing-rules.patch

Ja, ich nutze noch den 3er. In deinem Patch fehlt leider eine nötige Änderung in movieplayer_setup.cpp, die in einem #ifndef ENABLE_MOVIEPLAYER2 steckt. Ich vermute mal, dass du den MP2 nutzt und es dir deshalb nicht aufgefallen ist.

Genau.

Patch 0004
0004-audioplayer-don-t-free-curl_handle-too-early.patch