Patchwork List of typos.

login
register
mail settings
Submitter Georg-Johann Lay
Date July 7, 2013, 5:54 p.m.
Message ID <51D9AB5A.8000303@gcc.gnu.org>
Download mbox | patch
Permalink /patch/257353/
State New
Headers show

Comments

Georg-Johann Lay - July 7, 2013, 5:54 p.m.
Ondrej Bilka schrieb:

> http://kam.mff.cuni.cz/~ondra/gcc_misspell.patch

This is wrong:

@@ -10834,7 +10834,7 @@ avr_convert_to_type (tree type, tree expr)
     XOP[2]  # Bytes to copy

     Return TRUE  if the expansion is accomplished.
-   Return FALSE if the operand compination is not supported.  */
+   Return FALSE if the operand compilation is not supported.  */

Should be "combination" not "compilation".


index 406617f..c7a7f7b 100644

-/* Expand SYMBOL into its corresponding far-addresse symbol.
+/* Expand SYMBOL into its corresponding far-addressee symbol.

Is this correct or should be "far-address"?


@@ -5336,7 +5336,7 @@ pa_print_operand (FILE *file, rtx x, int code)
  		   && GET_CODE (XEXP (XEXP (x, 0), 1)) == REG)
  	    {
  	      /* Because the REG_POINTER flag can get lost during reload,
-		 pa_legitimate_address_p canonicalizes the order of the
+		 pa_legitimate_address_p canonizes the order of the

I am not sure about this one and many others. "canonicalize" sounds ok 
to me, so does "canonicalization". "canonize" sounds odd to me. For 
example the following which should change none or both:

-/* Canonicalize the filename NAME by canonicalizing directory
+/* Canonicalize the filename NAME by canonizing directory

-   * Unique vinsn derivates from CALL, ASM, JUMP (for a while) and other
+   * Unique vinsn derivatives from CALL, ASM, JUMP (for a while) and other

Shouldn't this be "derives"?


-      /* Merge c_expres found or unify live register sets from different
+      /* Merge c_express found or unify live register sets from different

Some lines above in sel-sched.c there is "C_EXPRes". Again, change none 
or both (none seems fine here):


-  /* Test exponentials and their signs.  A buggy lexer is more likely
+  /* Test exponential and their signs.  A buggy lexer is more likely

Should both be plural or singular. "exponents" sounds good to me.


-   Roger Sayle <...@eyesopen.com>
+   Roger Sayle <...@eyes open.com>

Don't change email addresses!


Johann
Oleg Endo - July 7, 2013, 7:57 p.m.
On Sun, 2013-07-07 at 19:54 +0200, Georg-Johann Lay wrote:
> Ondrej Bilka schrieb:
> 
> > http://kam.mff.cuni.cz/~ondra/gcc_misspell.patch
> 

Below are some other hunks that look suspicious...
(trying not to duplicate the things already mentioned by others)

- * 1) It means that finalizers, and all methods calle by them,
+ * 1) It means that finalizers, and all methods callee by them,

-> 'called'


-      /*  SET_ACCESS, we want to set an explicte set of permissions, do not
+      /*  SET_ACCESS, we want to set an explicate set of permissions, do not

-> 'explicit'


    In Objective-C, there are two additional variants:
 
    foreach-statement:
-     for ( expression in expresssion ) statement
+     for ( expression in expressions ) statement
      for ( declaration in expression ) statement

Really?  I'm not so sure.


-   configury */
+   configure */

-> 'configury'


-	calll    #gettlsoff(ADDR)@(gr8, gr0)
+	call    #gettlsoff(ADDR)@(gr8, gr0)


-	calll   #gettlsoff(ADDR)@(gr8, gr0)
+	call   #gettlsoff(ADDR)@(gr8, gr0)

The original 'calll' is correct (see frv.md).


-   have the same priority - candidate is best if its dependees were
+   have the same priority - candidate is best if its dependencies were

-> 'dependees'


-   does not look at other present displacement addressings around it.
+   does not look at other present displacement addressing around it.

