[Qemu-devel] [PATCH] tests: Fix typos in comments and help message (found by codespell)

[Qemu-devel] [PATCH] tests: Fix typos in comments and help message (found by codespell)

[Stefan Weil]

Fix also a grammar issue.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
---
 tests/bios-tables-test.c              | 2 +-
 tests/docker/Makefile.include         | 2 +-
 tests/docker/docker.py                | 4 ++--
 tests/guest-debug/test-gdbstub.py     | 2 +-
 tests/qemu-iotests/common.qemu        | 2 +-
 tests/tcg/Makefile.include            | 2 +-
 tests/tcg/Makefile.probe              | 2 +-
 tests/tcg/mips/mips64-dsp/subq_s_pw.c | 2 +-
 8 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/tests/bios-tables-test.c b/tests/bios-tables-test.c
index 4e24930c4b..af4b1fb6bd 100644
--- a/tests/bios-tables-test.c
+++ b/tests/bios-tables-test.c
@@ -390,7 +390,7 @@ try_again:
         if (g_file_test(aml_file, G_FILE_TEST_EXISTS)) {
             exp_sdt.aml_file = aml_file;
         } else if (*ext != '\0') {
-            /* try fallback to generic (extention less) expected file */
+            /* try fallback to generic (extension less) expected file */
             ext = "";
             g_free(aml_file);
             goto try_again;
diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
index b2a7e761cc..d865dcfdaa 100644
--- a/tests/docker/Makefile.include
+++ b/tests/docker/Makefile.include
@@ -37,7 +37,7 @@ docker-qemu-src: $(DOCKER_SRC_COPY)
 docker-image: ${DOCKER_TARGETS}
 
 # General rule for building docker images. If we are a sub-make
-# invoked with SKIP_DOCKER_BUILD we still check the image is upto date
+# invoked with SKIP_DOCKER_BUILD we still check the image is up to date
 # though
 ifdef SKIP_DOCKER_BUILD
 docker-image-%: $(DOCKER_FILES_DIR)/%.docker
diff --git a/tests/docker/docker.py b/tests/docker/docker.py
index 69e7130db7..b82926596c 100755
--- a/tests/docker/docker.py
+++ b/tests/docker/docker.py
@@ -99,7 +99,7 @@ def _get_so_libs(executable):
     return libs
 
 def _copy_binary_with_libs(src, dest_dir):
-    """Copy a binary executable and all its dependant libraries.
+    """Copy a binary executable and all its dependent libraries.
 
     This does rely on the host file-system being fairly multi-arch
     aware so the file don't clash with the guests layout."""
@@ -261,7 +261,7 @@ class SubCommand(object):
     name = None # Subcommand name
     def shared_args(self, parser):
         parser.add_argument("--quiet", action="store_true",
-                            help="Run quietly unless an error occured")
+                            help="Run quietly unless an error occurred")
 
     def args(self, parser):
         """Setup argument parser"""
diff --git a/tests/guest-debug/test-gdbstub.py b/tests/guest-debug/test-gdbstub.py
index 474d2c5c65..0e4ac01426 100644
--- a/tests/guest-debug/test-gdbstub.py
+++ b/tests/guest-debug/test-gdbstub.py
@@ -122,7 +122,7 @@ class CatchBreakpoint(gdb.Breakpoint):
 
 
 def run_test():
-    "Run throught the tests one by one"
+    "Run through the tests one by one"
 
     print ("Checking we can step the first few instructions")
     step_ok = 0
diff --git a/tests/qemu-iotests/common.qemu b/tests/qemu-iotests/common.qemu
index f285484951..dadde2a266 100644
--- a/tests/qemu-iotests/common.qemu
+++ b/tests/qemu-iotests/common.qemu
@@ -257,7 +257,7 @@ function _launch_qemu()
 }
 
 
-# Silenty kills the QEMU process
+# Silently kills the QEMU process
 #
 # If $wait is set to anything other than the empty string, the process will not
 # be killed but only waited for, and any output will be forwarded to stdout. If
