[ANNOUNCE] VDR developer version 2.1.5

holymoly · 23. Februar 2014

aus der ML

Zitat

VDR developer version 2.1.5 is now available at

ftp://ftp.tvdr.de/vdr/Developer/vdr-2.1.5.tar.bz2

A 'diff' against the previous version is available at

ftp://ftp.tvdr.de/vdr/Developer/vdr-2.1.4-2.1.5.diff

MD5 checksums:

ce561eef64c13e24e4817f70a6d9d5b0 vdr-2.1.5.tar.bz2
f433e78d90bc414bd9d858ca6e58e539 vdr-2.1.4-2.1.5.diff

WARNING:
========

This is a *developer* version. Even though *I* use it in my productive
environment, I strongly recommend that you only use it under controlled
conditions and for testing and debugging.

The changes since version 2.1.4:

- Now checking whether the primary device actually has a decoder before retuning the
current channel after a change in its parameters. This fixes broken recordings on
the primary device on "headless" systems.
- Increased MIN_TS_PACKETS_FOR_FRAME_DETECTOR to 100 and introduced counting the number
of actual video TS packets in cTsPayload in order to be able to record channels that
sometimes need even more than 10 TS packets for detecting frame borders (reported by
Oliver Endriss).
- Fixed sorting recordings by time in the Recordings menu if "Setup/OSD/Recording
directories" is set to "no".
- Fixed clearing non-editable members in the channel editor (thanks to Rolf Ahrenberg).
- Updated the Estonian OSD texts (thanks to Arthur Konovalov).
- Further clarified the semantics of cCamSlot::Decrypt().
- Fixed flickering if subtitles are active while the OSD demo is running.
- Fixed numbering frames. Previously they were numbered starting from 1, while it
is apparently standard to number them from 0. Any existing recordings with editing
marks (which will now be off by one) can still be cut with all VDR versions from
1.7.32, because these will automatically adjust editing marks to I-frames.
Users of stable releases shouldn't notice any problems.
- Fixed a possible crash in the OSD demo (reported by Christopher Reimer).
- Fixed some compiler warnings with Clang 3.4.1 (reported by Paul Menzel).
- Added LinkageTypePremiere to libsi/si.h and eit.c to avoid a compiler warning with
Clang 3.4.1 (suggested by Tony Houghten).
- Replaced the NULL pointer assignment in ~cReceiver() to force a segfault with
a call to abort() (suggested by Tony Houghten).
- Fixed learning keyboard remote control codes (thanks to Lars Hanisch).
- Improved PAT/PMT scanning to speed up initial tuning to encrypted channels on
transponders with many PAT entries (reported by Mariusz Bialonczyk).
- Fixed the replay progress display for very long recordings.
- Fixed detecting broken video data streams when recording.
- Fixed handling frame detection buffer length (reported by Eike Sauer).

Have fun!

Klaus

Alles anzeigen

S:oren · 23. Februar 2014

Vielen Dank erstmal fuer die neue Version.

Zitat von holymoly

- Fixed some compiler warnings with Clang 3.4.1 (reported by Paul Menzel).

Dafuer gibts jetzt leider Errors mit gcc (Debian 4.6.3-14)

Code

ci.c: In member function ‘void cCiDateTime::SendDateTime()’:
ci.c:867:18: error: expected primary-expression before ‘.’ token
ci.c:867:37: error: expected primary-expression before ‘.’ token
ci.c:867:67: error: expected primary-expression before ‘.’ token
ci.c:867:96: error: expected primary-expression before ‘.’ token
ci.c:867:125: error: expected primary-expression before ‘.’ token
ci.c: In member function ‘virtual void cCiMMI::Process(int, const uint8_t*)’:
ci.c:1007:47: error: expected primary-expression before ‘.’ token
ci.c:1007:71: error: expected primary-expression before ‘.’ token

Gruss,
S:oren

TheChief · 23. Februar 2014

Bei mir ist alles problemlos durchgelaufen (gcc (Debian 4.7.2-5) 4.7.2). Danke für das Update.

3PO · 23. Februar 2014

