[PATCH] Fix sis driver to compile with -Werror=format-security

Alan Cox alan at lxorguk.ukuu.org.uk
Tue Dec 23 10:22:04 PST 2008


> > better patch. On the other hand, if you guys feel there is nothing to be
> > fixed here I'll just keep the patch for the Mandriva package and let it go.
> 
> Change str to a macro and use that:
> 
> #define MSG_SEP "**************************************************\n"
> ...
> xf86DrvMsg(pScrn->scrnIndex, X_ERROR, MSG_SEP);
> ...
> #undef MSG_STR

GAK

Thats horrible on so many levels

1. It uglifiers the code enormously
2. You are trying to hide a compiler thing by confusing the tools, and
the tools will get smarter and work that out

Alan



More information about the xorg mailing list