[radio-1.1.0] RDS/Radiotext und AAC-Audio

FireFly · January 23, 2022 at 6:28 PM

Quote from HelmutB

Ich habe festgestellt, das das radio-plugin mit asprintf() arbeitet um den Radiotext zu speichern. Leider wird aber fast überall auf ein free() der nicht mehr benötigten Daten vergessen.

VDR löst das sehr schön mit der cString-Klasse, die man auch in Plugins verwenden kann und sich dann nicht mehr um das free() kümmern muss

HelmutB † · January 23, 2022 at 8:08 PM

FireFly cStrings wären auch eine Möglichkeit. Da eine Radiotextzeile aber nur aus max. 64 Zeichen besteht und man die Anzahl der zu speichernden Texte selbst festlegt, habe ich im Extension-Patch von char* auf char-Arrays umgestellt und schreibe direkt in diese Buffer.

Helmut

Uwe · January 24, 2022 at 4:37 PM

Quote from HelmutB

Hier die Version 7. Ich habe nun einen Pat/Pmt Parser eingebaut, damit kann das Plugin externe RDS-Pids selbst erkennen und der VDR muß nicht mehr gepatched werden. Das "bitrate" diff von oben ist natürlich enthalten, und das Default für die verbosity ist von "1" auf "0" geändert - Radiotext Debuging müss nun mit "-v 1" aktiviert werden.

Sonst ist kein weiterer Patch notwendig, der v6-Extension-Patch von Post #13 kann (oder sollte?) zusätzlich auch noch angewendet werden.

Helmut

PS: Ich habe mit vdr-2.5.6 und 2.4.7 getestet, vdr-2.2.0 sollte aber auch passen.

Bekomme hier ein Fehler beim bauen des Plugins mit vdr-2.6.0 ....

Code

*** Plugin radio:
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wno-unused-result -c -DPLUGIN_NAME_I18N='"radio"'  -DGIT_REV='"2.6.0"' -I/home/pi/vdr/include -o radio.o radio.c
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wno-unused-result -c -DPLUGIN_NAME_I18N='"radio"'  -DGIT_REV='"2.6.0"' -I/home/pi/vdr/include -o radioaudio.o radioaudio.c
radioaudio.c: In member function 'void cRadioAudio::HandleRdsPids(const int*, int)':
radioaudio.c:1962:22: warning: unused variable 'j' [-Wunused-variable]
 1962 |             for (int j = 0; !found && a[i]; i++)
      |                      ^
