Patch to add ATTRIBUTE_PRINTF to errors.h

Gabriel Dos Reis gdr@integrable-solutions.net
Wed May 21 00:46:00 GMT 2003


"Kaveh R. Ghazi" <ghazi@caip.rutgers.edu> writes:

| The comment in errors.h notwithstanding, all of the variadic functions
| in errors.c should be using ATTRIBUTE_PRINTF since they all simply
| pass along to vfprintf.

That is not true.  Many non-printf format specifiers are used, see
usage in the C++ front-end.  

The trend is that we should also use them in the C front-end.

| The errors.h file is mostly used by generator programs, but there are

The errors.h file should just go away in favor of diagnostic.h. 

| a few cases like rtl.c in which it is used with both generator
| programs and the gcc backend which relies on the extended functions in
| diagnostic.c.  I would say that these dual use files >must not< use
| any of the extensions in the diagnostic framework because if it did
| then when it is linked with a generator program it won't work!

That is not the right way to proceed.  A better way is to strip
diagnostic.[hc] of the tree thingies and make it available for use by
other parts.  There is no point in duplicating functionalities ad infinitum.

| So this patch enforces that dictum.  Bootstrapped on
| sparc-sun-solaris2.7, no new warnings.
| 
| Ok for mainline?

No for the reasons stated above.

-- Gaby



More information about the Gcc-patches mailing list