Bei mir auch.

Code

vdr01_64 vdr-2.1.5 # gcc -v
Es werden eingebaute Spezifikationen verwendet.
COLLECT_GCC=/usr/x86_64-pc-linux-gnu/gcc-bin/4.7.3/gcc
COLLECT_LTO_WRAPPER=/usr/libexec/gcc/x86_64-pc-linux-gnu/4.7.3/lto-wrapper
Ziel: x86_64-pc-linux-gnu
Konfiguriert mit: /tmp/portage/sys-devel/gcc-4.7.3-r1/work/gcc-4.7.3/configure --host=x86_64-pc-linux-gnu --build=x86_64-pc-linux-gnu --prefix=/usr --bindir=/usr/x86_64-pc-linux-gnu/gcc-bin/4.7.3 --included6_64-pc-linux-gnu/4.7.3/include --datadir=/usr/share/gcc-data/x86_64-pc-linux-gnu/4.7.3 --mandir=/usr/share/gcc-data/x86_64-pc-linux-gnu/4.7.3/man --infodir=/usr/share/gcc-data/x86_64-pc-linux-gnu/4.7.3/infde-dir=/usr/lib/gcc/x86_64-pc-linux-gnu/4.7.3/include/g++-v4 --with-python-dir=/share/gcc-data/x86_64-pc-linux-gnu/4.7.3/python --enable-languages=c,c++,fortran --enable-obsolete --enable-secureplt --disablstem-zlib --enable-nls --without-included-gettext --enable-checking=release --with-bugurl=https://bugs.gentoo.org/ --with-pkgversion='Gentoo 4.7.3-r1 p1.4, pie-0.5.5' --enable-libstdcxx-time --enable-sharedposix --enable-__cxa_atexit --enable-clocale=gnu --enable-multilib --with-multilib-list=m32,m64 --disable-altivec --disable-fixed-point --enable-targets=all --disable-libgcj --enable-libgomp --disable-libmubssp --enable-lto --without-cloog --without-ppl
Thread-Modell: posix
gcc-Version 4.7.3 (Gentoo 4.7.3-r1 p1.4, pie-0.5.5)
vdr01_64 vdr-2.1.5 #

Copperhead · 23. Februar 2014

Hier gibts keine Probleme:

Code

% gcc --version
gcc (GCC) 4.8.2 20140206 (prerelease)
Copyright (C) 2013 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Uwe · 23. Februar 2014

Vielen Dank für die VDR Version, Klaus.

Viele Grüße, Uwe

PS: Hier auch error:

Code

gcc (Ubuntu/Linaro 4.6.3-1ubuntu5) 4.6.3

Hat schon jemand eine Lösung?

S:oren · 23. Februar 2014

Zitat von Uwe

Hat schon jemand eine Lösung?

Die Aenderung wieder rueckgaengig machen:

Diff

--- vdr-2.1.5/ci.c.orig	2014-02-23 16:22:23.929700000 +0100
+++ vdr-2.1.5/ci.c	2014-02-23 18:17:40.379700000 +0100
@@ -864,7 +864,7 @@ void cCiDateTime::SendDateTime(void)
 #pragma pack(1)
      struct tTime { uint16_t mjd; uint8_t h, m, s; short offset; };
 #pragma pack()
-     tTime T = { .mjd = htons(MJD), .h = DEC2BCD(tm_gmt.tm_hour), .m = DEC2BCD(tm_gmt.tm_min), .s = DEC2BCD(tm_gmt.tm_sec), .offset = short(htons(tm_loc.tm_gmtoff / 60)) };
+     tTime T = { mjd : htons(MJD), h : DEC2BCD(tm_gmt.tm_hour), m : DEC2BCD(tm_gmt.tm_min), s : DEC2BCD(tm_gmt.tm_sec), offset : short(htons(tm_loc.tm_gmtoff / 60)) };
      bool OldDumpTPDUDataTransfer = DumpTPDUDataTransfer;
      DumpTPDUDataTransfer &= DumpDateTime;
      if (DumpDateTime)