-> 'addressings' (as in addressing modes)


-  CM_SMALL,	/* Makes various assumpation about sizes of code and
+  CM_SMALL,	/* Makes various assumption about sizes of code and

-  CM_SMALL_PIC,	/* Makes various assumpation about sizes of code and
+  CM_SMALL_PIC,	/* Makes various assumption about sizes of code and

-> 'assumptions'


           /* If we had fewer function args than explicit template args,
-             just use the explicits.  */
+             just use the explicit.  */

-> 'explicit ones'


-	     array reference if the where and elswhere destinations
+	     array reference if the where and Elsinore destinations

-> 'elsewhere'


-   We provide accestor to the inline_summary datastructure and
+   We provide accessor to the inline_summary datastructure and

-> probably 'ancestor'


-/* The array used to find duplications in conflict vectors of
+/* The array used to find duplication in conflict vectors of

-/* Remove duplications in conflict vector of OBJ.  */
+/* Remove duplication in conflict vector of OBJ.  */

-/* Process operator duplications in insn with ID.  We do it after the
+/* Process operator duplication in insn with ID.  We do it after the

-> 'duplicates' maybe?


-   function we iterate decompressions until no data remains.  */
+   function we iterate decompression's until no data remains.  */

-> 'decompressions'


-   TODO: Make into some kind of configury-generated table.  */
+   TODO: Make into some kind of configure-generated table.  */

-> 'configury-generated'


-	 point of view as prefetch withouth dependecies will have a
+	 point of view as prefetch withouth dependencies will have a

-> missed 'without'


-   * Unique vinsn derivates from CALL, ASM, JUMP (for a while) and other
+   * Unique vinsn derivatives from CALL, ASM, JUMP (for a while) and other

-> maybe 'deviates' ?


-/* Find the set of registers that are unavailable for storing expres
+/* Find the set of registers that are unavailable for storing express

-   that are not available for storing expres while moving ORIG_OPS up on the
+   that are not available for storing express while moving ORIG_OPS up on the

-      /* Merge c_expres found or unify live register sets from different
+      /* Merge c_express found or unify live register sets from different


-> maybe 'expression' ?


-/* { dg-final { scan-assembler "calll.*#gettlsoff\\(0\\)" } } */
+/* { dg-final { scan-assembler "call.*#gettlsoff\\(0\\)" } } */

-> see above for the 'calll' frv case.  This breaks the test case.
Do not change things inside /* { } */ comments in test cases.


-/* PR target/50749: Verify that subsequent post-increment addressings
+/* PR target/50749: Verify that subsequent post-increment addressing

-/* PR target/50749: Verify that subsequent pre-decrement addressings
+/* PR target/50749: Verify that subsequent pre-decrement addressing

-/* PR target/50749: Verify that subsequent post-increment addressings
+/* PR target/50749: Verify that subsequent post-increment addressing

-/* PR target/50749: Verify that subsequent pre-decrement addressings
+/* PR target/50749: Verify that subsequent pre-decrement addressing

-> 'addressings' (as in addressing modes)


-	/* For buitins that are likely expanded to nothing or
+	/* For builtin's that are likely expanded to nothing or

-> 'builtins'


    BASE must be either a declaration or a memory reference that has correct
-   alignment ifformation embeded in it (e.g. a pre-existing one in SRA).  */
+   alignment ifformation embedded in it (e.g. a pre-existing one in SRA).  */

-> missed 'information' I guess...


-     via commutating the addition and contracting operations to zero
+     via commuting the addition and contracting operations to zero

-> 'commutating'


-	 and one can be commutated try that for the scalar stmts
+	 and one can be commuted try that for the scalar stmts

-> 'commutated'


-   http://www.ddj.com/articles/1997/9701/9701o/9701o.htm?topic=algoritms
+   http://www.ddj.com/articles/1997/9701/9701o/9701o.htm?topic=algorithms

both links do 404 anyway ;)


-   Written by Cary Coutant <ccoutant@google.com>.
+   Written by Cary Coutant <accountant@google.com>.

