Patchwork Fix up English grammar in an error msg

login
register
mail settings
Submitter Marek Polacek
Date Jan. 18, 2013, 11:38 p.m.
Message ID <20130118233830.GC8948@redhat.com>
Download mbox | patch
Permalink /patch/213760/
State New
Headers show

Comments

Marek Polacek - Jan. 18, 2013, 11:38 p.m.
As the subject says.  Ok for trunk?

2013-01-19  Marek Polacek  <polacek@redhat.com>

	* cfgloop.c (verify_loop_structure): Fix up grammar.


	Marek
Gerald Pfeifer - Jan. 19, 2013, 3:27 p.m.
On Sat, 19 Jan 2013, Marek Polacek wrote:
> As the subject says.  Ok for trunk?
> 
> 2013-01-19  Marek Polacek  <polacek@redhat.com>
> 
> 	* cfgloop.c (verify_loop_structure): Fix up grammar.

You generally don't have to ask for approval for changes like this.

Strictly speaking trunk is in "regression fixes and docs only"
mode, and this was last touched in r101561 / 2005-07-03, but...

Gerald
Mike Stump - Jan. 20, 2013, 8:16 p.m.
On Jan 19, 2013, at 7:27 AM, Gerald Pfeifer <gerald@pfeifer.com> wrote:
> On Sat, 19 Jan 2013, Marek Polacek wrote:
>> As the subject says.  Ok for trunk?
>> 
>> 2013-01-19  Marek Polacek  <polacek@redhat.com>
>> 
>> 	* cfgloop.c (verify_loop_structure): Fix up grammar.
> 
> You generally don't have to ask for approval for changes like this.
> 
> Strictly speaking trunk is in "regression fixes and docs only"
> mode, and this was last touched in r101561 / 2005-07-03, but…

:-)  This is safe polish and we have plenty of time…  I think it is good for a .0 release, but not a .1 release.

Patch

--- gcc/cfgloop.c.mp	2013-01-19 00:34:50.857710526 +0100
+++ gcc/cfgloop.c	2013-01-19 00:35:03.310764236 +0100
@@ -1344,7 +1344,7 @@  verify_loop_structure (void)
       for (j = 0; j < loop->num_nodes; j++)
 	if (!flow_bb_inside_loop_p (loop, bbs[j]))
 	  {
-	    error ("bb %d do not belong to loop %d",
+	    error ("bb %d does not belong to loop %d",
 		   bbs[j]->index, loop->num);
 	    err = 1;
 	  }