Schönheitsfehler im LCD bei aktiven Flexmenü

[bruno71]

Wenn man das Flexmenü startet ist im LCD der D-Box oben noch etwas vom Kanalname sichtbar.

Mfg bruno71

[Gaucho316]

In den Zeilen 77 und 85 der Datei apps/tuxbox/plugins/shellexec/lcd.c ist das Zeichnen des Hintergrundbildes auskommentiert. Warum, weiß ich nicht. Vielleicht kann das mal jemand reinnehmen und ausprobieren. Ich nutze das Flexmenü nicht.

[Mourice]

In den Zeilen 77 und 85 der Datei apps/tuxbox/plugins/shellexec/lcd.c ist das Zeichnen des Hintergrundbildes auskommentiert. Warum, weiß ich nicht. Vielleicht kann das mal jemand reinnehmen und ausprobieren. Ich nutze das Flexmenü nicht.

Hi,

dann läuft shellexec überhaupt nicht mehr ... segfault.

MfG

[Gaucho316]

Schade, aber nun weiß ich wenigstens, warum der Code auskommentiert ist.

Ich habe mir die Funktion LCD_paintIcon() jetzt mal genauer angesehen. Wenn ich das richtig verstehe, geht die von einem Bild im RAW-Format aus. Übergeben wird da ja aber ein PNG. Deshalb fliegt uns das wohl um die Ohren. Aber macht nichts, ich habe da noch eine andere Idee im Kopf, die in Neutrino ansetzt, da ich absolut keine Lust habe, mich mit der Plugin-Programmierung auseinanderzusetzen. Ich werde das die Tage mal umsetzen.

[bruno71]

Mir ist noch aufgefallen, dass das Problem nur auftritt wenn man die EPG-Anzeige im LCD aktiviert hat.

[Mourice]

Mir ist noch aufgefallen, dass das Problem nur auftritt wenn man die EPG-Anzeige im LCD aktiviert hat.

Hi,

ich kann es noch etwas "eingrenzen".

Es passiert nur, wenn mehr als der Sender-Namen angezeigt werden soll, und der Sender-Name zu lang für eine Zeile ist (als Beispiel: RTL Television),
also ein Umbruch gemacht werden muss.

MfG

[GetAway]

Fix
shellexec_fix_lcd_fragments.diff

[Gaucho316]

Ich hatte mir das so gedacht, dass bei Plugins, die das LCD nutzen, nicht automatisch der TV-Modus vor dem Starten des Plugins gesetzt wird. Dann sollte das Flexmenü wie früher aussehen, da der Menü-Modus für die LCD-Anzeige aktiv bleibt und man statt des Fortschrittsbalkens den Neutrino-Schriftzug sieht. Dann muss auch am Plugin nichts geändert werden.

Neutrino: don't set LCD toTV mode before plugin execution if plugin uses LCD
Link entfernt, da Patch im CVS

P.S.: "Kaputt" gemacht hatte ich das übrigens mit dem folgenden Patch.
http://cvs.tuxbox.org/lists/tuxbox-cvs- ... 00018.html

[GetAway]

Dann gibt es aber keine Kopfzeile etc. mehr, die sich das Plugin eventuell holt.
Ich würde es so lassen, das müssen die Plugins selbst regeln.

[Gaucho316]

Mir soll's egal sein. Ich wollte eigentlich nur den alten, funktionierenden Stand wiederherstellen, so dass der Schriftzug "Neutrino" wieder da steht wie bisher. Dann sieht es mehr nach Menü aus.

[Gaucho316]

Und nun?

Dann gibt es aber keine Kopfzeile etc. mehr, die sich das Plugin eventuell holt.

Was meintest du eigentlich damit?

[GetAway]

Siehe Posting 1 und ähnliches, welches sich im LCD Menü einstellen lässt.

[Gaucho316]

Ich kapier's trotzdem noch nicht. Mein Patch ändert an dem Prinzip ja nichts, dass Plugins auf der LCD-Anzeige malen können, was sie wollen. Nur der Ausgangszustand ist eben wie früher der Neutrino-Schriftzug und nicht der Fortschrittsbalken, der so oder so nicht mehr aktualisiert wird, wenn ein Plugin die Kontrolle über die LCD-Anzeige hat.

Was bewirkt dein Patch eigentlich? Ist an der Stelle, an der früher der Neutrino-Schriftzug zu sehen war und an der mit dem aktuellen CVS-Stand der Fortschrittsbalken und ggf. die Textreste angezeigt werden, dann gähnende Leere?

[GetAway]

Beim LCD_Init() wird jetzt maximal nur die Anzahl von LCD_MAX_LINECOPY
gesaved und nicht mehr pauschal 2x8 Zeilen. Deswegen gabs die Fragmente
in Zeile 16, die durch das wiederherstellen auftraten. In der Zeile waren ein
paar Pixel vom Sendernamen aktiv.

[Gaucho316]

Dann beißen sich unsere Patches also gar nicht und können beide eingecheckt werden?

[GetAway]

Na ich weiß nicht. Der schöne Fortschrittsbalken.

[Mourice]

Hi,

habe Eure beiden Patche getestet --> funktioniert alles "sauber".

Bezüglich der Kopfzeile ...

Das müsst Ihr beiden schon unter Euch ausmachen ...

MfG

[Gaucho316]

Ich denke, beide Patches aus diesem Thread können auch ins CVS.

[GetAway]

jup

[rhabarber1848]

committed:

shellexec_fix_lcd_fragments.diff

neutrino_dont-set-lcd-to-tv-mode-before-plugin-exec-if-plugin-uses-lcd_2012-05-30_1814.diff

http://article.gmane.org/gmane.comp.vid ... x.scm/3440
http://article.gmane.org/gmane.comp.vid ... x.scm/3441
http://article.gmane.org/gmane.comp.vid ... x.scm/3442