Patch to fix diagnostic format warnings in cp/ dir

Kaveh R. Ghazi ghazi@caip.rutgers.edu
Wed Jul 2 13:07:00 GMT 2003


 > From: Gabriel Dos Reis <gdr@integrable-solutions.net>
 > 
 > "Kaveh R. Ghazi" <ghazi@caip.rutgers.edu> writes:
 > 
 > |  >  > From: Gabriel Dos Reis <gdr@integrable-solutions.net>
 > |  >  > 
 > |  >  > the fix is to supply the location info directly, i.e.: 
 > |  >  > 
 > |  >  >   cp_error ("%Hno specifiers", &DECL_SOURCE_LOCATION (extra_arg));
 >           ^^^^^^^^  
 > |  >  > 
 > |  >  > -- Gaby
 > |  > 
 > |  > Ok?
 > |  > 
 > |  > 2003-06-20  Kaveh R. Ghazi  <ghazi@caip.rutgers.edu>
 > | 
 > | Actually following this course yields several extra testsuite errors.
 > | E.g
 > | 
 > | [...]
 > | 
 > | Some are like this:
 > |  > g++.jason/crash10.C:8: internal compiler error: unexpected letter
 > |  >    `H' in locate_error
 > | 
 > | So the cp_*_at functions don't seems to accept %H because of the
 > | locate_error() wrapper.
 > 
 > Yes, that is why I did suggest using cp_error -- not cp_error_at -- in
 > the situation you descirbed (e.g. trailing arguments)

Well I could not find any `cp_error()' function in the sources, that's
why I thought you meant `cp_error_at()'.

Did you mean plain `error()' ?

		--Kaveh
--
Kaveh R. Ghazi			ghazi@caip.rutgers.edu



More information about the Gcc-patches mailing list