Re: [Linuxptp-devel] [PATCH 2/3] Set printf format attribute for print().

2014-02-12 Thread Richard Cochran
On Wed, Feb 12, 2014 at 10:21:46AM +0100, Miroslav Lichvar wrote: > > Should I respin the series with this change or will you amend it? I'll fix it up. Thanks, Richard -- Android apps run on BlackBerry 10 Introducing th

Re: [Linuxptp-devel] [PATCH 2/3] Set printf format attribute for print().

2014-02-12 Thread Miroslav Lichvar
On Tue, Feb 11, 2014 at 09:40:01PM +0100, Richard Cochran wrote: > With this series applied, this patch triggers a new warning on one of > my test builds (at least I assume it was this patch). > > cc1: warnings being treated as errors > /home/richard/git/linuxptp/clockadj.c: In function 'sysclk_se

Re: [Linuxptp-devel] [PATCH 2/3] Set printf format attribute for print().

2014-02-11 Thread Richard Cochran
On Tue, Feb 11, 2014 at 02:38:34PM +0100, Miroslav Lichvar wrote: > This allows gcc to check format strings used in pr_* calls. With this series applied, this patch triggers a new warning on one of my test builds (at least I assume it was this patch). cc1: warnings being treated as errors /home/r

[Linuxptp-devel] [PATCH 2/3] Set printf format attribute for print().

2014-02-11 Thread Miroslav Lichvar
This allows gcc to check format strings used in pr_* calls. Signed-off-by: Miroslav Lichvar --- print.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/print.h b/print.h index 56b2104..f5ed248 100644 --- a/print.h +++ b/print.h @@ -25,6 +25,9 @@ #define PRINT_LEVEL_MIN LOG_EMERG #define