@@ -1004,7 +1004,7 @@ void cCiMMI::Process(int Length, const u
                  case DCC_SET_MMI_MODE:
                       if (l == 2 && *++d == MM_HIGH_LEVEL) {
                          struct tDisplayReply { uint8_t id; uint8_t mode; };
-                         tDisplayReply dr = { .id = DRI_MMI_MODE_ACK, .mode = MM_HIGH_LEVEL };
+                         tDisplayReply dr = { id : DRI_MMI_MODE_ACK, mode : MM_HIGH_LEVEL };
                          dbgprotocol("Slot %d: ==> Display Reply (%d)\n", Tc()->CamSlot()->SlotNumber(), SessionId());
                          SendData(AOT_DISPLAY_REPLY, 2, (uint8_t *)&dr);
                          }

Alles anzeigen

Gruss,
S:oren

3PO · 23. Februar 2014

Zitat

Die Aenderung wieder rueckgaengig machen:

Also dann braucht man auch keine Developer Version zu verwenden.
Da wäre es doch sinnvoller gcc upzudaten.

Uwe · 23. Februar 2014

Super, Dank Dir S:oren!

S:oren · 23. Februar 2014

Zitat von 3PO

Also dann braucht man auch keine Developer Version zu verwenden.

Ich sehe einigen Wert darin, wenn mehrere Leute auch auf exotischeren Systemen die aktuelle Developer-Version von VDR testen. Dieses Ansinnen unterstuetze ich jedenfalls gerne mit einem kleinen Patch.

Ob Klaus das in der naechsten VDR-Version wieder aendert, ist doch seine Sache. Muss er doch sowieso selbst entscheiden, ob ihm die Vermeidung von Warnings auf einem eher exotischen Compiler das Aufgeben der out-of-the-box-Unterstuetzung grosser Distributionen (Debian stable, Linaro) wert ist.

Vielleicht findet sich ja auch eine allseits compatible Loesung...

Gruss,
S:oren

Moorviper · 23. Februar 2014

Zitat

...Aufgeben der out-of-the-box-Unterstuetzung grosser Distributionen (Debian stable, Linaro) wert ist.

????

Benutze am server debian stable und da hab ich

Zitat

# gcc --version
gcc (Debian 4.7.2-5) 4.7.2
Copyright (C) 2012 Free Software Foundation, Inc.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Alles anzeigen

und keine Probleme

S:oren · 23. Februar 2014

Zitat von Moorviper

Benutze am server debian stable und da hab ich

Nun ja, ich habe auch debian wheezy und

Code

# gcc -v
[...]
Target: arm-linux-gnueabi
Thread model: posix
gcc version 4.6.3 (Debian 4.6.3-14)

Schoen fuer alle, wo es funktioniert. Alle anderen koennen sich ja ueberlegen, ob sie einen neueren gcc installieren oder den Patch verwenden wollen...

Gruss,
S:oren

kls · 23. Februar 2014

Nachdem es bei mir mit gcc 4.7.1 problemlos lief hatte ich nicht mit größeren Problemen gerechnet.
Da es aber mit Clang nur eine Warnung gibt, mit älteren gcc-Versionen aber gar nicht geht, werde ich das in der nächsten Version wieder zurück ändern.
gcc hat Vorrang ;-).

Klaus

rudirabbit · 23. Februar 2014

Es geht hier nur wie man ein Struct füllt ?

Code