As Georg-Johann mentioned, do not touch email addresses.


-  /* This definition was pre-empted by a definition in a regular
+  /* This definition was pre-emptied by a definition in a regular

-  /* This definition was pre-empted by a definition in another IR file.  */
+  /* This definition was pre-emptied by a definition in another IR file.  */

-> 'preempted' ?


-  set it true anyway, since malloc copes with non-contiguities.
+  set it true anyway, since malloc copes with non-continuities.

-> 'non-contiguities'


-/* mstate bit set if continguous morecore disabled or failed */
+/* mstate bit set if continuous morecore disabled or failed */

-> 'contiguous'


-/* Chebyshev polynom coeficients for (exp(x)-1)/x */
+/* Chebyshev polynom coefficients for (exp(x)-1)/x */

-> 'polynomial'


-  // Randomize the colour, just for asthetics =)
+  // Randomize the colour, just for aesthetics =)

-> missed 'color' (in a couple of places actually)
Ondrej Bilka - July 8, 2013, 1:44 p.m.
On Sun, Jul 07, 2013 at 07:54:34PM +0200, Georg-Johann Lay wrote:
> Ondrej Bilka schrieb:
> 
> >http://kam.mff.cuni.cz/~ondra/gcc_misspell.patch
> 
> This is wrong:
> 
> @@ -10834,7 +10834,7 @@ avr_convert_to_type (tree type, tree expr)
>     XOP[2]  # Bytes to copy
> 
>     Return TRUE  if the expansion is accomplished.
> -   Return FALSE if the operand compination is not supported.  */
> +   Return FALSE if the operand compilation is not supported.  */
> 
> Should be "combination" not "compilation".
> 
> 
> index 406617f..c7a7f7b 100644
> --- a/gcc/config/frv/frv-opts.h
> +++ b/gcc/config/frv/frv-opts.h
> @@ -1,4 +1,4 @@
> -/* Frv option-handling defitions.
> +/* Frv option-handling deviations.
> 
> Should be "definitions" instead of "deviations".
> 
> 
> -/* Expand SYMBOL into its corresponding far-addresse symbol.
> +/* Expand SYMBOL into its corresponding far-addressee symbol.
> 
> Is this correct or should be "far-address"?
> 
> 
> @@ -5336,7 +5336,7 @@ pa_print_operand (FILE *file, rtx x, int code)
>  		   && GET_CODE (XEXP (XEXP (x, 0), 1)) == REG)
>  	    {
>  	      /* Because the REG_POINTER flag can get lost during reload,
> -		 pa_legitimate_address_p canonicalizes the order of the
> +		 pa_legitimate_address_p canonizes the order of the
> 
> I am not sure about this one and many others. "canonicalize" sounds
> ok to me, so does "canonicalization". "canonize" sounds odd to me.
> For example the following which should change none or both:
> 
> -/* Canonicalize the filename NAME by canonicalizing directory
> +/* Canonicalize the filename NAME by canonizing directory
> 
> -   * Unique vinsn derivates from CALL, ASM, JUMP (for a while) and other
> +   * Unique vinsn derivatives from CALL, ASM, JUMP (for a while) and other
> 
> Shouldn't this be "derives"?
> 
> 
> -      /* Merge c_expres found or unify live register sets from different
> +      /* Merge c_express found or unify live register sets from different
> 
> Some lines above in sel-sched.c there is "C_EXPRes". Again, change
> none or both (none seems fine here):
> 
oops it was bug, fixed.
> 
> -  /* Test exponentials and their signs.  A buggy lexer is more likely
> +  /* Test exponential and their signs.  A buggy lexer is more likely
> 
> Should both be plural or singular. "exponents" sounds good to me.
> 
it is also in string bellow so probably we need change or keep both.

> 
> -   Roger Sayle <...@eyesopen.com>
> +   Roger Sayle <...@eyes open.com>
> 
> Don't change email addresses!
bug, fixed
Ondrej Bilka - July 8, 2013, 2:12 p.m.
On Sun, Jul 07, 2013 at 09:57:05PM +0200, Oleg Endo wrote:
> On Sun, 2013-07-07 at 19:54 +0200, Georg-Johann Lay wrote:
> > Ondrej Bilka schrieb:
> > 
> > > http://kam.mff.cuni.cz/~ondra/gcc_misspell.patch
> >
I fixed most comments, put it here so you can diff these two files.
http://kam.mff.cuni.cz/~ondra/gcc_misspell_fixed.patch 

> 


>     BASE must be either a declaration or a memory reference that has correct
> -   alignment ifformation embeded in it (e.g. a pre-existing one in SRA).  */
> +   alignment ifformation embedded in it (e.g. a pre-existing one in SRA).  */
> 
> -> missed 'information' I guess...
>

