[PATCH] RCinput rework, mit besserem Repeat-Handling.

[GetAway]

Innerhalb der Audiosettings bleibt bei einem Wechsel der Volumesteuerung von avs/lirc/ost der Lautstärkebalken zum Justieren der Lautstärke nicht mehr länger als einen Bruchteil einer Sekunde sichtbar und verschwindet sofort.

@seife
Das erinnert doch stark an den Fehler den PauleFoul etwas höher im Thread gepostet
hatte. Kann natürlich sein das ich mit meiner Vermutung, das es am Autorepeat liegt,
komplett falsch liege. Könntest Du dir das bitte mal anschauen? Thx

Beim Movieplayer popt beim "?" die Infobar nur für 1sec auf...

Ist das auch ein FB Prob?? Könnst Du Dir (Seife) das mal bitte anschauen?

Kann ich bestätigen.
Genause wenn man mit der gelben Farbtaste zwischen Play und Pause wechselt.

[mb405]

nein.
im menü ist 55 und 20 eingestellt

[neutrino] setRepeat: delay=55 period=20 use kernel-repeat: yes

[seife]

Innerhalb der Audiosettings bleibt bei einem Wechsel der Volumesteuerung von avs/lirc/ost der Lautstärkebalken zum Justieren der Lautstärke nicht mehr länger als einen Bruchteil einer Sekunde sichtbar und verschwindet sofort.

@seife
Das erinnert doch stark an den Fehler den PauleFoul etwas höher im Thread gepostet
hatte. Kann natürlich sein das ich mit meiner Vermutung, das es am Autorepeat liegt,
komplett falsch liege. Könntest Du dir das bitte mal anschauen? Thx

Das sieht schon danach aus, aber ich habe auf die Schnelle den Schuldigen nicht gefunden. Muss ich mal am Gerät debuggen.

[seife]

nein.
im menü ist 55 und 20 eingestellt

[neutrino] setRepeat: delay=55 period=20 use kernel-repeat: yes

Das passt doch wunderbar. Anscheinend hat der 2.4er Kernel da halt irgendwo einen Faktor 10 drin. Kann auch an meinem Patch liegen, ich habe den auf 2.4 nie gebootet (oder erinnere mich wenigstens nicht mehr dran )

[seife]

@seife

Innerhalb der Audiosettings bleibt bei einem Wechsel der Volumesteuerung von avs/lirc/ost der Lautstärkebalken zum Justieren der Lautstärke nicht mehr länger als einen Bruchteil einer Sekunde sichtbar und verschwindet sofort.
Nur durch das Weiterschalten der Volumesteuerungsauswahl blitzt der Lautstärkebalken kurz auf.
Wählt man dann allerdings die +/- Tasten der FB zum einstellen, wird der Lautstärkebalken wieder sichtbar und verschwindet nicht mehr.

Fixed.

[GetAway]

Fixed.

@seife. Es funktioniert fast so wie es sollte.

OT:
Es befindet sich noch
irgendwo ein 2. Bug im Code. Die Volumeneinstellung für OST wird nicht gespeichert,
bzw. springt beim erneuter Auswahl auf OST immer auf 100%. Ist wohl nicht aufgefallen,
weil man es ja nicht sehen konnte. AVS ist i.O.

[seife]

Ich glaube nicht, das der 2.bug was mit dem Topic zu tun hat Ich würde sagen, das war schon vorher so.

[GetAway]

JA Ja , wäre nett wenn du trotzdem mal draufschauen würdest.

[seife]

Ich habe drauf geschaut.
Nein, ich fixe es nicht.
Wenn auf LIRC geschaltet wird, dann wird die Lautstärke auf "100 - PCM-Absenkung" gesetzt, und dem controld gesagt, er soll "OST" machen. Da der controld keinen Unterschied zwischen LIRC und OST macht, kann er sich die alte Lautstärke nicht merken, deswegen wird, wenn du wieder auf ost schaltest, die Lautstärke beibehalten.

Theoretisch könnte man das fixen. Da man aber normalerweise nur einmal die Settings einstellt, ist es IMVHO den Aufwand nicht wert.

[GetAway]

@seife

Wenn bei der EPG-Komplettvorschau länger auf -/+ zum Scrollen gedrückt wird, schaltet sich die
Volumensteuerung ein. Zusätzlich wird die rechte obere Ecke, in der das Mute-Icon seinen Platz
hat, gelöscht. Könntest Du Dir das mal anschauen. Repeathandling auf -/+ bleibt erwünscht.

[GetAway]

Ich nochmal.

Es sollte wahrscheinlich so funktionieren wie bei der Infobar.
Während die Infobar eingeblendet ist, kann man die Lautstärke regeln.

Zum Blättern durch die EPG-Komplettvorschau sind die Grüne- und Gelbe-Taste
vorbelegt. Das sollte auch reichen.

[GetAway]

Dieser Patch fixt das Problem das die EPG-Komplettvorschau umblättert, während
man mit -/+ die Lautstärke regelt.

Das andere Problem, das der Bereich des Mute-Icon (oben Rechts) gelöscht wird,
ist woanders zu suchen und zentral gelagert. Das passiert grundsätzlich immer
bei der Volumenregelung und fällt nur auf, wenn, wie in diesem Fall, eine Grafik bis zu
dieser Ecke reicht. Logischerweise sollte der Bereich nur gelöscht/wiederhergestellt werden,
wenn das Mute-Icon auch eingeblendet ist/war, bzw. nach Volume=0, wenn laut Settings
Mute-Icon bei Lautstärke 0 auf "ein" steht!!

