This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[C++ PATCH] Fix 3 diagnostic whitespace issues
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Jason Merrill <jason at redhat dot com>, Nathan Sidwell <nathan at acm dot org>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Mon, 13 Feb 2017 20:50:16 +0100
- Subject: [C++ PATCH] Fix 3 diagnostic whitespace issues
- Authentication-results: sourceware.org; auth=none
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
Hi!
And lastly this patch fixes 3 diagnostics issues found by that script
in the C++ FE. Bootstrapped/regtested on x86_64-linux and i686-linux,
ok for trunk?
2017-02-13 Jakub Jelinek <jakub@redhat.com>
* init.c (warn_placement_new_too_small): Add missing space in
diagnostics.
* parser.c (cp_parser_oacc_declare): Likewise.
* mangle.c (maybe_check_abi_tags): Likewise.
--- gcc/cp/init.c.jj 2017-01-31 09:26:02.000000000 +0100
+++ gcc/cp/init.c 2017-02-13 11:39:57.145924623 +0100
@@ -2606,7 +2606,7 @@ warn_placement_new_too_small (tree type,
exact_size ?
"placement new constructing an object of type %qT "
"and size %qwu in a region of type %qT and size %qwi"
- : "placement new constructing an object of type %qT"
+ : "placement new constructing an object of type %qT "
"and size %qwu in a region of type %qT and size "
"at most %qwu",
type, bytes_need, TREE_TYPE (oper),
--- gcc/cp/parser.c.jj 2017-02-10 21:35:26.000000000 +0100
+++ gcc/cp/parser.c 2017-02-13 11:41:21.162795860 +0100
@@ -36177,7 +36177,7 @@ cp_parser_oacc_declare (cp_parser *parse
|| !DECL_EXTERNAL (decl)))
{
error_at (loc,
- "%qD must be a global variable in"
+ "%qD must be a global variable in "
"%<#pragma acc declare link%>",
decl);
error = true;
--- gcc/cp/mangle.c.jj 2017-01-19 16:58:17.000000000 +0100
+++ gcc/cp/mangle.c 2017-02-13 11:40:41.354330685 +0100
@@ -4274,8 +4274,9 @@ maybe_check_abi_tags (tree t, tree for_d
for_decl, flag_abi_version, warn_abi_version);
else
warning_at (DECL_SOURCE_LOCATION (t), OPT_Wabi,
- "the mangled name of the initialization guard variable for"
- "%qD changes between -fabi-version=%d and -fabi-version=%d",
+ "the mangled name of the initialization guard variable "
+ "for %qD changes between -fabi-version=%d and "
+ "-fabi-version=%d",
t, flag_abi_version, warn_abi_version);
}
}
Jakub