PR 62170 describes a problem with how the quoting in pp_format interacts with the "aka" information for typedefs in %qT for the C family of frontends, and also now for %qH and %qI in the C++ frontend: we print: 'Py_ssize_t* {aka int*}' ^^^^^^^^^^^^^^^^^^^^^^ colorized as "quote" i.e. '[START_COLOR]Py_ssize_t* {aka int*}[END_COLOR]' when we should print: 'Py_ssize_t*' {aka 'int*'} ^^^^^^^^^^^ ^^^^ colorized as "quote" i.e. '[START_COLOR]Py_ssize_t*[END_COLOR]' {aka '[START_COLOR]int*[END_COLOR]'} where the opening and closing quote characters and colorization are added by the 'q' handling within pp_format. This patch fixes the quoting by updating the %T handling in C and C++ and the %H/%I handling in C++ to insert the quoting appropriately. It converts the "quote" param of the pp_format_decoder callback from bool to bool *, allowing for the %T and %H/%I handlers to write false back to it, to avoid printing the closing quote for the cases like the above where the trailing closing quote isn't needed. It introduces pp_begin_quote/pp_end_quote to simplify this. These take a "bool show_color", rather than using "pp_show_color (pp)" since cxx_pp's pp_show_color isn't currently initialized (since cxx_initialize_diagnostics happens before diagnostic_color_init). gcc/c/ChangeLog: PR c++/62170 * c-objc-common.c (c_tree_printer): Convert penultimate param from bool to bool *. Within '%T' handling, if showing an "aka", use "quoted" param to add appropriate quoting. gcc/cp/ChangeLog: PR c++/62170 * error.c (type_to_string): Add leading comment. Add params "postprocessed", "quote", and "show_color", using them to fix quoting of the "aka" for types involving typedefs. (arg_to_string): Update for new params to type_to_string. (cxx_format_postprocessor::handle): Likewise. (cp_printer): Convert penultimate param from bool to bool *. Update call to type_to_string and calls to defer_phase_2_of_type_diff. gcc/fortran/ChangeLog: PR c++/62170 * error.c (gfc_notify_std): Convert "quoted" param from bool to bool *. gcc/ChangeLog: PR c++/62170 * pretty-print.c (pp_format): Move quoting implementation to pp_begin_quote and pp_end_quote. Update pp_format_decoder call to pass address of "quote" local. (pp_begin_quote): New function. (pp_end_quote): New function. * pretty-print.h (printer_fn): Convert penultimate param from bool to bool *. (pp_begin_quote): New decl. (pp_end_quote): New decl. * tree-diagnostic.c (default_tree_printer): Convert penultimate param from bool to bool *. * tree-diagnostic.h (default_tree_printer): Likewise. gcc/testsuite/ChangeLog: PR c++/62170 * g++.dg/diagnostic/aka1.C: Update expected error messages to reflect fixes to quoting. * g++.dg/diagnostic/aka2.C: New test case. * g++.dg/parse/error55.C: Update expected error messages to reflect fixes to quoting. * gcc.dg/diag-aka-1.c: Likewise. * gcc.dg/diag-aka-2.c: New test case. * gcc.dg/pr13804-1.c: Update expected error messages to reflect fixes to quoting. * gcc.dg/pr56980.c: Likewise. * gcc.dg/pr65050.c: Likewise. * gcc.dg/redecl-14.c: Likewise. * gcc.dg/utf16-4.c Likewise. * gcc.target/i386/sse-vect-types.c (__m128d): Likewise. * obj-c++.dg/invalid-type-1.mm: Likewise. * objc.dg/proto-lossage-4.m: Likewise. From-SVN: r255076
Name |
Last commit
|
Last update |
---|---|---|
.. | ||
aka1.C | Loading commit data... | |
aka2.C | Loading commit data... | |
bad-binary-ops.C | Loading commit data... | |
bindings1.C | Loading commit data... | |
bitfld1.C | Loading commit data... | |
bitfld2.C | Loading commit data... | |
disable.C | Loading commit data... | |
duplicate1.C | Loading commit data... | |
expr1.C | Loading commit data... | |
friend1.C | Loading commit data... | |
inhibit-warn-1.C | Loading commit data... | |
inhibit-warn-2.C | Loading commit data... | |
member-decl-1.C | Loading commit data... | |
method1.C | Loading commit data... | |
nested-name-1.C | Loading commit data... | |
param-type-mismatch.C | Loading commit data... | |
parm1.C | Loading commit data... | |
pr13981.C | Loading commit data... | |
pr58979.C | Loading commit data... | |
pr59838.C | Loading commit data... | |
pr60862.C | Loading commit data... | |
pr61004.C | Loading commit data... | |
pr70105.C | Loading commit data... | |
pr71075.C | Loading commit data... | |
pr72803.C | Loading commit data... | |
pr77949.C | Loading commit data... | |
pr79300.C | Loading commit data... | |
pr79304.C | Loading commit data... | |
pr80473.C | Loading commit data... | |
ref1.C | Loading commit data... | |
ref2.C | Loading commit data... | |
string-literal-concat.C | Loading commit data... | |
unclosed-extern-c.C | Loading commit data... | |
unclosed-function.C | Loading commit data... | |
unclosed-namespace.C | Loading commit data... | |
unclosed-struct.C | Loading commit data... | |
variadic1.C | Loading commit data... | |
wrong-tag-1.C | Loading commit data... |