[PATCH] Two small translation related tweaks

Jakub Jelinek jakub@redhat.com
Fri Feb 24 20:52:00 GMT 2017


Hi!

After hinting Marek to use const char *const in his G_ addition patch,
I've grepped for other similar cases.  The number_of_iterations_exit
code doesn't make sense after we ended up (change from 6.x) with
only a single wording, and in the C++ FE the addition of const allows
-Wformat to see those format strings.

Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?

2017-02-24  Jakub Jelinek  <jakub@redhat.com>

	* tree-ssa-loop-niter.c (number_of_iterations_exit): Simplify warning.
cp/
	* call.c (build_op_delete_call): Make msg1 and msg2 const.

--- gcc/tree-ssa-loop-niter.c.jj	2017-01-24 23:29:05.000000000 +0100
+++ gcc/tree-ssa-loop-niter.c	2017-02-24 15:34:24.119941951 +0100
@@ -2362,13 +2362,9 @@ number_of_iterations_exit (struct loop *
     return true;
 
   if (warn)
-    {
-      const char *wording;
-
-      wording = N_("missed loop optimization, the loop counter may overflow");
-      warning_at (gimple_location_safe (stmt),
-		  OPT_Wunsafe_loop_optimizations, "%s", gettext (wording));
-    }
+    warning_at (gimple_location_safe (stmt),
+		OPT_Wunsafe_loop_optimizations,
+		"missed loop optimization, the loop counter may overflow");
 
   return false;
 }
--- gcc/cp/call.c.jj	2017-02-23 08:40:30.000000000 +0100
+++ gcc/cp/call.c	2017-02-24 15:31:49.041955772 +0100
@@ -6225,10 +6225,10 @@ build_op_delete_call (enum tree_code cod
 	 allocation function, the program is ill-formed."  */
       if (second_parm_is_size_t (fn))
 	{
-	  const char *msg1
+	  const char *const msg1
 	    = G_("exception cleanup for this placement new selects "
 		 "non-placement operator delete");
-	  const char *msg2
+	  const char *const msg2
 	    = G_("%qD is a usual (non-placement) deallocation "
 		 "function in C++14 (or with -fsized-deallocation)");
 

	Jakub



More information about the Gcc-patches mailing list