Commit-Wünsche für das CVS II

[Gaucho316]

Neutrino EPG Viewer: reset scroll position after rec dir selection
Link entfernt, da Patch in Git

[GetAway]

Patch 0006
0006-Y-Web-Y_VLC.js-fix-vlc-version-parse.patch

[Gaucho316]

Wenn man Bei Sofortaufn. Verzeichnisauswahl (recording_choose_direct_rec_dir) aktiviert hat, werden nach der Verzeichnisauswahl in der EPG-Liste der Header und nach dem Abbruch der Verzeichnisauswahl zusätzlich die Buttonleiste nicht gezeichnet.

Neutrino event list: fix paint after choosing recording directory
Link entfernt, da Patch in Git


Und außerdem passt die Bezeichnung Bei Sofortaufn. Verzeichnisauswahl überhaupt nicht, da diese Einstellung auch in der EPG-Liste, im EPG-Infofenster und im EPG Plus beachtet wird. Ich hab's mal in Verzeichnisauswahl anzeigen umbenannt.

Neutrino: change locale string recordingmenu.choose_direct_rec_dir
Link entfernt, da Patch in Git


Vielleicht hat ja jemand Lust, die folgenden 2 Patches für Tuxwetter zu portieren. Ich selbst nutze das Plugin nicht.
http://git.coolstreamtech.de/?p=cst-pub ... 681fc5958e
http://git.coolstreamtech.de/?p=cst-pub ... 46a9ead911

Patch 0006
0006-Y-Web-Y_VLC.js-fix-vlc-version-parse.patch

Y_Version.txt

Code: Alles auswählen

date=04.09.2013

[Gaucho316]

Neutrino EPG Plus: remove duplicate code
Link entfernt, da Patch in Git

Neutrino EPG Viewer: add missing hide() if no epg info found
Link entfernt, da Patch in Git

[Gaucho316]

Hier noch ein portierter Patch für die neue Kanalliste:

Neutrino channel list: fix paint details with different font size
Link zum Patch entfernt

Edit: Irgendwie hilft das nicht so, wie ich mir das vorgestellt hatte. Ich habe eine andere Lösung im Kopf. Ich probiere heute Abend aus, ob das so funktioniert, wie ich mir das denke.

[Gaucho316]

Dieser Fehler ist mir aufgefallen, als ich den unteren Patch erstellt habe.

Neutrino OSD setup: fix count of entries in font sizes groups
Link entfernt, da Patch in Git


Hier nun mein Vorschlag, der dafür sorgt, dass die Texte in der Kanalliste nicht mehr aus ihren Bereichen rauslaufen, wenn die Schriftgrößen für "Kanalliste" (FONT_TYPE_CHANNELLIST) und "Beschreibung" (FONT_TYPE_CHANNELLIST_DESCR) im Schriftuntermenü "Kanalliste" unterschiedlich eingestellt sind. Da sich aber die tatsächliche Höhe der beiden Schriften unterscheidet und die Schrift für den Beschreibungstext ein klein wenig höher ist, passt nun leider bei Standardeinstellungen eine Zeile weniger in die Liste der Kanäle. Stellt man für Beschreibung den Wert 19 ein, ist die Anzahl der Zeilen wie vorher.

Neutrino channel list: fix height calculation if font sizes differ
Link entfernt, da Patch in Git

[GetAway]

@Gaucho316

Übernehme die Fontgröße 19 doch gleich als Defaultwert.

Patch 0013:
0013-fontrenderer-init-colors-and-make-it-static.patch

[GetAway]

Patch 0014a: add 2 new udp-icons ported from yweb 2.8.0.x

0014a-yWeb-LiveTV-add-two-new-icons-udp-on-off.patch

Patch 0014b: use deamonized 'udpstreamts'
Thanks arno-neutrinoTV for reporting.
Thread: http://www.tuxbox-cvs.sourceforge.net/f ... dpstreamts
fix switching streamts <-> udpstreamts

0014b-yWeb-improve-LiveTV-udpstreaming.patch

Patch updated. Getestet mit gcc-3.4.6 / gcc-4.1.2
version bump boost 1.53.0
version bump binutils 2.23.52.0.1
0001-version-bump-boost-1.53.0-and-binutils-2.23.52.0.1.patch

[Gaucho316]

Wählt man einen kleineren horizontalen Schriftgrößenfaktor, sieht es so aus, als würden die Satfind-Signalbalken in der Infobar zu dem Wert rechts daneben gehören, obwohl sie eigentlich zu dem Wert links daneben gehören. Ich habe das mal korrigiert.

Neutrino infoviewer: fix paint of satfind progressbars
Link entfernt, da Patch in Git

Übernehme die Fontgröße 19 doch gleich als Defaultwert.

Nö. Ich denke, dass das jeder bei Bedarf umstellen sollte. Ich hab's jetzt auf 18 stehen. Ich wundere mich sowieso, dass der normale Text höher als der fette Text ist. Da stimmt wohl etwas im Font nicht.

[GetAway]

Patch 0016
0016-auto-enumerate-FONT_TYPE_COUNT-instead-of-defining-it.patch

[Gaucho316]

Je kleiner der horizontale Schriftgrößenfaktor ist, desto größer wird der Abstand zwischen der Zeitspalte und den restlichen Infos in der Timerliste. Ich habe das mal korrigiert.