diff --git a/tests/tcg/Makefile.include b/tests/tcg/Makefile.include
index 57470b2a2c..c581bd6ffc 100644
--- a/tests/tcg/Makefile.include
+++ b/tests/tcg/Makefile.include
@@ -2,7 +2,7 @@
 #
 # TCG tests (per-target rules)
 #
-# This Makefile fragement is included from the per-target
+# This Makefile fragment is included from the per-target
 # Makefile.target so will be invoked for each linux-user program we
 # build. We have two options for compiling, either using a configured
 # guest compiler or calling one of our docker images to do it for us.
diff --git a/tests/tcg/Makefile.probe b/tests/tcg/Makefile.probe
index 15c0412657..9dc654663d 100644
--- a/tests/tcg/Makefile.probe
+++ b/tests/tcg/Makefile.probe
@@ -2,7 +2,7 @@
 #
 # TCG Compiler Probe
 #
-# This Makefile fragement is included multiple times in the main make
+# This Makefile fragment is included multiple times in the main make
 # script to probe for available compilers. This is used to build up a
 # selection of required docker targets before we invoke a sub-make for
 # each target.
diff --git a/tests/tcg/mips/mips64-dsp/subq_s_pw.c b/tests/tcg/mips/mips64-dsp/subq_s_pw.c
index e8e0b0567e..4c080b785a 100644
--- a/tests/tcg/mips/mips64-dsp/subq_s_pw.c
+++ b/tests/tcg/mips/mips64-dsp/subq_s_pw.c
@@ -24,7 +24,7 @@ int main(void)
     rt = 0x123456789ABCDEF1;
     rs = 0x123456789ABCDEF2;
     result =  0x0000000000000001;
-    /* This time we do not set dspctrl, but it setted in pre-action. */
+    /* This time we do not set dspctrl, but set it in pre-action. */
     dspresult = 0x1;
 
     __asm
-- 
2.11.0

Re: [Qemu-devel] [PATCH] tests: Fix typos in comments and help message (found by codespell)

[Alex Bennée]

Stefan Weil <sw@weilnetz.de> writes:

> Fix also a grammar issue.
>
> Signed-off-by: Stefan Weil <sw@weilnetz.de>

*blush* 5 of those are my spelling mistakes, so have an embarrassed:

Reviewed-by: Alex Bennée <alex.bennee@linaro.org>

