Re: [PATCH] Remove warning 'format not a string literal' in libsvn_client

[Daniel Näslund]
> if (! original_value)
> - header_fmt = _("Added: %s%s");
> + SVN_ERR(file_printf_from_utf8(file, encoding, "Added: %s%s",
> + propchange->name, APR_EOL_STR));

Julian talked about this code already, but just to point out: your
change is incorrect, because it removes the _() functionality, i.e.,
string localisation. You have to be careful about these things.

...Indeed, we use quite a lot of _() with printf format strings (see
all the references to 'c-format' in tools/po/po-update.sh). Why
aren't these _all_ warnings with your compiler?

-- 
Peter Samuelson | org-tld!p12n!peter | http://p12n.org/
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2414768