Re: [Qemu-devel] [PATCH] Fix typos and misspellings

[Stefan Weil]

Am 18.03.2013 18:20, schrieb Peter Maydell:
> Fix various typos and misspellings. The bulk of these were found with
> codespell.
>
> Signed-off-by: Peter Maydell <address@hidden>
> ---
> I noticed the PCI typo while reading the file so I figured I'd run codespell
> on the tree while I was at it. (Oddly codespell doesn't complain about the
> typos in pci_host.c.)

Codespell uses a list of common typos. It's possible to extend this
list - just send a patch to Codespell's author.

>
>  docs/usb-storage.txt     |    4 ++--
>  hw/arm-misc.h            |    2 +-
>  hw/pci/pci_host.c        |    2 +-
>  hw/sdhci.c               |    2 +-
>  include/qom/object.h     |    2 +-
>  monitor.c                |    8 ++++----
>  net/tap.c                |    2 +-
>  qemu-options.hx          |    2 +-
>  qga/commands-posix.c     |    2 +-
>  target-i386/translate.c  |    2 +-
>  target-s390x/translate.c |    2 +-
>  11 files changed, 15 insertions(+), 15 deletions(-)
>
> diff --git a/docs/usb-storage.txt b/docs/usb-storage.txt
> index fa93111..c5a3866 100644
> --- a/docs/usb-storage.txt
> +++ b/docs/usb-storage.txt
> @@ -5,7 +5,7 @@ qemu usb storage emulation
>  QEMU has three devices for usb storage emulation.
>  
>  Number one emulates the classic bulk-only transport protocol which is
> -used by 99% of the usb sticks on the marked today and is called
> +used by 99% of the usb sticks on the market today and is called
>  "usb-storage".  Usage (hooking up to xhci, other host controllers work
>  too):
>  
> @@ -36,7 +36,7 @@ It's called "usb-bot".  It shares most code with 
> "usb-storage", and
>  the guest will not be able to see the difference.  The qemu command
>  line interface is simliar to usb-uas though, i.e. no automatic scsi
>  disk creation.  It also features support for up to 16 LUNs.  The LUN
> -numbers must be continous, i.e. for three devices you must use 0+1+2.
> +numbers must be continuous, i.e. for three devices you must use 0+1+2.
>  The 0+1+5 numbering from the "usb-uas" example isn't going to work
>  with "usb-bot".
>  
> diff --git a/hw/arm-misc.h b/hw/arm-misc.h
> index cba7553..7b2b02d 100644
> --- a/hw/arm-misc.h
> +++ b/hw/arm-misc.h
> @@ -14,7 +14,7 @@
>  #include "exec/memory.h"
>  #include "hw/irq.h"
>  
> -/* The CPU is also modeled as an interrupt controller.  */
> +/* The CPU is also modelled as an interrupt controller.  */
>  #define ARM_PIC_CPU_IRQ 0
>  #define ARM_PIC_CPU_FIQ 1
>  qemu_irq *arm_pic_init_cpu(ARMCPU *cpu);
> diff --git a/hw/pci/pci_host.c b/hw/pci/pci_host.c
> index daca1c1..12254b1 100644
> --- a/hw/pci/pci_host.c
> +++ b/hw/pci/pci_host.c
> @@ -38,7 +38,7 @@ do { printf("pci_host_data: " fmt , ## __VA_ARGS__); } 
> while (0)
>   * bit  0 -  7: offset in configuration space of a given pci device
>   */
>  
> -/* the helper functio to get a PCIDeice* for a given pci address */
> +/* the helper function to get a PCIDevice* for a given pci address */
>  static inline PCIDevice *pci_dev_find_by_addr(PCIBus *bus, uint32_t addr)
>  {
>      uint8_t bus_num = addr >> 16;
> diff --git a/hw/sdhci.c b/hw/sdhci.c
> index 93feada..4a29e6c 100644
> --- a/hw/sdhci.c
> +++ b/hw/sdhci.c
> @@ -763,7 +763,7 @@ static void sdhci_do_adma(SDHCIState *s)
>          }
>      }
>  
> -    /* we have unfinished bussiness - reschedule to continue ADMA */
> +    /* we have unfinished business - reschedule to continue ADMA */
>      qemu_mod_timer(s->transfer_timer,
>                     qemu_get_clock_ns(vm_clock) + SDHC_TRANSFER_DELAY);
>  }
> diff --git a/include/qom/object.h b/include/qom/object.h
> index cf094e7..d0f99c5 100644
> --- a/include/qom/object.h
> +++ b/include/qom/object.h
> @@ -202,7 +202,7 @@ typedef struct InterfaceInfo InterfaceInfo;
>   * Methods are always <emphasis>virtual</emphasis>. Overriding a method in
>   * #TypeInfo.class_init of a subclass leads to any user of the class obtained
>   * via OBJECT_GET_CLASS() accessing the overridden function.
> - * The original function is not automatically invoked. It is the 
> responsability
> + * The original function is not automatically invoked. It is the 
> responsibility
>   * of the overriding class to determine whether and when to invoke the method
>   * being overridden.
>   *
> diff --git a/monitor.c b/monitor.c
> index 112e920..99bbb17 100644
> --- a/monitor.c
> +++ b/monitor.c
> @@ -3541,10 +3541,10 @@ static const mon_cmd_t *qmp_find_cmd(const char 
> *cmdname)
>   * If @cmdline is blank, return NULL.
>   * If it can't be parsed, report to @mon, and return NULL.
>   * Else, insert command arguments into @qdict, and return the command.
> - * If sub-command table exist, and if @cmdline contains addtional string for
> - * sub-command, this function will try search sub-command table. if no
> - * addtional string for sub-command exist, this function will return the 
> found
> - * one in @table.
> + * If a sub-command table exists, and if @cmdline contains an additional 
> string
> + * for a sub-command, this function will try to search the sub-command table.
> + * If no additional string for a sub-command is present, this function will
> + * return the command found in @table.
>   * Do not assume the returned command points into @table!  It doesn't
>   * when the command is a sub-command.
>   */
> diff --git a/net/tap.c b/net/tap.c
> index daab350..ce79699 100644
> --- a/net/tap.c
> +++ b/net/tap.c
> @@ -696,7 +696,7 @@ int net_init_tap(const NetClientOptions *opts, const char 
> *name,
>      /* QEMU vlans does not support multiqueue tap, in this case peer is set.
>       * For -netdev, peer is always NULL. */
>      if (peer && (tap->has_queues || tap->has_fds || tap->has_vhostfds)) {
> -        error_report("Multiqueue tap cannnot be used with QEMU vlans");
> +        error_report("Multiqueue tap cannot be used with QEMU vlans");
>          return -1;
>      }
>  
> diff --git a/qemu-options.hx b/qemu-options.hx
> index 30fb85d..c68a46e 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -1119,7 +1119,7 @@ is a TCP port number, not a display number.
>  @item websocket
>  
>  Opens an additional TCP listening port dedicated to VNC Websocket 
> connections.
> -By defintion the Websocket port is address@hidden If @var{host} is
> +By definition the Websocket port is address@hidden If @var{host} is
>  specified connections will only be allowed from this host.
>  As an alternative the Websocket port could be specified by using
>  @address@hidden
> diff --git a/qga/commands-posix.c b/qga/commands-posix.c
> index d7da850..3b5c536 100644
> --- a/qga/commands-posix.c
> +++ b/qga/commands-posix.c
> @@ -120,7 +120,7 @@ void qmp_guest_shutdown(bool has_mode, const char *mode, 
> Error **err)
>          return;
>      }
>  
> -    /* succeded */
> +    /* succeeded */
>  }
>  
>  int64_t qmp_guest_get_time(Error **errp)
> diff --git a/target-i386/translate.c b/target-i386/translate.c
> index 705147a..c9cc2ff 100644
> --- a/target-i386/translate.c
> +++ b/target-i386/translate.c
> @@ -4746,7 +4746,7 @@ static target_ulong disas_insn(CPUX86State *env, 
> DisasContext *s,
>              }
>              s->pc++;
>  
> -            /* 4.1.1-4.1.3: No preceeding lock, 66, f2, f3, or rex prefixes. 
> */
> +            /* 4.1.1-4.1.3: No preceding lock, 66, f2, f3, or rex prefixes. 
> */
>              if (prefixes & (PREFIX_REPZ | PREFIX_REPNZ
>                              | PREFIX_LOCK | PREFIX_DATA)) {
>                  goto illegal_op;
> diff --git a/target-s390x/translate.c b/target-s390x/translate.c
> index 88e481c..a4f2194 100644
> --- a/target-s390x/translate.c
> +++ b/target-s390x/translate.c
> @@ -1142,7 +1142,7 @@ struct DisasInsn {
>  };
>  
>  /* ====================================================================== */
> -/* Miscelaneous helpers, used by several operations.  */
> +/* Miscellaneous helpers, used by several operations.  */
>  
>  static void help_l2_shift(DisasContext *s, DisasFields *f,
>                            DisasOps *o, int mask)


Reviewed-by: Stefan Weil <address@hidden>