Re: [PATCH] Fix typo and miswordings in three error messages

[Gerald Pfeifer]

Hi Benno,

I tested your fixes on i386-unknown-freebsd10.1 -- you never
know, after some of these could have been reflected in some
test case for example -- and committed them on your behalf.

Sorry for none of us looking into this for so long.  If you
find further typos or phrases that need improvement, please
do not hestiate to let us know.  Providing patches, including
a ChangeLog entry as you have done, is even better!

Thank you,
Gerald

On Thu, 6 Feb 2014, Benno Schulenberg wrote:
> Updating a bit the Dutch translations of GCC's messages,
> I noticed the following mistakes in three msgids:
> 
>   "only displayed one"  ==>  "displayed only once"
>   "none class-method"  ==>  "non-class method"
>   "incorect"  ==>  "incorrect"
> 
> Below patch fixes those.  I'm not entirely sure about
> the second fix, but the "none" doesn't make any sense.
> When found okay, please apply.
> 
> 
> 2014-02-05  Benno Schulenberg  
> 
>   * gcov.c (find_source): Fix miswording in error message.
>   * config/i386/i386.c (ix86_handle_cconv_attribute): Likewise.
>   (ix86_expand_sse_comi_round): Fix typo in error message.
> 
> 
> Index: gcov.c
> ===
> --- gcov.c(revision 207551)
> +++ gcov.c(working copy)
> @@ -1141,7 +1141,7 @@
>if (!info_emitted)
>   {
> fnotice (stderr,
> -"(the message is only displayed one per source file)\n");
> +"(the message is displayed only once per source file)\n");
> info_emitted = 1;
>   }
>sources[idx].file_time = 0;
> Index: config/i386/i386.c
> ===
> --- config/i386/i386.c(revision 207551)
> +++ config/i386/i386.c(working copy)
> @@ -5446,7 +5446,7 @@
>else if (is_attribute_p ("thiscall", name))
>  {
>if (TREE_CODE (*node) != METHOD_TYPE && pedantic)
> - warning (OPT_Wattributes, "%qE attribute is used for none class-method",
> + warning (OPT_Wattributes, "%qE attribute is used for non-class method",
>name);
>if (lookup_attribute ("stdcall", TYPE_ATTRIBUTES (*node)))
>   {
> @@ -34230,7 +34230,7 @@
>  }
>if (INTVAL (op2) < 0 || INTVAL (op2) >= 32)
>  {
> -  error ("incorect comparison mode");
> +  error ("incorrect comparison mode");
>return const0_rtx;
>  }

Re: [PATCH] Fix typo and miswordings in three error messages

[Benno Schulenberg]

On 2014-02-06 21:18, Benno Schulenberg wrote:
> Updating a bit the Dutch translations of GCC's messages,
> I noticed the following mistakes in three msgids:
> 
>   "only displayed one"  ==>  "displayed only once"
>   "none class-method"  ==>  "non-class method"
>   "incorect"  ==>  "incorrect"

Ping?

> Below patch fixes those.  I'm not entirely sure about
> the second fix, but the "none" doesn't make any sense.
> When found okay, please apply.
> 
> 
> 2014-02-05  Benno Schulenberg  
> 
>   * gcov.c (find_source): Fix miswording in error message.
>   * config/i386/i386.c (ix86_handle_cconv_attribute): Likewise.
>   (ix86_expand_sse_comi_round): Fix typo in error message.
> 
> 
> Index: gcov.c
> ===
> --- gcov.c(revision 207551)
> +++ gcov.c(working copy)
> @@ -1141,7 +1141,7 @@
>if (!info_emitted)
>   {
> fnotice (stderr,
> -"(the message is only displayed one per source file)\n");
> +"(the message is displayed only once per source file)\n");
> info_emitted = 1;
>   }
>sources[idx].file_time = 0;
> Index: config/i386/i386.c
> ===
> --- config/i386/i386.c(revision 207551)
> +++ config/i386/i386.c(working copy)
> @@ -5446,7 +5446,7 @@
>else if (is_attribute_p ("thiscall", name))
>  {
>if (TREE_CODE (*node) != METHOD_TYPE && pedantic)
> - warning (OPT_Wattributes, "%qE attribute is used for none class-method",
> + warning (OPT_Wattributes, "%qE attribute is used for non-class method",
>name);
>if (lookup_attribute ("stdcall", TYPE_ATTRIBUTES (*node)))
>   {
> @@ -34230,7 +34230,7 @@
>  }
>if (INTVAL (op2) < 0 || INTVAL (op2) >= 32)
>  {
> -  error ("incorect comparison mode");
> +  error ("incorrect comparison mode");
>return const0_rtx;
>  }
>  

[PATCH] Fix typo and miswordings in three error messages

[Benno Schulenberg]

Updating a bit the Dutch translations of GCC's messages,
I noticed the following mistakes in three msgids:

  "only displayed one"  ==>  "displayed only once"
  "none class-method"  ==>  "non-class method"
  "incorect"  ==>  "incorrect"

Below patch fixes those.  I'm not entirely sure about
the second fix, but the "none" doesn't make any sense.
When found okay, please apply.


2014-02-05  Benno Schulenberg  

* gcov.c (find_source): Fix miswording in error message.
* config/i386/i386.c (ix86_handle_cconv_attribute): Likewise.
(ix86_expand_sse_comi_round): Fix typo in error message.


Index: gcov.c
===
--- gcov.c  (revision 207551)
+++ gcov.c  (working copy)
@@ -1141,7 +1141,7 @@
   if (!info_emitted)
{
  fnotice (stderr,
-  "(the message is only displayed one per source file)\n");
+  "(the message is displayed only once per source file)\n");
  info_emitted = 1;
}
   sources[idx].file_time = 0;
Index: config/i386/i386.c
===
--- config/i386/i386.c  (revision 207551)
+++ config/i386/i386.c  (working copy)
@@ -5446,7 +5446,7 @@
   else if (is_attribute_p ("thiscall", name))
 {
   if (TREE_CODE (*node) != METHOD_TYPE && pedantic)
-   warning (OPT_Wattributes, "%qE attribute is used for none class-method",
+   warning (OPT_Wattributes, "%qE attribute is used for non-class method",
 name);
   if (lookup_attribute ("stdcall", TYPE_ATTRIBUTES (*node)))
{
@@ -34230,7 +34230,7 @@
 }
   if (INTVAL (op2) < 0 || INTVAL (op2) >= 32)
 {
-  error ("incorect comparison mode");
+  error ("incorrect comparison mode");
   return const0_rtx;
 }