]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
Fix up some further cases of missing or extraneous spaces in diagnostics
authorJakub Jelinek <jakub@redhat.com>
Sat, 22 Mar 2025 07:39:38 +0000 (08:39 +0100)
committerJakub Jelinek <jakub@gcc.gnu.org>
Sat, 22 Mar 2025 07:40:56 +0000 (08:40 +0100)
Given the recent PR119406 I've tried to grep for concatenated string
literals without space at the end of one line and at the start of next line,
unless it was obviously intentional.
Furthermore, I've then looked through gcc.pot looking for 2 adjacent spaces
and looking back if that wasn't the case of "something "
" with spaces at both sides".

Here is the result from that.

I think just the c.opt change needs an explanation, the "" in the
description is simply eaten up somewhere during the option processing and
gcc -v --help before this patch was displaying
  -Wdeprecated-literal-operator Warn about deprecated space between  and suffix in a user-defined literal operator.

2025-03-22  Jakub Jelinek  <jakub@redhat.com>

gcc/
* gimplify.cc (warn_switch_unreachable_and_auto_init_r): Add missing
space in the middle of diagnostics.
* tree-vect-stmts.cc (vectorizable_load): Add missing space in the
middle of debug dump message.
* sym-exec/sym-exec-state.cc (state::check_args_compatibility):
Likewise.
gcc/c-family/
* c.opt (Wdeprecated-literal-operator): Use \"\" rather than ""
in option description.
gcc/fortran/
* resolve.cc (resolve_procedure_expression): Remove extraneous space
from the middle of diagnostics.

gcc/c-family/c.opt
gcc/fortran/resolve.cc
gcc/gimplify.cc
gcc/sym-exec/sym-exec-state.cc
gcc/tree-vect-stmts.cc

index 6c6922ab2018f65997faeb8e34db2428529a9039..ad216983126e22c1d278ef06366138fc2b8f6e97 100644 (file)
@@ -666,7 +666,7 @@ type and the other is of a floating-point type.
 
 Wdeprecated-literal-operator
 C++ ObjC++ Var(warn_deprecated_literal_operator) Warning
-Warn about deprecated space between "" and suffix in a user-defined literal operator.
+Warn about deprecated space between \"\" and suffix in a user-defined literal operator.
 
 Wdeprecated-non-prototype
 C ObjC Var(warn_deprecated_non_prototype) Init(-1) Warning
index e016865a6bb42d6fdbae9c09f47e97db1c056341..cf9318ff763c5175bd5943d4100706f20a05857f 100644 (file)
@@ -1986,7 +1986,7 @@ resolve_procedure_expression (gfc_expr* expr)
   if (is_illegal_recursion (sym, gfc_current_ns))
     {
       if (sym->attr.use_assoc && expr->symtree->name[0] == '@')
-       gfc_warning (0, "Non-RECURSIVE procedure %qs from module %qs is "
+       gfc_warning (0, "Non-RECURSIVE procedure %qs from module %qs is"
                     " possibly calling itself recursively in procedure %qs. "
                     " Declare it RECURSIVE or use %<-frecursive%>",
                     sym->name, sym->module, gfc_current_ns->proc_name->name);
index 244c4ba4b63716d4ee12fa49dc86ba520f166fc6..354c3d663e7e38122e2d43ba8c057c06d12cc5ef 100644 (file)
@@ -2395,7 +2395,7 @@ warn_switch_unreachable_and_auto_init_r (gimple_stmt_iterator *gsi_p,
          const char *var_name_str = TREE_STRING_POINTER (var_name);
 
          warning_at (gimple_location (stmt), OPT_Wtrivial_auto_var_init,
-                     "%qs cannot be initialized with"
+                     "%qs cannot be initialized with "
                      "%<-ftrivial-auto-var_init%>",
                      var_name_str);
          break;
index fb7495a2c56bd849e8570ecac525591677822b22..d5cbb73694b0666bdb241d351d4a8525d4ab7f28 100644 (file)
@@ -265,7 +265,7 @@ state::check_args_compatibility (tree arg1, tree arg2, tree dest)
           || TREE_CODE (arg2) == INTEGER_CST))
     {
       if (dump_file && (dump_flags & TDF_DETAILS))
-       fprintf (dump_file, "Sym-Exec: Incompatible destination"
+       fprintf (dump_file, "Sym-Exec: Incompatible destination "
                            "and argument sizes.\n");
 
       return false;
index 17e3b1db894a0a8b6350d44f85a4a512498e2deb..813b2b77951158462800a3db6399f07842d0febc 100644 (file)
@@ -12576,7 +12576,7 @@ vectorizable_load (vec_info *vinfo,
 
                  if (dump_enabled_p ())
                    dump_printf_loc (MSG_NOTE, vect_location,
-                                    "vect_model_load_cost:"
+                                    "vect_model_load_cost: "
                                     "strided group_size = %d .\n",
                                     group_size);
                }