Re: [vdr] [ANNOUNCE] VDR version 2.0.7 released

On 24.01.2015 16:18, Joerg Bornkessel wrote:

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1

Am 20.01.2015 um 10:23 schrieb Klaus Schmidinger:

...

VDR version 2.0.7 is now available at

Hallo Klaus,

just for the statistic ;)

gcc-4.9.2 with CXXFLAGS=" -std=c++11"

several warnings

<snipp> -o device.o device.c channels.c: In static member function ‘static tChannelID tChannelID::FromString(const char*)’: channels.c:32:82: warning: format ‘%a’ expects argument of type ‘float*’, but argument 3 has type ‘char**’ [-Wformat=] int fields = sscanf(s, "%a[^-]-%d-%d-%d-%d", &sourcebuf, &nid, &tid, &sid, &rid);

         ^

channels.c: In member function ‘bool cChannel::Parse(const char*)’: channels.c:589:221: warning: format ‘%a’ expects argument of type ‘float*’, but argument 3 has type ‘char**’ [-Wformat=] ncy, &parambuf, &sourcebuf, &srate, &vpidbuf, &apidbuf, &tpidbuf, &caidbuf, &sid, &nid, &tid, &rid);

                          ^

channels.c:589:221: warning: format ‘%a’ expects argument of type ‘float*’, but argument 5 has type ‘char**’ [-Wformat=] channels.c:589:221: warning: format ‘%a’ expects argument of type ‘float*’, but argument 6 has type ‘char**’ [-Wformat=] channels.c:589:221: warning: format ‘%a’ expects argument of type ‘float*’, but argument 8 has type ‘char**’ [-Wformat=] channels.c:589:221: warning: format ‘%a’ expects argument of type ‘float*’, but argument 9 has type ‘char**’ [-Wformat=] channels.c:589:221: warning: format ‘%a’ expects argument of type ‘float*’, but argument 10 has type ‘char**’ [-Wformat=] channels.c:589:221: warning: format ‘%a’ expects argument of type ‘float*’, but argument 11 has type ‘char**’ [-Wformat=] ... -o dvbdevice.o dvbdevice.c diseqc.c: In member function ‘bool cDiseqc::Parse(const char*)’: diseqc.c:117:102: warning: format ‘%a’ expects argument of type ‘float*’, but argument 3 has type ‘char**’ [-Wformat=] nt fields = sscanf(s, "%a[^ ] %d %c %d %a[^\n]", &sourcebuf, &slof, &polarization, &lof, &commands);

                          ^

diseqc.c:117:102: warning: format ‘%a’ expects argument of type ‘float*’, but argument 7 has type ‘char**’ [-Wformat=] ... -o font.o font.c epg.c: In member function ‘bool tComponent::FromString(const char*)’: epg.c:35:82: warning: format ‘%a’ expects argument of type ‘float*’, but argument 6 has type ‘char**’ [-Wformat=] ... -o sources.o sources.c sources.c: In member function ‘bool cSource::Parse(const char*)’: sources.c:34:62: warning: format ‘%a’ expects argument of type ‘float*’, but argument 3 has type ‘char**’ [-Wformat=] if (2 == sscanf(s, "%a[^ ] %a[^\n]", &codeBuf, &description)) ^ sources.c:34:62: warning: format ‘%a’ expects argument of type ‘float*’, but argument 4 has type ‘char**’ [-Wformat=]

</snapp>

Well, "%a" has always been a format character for an automatically allocated string ("The GNU C Library Reference Manual", Vol. 1, Chapter 12.14.6, "Dynamically Allocating String Conversions"). Why would that change with -std=c++11?

and of course, warnings about invalid suffix on literal; C++11 requires a space between literal and string macro in dvbsubtitle.c remux.c

I've already adopted that for the next version.

Klaus