GIT-Archiv für VDR auf http://git.tvdr.de

kls · Dec 18th 2020

Also beim Macro (DEBUG) kann ich es gerade noch akzeptieren, denn da wird durch den Preprozessor etwas "leer gemacht". Die Änderungen in dvbspu.c und pat.c werde ich übernehmen, insbesondere da es in tools.h auch schon so gemacht ist (dort allerdings wegen des ternären Operators).

Wenn es in timers.c angemeckert wird, dann frage ich mich, warum es in z.B. osd.c nicht gemeldet wird

if ((Alignment & taTop) != 0)

;

Ich habe nicht vor, das an all diesen Stellen zu ändern, vor allem, weil ich das wohl auch künftig immer wieder aus Gewohnheit so schreiben würde ;-).

glotzipapa · Dec 18th 2020

Quote from HelmutB

glotzipapa : Falls du es testest, vielleicht erkennst du unpassende Werte (nicht zu lange laufen lassen, es wird viel ins syslog geschrieben).

Ich habe den Patch mal laufen lassen, anbei das Log.

So sieht übrigens ein karputer EPG-Eintrag aus

Code

C S19.2E-1-1089-12003 RTL Television
E 28538 1608303600 3600 4E B
T dolby digital
D yFE
G 81
X 2 03 deu deutsch
X 2 05 deu dolby digital
e
E 28539 1608307200 1800 4E B
T dolby digital
D 'Ê3g'Ë3gÍ3gðÏ3g(Ò3gÔ3gxÖ3gÈØ3gÚ3ghÜ3gÞ3g à3gžâ3gå3gtµNpµNèë3g
G 81
X 2 03 deu deutsch
X 2 05 deu dolby digital

Display More

Es wird übrigens nicht nur das EPG zerstört, sondern auch die channels.conf. Aber nur wenn die Kanalaktualisierung aktiviert ist.

HelmutB · Dec 18th 2020

Quote from kls

dann frage ich mich, warum es in z.B. osd.c nicht gemeldet wird

Das hat mich jetzt auch interessiert, und ein kleiner Test hat ergeben, dass -Wempty-body nur beim letzten leeren if/else Statement diese Warnung ausgibt:

if (a<b)

;

else if (a>b)

;

else

; // <<< "empty-body" Warnung

In osd.c daher beim "if" keine Warnung weil danach ein "else" folgt.

Helmut

HelmutB · Dec 18th 2020

glotzipapa : nach dem Aufruf von "convertCharacterTable()" stimmt irgendetwas nicht mehr:

Code: msg

Dec 18 17:05:22 raspi user.debug vdr: [203] *** ExtEvent getText: s=957
Dec 18 17:05:22 raspi user.debug vdr: [203]     ExtEvent[0]: l=234 i=234 t=723 <-- Textlänge 234 bytes
Dec 18 17:05:22 raspi user.debug vdr: [203]     ExtEvent conv: l=234 i=7 s=950 <-- Textlänge 7 bytes!
Dec 18 17:05:22 raspi user.debug vdr: [203] === ExtEvent Done: i=7 s=950
Dec 18 17:05:22 raspi user.debug vdr: [203] *** ExtEvent getText: s=925
Dec 18 17:05:22 raspi user.debug vdr: [203]     ExtEvent[0]: l=226 i=226 t=699 <-- Textlänge 226 bytes
Dec 18 17:05:22 raspi user.debug vdr: [203]     ExtEvent conv: l=226 i=0 s=925 <-- Textlänge 0 bytes!
Dec 18 17:05:22 raspi user.debug vdr: [203] === ExtEvent Done: i=0 s=925

Vielleicht muss man auch noch die übegebenen Buffer und "FromCode" Pointer überprüfen.

Verwendest du VDR_CHARSET_OVERRIDE ? Wenn Ja, probier es einmal ohne, falls Nein, definiere einen - z.B "ISO8859-9"

Helmut

glotzipapa · Dec 18th 2020

Quote from HelmutB

Verwendest du VDR_CHARSET_OVERRIDE ? Wenn Ja, probier es einmal ohne, falls Nein, definiere einen - z.B "ISO8859-9"

Helmut

Also beim Start setze ich

Code

--chartab=ISO-8859-15

Ich habe das mal weggelassen, dann wird laut Log

Code

'ANSI_X3.4-1968'

gesetzt. Funktioniert aber immer noch nicht. VDR_CHARSET_OVERRIDE gibts IMO so nicht in jeder Distri.

HelmutB · Dec 18th 2020

Stimmt, VDR_CHARSET_OVERRIDE ist von Gentoo, es wird dann bei "--chartab=..." eingesetzt.

Hier noch ein erweiteter Patch, der die Pointer und Charsets, auch in "convertCharacterTable()", ausgibt.

Damit sollte man besser erkennen, wo die falsche Stringlänge herkommt.

Helmut

HelmutB · Dec 19th 2020

glotzipapa : Vergiss vorläufig den Patch vom letzten Post und versuche einmal den hier im Anhang.

Möglicherweise klappt das "iconf_open()" bei dir nicht. Dieser Fehlerfall wird in "convertCharacterTable()" aber nicht (mehr) berücksichtigt und damit wird die Description auch nicht in den Zielbuffer kopiert. Der Inhalt ist damit undefiniert.

Der kleine Patch sollte das korrigieren.

LG Helmut

gehlhajo · Dec 19th 2020

Quote from kls

Bitte ausgiebig testen.

Falls es noch von Belang ist: Gestern auf einem Raspi2 (Archlinux) gebaut. Bisher keine Probleme.

Auch von mir vielen Dank für die jahrelange Arbeit.

LG gehlhajo

glotzipapa · Dec 19th 2020

Quote from HelmutB

glotzipapa : Vergiss vorläufig den Patch vom letzten Post und versuche einmal den hier im Anhang.

Möglicherweise klappt das "iconf_open()" bei dir nicht. Dieser Fehlerfall wird in "convertCharacterTable()" aber nicht (mehr) berücksichtigt und damit wird die Description auch nicht in den Zielbuffer kopiert. Der Inhalt ist damit undefiniert.

Der kleine Patch sollte das korrigieren.

Bingo! Funktioniert!

Was mir beim bauen jetzt noch aufgefallen ist:

Code

o/de_DE.po: warning: Charset "ISO-8859-15" is not supported. msgfmt relies on iconv(),
and iconv() does not support "ISO-8859-15".
Installing GNU libiconv and then reinstalling GNU gettext
would fix this problem.
Continuing anyway.

Yocto Linux ist halt sehr minimalistisch, ich muss mal schauen, was da anders ist bei der libiconv

kls · Dec 21st 2020

Voraussichtlich letztes Update des GIT vor der Freigabe der Version 2.4.6.

GIT-Archiv für VDR auf http://git.tvdr.de

Participate now!

Share