This fixes only a-e. These are probably incomplete as I needed to
exclude lot of names that are variable names etc. 

I did selectin based on following file:
http://kam.mff.cuni.cz/~ondra/gcc_misspells 
>
> 
> -   http://www.ddj.com/articles/1997/9701/9701o/9701o.htm?topic=algoritms
> +   http://www.ddj.com/articles/1997/9701/9701o/9701o.htm?topic=algorithms
> 
> both links do 404 anyway ;)
> 
could you find what this was? I need to add another filter not touch
html which I will do probably tomorrow.

> 
> -  // Randomize the colour, just for asthetics =)
> +  // Randomize the colour, just for aesthetics =)
> 
> -> missed 'color' (in a couple of places actually)
> 
Wa- not on my list because my aspell thinks its valid.
Oleg Endo - July 8, 2013, 7:42 p.m.
On Mon, 2013-07-08 at 16:12 +0200, Ondřej Bílka wrote:
> On Sun, Jul 07, 2013 at 09:57:05PM +0200, Oleg Endo wrote:
> > On Sun, 2013-07-07 at 19:54 +0200, Georg-Johann Lay wrote:
> > > Ondrej Bilka schrieb:
> > > 
> > > > http://kam.mff.cuni.cz/~ondra/gcc_misspell.patch
> > >
> I fixed most comments, put it here so you can diff these two files.
> http://kam.mff.cuni.cz/~ondra/gcc_misspell_fixed.patch 
> 
> > 
> 
> 
> >     BASE must be either a declaration or a memory reference that has correct
> > -   alignment ifformation embeded in it (e.g. a pre-existing one in SRA).  */
> > +   alignment ifformation embedded in it (e.g. a pre-existing one in SRA).  */
> > 
> > -> missed 'information' I guess...
> >
> 
> This fixes only a-e. These are probably incomplete as I needed to
> exclude lot of names that are variable names etc. 
> 
> I did selectin based on following file:
> http://kam.mff.cuni.cz/~ondra/gcc_misspells 
> >
> > 
> > -   http://www.ddj.com/articles/1997/9701/9701o/9701o.htm?topic=algoritms
> > +   http://www.ddj.com/articles/1997/9701/9701o/9701o.htm?topic=algorithms
> > 
> > both links do 404 anyway ;)
> > 
> could you find what this was? I need to add another filter not touch
> html which I will do probably tomorrow.

It seems the original article is gone.  At least I can't find it easily
(the patch that added the link is from 2001...).  It's about Fibonacci
Heaps so there's plenty of material out there on the net.  The paper
mentioned in the comment can be found rather easily:

http://www.cs.princeton.edu/courses/archive/fall03/cs528/handouts/fibonacci%20heaps.pdf

However, maybe it's better to replace the DDJ link with this
http://en.wikipedia.org/wiki/Fibonacci_heap 