> ---
>  tests/bios-tables-test.c              | 2 +-
>  tests/docker/Makefile.include         | 2 +-
>  tests/docker/docker.py                | 4 ++--
>  tests/guest-debug/test-gdbstub.py     | 2 +-
>  tests/qemu-iotests/common.qemu        | 2 +-
>  tests/tcg/Makefile.include            | 2 +-
>  tests/tcg/Makefile.probe              | 2 +-
>  tests/tcg/mips/mips64-dsp/subq_s_pw.c | 2 +-
>  8 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/tests/bios-tables-test.c b/tests/bios-tables-test.c
> index 4e24930c4b..af4b1fb6bd 100644
> --- a/tests/bios-tables-test.c
> +++ b/tests/bios-tables-test.c
> @@ -390,7 +390,7 @@ try_again:
>          if (g_file_test(aml_file, G_FILE_TEST_EXISTS)) {
>              exp_sdt.aml_file = aml_file;
>          } else if (*ext != '\0') {
> -            /* try fallback to generic (extention less) expected file */
> +            /* try fallback to generic (extension less) expected file */
>              ext = "";
>              g_free(aml_file);
>              goto try_again;
> diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
> index b2a7e761cc..d865dcfdaa 100644
> --- a/tests/docker/Makefile.include
> +++ b/tests/docker/Makefile.include
> @@ -37,7 +37,7 @@ docker-qemu-src: $(DOCKER_SRC_COPY)
>  docker-image: ${DOCKER_TARGETS}
>
>  # General rule for building docker images. If we are a sub-make
> -# invoked with SKIP_DOCKER_BUILD we still check the image is upto date
> +# invoked with SKIP_DOCKER_BUILD we still check the image is up to date
>  # though
>  ifdef SKIP_DOCKER_BUILD
>  docker-image-%: $(DOCKER_FILES_DIR)/%.docker
> diff --git a/tests/docker/docker.py b/tests/docker/docker.py
> index 69e7130db7..b82926596c 100755
> --- a/tests/docker/docker.py
> +++ b/tests/docker/docker.py
> @@ -99,7 +99,7 @@ def _get_so_libs(executable):
>      return libs
>
>  def _copy_binary_with_libs(src, dest_dir):
> -    """Copy a binary executable and all its dependant libraries.
> +    """Copy a binary executable and all its dependent libraries.
>
>      This does rely on the host file-system being fairly multi-arch
>      aware so the file don't clash with the guests layout."""
> @@ -261,7 +261,7 @@ class SubCommand(object):
>      name = None # Subcommand name
>      def shared_args(self, parser):
>          parser.add_argument("--quiet", action="store_true",
> -                            help="Run quietly unless an error occured")
> +                            help="Run quietly unless an error occurred")
>
>      def args(self, parser):
>          """Setup argument parser"""
> diff --git a/tests/guest-debug/test-gdbstub.py b/tests/guest-debug/test-gdbstub.py
> index 474d2c5c65..0e4ac01426 100644
> --- a/tests/guest-debug/test-gdbstub.py
> +++ b/tests/guest-debug/test-gdbstub.py
> @@ -122,7 +122,7 @@ class CatchBreakpoint(gdb.Breakpoint):
>
>
>  def run_test():
> -    "Run throught the tests one by one"
> +    "Run through the tests one by one"
>
>      print ("Checking we can step the first few instructions")
>      step_ok = 0
> diff --git a/tests/qemu-iotests/common.qemu b/tests/qemu-iotests/common.qemu
> index f285484951..dadde2a266 100644
> --- a/tests/qemu-iotests/common.qemu
> +++ b/tests/qemu-iotests/common.qemu
> @@ -257,7 +257,7 @@ function _launch_qemu()
>  }
>
>
> -# Silenty kills the QEMU process
> +# Silently kills the QEMU process
>  #
>  # If $wait is set to anything other than the empty string, the process will not
>  # be killed but only waited for, and any output will be forwarded to stdout. If
> diff --git a/tests/tcg/Makefile.include b/tests/tcg/Makefile.include
> index 57470b2a2c..c581bd6ffc 100644
> --- a/tests/tcg/Makefile.include
> +++ b/tests/tcg/Makefile.include
> @@ -2,7 +2,7 @@
>  #
>  # TCG tests (per-target rules)
>  #
> -# This Makefile fragement is included from the per-target
> +# This Makefile fragment is included from the per-target
>  # Makefile.target so will be invoked for each linux-user program we
>  # build. We have two options for compiling, either using a configured
>  # guest compiler or calling one of our docker images to do it for us.
> diff --git a/tests/tcg/Makefile.probe b/tests/tcg/Makefile.probe
> index 15c0412657..9dc654663d 100644
> --- a/tests/tcg/Makefile.probe
> +++ b/tests/tcg/Makefile.probe
> @@ -2,7 +2,7 @@
>  #
>  # TCG Compiler Probe
>  #
> -# This Makefile fragement is included multiple times in the main make
> +# This Makefile fragment is included multiple times in the main make
>  # script to probe for available compilers. This is used to build up a
>  # selection of required docker targets before we invoke a sub-make for
>  # each target.
> diff --git a/tests/tcg/mips/mips64-dsp/subq_s_pw.c b/tests/tcg/mips/mips64-dsp/subq_s_pw.c
> index e8e0b0567e..4c080b785a 100644
> --- a/tests/tcg/mips/mips64-dsp/subq_s_pw.c
> +++ b/tests/tcg/mips/mips64-dsp/subq_s_pw.c
> @@ -24,7 +24,7 @@ int main(void)
>      rt = 0x123456789ABCDEF1;
>      rs = 0x123456789ABCDEF2;
>      result =  0x0000000000000001;
> -    /* This time we do not set dspctrl, but it setted in pre-action. */
> +    /* This time we do not set dspctrl, but set it in pre-action. */
>      dspresult = 0x1;
>
>      __asm


--
Alex Bennée