diff mbox

PING PATCH: break lines in announce_function

Message ID 20120516124641.GB21919@ours.starynkevitch.net
State New
Headers show

Commit Message

Basile Starynkevitch May 16, 2012, 12:46 p.m. UTC
Hello All,

I am pinging the patch http://gcc.gnu.org/ml/gcc-patches/2012-02/msg00474.html
below for trunk svn 187587
############################################################# patch
########################################################### gcc/ChangeLog entry
2011-05-16  Basile Starynkevitch  <basile@starynkevitch.net>

	* toplev.c (announce_function): Output newline periodically.
###########################################################

Ok for trunk?

Cheers

Comments

Richard Biener May 16, 2012, 1:02 p.m. UTC | #1
On Wed, May 16, 2012 at 2:46 PM, Basile Starynkevitch
<basile@starynkevitch.net> wrote:
> Hello All,
>
> I am pinging the patch http://gcc.gnu.org/ml/gcc-patches/2012-02/msg00474.html
> below for trunk svn 187587
> ############################################################# patch
> Index: gcc/toplev.c
> ===================================================================
> --- gcc/toplev.c        (revision 187587)
> +++ gcc/toplev.c        (working copy)
> @@ -229,6 +229,12 @@
>  {
>   if (!quiet_flag)
>     {
> +

Seems pretty arbitrary and with bogus whitespace here ^^^

So - why?  I like it the way it is.

Richard.

> +      static long count;
> +      count++;
> +      if (count % 8 == 0)
> +        putc('\n', stderr);
> +
>       if (rtl_dump_and_exit)
>        fprintf (stderr, "%s ",
>                 identifier_to_locale (IDENTIFIER_POINTER (DECL_NAME (decl))));
> ########################################################### gcc/ChangeLog entry
> 2011-05-16  Basile Starynkevitch  <basile@starynkevitch.net>
>
>        * toplev.c (announce_function): Output newline periodically.
> ###########################################################
>
> Ok for trunk?
>
> Cheers
> --
> Basile STARYNKEVITCH         http://starynkevitch.net/Basile/
> email: basile<at>starynkevitch<dot>net mobile: +33 6 8501 2359
> 8, rue de la Faiencerie, 92340 Bourg La Reine, France
> *** opinions {are only mines, sont seulement les miennes} ***
diff mbox

Patch

Index: gcc/toplev.c
===================================================================
--- gcc/toplev.c	(revision 187587)
+++ gcc/toplev.c	(working copy)
@@ -229,6 +229,12 @@ 
 {
   if (!quiet_flag)
     {
+
+      static long count;
+      count++;
+      if (count % 8 == 0)
+        putc('\n', stderr);
+
       if (rtl_dump_and_exit)
 	fprintf (stderr, "%s ",
 		 identifier_to_locale (IDENTIFIER_POINTER (DECL_NAME (decl))));