Commit-Wünsche für das CVS II

[Gaucho316]

Mir ist aufgefallen, daß in der Audioauswahl bei AC3-Tonspuren die AC3-Kennung doppelt vorkommen kann. Im Netz von KDG ist das bspw. bei RTL der Fall. Der folgende Patch korrigiert dieses Verhalten, indem vor dem Anhängen von " (AC3)" an den Tonspurnamen geprüft wird, ob das Wort "AC3" schon vorkommt.

Neutrino: prevent double AC3 in audio names
Link entfernt, da Patch im CVS


Beim Beheben des obigen Fehlers im nhttpd fiel mir auf, dass beim Aufruf von
http://dbox/control/zapto?getallpids
und
http://dbox/y/cgi?execute=func:get_audi ... s_dropdown
auf RTL überhaupt keine AC3-Tonspur angezeigt wird. Wenn man den Code ein wenig umstellt, geht's. Der folgende Patch löst also beide Probleme.

nhttpd: fix APID display and prevent double AC3 in audio names
Link entfernt, da Patch im CVS

Edit: Ich habe den Patch eben aktualisiert. Die Überprüfung des ersten Zeichens des Component Descriptors kann entfallen, da möglicherweise vorhandene Steuerzeichen schon seit einiger Zeit vom sectionsd entfernt werden.

[rhabarber1848]

committed:

usermenu_fix-memleaks_2012-07-01_1911.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3522

locale_update-channel-history-locales_2012-07-18_1940.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3523

neutrino_no-need-to-inherit-from-cmenuwidget_2012-07-20_1803.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3524
http://article.gmane.org/gmane.comp.vid ... x.scm/3525
http://article.gmane.org/gmane.comp.vid ... x.scm/3526

remotecontrol_prevent-double-ac3-in-audio-names_2012-07-21_1734.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3527

nhttpd_fix-apid-display-and-prevent-double-ac3-in-audio-names_2012-07-21_2314.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3528

[Gaucho316]

Seen in Neutrino-HD: use .empty() instead of .size()
use_empty_instead_of_size_2012_06_27_1522.diff

Bei diesem Commit sind zwei Änderungen leider falsch.

In bookmarkmanager.cpp wurde in Zeile 253

Code: Alles auswählen

if(selected==bookmarks.size() && !(bookmarks.empty()))

in

Code: Alles auswählen

if (!(bookmarks.empty() && selected==bookmarks.size()))

geändert. Das muss aber

Code: Alles auswählen

if (!bookmarks.empty() && selected==bookmarks.size())

lauten.

Und in channellist.cpp wurde in Zeile 714

Code: Alles auswählen

