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: Patch for %E formatting for C


"Joseph S. Myers" <jsm@polyomino.org.uk> writes:

| This patch fixes the problem of %E formats for C including a bogus
| "({anonymous})" in the output.  (I don't think the PR requested in
| <http://gcc.gnu.org/ml/gcc-patches/2004-09/msg02087.html> was ever
| opened.)
| 
| There is still a small patch to pretty-print.c in bug 16119 which
| needs review by someone suitably familiar with the code; the bug
| doesn't explain what was meant by "exposed glitches" or give a
| testcase for it.

For some reaons, I missed that report.

I'm not sure I understand why the reporter thinks it is a bug for
pp_base_clear_output_area to reset the line lneght to zero.  I suspect
that the origin of his problem is elsewhere.

| -   %E: An expression,
| +   %E: an identifier or expression,

Also, %E really is for expressions, and identifiers are also
expressions.  There is no need to introduce artifical distinction and
comment #4 is right.

-- Gaby


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