[vdr] Logs from building VDR 2.1.3 with Clang 3.4.1

2014-02-08 Thread Paul Menzel
Dear VDR folks, building VDR 2.1.3 with Clang 3.4.1 the warnings below are shown. Most warnings look like they can be ignored. Maybe you can spot something, which should be fixed. $ clang --version Debian clang version 3.4-1 (tags/RELEASE_34/final) (based on LLVM 3.4) Ta

Re: [vdr] Logs from building VDR 2.1.3 with Clang 3.4.1

2014-02-08 Thread Klaus Schmidinger
On 08.02.2014 09:59, Paul Menzel wrote: Dear VDR folks, building VDR 2.1.3 with Clang 3.4.1 the warnings below are shown. Most warnings look like they can be ignored. Maybe you can spot something, which should be fixed. $ clang --version Debian clang version 3.4-1 (tags/RELE

Re: [vdr] Logs from building VDR 2.1.3 with Clang 3.4.1

2014-02-08 Thread Tony Houghton
On Sat, 08 Feb 2014 13:51:10 +0100 Klaus Schmidinger wrote: > > channels.c:45:119: warning: data argument not used by format > > string [-Wformat-extra-args] > >snprintf(buffer, sizeof(buffer), rid ? "%s-%d-%d-%d-%d" : > > "%s-%d-%d-%d", *cSource::ToString(source), nid, tid

Re: [vdr] Logs from building VDR 2.1.3 with Clang 3.4.1

2014-02-08 Thread Klaus Schmidinger
On 08.02.2014 14:34, Tony Houghton wrote: On Sat, 08 Feb 2014 13:51:10 +0100 Klaus Schmidinger wrote: channels.c:45:119: warning: data argument not used by format string [-Wformat-extra-args] snprintf(buffer, sizeof(buffer), rid ? "%s-%d-%d-%d-%d" : "%s-%d-%d-%d", *cSou

Re: [vdr] Logs from building VDR 2.1.3 with Clang 3.4.1

2014-02-08 Thread Tony Houghton
On Sat, 08 Feb 2014 15:17:09 +0100 Klaus Schmidinger wrote: > On 08.02.2014 14:34, Tony Houghton wrote: > > > > The warning is justified, because if rid is 0 it's still there as an > > argument, but just happens to have a value of 0. I think you can make > > snprintf "consume" it without printing

Re: [vdr] Logs from building VDR 2.1.3 with Clang 3.4.1

2014-02-08 Thread Klaus Schmidinger
On 08.02.2014 16:10, Tony Houghton wrote: On Sat, 08 Feb 2014 15:17:09 +0100 Klaus Schmidinger wrote: On 08.02.2014 14:34, Tony Houghton wrote: The warning is justified, because if rid is 0 it's still there as an argument, but just happens to have a value of 0. I think you can make snprintf