Cheers,
Oleg
Ondrej Bilka - July 10, 2013, 2 p.m.
On Mon, Jul 08, 2013 at 09:42:14PM +0200, Oleg Endo wrote:
> On Mon, 2013-07-08 at 16:12 +0200, Ondřej Bílka wrote:
> > On Sun, Jul 07, 2013 at 09:57:05PM +0200, Oleg Endo wrote:
> > > On Sun, 2013-07-07 at 19:54 +0200, Georg-Johann Lay wrote:
> > > 
> > > -   http://www.ddj.com/articles/1997/9701/9701o/9701o.htm?topic=algoritms
> > > +   http://www.ddj.com/articles/1997/9701/9701o/9701o.htm?topic=algorithms
> > > 
> > > both links do 404 anyway ;)
> > > 
> > could you find what this was? I need to add another filter not touch
> > html which I will do probably tomorrow.
> 
> It seems the original article is gone.  At least I can't find it easily
> (the patch that added the link is from 2001...).  It's about Fibonacci
> Heaps so there's plenty of material out there on the net.  The paper
> mentioned in the comment can be found rather easily:
> 
> http://www.cs.princeton.edu/courses/archive/fall03/cs528/handouts/fibonacci%20heaps.pdf
> 
> However, maybe it's better to replace the DDJ link with this
> http://en.wikipedia.org/wiki/Fibonacci_heap 
> 
I prefer fix issues one class at time. I added script that checks dead
links by extracting http addresses from sources and checking them by curl -f.
It found:

$ ./stylepp/script/stylepp_warn_dead_link

Dead link: http://anubis.dkuug.dk/jtc1/sc22/wg21/docs/cwg_closed.html#46
Dead link: http://citeseer.nj.nec.com/15361.html
Dead link: http://dwarf.freestandards.org
Dead link: http://fedora.linux.duke.edu/fc1_x86_64
Dead link: http://gcc.gnu.org/ml/gcc-patches/2004-02/msg01307.html
Dead link: http://gcc.gnu.org/ml/gcc-patches/2004-02/msg01518.html
Dead link: http://gcc.gnu.org/onlinedocs/libstdc++/documentation.html
Dead link: http://rogue.colorado.edu/EPIC7/
Dead link: http://sourcery.mentor.com/public/cxx-abi/exceptions.pdf
Dead link: http://www.beunited.org/articles/jbq/nasm.shtml
Dead link: http://www.netlib.org/specfun/algama
Dead link: http://www.netlib.org/specfun/erf
Dead link: http://www.netlib.org/specfun/gamma

If somebody wants to find replacements for them it would be welcome.
Jonathan Wakely - July 10, 2013, 2:23 p.m.
On 10 July 2013 15:00, Ondřej Bílka wrote:
>>
> I prefer fix issues one class at time. I added script that checks dead
> links by extracting http addresses from sources and checking them by curl -f.
> It found:
>
> $ ./stylepp/script/stylepp_warn_dead_link
>
> Dead link: http://anubis.dkuug.dk/jtc1/sc22/wg21/docs/cwg_closed.html#46

http://www.open-std.org/jtc1/sc22/wg21/docs/cwg_closed.html#46

> Dead link: http://citeseer.nj.nec.com/15361.html
> Dead link: http://dwarf.freestandards.org

http://www.dwarfstd.org/

> Dead link: http://fedora.linux.duke.edu/fc1_x86_64
> Dead link: http://gcc.gnu.org/ml/gcc-patches/2004-02/msg01307.html
> Dead link: http://gcc.gnu.org/ml/gcc-patches/2004-02/msg01518.html

Strange, I don't know why those messages are missing, they're in the index.

> Dead link: http://gcc.gnu.org/onlinedocs/libstdc++/documentation.html

Where is that link?  It should probably be
http://gcc.gnu.org/onlinedocs/libstdc++/

> Dead link: http://rogue.colorado.edu/EPIC7/
> Dead link: http://sourcery.mentor.com/public/cxx-abi/exceptions.pdf

Probably http://mentorembedded.github.io/cxx-abi/abi-eh.html

> Dead link: http://www.beunited.org/articles/jbq/nasm.shtml
> Dead link: http://www.netlib.org/specfun/algama
> Dead link: http://www.netlib.org/specfun/erf
> Dead link: http://www.netlib.org/specfun/gamma
>
> If somebody wants to find replacements for them it would be welcome.
David Malcolm - July 10, 2013, 2:47 p.m.
On Wed, 2013-07-10 at 16:00 +0200, Ondřej Bílka wrote:

> Dead link: http://fedora.linux.duke.edu/fc1_x86_64

FWIW, this link occurs in historical notes at the top of these files:
  zlib/contrib/inflate86/inffas86.c
  zlib/contrib/masmx64/inffas8664.c
dated Dec-29-2003 describing specific testing done at that time, and
thus although the link is dead, it seems wrong to me to go changing it
(also, given that it's an embedded copy of a 3rd-party library, this
would be more for upstream zlib).

Hope this is helpful
Dave
Jonathan Wakely - July 10, 2013, 3:01 p.m.
On 10 July 2013 15:47, David Malcolm wrote:
> On Wed, 2013-07-10 at 16:00 +0200, Ondřej Bílka wrote:
>
>> Dead link: http://fedora.linux.duke.edu/fc1_x86_64
>
> FWIW, this link occurs in historical notes at the top of these files:
>   zlib/contrib/inflate86/inffas86.c
>   zlib/contrib/masmx64/inffas8664.c
> dated Dec-29-2003 describing specific testing done at that time, and
> thus although the link is dead, it seems wrong to me to go changing it
> (also, given that it's an embedded copy of a 3rd-party library, this
> would be more for upstream zlib).
>
> Hope this is helpful

Thanks.  Ondřej, as stated a few times now, anything in zlib,
boehm-gc, etc. is maintained outside the GCC project.  Do you have any
way to filter out certain directories from being checked by your
scripts?
Ondrej Bilka - July 10, 2013, 3:22 p.m.
On Wed, Jul 10, 2013 at 04:01:54PM +0100, Jonathan Wakely wrote:
> On 10 July 2013 15:47, David Malcolm wrote:
> > On Wed, 2013-07-10 at 16:00 +0200, Ondřej Bílka wrote:
> >
> >> Dead link: http://fedora.linux.duke.edu/fc1_x86_64
> >
> > FWIW, this link occurs in historical notes at the top of these files:
> >   zlib/contrib/inflate86/inffas86.c
> >   zlib/contrib/masmx64/inffas8664.c
> > dated Dec-29-2003 describing specific testing done at that time, and
> > thus although the link is dead, it seems wrong to me to go changing it
> > (also, given that it's an embedded copy of a 3rd-party library, this
> > would be more for upstream zlib).
> >
> > Hope this is helpful
> 
> Thanks.  Ondřej, as stated a few times now, anything in zlib,
> boehm-gc, etc. is maintained outside the GCC project.  Do you have any
> way to filter out certain directories from being checked by your
> scripts?

I have (place .indent.off file to directory) but I do not know which ones 
except these are external. Closest what I could find is
http://gcc.gnu.org/onlinedocs/gccint/Top-Level.html#Top-Level

I added boehm-gc,zlib and gcc/testsuite. Are there others?
Jonathan Wakely - July 10, 2013, 4:27 p.m.
On 10 July 2013 16:22, Ondřej Bílka wrote:
>> Thanks.  Ondřej, as stated a few times now, anything in zlib,
>> boehm-gc, etc. is maintained outside the GCC project.  Do you have any
>> way to filter out certain directories from being checked by your
>> scripts?
>
> I have (place .indent.off file to directory) but I do not know which ones
> except these are external. Closest what I could find is
> http://gcc.gnu.org/onlinedocs/gccint/Top-Level.html#Top-Level
>
> I added boehm-gc,zlib and gcc/testsuite. Are there others?

Joseph listed some more at http://gcc.gnu.org/ml/gcc/2013-07/msg00119.html

Patch

--- a/gcc/config/frv/frv-opts.h
+++ b/gcc/config/frv/frv-opts.h
@@ -1,4 +1,4 @@ 
-/* Frv option-handling defitions.
+/* Frv option-handling deviations.

Should be "definitions" instead of "deviations".