linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* scripts/spelling.txt: Fix "mistake" version of "synchronization"
@ 2021-09-26  6:55 Sven Eckelmann
  2021-09-26 11:52 ` Colin Ian King
  0 siblings, 1 reply; 2+ messages in thread
From: Sven Eckelmann @ 2021-09-26  6:55 UTC (permalink / raw)
  To: Andrew Morton; +Cc: kernel-janitors, linux-kernel, Colin King, Sven Eckelmann

If both "mistake" version and "correction" version are the same, a warning
message is created by checkpatch which is impossible to fix. But it was
noticed that Colan Ian King created a commit e6c0a0889b80 ("ALSA: aloop:
Fix spelling mistake "synchronization" -> "synchronization"") which
suggests that this spelling mistake was fixed by replacing the word
"synchronization" with itself. But the actual diff shows that the mistake
in the code was "sychronization". It is rather likely that the "mistake"
in spelling.txt should have been the latter.

Fixes: 2e74c9433ba8 ("scripts/spelling.txt: add more spellings to spelling.txt")
Signed-off-by: Sven Eckelmann <sven@narfation.org>
---
This was already reported last week in https://lore.kernel.org/all/1930750.yhyOXdeGKK@sven-l14/

diff --git a/scripts/spelling.txt b/scripts/spelling.txt
index fd8f07317b8e1c572e1e0f6bb4f9e01dff96c17c..acf6ea71129921ad77a201d13761382ee032483e 100644
--- a/scripts/spelling.txt
+++ b/scripts/spelling.txt
@@ -1450,7 +1450,7 @@ syfs||sysfs
 symetric||symmetric
 synax||syntax
 synchonized||synchronized
-synchronization||synchronization
+sychronization||synchronization
 synchronuously||synchronously
 syncronize||synchronize
 syncronized||synchronized

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: scripts/spelling.txt: Fix "mistake" version of "synchronization"
  2021-09-26  6:55 scripts/spelling.txt: Fix "mistake" version of "synchronization" Sven Eckelmann
@ 2021-09-26 11:52 ` Colin Ian King
  0 siblings, 0 replies; 2+ messages in thread
From: Colin Ian King @ 2021-09-26 11:52 UTC (permalink / raw)
  To: Sven Eckelmann, Andrew Morton; +Cc: kernel-janitors, linux-kernel



On 26/09/2021 07:55, Sven Eckelmann wrote:
> If both "mistake" version and "correction" version are the same, a warning
> message is created by checkpatch which is impossible to fix. But it was
> noticed that Colan Ian King created a commit e6c0a0889b80 ("ALSA: aloop:
> Fix spelling mistake "synchronization" -> "synchronization"") which
> suggests that this spelling mistake was fixed by replacing the word
> "synchronization" with itself. But the actual diff shows that the mistake
> in the code was "sychronization". It is rather likely that the "mistake"
> in spelling.txt should have been the latter.
> 
> Fixes: 2e74c9433ba8 ("scripts/spelling.txt: add more spellings to spelling.txt")
> Signed-off-by: Sven Eckelmann <sven@narfation.org>

Good catch. My bad.

Reviewed-by: Colin Ian King <colin.king@canonical.com>

> ---
> This was already reported last week in https://lore.kernel.org/all/1930750.yhyOXdeGKK@sven-l14/
> 
> diff --git a/scripts/spelling.txt b/scripts/spelling.txt
> index fd8f07317b8e1c572e1e0f6bb4f9e01dff96c17c..acf6ea71129921ad77a201d13761382ee032483e 100644
> --- a/scripts/spelling.txt
> +++ b/scripts/spelling.txt
> @@ -1450,7 +1450,7 @@ syfs||sysfs
>  symetric||symmetric
>  synax||syntax
>  synchonized||synchronized
> -synchronization||synchronization
> +sychronization||synchronization
>  synchronuously||synchronously
>  syncronize||synchronize
>  syncronized||synchronized
> 


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-09-26 11:52 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-26  6:55 scripts/spelling.txt: Fix "mistake" version of "synchronization" Sven Eckelmann
2021-09-26 11:52 ` Colin Ian King

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).