From patchwork Tue Dec 27 17:19:05 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: =?utf-8?q?Marc-Andr=C3=A9_Lureau?= X-Patchwork-Id: 9489721 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 7A0D162AAF for ; Tue, 27 Dec 2016 17:19:50 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 6C2361FF26 for ; Tue, 27 Dec 2016 17:19:50 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 5F6592094F; Tue, 27 Dec 2016 17:19:50 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 2014E1FF26 for ; Tue, 27 Dec 2016 17:19:46 +0000 (UTC) Received: from localhost ([::1]:55144 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cLvPd-0001gU-5F for patchwork-qemu-devel@patchwork.kernel.org; Tue, 27 Dec 2016 12:19:45 -0500 Received: from eggs.gnu.org ([2001:4830:134:3::10]:60349) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cLvPA-0001gC-Th for qemu-devel@nongnu.org; Tue, 27 Dec 2016 12:19:21 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1cLvP5-0000EL-Lc for qemu-devel@nongnu.org; Tue, 27 Dec 2016 12:19:16 -0500 Received: from mx5-phx2.redhat.com ([209.132.183.37]:48331) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1cLvP5-0000Dl-3H for qemu-devel@nongnu.org; Tue, 27 Dec 2016 12:19:11 -0500 Received: from zmail17.collab.prod.int.phx2.redhat.com (zmail17.collab.prod.int.phx2.redhat.com [10.5.83.19]) by mx5-phx2.redhat.com (8.14.4/8.14.4) with ESMTP id uBRHJ5SZ043694; Tue, 27 Dec 2016 12:19:05 -0500 Date: Tue, 27 Dec 2016 12:19:05 -0500 (EST) From: =?utf-8?Q?Marc-Andr=C3=A9?= Lureau To: ka7@la-evento.com, qemu-devel@nongnu.org Message-ID: <1958441218.7403706.1482859145031.JavaMail.zimbra@redhat.com> In-Reply-To: <5a3cff0afa557527bb1ab6692d4e096c@rc.la-evento.com> References: <5a3cff0afa557527bb1ab6692d4e096c@rc.la-evento.com> MIME-Version: 1.0 X-Originating-IP: [10.36.116.91] X-Mailer: Zimbra 8.0.6_GA_5922 (ZimbraWebClient - FF50 (Linux)/8.0.6_GA_5922) Thread-Topic: qemu: bunch of spelling-fixes Thread-Index: EiwY41c1nvm2h6iSDG7wMGXhuRjrlQ== X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x [fuzzy] X-Received-From: 209.132.183.37 Subject: Re: [Qemu-devel] qemu: bunch of spelling-fixes X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+patchwork-qemu-devel=patchwork.kernel.org@nongnu.org Sender: "Qemu-devel" X-Virus-Scanned: ClamAV using ClamSMTP Hi, ----- Original Message ----- > Hi, > > I used misspell_fixer [1], [2] on qemu sources. (used > http://git.qemu.org/qemu.git ) > found a bunch of typos, patches with spelling-fixes attached. > Nice tool, thanks for the links. > so.. i did all these stuff, before read and fully understood the full > "howto submit patch". > > phew. sounds too complicated.. sitting on wrong machine, wrong email > addresses and want to keep my personal footprint out there as small as > possible. > And hey, this are just typo-fixes, no rocket-science. It was even easier > to get the fixes into openssl > > But throwing the stuff as its done not into /dev/null is also not good. > So i picked someone at random from the trivial-mailinglist... tada... > If you don't have the time to follow http://qemu-project.org/Contribute/SubmitAPatch, perhaps someone else will do it for you, or not. > > I'm attaching 2 patches. > 1) is just for comments and texts. ( all the .txt, CHANGELOG, > /*comment*/ stuff ) > after applying, the compiled binary should [3] still be same. > 2) some printf() related output. > > ... and I did not test the stuff. Not a good idea :) > > lg > .klemens > > 1] https://github.com/ka7/misspell_fixer > it also relays heavily to the spelling-dictionary of debian-linter > 2] https://github.com/ka7/misspell_fixer/tree/master/motivation > 3] https://wiki.debian.org/ReproducibleBuilds/About From 45bc74375b60db2e82849a186f89af8d1dd05e53 Mon Sep 17 00:00:00 2001 From: klemens Date: Sun, 25 Dec 2016 21:52:46 +0100 Subject: [PATCH 2/2] spelling fixes, might have an side-effect ( print() .. ) --- scripts/tracetool/__init__.py | 2 +- target/ppc/translate_init.c | 2 +- tests/docker/docker.py | 4 ++-- tests/guest-debug/test-gdbstub.py | 2 +- tests/vhost-user-bridge.c | 2 +- 5 files changed, 6 insertions(+), 6 deletions(-) diff --git a/scripts/tracetool/__init__.py b/scripts/tracetool/__init__.py index 365446f..f7d139f 100644 --- a/scripts/tracetool/__init__.py +++ b/scripts/tracetool/__init__.py @@ -35,7 +35,7 @@ def error(*lines): def out(*lines, **kwargs): """Write a set of output lines. - You can use kwargs as a shorthand for mapping variables when formating all + You can use kwargs as a shorthand for mapping variables when formatting all the strings in lines. """ lines = [ l % kwargs for l in lines ] diff --git a/target/ppc/translate_init.c b/target/ppc/translate_init.c index 626e031..cd3ba19 100644 --- a/target/ppc/translate_init.c +++ b/target/ppc/translate_init.c @@ -9861,7 +9861,7 @@ static void ppc_cpu_realizefn(DeviceState *dev, Error **errp) #if !defined(CONFIG_USER_ONLY) if (env->tlb.tlb6) { printf(" %d %s TLB in %d ways\n", - env->nb_tlb, env->id_tlbs ? "splitted" : "merged", + env->nb_tlb, env->id_tlbs ? "split" : "merged", env->nb_ways); } #endif diff --git a/tests/docker/docker.py b/tests/docker/docker.py index 37d8319..61b9504 100755 --- a/tests/docker/docker.py +++ b/tests/docker/docker.py @@ -83,7 +83,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.""" @@ -196,7 +196,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 31ba6c9..88463f7 100644 --- a/tests/guest-debug/test-gdbstub.py +++ b/tests/guest-debug/test-gdbstub.py @@ -121,7 +121,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/vhost-user-bridge.c b/tests/vhost-user-bridge.c index 775e031..ebc4384 100644 --- a/tests/vhost-user-bridge.c +++ b/tests/vhost-user-bridge.c @@ -1426,7 +1426,7 @@ main(int argc, char *argv[]) out: fprintf(stderr, "Usage: %s ", argv[0]); fprintf(stderr, "[-c] [-u ud_socket_path] [-l lhost:lport] [-r rhost:rport]\n"); - fprintf(stderr, "\t-u path to unix doman socket. default: %s\n", + fprintf(stderr, "\t-u path to unix domain socket. default: %s\n", DEFAULT_UD_SOCKET); fprintf(stderr, "\t-l local host and port. default: %s:%s\n", DEFAULT_LHOST, DEFAULT_LPORT); -- 2.1.4