tTime T = { .mjd = htons(MJD), .h = DEC2BCD(tm_gmt.tm_hour), ..

Wenn der Compiler meckert sollte auch das funktionieren ??

Code

tTime T = {T.mjd = htons(MJD),T.h = DEC2BCD(tm_gmt.tm_hour), ..

S:oren · 23. Februar 2014

Zitat von rudirabbit

Wenn der Compiler meckert sollte auch das funktionieren ??

Sieht schon mal gut aus.

Diff

--- vdr-2.1.5/ci.c.orig	2014-02-23 16:22:23.929700000 +0100
+++ vdr-2.1.5/ci.c	2014-02-23 20:04:20.849700000 +0100
@@ -864,7 +864,7 @@ void cCiDateTime::SendDateTime(void)
 #pragma pack(1)
      struct tTime { uint16_t mjd; uint8_t h, m, s; short offset; };
 #pragma pack()
-     tTime T = { .mjd = htons(MJD), .h = DEC2BCD(tm_gmt.tm_hour), .m = DEC2BCD(tm_gmt.tm_min), .s = DEC2BCD(tm_gmt.tm_sec), .offset = short(htons(tm_loc.tm_gmtoff / 60)) };
+     tTime T = { T.mjd = htons(MJD), T.h = DEC2BCD(tm_gmt.tm_hour), T.m = DEC2BCD(tm_gmt.tm_min), T.s = DEC2BCD(tm_gmt.tm_sec), T.offset = short(htons(tm_loc.tm_gmtoff / 60)) };
      bool OldDumpTPDUDataTransfer = DumpTPDUDataTransfer;
      DumpTPDUDataTransfer &= DumpDateTime;
      if (DumpDateTime)
@@ -1004,7 +1004,7 @@ void cCiMMI::Process(int Length, const u
                  case DCC_SET_MMI_MODE:
                       if (l == 2 && *++d == MM_HIGH_LEVEL) {
                          struct tDisplayReply { uint8_t id; uint8_t mode; };
-                         tDisplayReply dr = { .id = DRI_MMI_MODE_ACK, .mode = MM_HIGH_LEVEL };
+                         tDisplayReply dr = { dr.id = DRI_MMI_MODE_ACK, dr.mode = MM_HIGH_LEVEL };
                          dbgprotocol("Slot %d: ==> Display Reply (%d)\n", Tc()->CamSlot()->SlotNumber(), SessionId());
                          SendData(AOT_DISPLAY_REPLY, 2, (uint8_t *)&dr);
                          }

Alles anzeigen

Das compiliert schon mal mit gcc-4.6.3. Die Funktion kann ich mangels CI allerdings nicht testen...

Gruss,
S:oren

UFO · 23. Februar 2014

Im Zweifelsfall würde ich explizite Zuweisungen daraus machen.

Mir ist allerdings nicht klar, wieso die ".xyz"-Syntax nicht akzeptiert werden sollte.
Riecht nach Compiler-Fehler...

CU
Oliver

lancs · 24. Februar 2014

von mir auch ein großes Danke für das Update..... funzt prächtig

Code

gcc version 4.7.2 (Debian 4.7.2-5)

Frank

kls · 24. Februar 2014

Zitat von rudirabbit
Wenn der Compiler meckert sollte auch das funktionieren ??
Code
tTime T = {T.mjd = htons(MJD),T.h = DEC2BCD(tm_gmt.tm_hour), ..

Dann kann man es aber auch gleich explizit hinschreiben, denn damit spart man nichts, da man den Variablennamen ja x-mal angeben muß. Der Gag bei dieser Notation ist ja gerade, daß man den Variablennamen nur *einmal* angeben muß.

Klaus

kls · 24. Februar 2014

Zitat von UFO

Mir ist allerdings nicht klar, wieso die ".xyz"-Syntax nicht akzeptiert werden sollte.
Riecht nach Compiler-Fehler...

Würde ich auch fast vermuten.
Mit welchem Compiler wird auf solchen Systemen eigentlich der Treiber übersetzt?
Dort finden sich auch reihenweise solcher Konstrukte, z.B. in usb/ttusb-budget/dvb-ttusb-budget.c:

Code

static struct usb_driver ttusb_driver = {
      .name             = "ttusb",
      .probe            = ttusb_probe,
      .disconnect       = ttusb_disconnect,
      .id_table         = ttusb_table,
};

Klaus

S:oren · 24. Februar 2014

Zitat von kls

Mit welchem Compiler wird auf solchen Systemen eigentlich der Treiber übersetzt?

Mit derselben gcc-Version. Der Treiber ist aber plain C, der vdr C++.

Gruss,
S:oren

[ANNOUNCE] VDR developer version 2.1.5

Jetzt mitmachen!

Teilen