Neutrino timer list: calculate width of time column
Link zum Patch entfernt

Edit: Ich veröffentliche heute Abend noch eine neue Version dieses Patches, die verhindert, dass time_width unnötigerweise andauernd neu berechnet wird.


sectionsd: fix memleak with unused events
Link zum Patch entfernt

Edit: Ich sehe gerade, dass das für Neutrino unnötig ist, weil Boost sich darum kümmert. Mir war bis eben nicht bewusst, dass Boost in Neutrino-MP nicht mehr benutzt wird.

[seife]

Boost wird benutzt, und ich kann mir (ohne den Code gelesen zu haben) nicht vorstellen wie es sich darum kümmern soll, wenn es das objekt niemals "behandelt" hat.

[Gaucho316]

Hier nun die verbesserte Version des Patches für die Timerliste zusammen mit zwei anderen.

Neutrino timer list: calculate width of time column
Link entfernt, da Patch in Git

Neutrino infoviewer: move global vars into class
Link entfernt, da Patch in Git

sectionsd: remove unnecessary SIevent creation
Link entfernt, da Patch in Git

[Gaucho316]

Neutrino infoviewer: satfind values are not UTF-8 encoded
Link entfernt, da Patch in Git

[Gaucho316]

Neutrino file browser: fix typo in SMS key input
Link entfernt, da Patch in Git

Neutrino: fix getting width of UTF-8 strings in some classes
Link entfernt, da Patch in Git

Neutrino EPG Viewer: properly encode extended information
Link entfernt, da Patch in Git

[Gaucho316]

Neutrino infoviewer: remove newline from the end of infotext
Link entfernt, da Patch in Git

[GetAway]

Es wäre nicht schlecht, die angesammelten Patche mal zu committen.
Das freut bestimmt auch diejenigen die nicht patchen können.

Vielen Dank.

[dbt]

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

[GetAway]

@dbt
Mal wieder vielen Dank.

Patch 0001
0001-xmlparse-Possible-Memory-leak-tag.patch

Patch 0002
0002-xmlparse.cpp-fix-workaround-for-crash-while-parsing-.patch

Patch 0003
0003-xmlinterface.cpp-ignore-characters-less-than-0x20-wh.patch

Patch 0004
0004-audioplayer.cpp-add-urls-from-m3u.patch

[Gaucho316]

Auch von mir ein großes Dankeschön an dbt fürs Einchecken.


Ich habe bei mir ja nun die Schriftgröße für die Beschreibungen in der Kanalliste auf 18 gestellt. Der Text geht dann oftmals bis direkt an den Scrollbalken ran. Das sieht irgendwie blöd aus. Ich habe den Anfang des Textes deshalb ein wenig nach links verschoben. Das sieht besser aus, auch bei anderen Schriftgrößen. Außerdem ist nun der Abstand zum Trennpunkt zwischen dem Kanalnamen und der Beschreibung zu beiden Seiten gleich groß. Das war vorher nicht der Fall.

Neutrino channel list: fix start pos of left aligned event description
Link entfernt, da Patch in Git

[GetAway]

Patch 0006
Patch entfernt. Wird nochmal überarbeitet.

[seife]

Patch 0006
0006-performance-Prefer-prefix-operators-for-non-primitiv.patch

zwei drittel davon ist humbug: praktisch alle die nicht direkt im for(...)-Schleifenkopf drin stehen zum Beispiel.

[GetAway]

OK, werde ich nochmal überarbeiten.

Patch 0006 überarbeitet. Sind noch ein paar dazugekommen.
Was da jetzt irgendwo Humbug sein soll, muss man mir mal erklären. Die Operanden befinden sich alle
im Schleifenkopf bzw zusätzlich in der Schleife selbst. Ich bin ja kein Profi und habe dafür ein Prüftool
genommen. CppCheck sollte ja bekannt sein. Klar arbeitet das auch nicht ganz fehlerfrei, aber logisch
erscheinen mir alle. In Neutrino-HD kann man 95% davon gegenchecken und kaputt macht es auch nichts.
http://git.coolstreamtech.de/?p=cst-pub ... b37514ae35#

0006-performance-Prefer-prefix-operators-for-non-primitiv.patch

[GetAway]

Patch 0007
0007-use-var-passed-by-reference.patch

[doc]

OK, werde ich nochmal überarbeiten.

Patch 0006 überarbeitet. Sind noch ein paar dazugekommen.
Was da jetzt irgendwo Humbug sein soll, muss man mir mal erklären. Die Operanden befinden sich alle
im Schleifenkopf bzw zusätzlich in der Schleife selbst.

Da in der Regel wohl immer Integer als Werte benutzt werden dürfte sich der Performancegewinn als sehr gering erweisen, an der Stelle ist es daher doch relativ egal ob man Postkrement oder Dekrement benutzt. Interessanter wird es bei Iteratoren oder Klassen die an dieser Stelle auch benutzt werden könnten. Durch den Wegfall des temporären Anlegens einer Kopie merkt man bei der Verwendung von Dekrement schon eine Verbesserung von Speicherverbrauch und CPU Zeit.

Der Patch ist nicht falsch, trotzdem hier und da inkonsequent, es gibt immer noch Stellen wo man mit Postkrement im Schleifenkopf arbeitet.
http://bytes.com/topic/c/answers/63485- ... ad-i-loops