Re: [Qemu-trivial] [Qemu-devel] [PATCH] Spelling fixes in comments (it's -> its)

["Andreas Färber" <afaerber@suse.de>]

Am 26.02.2012 14:35, schrieb Stefan Weil:
> * it's -> its (fixed for all files)
> * dont -> don't (only fixed in a line which was touched by the previous fix)
> 
> Signed-off-by: Stefan Weil <sw@weilnetz.de>

Reviewed-by: Andreas Färber <afaerber@suse.de>

Except for one omission:

> ---
>  configure               |    2 +-
>  hw/exynos4210_mct.c     |    2 +-
>  hw/usb-ccid.c           |    2 +-
>  include/qemu/object.h   |    4 ++--
>  kvm-all.c               |    2 +-
>  libcacard/vscclient.c   |    2 +-
>  linux-user/signal.c     |    2 +-
>  qemu-file.h             |    2 +-
>  target-mips/op_helper.c |    2 +-
>  9 files changed, 10 insertions(+), 10 deletions(-)

> diff --git a/target-mips/op_helper.c b/target-mips/op_helper.c
> index c51b9cb..7d6773d 100644
> --- a/target-mips/op_helper.c
> +++ b/target-mips/op_helper.c
> @@ -1930,7 +1930,7 @@ target_ulong helper_evpe(void)
>  
>      do {
>          if (other_cpu != env
> -           /* If the VPE is WFI, dont distrub it's sleep.  */
> +           /* If the VPE is WFI, don't distrub its sleep.  */

While at it, please also fix "disturb". :)

Also I've generally wondered if there is a reason for having two spaces
before */? Some do, some don't.

Andreas

>             && !mips_vpe_is_wfi(other_cpu)) {
>              /* Enable the VPE.  */
>              other_cpu->mvp->CP0_MVPControl |= (1 << CP0MVPCo_EVP);

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg