[PATCH] Fix formatting of init_target_chars calls

Jakub Jelinek jakub@redhat.com
Thu Feb 1 13:35:00 GMT 2007


Hi!

While looking at PR middle-end/30473, I have noticed a formatting glitch.
Is this something we want to fix on the trunk?

2007-02-01  Jakub Jelinek  <jakub@redhat.com>

	* builtins.c (expand_builtin_printf, expand_builtin_fprintf,
	expand_builtin_sprintf, fold_builtin_sprintf,
	maybe_emit_sprintf_chk_warning, fold_builtin_sprintf_chk,
	fold_builtin_snprintf_chk, fold_builtin_printf,
	fold_builtin_fprintf): Fix formatting of init_target_chars call.

--- gcc/builtins.c.jj	2007-02-01 11:03:12.000000000 +0100
+++ gcc/builtins.c	2007-02-01 14:20:20.000000000 +0100
@@ -5108,7 +5108,7 @@ expand_builtin_printf (tree exp, rtx tar
   if (fmt_str == NULL)
     return 0;
 
-  if (!init_target_chars())
+  if (!init_target_chars ())
     return 0;
 
   /* If the format specifier was "%s\n", call __builtin_puts(arg).  */
@@ -5224,7 +5224,7 @@ expand_builtin_fprintf (tree exp, rtx ta
   if (fmt_str == NULL)
     return 0;
 
-  if (!init_target_chars())
+  if (!init_target_chars ())
     return 0;
 
   /* If the format specifier was "%s", call __builtin_fputs(arg,fp).  */
@@ -5316,7 +5316,7 @@ expand_builtin_sprintf (tree arglist, rt
   if (fmt_str == NULL)
     return 0;
 
-  if (!init_target_chars())
+  if (!init_target_chars ())
     return 0;
 
   /* If the format doesn't contain % args or %%, use strcpy.  */
@@ -10562,7 +10563,7 @@ fold_builtin_sprintf (tree arglist, int 
   call = NULL_TREE;
   retval = NULL_TREE;
 
-  if (!init_target_chars())
+  if (!init_target_chars ())
     return 0;
 
   /* If the format doesn't contain % args or %%, use strcpy.  */
@@ -10901,7 +10910,7 @@ maybe_emit_sprintf_chk_warning (tree exp
   if (fmt_str == NULL)
     return;
 
-  if (!init_target_chars())
+  if (!init_target_chars ())
     return;
 
   /* If the format doesn't contain % args or %%, we know its size.  */
@@ -11363,7 +11372,7 @@ fold_builtin_sprintf_chk (tree arglist, 
 
   len = NULL_TREE;
 
-  if (!init_target_chars())
+  if (!init_target_chars ())
     return 0;
 
   /* Check whether the format is a literal string constant.  */
@@ -11488,7 +11497,7 @@ fold_builtin_snprintf_chk (tree arglist,
 	return 0;
     }
 
-  if (!init_target_chars())
+  if (!init_target_chars ())
     return 0;
 
   /* Only convert __{,v}snprintf_chk to {,v}snprintf if flag is 0
@@ -11572,7 +11581,7 @@ fold_builtin_printf (tree fndecl, tree a
       fn_puts = implicit_built_in_decls[BUILT_IN_PUTS];
     }
 
-  if (!init_target_chars())
+  if (!init_target_chars ())
     return 0;
 
   if (strcmp (fmt_str, target_percent_s) == 0 || strchr (fmt_str, target_percent) == NULL)
@@ -11733,7 +11742,7 @@ fold_builtin_fprintf (tree fndecl, tree 
       fn_fputs = implicit_built_in_decls[BUILT_IN_FPUTS];
     }
 
-  if (!init_target_chars())
+  if (!init_target_chars ())
     return 0;
 
   /* If the format doesn't contain % args or %%, use strcpy.  */

	Jakub



More information about the Gcc-patches mailing list