Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

re: CVS commit: src/share/mk



   matthew green wrote:
   >    
   >    Module Name:    src
   >    Committed By:   elad
   >    Date:           Sun Oct  8 17:16:51 UTC 2006
   >    
   >    Modified Files:
   >            src/share/mk: bsd.README bsd.sys.mk
   >    
   >    Log Message:
   >    PR/28368: Peter Postma: unrecognized option `-Wnetbsd-format-audit'
   >    
   >    Apply patch from rivo nurges in #NetBSD-code, thanks!
   > 
   > 
   > hmmm, i'm not sure this is the right solution.  i think we want to
   > re-port this code to newer GCC rather than abandon it entirely.
   > 
   > please back this out, re-open the PR and assign it to me.
   
   This was discussed as not being useful anymore and okay'd by several
   developers. I don't mind re-opening it and assigning to you, but out of
   curiosity I must ask: what are you going to "audit" with this thing?


the netbsd source tree.  this option increase the scope of what
GCC checks for printf-like format strings.  it was added to netbsd
gcc by sommerfeld to address cases where the format string is not
literally part of the function call, eg:

        const char foo[]="%s %d";
        printf(foo, 1, "2");

see, eg, src/gnu/dist/gcc/c-common.c revs 1.9 & 1.10 (note this
file is in the Attic...)


hmm, it occurs to me that this may be what GCC does now anyway.
in any case, please re-open and assign to me, and i will deal
with it.

thanks.


.mrg.



Home | Main Index | Thread Index | Old Index