2021-09-26 06:57:13

by Sven Eckelmann

[permalink] [raw]
Subject: scripts/spelling.txt: Fix "mistake" version of "synchronization"

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 <[email protected]>
---
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


2021-09-26 11:53:19

by Colin King

[permalink] [raw]
Subject: Re: scripts/spelling.txt: Fix "mistake" version of "synchronization"



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 <[email protected]>

Good catch. My bad.

Reviewed-by: Colin Ian King <[email protected]>

> ---
> 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
>