radioaudio.c:1964:22: warning: unused variable 'j' [-Wunused-variable]
 1964 |             for (int j = 0; !found && d[i]; i++)
      |                      ^
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wno-unused-result -c -DPLUGIN_NAME_I18N='"radio"'  -DGIT_REV='"2.6.0"' -I/home/pi/vdr/include -o radioskin.o radioskin.c
radioaudio.c: In member function 'virtual void cRTplusList::Load()':
radioaudio.c:3372:62: warning: '%s' directive output may be truncated writing up to 79 bytes into a region of size between 56 and 74 [-Wformat-truncation=]
 3372 |                     snprintf(text, sizeof(text), "%02d:%02d\t%s\t\t%s",
      |                                                              ^~
 3373 |                             ts->tm_hour, ts->tm_min, ctitle,
      |                                                      ~~~~~~
radioaudio.c:3372:29: note: 'snprintf' output 9 or more bytes (assuming 88) into a destination of size 80
 3372 |                     snprintf(text, sizeof(text), "%02d:%02d\t%s\t\t%s",
      |                     ~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 3373 |                             ts->tm_hour, ts->tm_min, ctitle,
      |                             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 3374 |                             Convert(rtp_content.item_Artist[i]));
      |                             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
radioaudio.c:3385:58: warning: '%s' directive output may be truncated writing up to 79 bytes into a region of size between 56 and 74 [-Wformat-truncation=]
 3385 |                 snprintf(text, sizeof(text), "%02d:%02d\t%s\t\t%s", ts->tm_hour,
      |                                                          ^~
 3386 |                         ts->tm_min, ctitle,
      |                                     ~~~~~~
radioaudio.c:3385:25: note: 'snprintf' output 9 or more bytes (assuming 88) into a destination of size 80
 3385 |                 snprintf(text, sizeof(text), "%02d:%02d\t%s\t\t%s", ts->tm_hour,
      |                 ~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 3386 |                         ts->tm_min, ctitle,
      |                         ~~~~~~~~~~~~~~~~~~~
 3387 |                         Convert(rtp_content.item_Artist[i]));
      |                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wno-unused-result -c -DPLUGIN_NAME_I18N='"radio"'  -DGIT_REV='"2.6.0"' -I/home/pi/vdr/include -o radiotools.o radiotools.c
radiotools.c: In function 'bool ParseMpaFrameHeader(const uchar*, uint32_t*, int*, char**)':
radiotools.c:250:9: warning: unused variable 'A' [-Wunused-variable]
  250 |     int A = (info >> 21) & 0x7FF;
      |         ^
radiotools.c:253:9: warning: unused variable 'D' [-Wunused-variable]
  253 |     int D = (info >> 16) & 0x1;
      |         ^
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wno-unused-result -c -DPLUGIN_NAME_I18N='"radio"'  -DGIT_REV='"2.6.0"' -I/home/pi/vdr/include -o radioepg.o radioepg.c
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wno-unused-result -c -DPLUGIN_NAME_I18N='"radio"'  -DGIT_REV='"2.6.0"' -I/home/pi/vdr/include -o inforx.o inforx.c
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wno-unused-result -c -DPLUGIN_NAME_I18N='"radio"'  -DGIT_REV='"2.6.0"' -I/home/pi/vdr/include -o rdspatpmt.o rdspatpmt.c
rdspatpmt.c:5:10: fatal error: vdr/libsi/descriptor.h: Datei oder Verzeichnis nicht gefunden
    5 | #include <vdr/libsi/descriptor.h>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make[1]: *** [Makefile:72: rdspatpmt.o] Fehler 1
make[1]: *** Es wird auf noch nicht beendete Prozesse gewartet....
inforx.c: In function 'int info_request(int, int)':
inforx.c:144:55: warning: '%s' directive output may be truncated writing up to 64 bytes into a region of size between 39 and 121 [-Wformat-truncation=]
  144 |         snprintf(temp[1], 2 * RT_MEL, "%02d:%02d  %s: %s", ts->tm_hour,
      |                                                       ^~
  145 |                 ts->tm_min, RTP_Artist, RTP_Title);
      |                                         ~~~~~~~~~
inforx.c:144:17: note: 'snprintf' output between 10 and 156 bytes into a destination of size 130
  144 |         snprintf(temp[1], 2 * RT_MEL, "%02d:%02d  %s: %s", ts->tm_hour,
      |         ~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  145 |                 ts->tm_min, RTP_Artist, RTP_Title);
      |                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Display More

Code

$ ls -la
insgesamt 8
drwxr-xr-x 2 pi pi 4096 24. Jan 16:31 .
drwxr-xr-x 4 pi pi 4096 17. Jan 19:14 ..
lrwxrwxrwx 1 pi pi   24 24. Jan 16:31 descriptor.h -> ../../libsi/descriptor.h
lrwxrwxrwx 1 pi pi   21 24. Jan 16:31 headers.h -> ../../libsi/headers.h
lrwxrwxrwx 1 pi pi   21 24. Jan 16:31 section.h -> ../../libsi/section.h
lrwxrwxrwx 1 pi pi   16 24. Jan 16:31 si.h -> ../../libsi/si.h
lrwxrwxrwx 1 pi pi   18 24. Jan 16:31 util.h -> ../../libsi/util.h
$ pwd
/home/pi/vdr/include/libsi

Display More

Edit: So klappt es:

C

/*
 * rdspatpmt.c: PAT section filter
 */

- #include <vdr/libsi/descriptor.h>
+ #include <libsi/descriptor.h>
#include "rdspatpmt.h"
#include "radioaudio.h"

#define PMT_SCAN_TIMEOUT  2000 // ms

Gruß

Uwe

wolfi.m · January 24, 2022 at 5:18 PM

Hallo,

Quote from HelmutB

PS: Ich habe mit vdr-2.5.6 und 2.4.7 getestet, vdr-2.2.0 sollte aber auch passen.

hatte den gleichen Fehler im PPA.
Nach anpassen der include #include <libsi/descriptor.h> Build mit VDR-2.2.0 im PPA Ok!

Gruss

Wolfgang

HelmutB † · January 24, 2022 at 5:31 PM

Danke für die Info!

Ich habe pat.c aus vdr-2.5.6 als Vorlage genommen, vermutlich deshalb. Es hat bei mir aber weder ein einfaches make oder auch ein gentoo emerge nie Probleme damit gehabt. ich werde es aber auf jeden Fall ausbessern.

Helmut

Edit: Ich sehe gerade, dass in pat.c #include "libsi/descriptor.h"steht und frage mich, warum - und vor allem, wann ich das geändert habe :).

HelmutB † · January 24, 2022 at 10:19 PM

Mit #include "libsi/descriptor.hhabe ich keinen Erfolg.

Bei mir sind nach einer VDR Installation die Headerdatein in /usr/include/vdr bzw. /usr/include/vdr/libsi zu finden, in den CFLAGS steht -I/usr/include, daher passt das <vdr/libsi/..> beim #include.

Uwe Hast du die Header Dateien nur lokal im VDR Source-Verzeichnis? -I/home/pi/vdr/include führt dann zu den von make erstellten lokalen Verzeichnissen include/vdr und include/libsi. Darin sind nur Symlinks, die auf /home/pi/vdr zurückführen. Mit -I/home/pi sollte es aber funktionieren. wolfi.m vielleicht ist es bei dir ähnlich - kein installiertes "vdr-devel" Paket?

Helmut

FireFly · January 25, 2022 at 12:06 AM

Für opensuse musste ich es auch in #include "libsi/descriptor.h"abändern. Auch etliche andere Plugins nutzen das so:

Code

PLUGINS/src> grep -R "#include" *|grep libsi
burn/scanner.c:#include <libsi/si.h>
dvbhddevice/dvbhdffdevice.c:#include <libsi/si.h>
epgsearch/conflictcheck.c:/*#include <libsi/si.h>*/
vnsiserver/demuxer.c:#include <libsi/si.h>
vnsiserver/videoinput.c:#include <libsi/section.h>
vnsiserver/videoinput.c:#include <libsi/descriptor.h>

wolfi.m · January 25, 2022 at 12:34 AM

Hallo,

Quote from HelmutB

wolfi.m vielleicht ist es bei dir ähnlich - kein installiertes "vdr-devel" Paket?

build der Pakete erfolgt in Launchpad. ...vdr-dev ist als depends bei allen Plugins im file debian/control enthalten.

Wie bei FireFly ist auch unter debian/ubuntu der Pfad libsi/descriptor.h

Code

 wolfi@Ku-Aspire:~/Git/1_gitlab/five/v$ grep -rnwi './' -e 'descriptor.h'
./vdr-plugin-eepg-0.0.6pre.git20150215/dish.h:14:#include <libsi/descriptor.h>
./vdr-plugin-eepg-0.0.6pre.git20150215/eit2.h:4:#include <libsi/descriptor.h>
./vdr-plugin-eepg-0.0.6pre.git20150215/eepg.c:37:#include <libsi/descriptor.h>
./vdr-plugin-eepg-0.0.6pre.git20150215/dish.c:15://#include <libsi/descriptor.h>
./vdr-plugin-radio-1.1.0/rdspatpmt.c:5:#include <libsi/descriptor.h>
./vdr-plugin-vnsiserver-1.8.0/videoinput.c:37:#include <libsi/descriptor.h>
./vdr-plugin-xine-0.9.5+git20160102/vdr172remux.c:33:#include "libsi/descriptor.h"

Gruss

Wolfgang

HelmutB † · January 25, 2022 at 1:27 AM

Interessant. Ich habe gerade in vdr-epgsearch nachgesehen. #include <libsi/si.h>* ist so im Code, aber bei gentoo wird vor dem make dieses include durch ein eclass-script in vdr/libsigeändert.

Code

fix_vdr_libsi_include() {
    eqawarn "Fixing include of libsi-headers"
    local f
    for f; do
        sed -i "${f}" \
            -e '/#include/s:"\(.*libsi.*\)":<\1>:' \
            -e '/#include/s:<.*\(libsi/.*\)>:<vdr/\1>:'
    done
}

Das muss Distributionsabhängig sein. In Ubuntu z.B. liegen die include Dateien unter /usr/include/vdr und /usr/include/libsi.

Wie es aussieht, ist damit für die Mehrheit <libsi/.....> richtig. Ich werde es so anpassen und mir für gentoo eine andere Lösung überlegen.

Helmut

Uwe · January 25, 2022 at 12:36 PM

Quote from HelmutB

Mit #include "libsi/descriptor.hhabe ich keinen Erfolg.

Bei mir sind nach einer VDR Installation die Headerdatein in /usr/include/vdr bzw. /usr/include/vdr/libsi zu finden, in den CFLAGS steht -I/usr/include, daher passt das <vdr/libsi/..> beim #include.

Uwe Hast du die Header Dateien nur lokal im VDR Source-Verzeichnis? -I/home/pi/vdr/include führt dann zu den von make erstellten lokalen Verzeichnissen include/vdr und include/libsi. Darin sind nur Symlinks, die auf /home/pi/vdr zurückführen. Mit -I/home/pi sollte es aber funktionieren. wolfi.m vielleicht ist es bei dir ähnlich - kein installiertes "vdr-devel" Paket?

Helmut

Hallo Helmut,

erstmal, vielen Dank für den tollen Patch!

Ja, ich habe aus Bequemlichkeit die Sourcen der Plugins in den Sourcen von VDR(git)... (~/vdr/PLUGINS/src), aber die Header werden jedes mal mit make install unter Debian auch ins "System" installiert.

Hier befinden sich die speziellen Header aus dem Ordner "libsi", hier:

Code

pi@rpi4:/usr/include/libsi $ ls -la
insgesamt 160
drwxr-xr-x  2 pi   pi    4096 24. Jan 17:01 .
drwxr-xr-x 80 root root 16384 18. Jan 22:26 ..
-rw-r--r--  1 pi   pi   21816 17. Jan 18:19 descriptor.h
-rw-r--r--  1 pi   pi   73931 17. Jan 18:19 headers.h
-rw-r--r--  1 pi   pi    8085 17. Jan 18:19 section.h
-rw-r--r--  1 pi   pi   23362 17. Jan 18:19 si.h
-rw-r--r--  1 pi   pi    5907 17. Jan 18:19 util.h
pi@rpi4:/usr/include/libsi $ pwd
/usr/include/libsi

Display More

Die restlichen Header befinden sich hier:

Code

pi@rpi4:/usr/include/vdr $ ls -la
insgesamt 608
drwxr-xr-x  2 pi   pi    4096 24. Jan 17:01 .
drwxr-xr-x 80 root root 16384 18. Jan 22:26 ..
-rw-r--r--  1 pi   pi     687 17. Jan 18:19 args.h
-rw-r--r--  1 pi   pi    1977 17. Jan 18:19 audio.h
-rw-r--r--  1 pi   pi   12552 17. Jan 18:19 channels.h
-rw-r--r--  1 pi   pi   26063 17. Jan 18:19 ci.h
-rw-r--r--  1 pi   pi   10907 17. Jan 18:19 config.h
-rw-r--r--  1 pi   pi    1497 17. Jan 18:19 cutter.h
-rw-r--r--  1 pi   pi   49029 17. Jan 18:19 device.h
-rw-r--r--  1 pi   pi    5909 17. Jan 18:19 diseqc.h
-rw-r--r--  1 pi   pi     766 17. Jan 18:19 dvbci.h
-rw-r--r--  1 pi   pi   10926 17. Jan 18:19 dvbdevice.h
-rw-r--r--  1 pi   pi    2864 17. Jan 18:19 dvbplayer.h
-rw-r--r--  1 pi   pi    4978 17. Jan 18:19 dvbspu.h
-rw-r--r--  1 pi   pi    1599 17. Jan 18:19 dvbsubtitle.h
-rw-r--r--  1 pi   pi    2100 17. Jan 18:19 eit.h
-rw-r--r--  1 pi   pi     846 17. Jan 18:19 eitscan.h
-rw-r--r--  1 pi   pi   16036 17. Jan 18:19 epg.h
-rw-r--r--  1 pi   pi    5480 17. Jan 18:19 filter.h
-rw-r--r--  1 pi   pi    5974 17. Jan 18:19 font.h
-rw-r--r--  1 pi   pi    4680 17. Jan 18:19 i18n.h
-rw-r--r--  1 pi   pi     769 17. Jan 18:19 interface.h
-rw-r--r--  1 pi   pi    3803 17. Jan 18:19 keys.h
-rw-r--r--  1 pi   pi     613 17. Jan 18:19 lirc.h
-rw-r--r--  1 pi   pi    9728 17. Jan 18:19 menu.h
-rw-r--r--  1 pi   pi    6370 17. Jan 18:19 menuitems.h
-rw-r--r--  1 pi   pi    8423 17. Jan 18:19 mtd.h
-rw-r--r--  1 pi   pi     553 17. Jan 18:19 nit.h
-rw-r--r--  1 pi   pi    4589 17. Jan 18:19 osdbase.h
-rw-r--r--  1 pi   pi   60022 17. Jan 18:19 osd.h
-rw-r--r--  1 pi   pi    2741 17. Jan 18:19 pat.h
-rw-r--r--  1 pi   pi    7600 17. Jan 18:19 player.h
-rw-r--r--  1 pi   pi    3382 17. Jan 18:19 plugin.h
-rw-r--r--  1 pi   pi    9704 17. Jan 18:19 positioner.h
-rw-r--r--  1 pi   pi    4621 17. Jan 18:19 receiver.h
-rw-r--r--  1 pi   pi    1755 17. Jan 18:19 recorder.h
-rw-r--r--  1 pi   pi   25816 17. Jan 18:19 recording.h
-rw-r--r--  1 pi   pi    3624 17. Jan 18:19 remote.h
-rw-r--r--  1 pi   pi   21828 17. Jan 18:19 remux.h
-rw-r--r--  1 pi   pi    5509 17. Jan 18:19 ringbuffer.h
-rw-r--r--  1 pi   pi    1092 17. Jan 18:19 sdt.h
-rw-r--r--  1 pi   pi    1120 17. Jan 18:19 sections.h
-rw-r--r--  1 pi   pi    5337 17. Jan 18:19 shutdown.h
-rw-r--r--  1 pi   pi     792 17. Jan 18:19 skinclassic.h
-rw-r--r--  1 pi   pi     798 17. Jan 18:19 skinlcars.h
-rw-r--r--  1 pi   pi   28256 17. Jan 18:19 skins.h
-rw-r--r--  1 pi   pi     785 17. Jan 18:19 skinsttng.h
-rw-r--r--  1 pi   pi    1978 17. Jan 18:19 sourceparams.h
-rw-r--r--  1 pi   pi    2586 17. Jan 18:19 sources.h
-rw-r--r--  1 pi   pi    1306 17. Jan 18:19 spu.h
-rw-r--r--  1 pi   pi    7928 17. Jan 18:19 status.h
-rw-r--r--  1 pi   pi    2260 17. Jan 18:19 svdrp.h
-rw-r--r--  1 pi   pi    3177 17. Jan 18:19 themes.h
-rw-r--r--  1 pi   pi   13388 17. Jan 18:19 thread.h
-rw-r--r--  1 pi   pi   12531 17. Jan 18:19 timers.h
-rw-r--r--  1 pi   pi   36715 17. Jan 18:19 tools.h
-rw-r--r--  1 pi   pi     950 17. Jan 18:19 transfer.h
-rw-r--r--  1 pi   pi    6901 17. Jan 18:19 videodir.h

Display More

Gruß

Uwe

HelmutB † · January 25, 2022 at 9:09 PM

Uwe bei Dir (und auch bei allen anderen) passt die Installation. So ist sie von VDR vorgesehen. Gentoo dürfte mittlerweile die einzige Distribution sein, die libsi unter /usr/include/vdr ablegt (Debian hat es vor vielen jahren auch noch so gemacht -> [ANNOUNCE] VDR developer version 1.7.36)

Im Anhang ein Patch der den VDR Include-Pfad berücksichtigt, und ein etwas "harter" Fix im Makefile für Gentoo.

Kannst du testen, ob es Nebenwirkungen gibt.

Diff

--- a/rdspatpmt.c    2022-01-22 22:45:05.550820255 +0100
+++ b/rdspatpmt.c    2022-01-25 18:32:02.780010515 +0100
@@ -2,7 +2,7 @@
  * rdspatpmt.c: PAT section filter
  */
 
-#include <vdr/libsi/descriptor.h>
+#include <libsi/descriptor.h>
 #include "rdspatpmt.h"
 #include "radioaudio.h"
 
--- a/Makefile    2022-01-22 18:36:12.770294390 +0100
+++ b/Makefile    2022-01-25 20:43:24.670288111 +0100
@@ -55,6 +55,11 @@ SOFILE = libvdr-$(PLUGIN).so
 
 INCLUDES +=
 
+### fix (or hack) for gentoo's 'libsi' include directory
+ifneq (,$(wildcard /usr/include/vdr/libsi/*.h))
+INCLUDES += -I/usr/include/vdr
+endif
+
 DEFINES += -DPLUGIN_NAME_I18N='"$(PLUGIN)"' \
     $(if $(GIT_REV), -DGIT_REV='"$(GIT_REV)"')

Display More

LG Helmut

Uwe · January 25, 2022 at 9:25 PM

Bei mir funktioniert dies nun sehr gut, wenn alles neu erstellt, sowie neu gebaut wird… 👍

Hier mal die Ausgaben:

Code

*** Plugin radio:
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wno-unused-result -c -DPLUGIN_NAME_I18N='"radio"'  -DGIT_REV='"2.6.0"' -I/home/pi/vdr/include -o radio.o radio.c
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wno-unused-result -c -DPLUGIN_NAME_I18N='"radio"'  -DGIT_REV='"2.6.0"' -I/home/pi/vdr/include -o radioaudio.o radioaudio.c
radioaudio.c: In member function 'void cRadioAudio::HandleRdsPids(const int*, int)':
radioaudio.c:1962:22: warning: unused variable 'j' [-Wunused-variable]
 1962 |             for (int j = 0; !found && a[i]; i++)
      |                      ^
radioaudio.c:1964:22: warning: unused variable 'j' [-Wunused-variable]
 1964 |             for (int j = 0; !found && d[i]; i++)
      |                      ^
radioaudio.c: In member function 'virtual void cRTplusList::Load()':
radioaudio.c:3372:62: warning: '%s' directive output may be truncated writing up to 79 bytes into a region of size between 56 and 74 [-Wformat-truncation=]
 3372 |                     snprintf(text, sizeof(text), "%02d:%02d\t%s\t\t%s",
      |                                                              ^~
 3373 |                             ts->tm_hour, ts->tm_min, ctitle,
      |                                                      ~~~~~~   
radioaudio.c:3372:29: note: 'snprintf' output 9 or more bytes (assuming 88) into a destination of size 80
 3372 |                     snprintf(text, sizeof(text), "%02d:%02d\t%s\t\t%s",
      |                     ~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 3373 |                             ts->tm_hour, ts->tm_min, ctitle,
      |                             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 3374 |                             Convert(rtp_content.item_Artist[i]));
      |                             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
radioaudio.c:3385:58: warning: '%s' directive output may be truncated writing up to 79 bytes into a region of size between 56 and 74 [-Wformat-truncation=]
 3385 |                 snprintf(text, sizeof(text), "%02d:%02d\t%s\t\t%s", ts->tm_hour,
      |                                                          ^~
 3386 |                         ts->tm_min, ctitle,
      |                                     ~~~~~~                
radioaudio.c:3385:25: note: 'snprintf' output 9 or more bytes (assuming 88) into a destination of size 80
 3385 |                 snprintf(text, sizeof(text), "%02d:%02d\t%s\t\t%s", ts->tm_hour,
      |                 ~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 3386 |                         ts->tm_min, ctitle,
      |                         ~~~~~~~~~~~~~~~~~~~
 3387 |                         Convert(rtp_content.item_Artist[i]));
      |                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wno-unused-result -c -DPLUGIN_NAME_I18N='"radio"'  -DGIT_REV='"2.6.0"' -I/home/pi/vdr/include -o radioskin.o radioskin.c
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wno-unused-result -c -DPLUGIN_NAME_I18N='"radio"'  -DGIT_REV='"2.6.0"' -I/home/pi/vdr/include -o radiotools.o radiotools.c
radiotools.c: In function 'bool ParseMpaFrameHeader(const uchar*, uint32_t*, int*, char**)':
radiotools.c:250:9: warning: unused variable 'A' [-Wunused-variable]
  250 |     int A = (info >> 21) & 0x7FF;
      |         ^
radiotools.c:253:9: warning: unused variable 'D' [-Wunused-variable]
  253 |     int D = (info >> 16) & 0x1;
      |         ^
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wno-unused-result -c -DPLUGIN_NAME_I18N='"radio"'  -DGIT_REV='"2.6.0"' -I/home/pi/vdr/include -o radioepg.o radioepg.c
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wno-unused-result -c -DPLUGIN_NAME_I18N='"radio"'  -DGIT_REV='"2.6.0"' -I/home/pi/vdr/include -o inforx.o inforx.c
inforx.c: In function 'int info_request(int, int)':
inforx.c:144:55: warning: '%s' directive output may be truncated writing up to 64 bytes into a region of size between 39 and 121 [-Wformat-truncation=]  144 |         snprintf(temp[1], 2 * RT_MEL, "%02d:%02d  %s: %s", ts->tm_hour,
      |                                                       ^~
  145 |                 ts->tm_min, RTP_Artist, RTP_Title);
      |                                         ~~~~~~~~~      
inforx.c:144:17: note: 'snprintf' output between 10 and 156 bytes into a destination of size 130
  144 |         snprintf(temp[1], 2 * RT_MEL, "%02d:%02d  %s: %s", ts->tm_hour,
      |         ~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  145 |                 ts->tm_min, RTP_Artist, RTP_Title);
      |                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wno-unused-result -c -DPLUGIN_NAME_I18N='"radio"'  -DGIT_REV='"2.6.0"' -I/home/pi/vdr/include -o rdspatpmt.o rdspatpmt.c
g++ -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wno-unused-result  -shared radio.o radioaudio.o radioskin.o radiotools.o radioepg.o inforx.o rdspatpmt.o -o libvdr-radio.so
xgettext -C -cTRANSLATORS --no-wrap --no-location -k -ktr -ktrNOOP --package-name=vdr-radio --package-version=1.1.0 --msgid-bugs-address='<see README>' -o po/radio.pot `ls inforx.c radioaudio.c radio.c radioepg.c radioskin.c radiotools.c rdspatpmt.c`
msgmerge -U --no-wrap --no-location --backup=none -q -N po/de_DE.po po/radio.pot
msgfmt -c -o po/de_DE.mo po/de_DE.po
msgmerge -U --no-wrap --no-location --backup=none -q -N po/fi_FI.po po/radio.pot
msgfmt -c -o po/fi_FI.mo po/fi_FI.po
msgmerge -U --no-wrap --no-location --backup=none -q -N po/fr_FR.po po/radio.pot
msgfmt -c -o po/fr_FR.mo po/fr_FR.po
msgmerge -U --no-wrap --no-location --backup=none -q -N po/hu_HU.po po/radio.pot
msgfmt -c -o po/hu_HU.mo po/hu_HU.po
msgmerge -U --no-wrap --no-location --backup=none -q -N po/it_IT.po po/radio.pot
msgfmt -c -o po/it_IT.mo po/it_IT.po

Display More

TomJoad · January 25, 2022 at 9:25 PM

Uwe hat oben schon darauf hingewiesen:

Code

radioaudio.c:1962:22: warning: unused variable 'j' [-Wunused-variable]
 1962 |             for (int j = 0; !found && a[i]; i++)

da und in den folgenden Zeilen muss i durch j ersetzt werden

HelmutB † · January 25, 2022 at 9:39 PM

TomJoad Danke für die Erinnerung! Im Anhang der Patch dazu.

LG Helmut

HelmutB † · March 5, 2022 at 12:06 AM

Ich habe inzwischen einen Klon des Radio-Plugins in https://github.com/siricco/vdr-plugin-radio angelegt, den AAC- und Extension-Patch übernommen und bin bei Version 9 angelangt.

Die Commits bis zum Stand Version 8 sind in der Branch 1.1.0-aac zu finden, es ist mir aber nicht ganz gelungen alle Commits auch direkt in den git-Stand vom 15.7.2018 zu übernehmen, deshalb gibt es im Master einen Commit der alles Fehlende zur v8 abdeckt.

Der aktuelle Stand kann über den Tag AAC_RDSv9als "tar.gz" heruntergeladen werden:

https://github.com/siricco/vdr-pl…s/tag/AAC_RDSv9

Und falls es wer brauchen kann - im Anhang nur das Diff zur Version 1.1.0.

Helmut

Dr. Seltsam · March 5, 2022 at 2:55 PM

Was für ein Ausgabedevice hast Du?

Mit softhddevice kriege ich zwar bei Live-Radio das dafür konfigurierte Stillpicture angezeigt, aber nicht bei Wiedergabe einer Radioaufnahme. Ich hatte gehofft, damit folgendes Problem umgehen zu können:

Wenn man im softhddevice-Plugin die Option "Black picture" aktiviert, startet die Wiedergabe der Radioaufzeichnung zwar mit einem Schwarzbild, aber sobald man in der Aufzeichnung "spult", kommt das Bild vom zuletzt aktiven TV-Sender.

HelmutB † · March 5, 2022 at 3:41 PM

Ich verwende xineliboutput. Hier gibt es bei Live das Problem, dass sowohl mit PlayPes() als mit StillPicture() zwar das Bild angezeigt wird, xinliboutput dann aber den Ton unterdrückt. Deshalb muß ich das Hintergrundbild ganz abschalten.
Ich habe es jetzt kurz getestet - bei der Wiedegabe von Aufnahmen habe ich auch kein Hintergrundbild, mit PlayPes() habe ich Ton, mit StillPicture() weder Ton noch Bild. Hast du beide Optionen ausprobiert?

Ich habe bei der Bildausgabe nichts geändert - zumindest nicht wissentlich. Ich werde es mir aber ansehen.

Helmut

Dr. Seltsam · March 5, 2022 at 4:14 PM

Das dürfte ein uraltes Problem sein, aber wenn es einer lösen kann, dann Du

Mit StillPicture kriege ich auch Ton bei Wiedergabe von Radio-Aufzeichnungen. ClearOnSwitch und BlackPicture sind bei softhddevice aktiviert.

HelmutB † · March 5, 2022 at 4:38 PM

Es scheint, als wäre das Hintergrundbild bei Wiedergabe überhaupt deaktiviert:

Code: radioimage.c

       if ((IsRadioOrReplay == 1) && imagepath && !imageShown) { // only live $20090905

Versuche einmal den Patch im Anhang.

Helmut

Dr. Seltsam · March 5, 2022 at 5:43 PM

das funktioniert!

Nun ist die Frage, warum wurde es damals deaktiviert? Aus der HISTORY geht die Deaktivierung für replay nicht hervor.

[radio-1.1.0] RDS/Radiotext und AAC-Audio

Participate now!

Users Viewing This Thread