if (channelList.getSize() != 0) {

in

Code: Alles auswählen

if (!chanlist.empty()) {

geändert. Das muss aber so bleiben, wie es war, denn channelList ist nicht gleich chanlist.

[rhabarber1848]

Bei diesem Commit sind zwei Änderungen leider falsch.

committed: http://article.gmane.org/gmane.comp.vid ... x.scm/3529

[Gaucho316]

Wenn man in channellist.cpp (und epgplus.cpp) auch empty() statt size() verwenden möchte, kann man das auf folgende Weise machen.

Channel list: add method isEmpty() and use it
Link entfernt, da Patch im CVS

[GetAway]

@Gaucho316

Danke für das Korrigieren der Klammern im Bookmarkmanager.

Und in channellist.cpp wurde in Zeile 714

Code: Alles auswählen

if (channelList.getSize() != 0) {

in

Code: Alles auswählen

if (!chanlist.empty()) {

geändert. Das muss aber so bleiben, wie es war, denn channelList ist nicht gleich chanlist.

Bist du dir sicher? Hast du dir angesehen was

Code: Alles auswählen

channelList.getSize()

macht?

Code: Alles auswählen

int CChannelList::getSize() const
{
	return this->chanlist.size();
}

Daraus läßt sich doch erkennbar

Code: Alles auswählen

chanlist.empty()

direkt abfragen, oder sehe ich da etwas falsch?
This ist doch ein Zeiger auf sich selbst.

Edit: Ich glaube ich habs kapiert; channelList ist ein neues Objekt und um die Methode für sich selbst aufzurufen wird "this" benutzt.
Das war ein dicker Bug, deswegen hatte ich auch Probleme mit meinen Programmverlauf. Das hatte ich aber schon gepostet.

[Gaucho316]

Edit: Ich glaube ich habs kapiert; channelList ist ein neues Objekt und um die Methode für sich selbst aufzurufen wird "this" benutzt.

Richtig, in Zeile 704 von channellist.cpp wird eine neue Instanz von CChannelList innerhalb von CChannelList angelegt. In Zeile 711 brauchen wir dann die Größe von chanlist der neuen Instanz.

Das war ein dicker Bug, deswegen hatte ich auch Probleme mit meinen Programmverlauf. Das hatte ich aber schon gepostet.

Ich hatte das Problem nicht, weil ich derzeit den JtG-Snap vom 27.06. verwende und der Code da noch nicht drin war. Damit wäre das auch geklärt.

[Gaucho316]

Ich habe mal eine Übersicht aller über verschiedene Threads verstreuten Patches erstellt, die meiner Meinung nach eingecheckt werden können.

Neutrino: lower font baseline for headers and vertically center header icons
http://forum.tuxbox.org/forum/viewtopic ... 45#p383845
http://forum.tuxbox.org/forum/viewtopic ... 49#p383949

Bouquet editor: small gui rework
http://forum.tuxbox.org/forum/viewtopic ... 45#p383845

CVCRControl: use enums and remove unneeded code
http://forum.tuxbox.org/forum/viewtopic ... 50#p383950

controld: fix video output setting
http://forum.tuxbox.org/forum/viewtopic ... 62#p383962

zapit: terminate sat name correctly
http://forum.tuxbox.org/forum/viewtopic ... 34#p384034

CStreamInfo2: get mode correctly and use enum
http://forum.tuxbox.org/forum/viewtopic ... 39#p384039

Ach ja, und bitte nicht den Patch drei Postings über diesem vergessen.

[rhabarber1848]

committed:

channellist_add-method-isempty-and-use-it_2012-07-22_1700.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3543

Ich habe mal eine Übersicht aller über verschiedene Threads verstreuten Patches erstellt, die meiner Meinung nach eingecheckt werden können.

Ebenfalls alle committed, commit-Log siehe die jeweiligen threads.

[Gaucho316]

sectionsdcontrol: add option to register neutrino as event client
http://forum.tuxbox.org/forum/viewtopic ... 34#p383734

nhttpd: add registerneutrino in controlapi for commandline
http://forum.tuxbox.org/forum/viewtopic ... 29#p384229

Channel list: don't overlay bouquet name with sat name
http://forum.tuxbox.org/forum/viewtopic ... 29#p384129

Plugin Tuxwetter: fix memory/resource leak
http://forum.tuxbox.org/forum/viewtopic ... 33#p384133

Plugin Logomask: fix resource leak
http://forum.tuxbox.org/forum/viewtopic ... 34#p384134

[Gaucho316]

Ich habe zwei Sachen aus Neutrino-HD übernommen. Der erste Patch erweitert die Methode CMenuWidget::addIntroItems(...) um die Möglichkeit, den angezeigten Button (Zurück oder Abbrechen) zu beeinflussen. Der zweite Patch benutzt diese Neuerung, um im Startpositionsauswahlmenü den Abbrechen-Button statt des Zurück-Buttons anzuzeigen. Ich finde nämlich auch, dass der Text "Abbrechen" dort besser passt.

CMenuWidget: add cancel button to addIntroItems()
Link entfernt, da Patch im CVS

Moviebrowser: use cancel button in start position menu
Link entfernt, da Patch im CVS

In den nächsten Tagen habe ich dann vor, die Methode addIntroItems(...) überall dort einzubauen, wo die "Intro Items" noch einzeln eingefügt werden. Und das ist so ziemlich überall der Fall.

[GetAway]

Satbaby hat einen weiteren möglichen Segfault in sectionsd.cpp gefixt.

http://git.coolstreamtech.de/?p=cst-pub ... 0a9704b3a6

Eventuell wäre dieser Fix auch noch etwas.
http://git.coolstreamtech.de/?p=cst-pub ... cb6b68d03b

[GetAway]

@Gaucho316

Wenn du gerade die Menü-Klasse anfasst, hätte ich noch 2 Kleinigkeiten.
Neuerdings liegt eine horizontale Verschiebung des Haupttitels zum Untertitel
vor. Das sieht unschön aus. Außerdem hätte ich den Vorschlag, die Ausrichtung
der Menüeinträge(Buttons) an der Trennlinie auszurichten. Damals, als der Pfeil
dazu kam, sind die Buttons nach links verschoben worden. Das könnte mal gerichtet
werden, so dass Haupttitel, Untertitel, Menüeinträge in einer Flucht sind.

[Gaucho316]

Wenn du gerade die Menü-Klasse anfasst, hätte ich noch 2 Kleinigkeiten.
Neuerdings liegt eine horizontale Verschiebung des Haupttitels zum Untertitel
vor. Das sieht unschön aus. Außerdem hätte ich den Vorschlag, die Ausrichtung
der Menüeinträge(Buttons) an der Trennlinie auszurichten. Damals, als der Pfeil
dazu kam, sind die Buttons nach links verschoben worden. Das könnte mal gerichtet
werden, so dass Haupttitel, Untertitel, Menüeinträge in einer Flucht sind.

Reicht das so?

CMenuWidget: fix icon offset calculation
Link entfernt, da Patch im CVS

[rhabarber1848]

sectionsdcontrol: add option to register neutrino as event client
nhttpd: add registerneutrino in controlapi for commandline
Channel list: don't overlay bouquet name with sat name
Plugin Tuxwetter: fix memory/resource leak
Plugin Logomask: fix resource leak

committed

[Gaucho316]

In deutsch.locale und english.locale ist mir aufgefallen, dass hinter

Code: Alles auswählen

AUDIOSelectMenue.head
eventfinder.start_search

zwei Leerzeichen stehen. Das sieht in der Audio-Auswahl bzw. in der EPG-Suche ziemlich dämlich aus. Ich wundere mich, dass mir das noch nicht eher aufgefallen ist.

[rhabarber1848]

committed:

menue_add-cancel-button-to-addintroitems_2012-08-27_1843.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3561

moviebrowser_use-cancel-button-in-start-position-menu_2012-08-27_1911.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3562

[GetAway]

Bitte bei den Plugins die Versionsangabe anpassen.

[rhabarber1848]

In deutsch.locale und english.locale ist mir aufgefallen

committed: http://article.gmane.org/gmane.comp.vid ... x.scm/3563

[rhabarber1848]

Bitte bei den Plugins die Versionsangabe anpassen.

http://article.gmane.org/gmane.comp.vid ... x.scm/3564
http://article.gmane.org/gmane.comp.vid ... x.scm/3565

[GetAway]

Wenn du gerade die Menü-Klasse anfasst, hätte ich noch 2 Kleinigkeiten.
Neuerdings liegt eine horizontale Verschiebung des Haupttitels zum Untertitel
vor. Das sieht unschön aus. Außerdem hätte ich den Vorschlag, die Ausrichtung
der Menüeinträge(Buttons) an der Trennlinie auszurichten. Damals, als der Pfeil
dazu kam, sind die Buttons nach links verschoben worden. Das könnte mal gerichtet
werden, so dass Haupttitel, Untertitel, Menüeinträge in einer Flucht sind.

Reicht das so?

CMenuWidget: fix icon offset calculation
menue_fix-icon-offset-calculation_2012-08-29_1957.diff

Perfekt.

[Gaucho316]

In den nächsten Tagen habe ich dann vor, die Methode addIntroItems(...) überall dort einzubauen, wo die "Intro Items" noch einzeln eingefügt werden. Und das ist so ziemlich überall der Fall.

Neutrino: use addIntroItems() in menus
Link entfernt, da Patch im CVS


Dann habe ich noch einen Patch für die Movieplayer-Einstellungen. Ich finde das Menü ziemlich überfrachtet. Deshalb habe ich die Streamingserver-Einstellungen in ein Untermenü verfrachtet.

Movieplayer setup: add submenu for streaming server settings
Link zum Patch entfernt

Ich habe hier noch eine andere Variante, die mir persönlich besser gefällt. Sucht euch eine aus.

Movieplayer setup: add submenu for streaming server settings
Link zum Patch entfernt

[GetAway]

Hättest du nicht schnell 2 Screenshots machen können?

[Gaucho316]

Hättest du nicht schnell 2 Screenshots machen können?

Hätte ich, hab ich aber nicht. Ich versuche es mal anders anzudeuten.

Variante 1:

Code: Alles auswählen

Zurück                                       Zurück
-----------------------------------          -----------------------------------
Streamingserver           aktiviert          Streamingserver IP              xxx
Streamingserver-Einstellungen          ->    Streamingserver Port            xxx
-----------------------------------          DVD Laufwerk                    xxx
Start-Verz.              /mnt/filme          Verzeichnis (VLC)               xxx
...                                          ...

Variante 2:

Code: Alles auswählen

Zurück                                       Zurück
-----------------------------------          -----------------------------------
Streamingserver-Einstellungen          ->    Streamingserver           aktiviert
-----------------------------------          -----------------------------------
Start-Verz.              /mnt/filme          Streamingserver IP              xxx
Start-Plugin               Teletext          Streamingserver Port            xxx
...                                          ...

[rhabarber1848]

Mir gefällt Variante 1 besser, da ich den Streamingserver nutze,
ihn gelegentlich jedoch ausschalte, um eine Direktaufnahme zu
machen. In Variante 1 ist das möglich, ohne ein weiteres Menü
öffnen zu müssen.