Re: "format not a string literal" warnings

On Mon, Nov 10, 2008 at 06:35:09AM -0800, Greg Stein wrote:
> When I've turned up the error reporting, I've seen those warnings,
> too. It also happens when we do something like:
>
> printf(timedate_format, arg1, arg2);
>
> You can't do much about those. But for constructs like you pointed
> out: definitely, let's add the "%s". It is much safer to use that over
> the lifetime of the code. Who knows where the argument might come from
> next week.

+1

Stefan

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-11-11 00:26:27 CET