If it's just a coding style patch, then I think it may be ill-advised. If you look at
how editres is implemented, most of the res_labels have formatting characters in them.
See for instance
The lines containing "label" are initializing the res_labels array.
On Fri, Dec 21, 2012 at 2:28 AM, Mats Lidell <matsl(a)xemacs.org> wrote:
I think it is just a coding style patch. It is bad practice to have
format string from a variable in case you'll get a % in there
somehow. So better guard against that an use a simple "%s" as
format. Maybe compilers even warn if you don't supply at least three
args to sprintf!?
I've never seen such a warning.
XEmacs-Beta mailing list