Re: [PATCH] Fix typos in comments throughout source tree

[Jonathan Wakely <jwakely at redhat dot com>]

On 03/04/17 20:06 +0100, Jonathan Wakely wrote:
> On 03/04/17 11:48 -0700, Mike Stump wrote:
> > [ dup, I sent the wrong from the wrong email address, you might not see it because of that ]
> >
> > > On Apr 3, 2017, at 11:47 AM, Mike Stump <mrs@mrs.kithrup.com> wrote:
> > >
> > > On Apr 3, 2017, at 10:53 AM, Jonathan Wakely <jwakely@redhat.com> wrote:
> > > > This fixes lots of typos throughout the tree. Can this be committed to
> > > > trunk as obvious?
> > >
> > > Yes, it can.  However, I like it when even this patch is reviewed.  I reviewed it, looks good.
> > >
> > > You missed initialisers -> initializers.
>
> Ah yes, in a couple of places.
>
> I think the original email about these typos only covered words
> beginning with the letters A to E, so there are bound to be lots more
> typos unfixed :-)
>
>
> > > Not sure I like datastructures, should be data structures?
>
> Will do. I also forgot to do something about the "accestor ???" part
>
>  We provide accestor to the inline_summary datastructure and
>  basic logic updating the parameters when inlining is performed.
>
> I don't know what that's supposed to say. "access"? "accessors"?

I went for "access".

Thanks for the review.


> > > figure out that poth parameters  -> figure out that both parameters
>
> I'll fix that.
>
> > > > I've bootstrapped the C and C++ compilers, but not Ada, and of course
> > > > not compiled all the target changes in gcc/config/*
> > > >
> > > > Is one big commit OK, or one per-ChangeLog? Or something else?
> > >
> > > I'd just do one big commit.
>
> Will do, thanks.