[systemd-devel] [PATCH] Add more _printf_'s for format-nonliterals

Zbigniew Jędrzejewski-Szmek zbyszek at in.waw.pl
Sun Dec 15 14:58:37 PST 2013


On Sun, Dec 15, 2013 at 11:49:55PM +0100, Lennart Poettering wrote:
> On Sat, 14.12.13 13:32, Thomas H.P. Andersen (phomes at gmail.com) wrote:
> 
> > +_printf_(3,0)
> >  int bus_error_setfv(sd_bus_error *e, const char *name, const char
> >  *format, va_list ap) {
> 
> This one should be on the prototype, and not on the actual function
> declaration, right? (Because it is non-static unlike the others) I mean,
> otherwise it's not useful to detect incorrect calls, right?
Ooops, I looked them over but I missed that.

> (Fixed this in git)

Zbyszek


More information about the systemd-devel mailing list