This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: add %J for diagnostics


Richard Henderson <rth@twiddle.net> writes:

| For technical reasons, mainline and tree-ssa branch differ in how
| line number information is stored.  Unfortunately, this difference
| is exposed enough places that it's making merging difficult.
| 
| I've been working on patches to reduce this interface difference,
| but in the process I noticed that our current diagnostic idiom:
| 
| 	error("%Hstuff", &DECL_SOURCE_LOCATION (decl), ...)
| 
| is needlessly verbose.  Indeed, is the bulk of the merge problem.
| 
| I've now streamlined this to
| 
| 	error("%Jstuff", decl, ...)
| 
| which is, as you can see from the patch, much more readable and
| managable.

RTH,  

Please hold on this patch.  The C++ front-end already has a
functionality similar to that:  It is named cp_error_at/cp_wraning_at.
If we're going to implement that for all front-end, I'd rather we make
C++'s functionality avaliable for all front-ends.

-- Gaby


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]