Code: Alles auswählen

Index: apps/tuxbox/neutrino/src/gui/epgplus.cpp
===================================================================
RCS file: /cvs/tuxbox/apps/tuxbox/neutrino/src/gui/epgplus.cpp,v
retrieving revision 1.45
diff -U3 -r1.45 epgplus.cpp
--- a/apps/tuxbox/neutrino/src/gui/epgplus.cpp	23 Feb 2009 20:06:41 -0000	1.45
+++ b/apps/tuxbox/neutrino/src/gui/epgplus.cpp	5 Mar 2009 10:23:59 -0000
@@ -1408,7 +1408,7 @@
 			if ( msg <= CRCInput::RC_MaxRC )
 				timeoutEnd = CRCInput::calcTimeoutEnd(g_settings.timing[SNeutrinoSettings::TIMING_CHANLIST]);
 
-			if (msg == g_settings.key_channelList_pagedown || msg == CRCInput::RC_yellow)
+			if (msg == CRCInput::RC_yellow)
 			{
 				if (this->channelList->getSize() > 0 )
 					{
@@ -1462,7 +1462,7 @@
 						}
 					}
 			}
-			else if (msg == g_settings.key_channelList_pageup || msg == CRCInput::RC_green)
+			else if (msg == CRCInput::RC_green)
 			{
 				if (this->channelList->getSize() > 0 )
 				{

[rhabarber1848]

Dieser Patch fixt das Problem das die EPG-Komplettvorschau umblättert, während
man mit -/+ die Lautstärke regelt.

committed: http://article.gmane.org/gmane.comp.vid ... ox.scm/727

[seife]

Dieser Patch ist falsch, g_settings.key_channelList_pageup kann was ganz anderes sein, als "+" und dann funktioniert es nicht mehr wie geplant.
Man müsste an dieser Stelle das repeat-Problem fixen.

[GetAway]

Ok, habe nochmal draufgeschaut. +/- wird zum Seitenweise Blättern gebraucht, wenn der Blättermodus Bouquetweise auf
gelb/grün über die Blaue Taste definiert wurde. Damit muss halt das Repeat rausfliegen und Lautstärkeregelung dadurch
ebenfalls.

Edit:
Da ist ebenfalls noch ein Bug. Der aktuelle Blättermodus (Bouquet) wird nicht angezeigt, wenn man EPG-Plus
erneut aufruft und dann auf "Optionen" geht.

[seife]

Ich glaube das Problem ist, dass settings.key_channelList_pageup / pagedown in der epgplus.cpp abgearbeitet werden, aber wenn das repeat-bit gesetzt ist, dann werden die durchgereicht an die anderen handler und somit landen sie dann im Volume-up/down handler.

Die Lösung wäre vermutlich, im epgplus.cpp die entsprechenden Events auch mit repeat- und / oder release-Bit als "handled" zu markieren (ich würde aber nicht unbedingt beim repeat auch hoch / runterscrollen, denn das ist doch eine eher langsame Operation und da will man nicht unbedingt repeat drin haben).

Das selbe müsste übrigens auch auf die rot/grün-Tasten zutreffen, allerdings haben die vermutlich keinen default-Handler, deswegen werden sie vermutlich einfach ignoriert. Das ist aber nur "aus Versehen" richtig und nicht mit Absicht

Es gibt halt auch geräte, die für Pageup/down eigene Tasten haben, und die macht dein Patch kaputt.

Andererseits ist es auf der dbox nicht so gravierend, insofern kann der Patch von mir aus erst mal drin bleiben, ich schau es mir dann mal an, wenn ich mal ein Wochenende nicht irgendwohin muss

[GetAway]

@seife

Kannst mal draufschauen. Läßt sich bestimmt noch besser lösen,
aber so funktionierts auch erstmal und g_settings.key_channelList_pageup
und g_settings.key_channelList_pagedown bleiben erhalten.

ignore_repeat_epgplus.diff

[seife]

Ja, sowas in der Art.

Lassen wir es im CVS mal wie es ist - die Hauptnutzer (dbox) haben erst mal einen Fix der für sie funktionieren sollte (es ist evtl. sogar praktisch, in der EPG-Ansicht die Lautstärke regeln zu können), und für die Maschinen mit pgup/pgdown-Tasten (TD) schau ich's mir nochmal an.

[seife]

So. Ich habe es jetzt mal gefixt, mit ausführlichem Kommentar, warum das so ist. Es ist im Prinzip genau dein Patch.

...und weils so schön war...
...nein, eigentlich weil der Code so unschön war, habe ich epgplus.{h,cpp} in ein Paar Schritten aufgeräumt, jetzt kompiliert es sogar mit gcc4 ohne Warnungen.
Weil ich mir da nicht überall 100% sicher war, sind das mehrere Schritte, wenn jetzt was neues kaputt sein sollte, diese Schritt für Schritt rückgängig machen und schauen ab wann es hakt.

[GetAway]

Sauber. Dank Dir.

Fehlt nur noch das Fixen der Optionseinstellung. Wenn auf Bouquet gestellt ist,
steht das nach Wiederaufruf der Optionsauswahl wieder auf "seitenweise"

[seife]

Sauber. Dank Dir.

Fehlt nur noch das Fixen der Optionseinstellung. Wenn auf Bouquet gestellt ist,
steht das nach Wiederaufruf der Optionsauswahl wieder auf "seitenweise"

Ich behaupte das ist Absicht, das wird nirgends gespeichert AFAICS (ist mir auch aufgefallen).