From mboxrd@z Thu Jan 1 00:00:00 1970 Received: by 2002:a5d:6089:0:0:0:0:0 with SMTP id w9csp2177205wrt; Fri, 14 Dec 2018 07:18:41 -0800 (PST) X-Google-Smtp-Source: AFSGD/XhpJF0FTjIOCzGsjscc6+SqoMW+IQdJ925M0ECvM7PTgd7s9QlIk4lMgca6S54e7jPu0vg X-Received: by 2002:aed:3bb8:: with SMTP id r53mr3174451qte.77.1544800721900; Fri, 14 Dec 2018 07:18:41 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1544800721; cv=none; d=google.com; s=arc-20160816; b=wSgxKbmTEusFsoeJisU4Vbc2NtU3aQjicTblqXqMPuQmXKuFatS0wmHiN54PsmLWK/ UETToAQPqHsCBzPQFTFcIQwB1kNx8Qw00mVs6GVc4d/v9AIRWBf/Zz0vOV7kj4Y4yHE/ Cg/Vcpf057MOgTyEu3p+Ld1T/Po8QooGjFqpIw5DZUzav6dhp1PteuBwfyDZXQUX6l03 cRynr8llXpAQScm1kACE71rDsLnQOUg3T2+YmcxdCukd4wdAUGptgvBOcfL13vmMeoIh GfcjIek3JBDb2ZTH8cRMsAYo/Ks1Jqvx3ersXNy8NMWKfeRjw4wcHk6KpFSdGvyF03wb AtkA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date; bh=Y+k/INGOJtD8HfCs8n0ZVdzo3iiG2oMBNkkk0LdYh40=; b=L+MschTrxPF/PhRqiaOwBAS9UI1CFZpYVJGv20qwnRVIl12f4NTNJTn//CmeFi+RhF RKDhp145mQBMRW+JsELrvmbeSYhbPwHdwa03GT+EMpu7skPrbWP7p5mDTapZif6SOPF+ lsmuBIcZDfD5eQ9AWw/CM523LYCHj45OZrVHf+Ig3lHFQodzhqRvnssbHbh430a6UaLO s8idH8gWNx5tMqCWSUIeLfYFBXYj/mnognCCZA/xC60E2Ii12SZrthcq8Sez45SSO1q4 82DC76XpYGzf4qrNbQQfmhmh4r72TapP+bfwOPqQW7SMx+wR5OZ5jMbE2hq3VsFb64nw rgIw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of mst@redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=mst@redhat.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from mx1.redhat.com (mx1.redhat.com. [209.132.183.28]) by mx.google.com with ESMTPS id s7si454334qvr.49.2018.12.14.07.18.41 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 14 Dec 2018 07:18:41 -0800 (PST) Received-SPF: pass (google.com: domain of mst@redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; Authentication-Results: mx.google.com; spf=pass (google.com: domain of mst@redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=mst@redhat.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id AB96831256C8; Fri, 14 Dec 2018 15:18:40 +0000 (UTC) Received: from redhat.com (ovpn-121-101.rdu2.redhat.com [10.10.121.101]) by smtp.corp.redhat.com (Postfix) with SMTP id CDA58105705F; Fri, 14 Dec 2018 15:17:56 +0000 (UTC) Date: Fri, 14 Dec 2018 10:17:56 -0500 From: "Michael S. Tsirkin" To: Paolo Bonzini Cc: qemu-devel@nongnu.org, Stefan Hajnoczi , Kevin Wolf , Max Reitz , Stefan Weil , Viktor Prutyanov , Richard Henderson , Peter Maydell , =?iso-8859-1?Q?Marc-Andr=E9?= Lureau , Amit Shah , Stefano Stabellini , Anthony Perard , Andrzej Zaborowski , Gerd Hoffmann , Eduardo Habkost , Marcel Apfelbaum , John Snow , Aurelien Jarno , Aleksandar Markovic , Stefan Markovic , Jason Wang , Jiri Pirko , Dmitry Fleytman , David Gibson , =?iso-8859-1?Q?Herv=E9?= Poussineau , Fam Zheng , Magnus Damm , Igor Mammedov , Riku Voipio , Laurent Vivier , Eric Blake , Thomas Huth , "Edgar E. Iglesias" , Alex =?iso-8859-1?Q?Benn=E9e?= , Daniel =?iso-8859-1?Q?P=2E_Berrang=E9?= , "open list:bochs" , "open list:Stellaris" , "open list:X86" , "open list:ppc4xx" Subject: Re: [PATCH 2/2] avoid TABs in files that only contain a few Message-ID: <20181214101725-mutt-send-email-mst@kernel.org> References: <20181213223737.11793-1-pbonzini@redhat.com> <20181213223737.11793-3-pbonzini@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20181213223737.11793-3-pbonzini@redhat.com> X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.46]); Fri, 14 Dec 2018 15:18:41 +0000 (UTC) X-TUID: ngvww2sT3rh3 On Thu, Dec 13, 2018 at 11:37:37PM +0100, Paolo Bonzini wrote: > Most files that have TABs only contain a handful of them. Change > them to spaces so that we don't confuse people. > > disas, standard-headers, linux-headers and libdecnumber are imported > from other projects and probably should be exempted from the check. > Outside those, after this patch the following files still contain both > 8-space and TAB sequences at the beginning of the line. Many of them > have a majority of TABs, or were initially committed with all tabs. > > bsd-user/i386/target_syscall.h > bsd-user/x86_64/target_syscall.h > crypto/aes.c > hw/audio/fmopl.c > hw/audio/fmopl.h > hw/block/tc58128.c > hw/display/cirrus_vga.c > hw/display/xenfb.c > hw/dma/etraxfs_dma.c > hw/intc/sh_intc.c > hw/misc/mst_fpga.c > hw/net/pcnet.c > hw/sh4/sh7750.c > hw/timer/m48t59.c > hw/timer/sh_timer.c > include/crypto/aes.h > include/disas/bfd.h > include/hw/sh4/sh.h > libdecnumber/decNumber.c > linux-headers/asm-generic/unistd.h > linux-headers/linux/kvm.h > linux-user/alpha/target_syscall.h > linux-user/arm/nwfpe/double_cpdo.c > linux-user/arm/nwfpe/fpa11_cpdt.c > linux-user/arm/nwfpe/fpa11_cprt.c > linux-user/arm/nwfpe/fpa11.h > linux-user/flat.h > linux-user/flatload.c > linux-user/i386/target_syscall.h > linux-user/ppc/target_syscall.h > linux-user/sparc/target_syscall.h > linux-user/syscall.c > linux-user/syscall_defs.h > linux-user/x86_64/target_syscall.h > slirp/cksum.c > slirp/if.c > slirp/ip.h > slirp/ip_icmp.c > slirp/ip_icmp.h > slirp/ip_input.c > slirp/ip_output.c > slirp/mbuf.c > slirp/misc.c > slirp/sbuf.c > slirp/socket.c > slirp/socket.h > slirp/tcp_input.c > slirp/tcpip.h > slirp/tcp_output.c > slirp/tcp_subr.c > slirp/tcp_timer.c > slirp/tftp.c > slirp/udp.c > slirp/udp.h > target/cris/cpu.h > target/cris/mmu.c > target/cris/op_helper.c > target/sh4/helper.c > target/sh4/op_helper.c > target/sh4/translate.c > tcg/sparc/tcg-target.inc.c > tests/tcg/cris/check_addo.c > tests/tcg/cris/check_moveq.c > tests/tcg/cris/check_swap.c > tests/tcg/multiarch/test-mmap.c > ui/vnc-enc-hextile-template.h > ui/vnc-enc-zywrle.h > util/envlist.c > util/readline.c > > The following have only TABs: > > bsd-user/i386/target_signal.h > bsd-user/sparc64/target_signal.h > bsd-user/sparc64/target_syscall.h > bsd-user/sparc/target_signal.h > bsd-user/sparc/target_syscall.h > bsd-user/x86_64/target_signal.h > crypto/desrfb.c > hw/audio/intel-hda-defs.h > hw/core/uboot_image.h > hw/sh4/sh7750_regnames.c > hw/sh4/sh7750_regs.h > include/hw/cris/etraxfs_dma.h > linux-user/alpha/termbits.h > linux-user/arm/nwfpe/fpopcode.h > linux-user/arm/nwfpe/fpsr.h > linux-user/arm/syscall_nr.h > linux-user/arm/target_signal.h > linux-user/cris/target_signal.h > linux-user/i386/target_signal.h > linux-user/linux_loop.h > linux-user/m68k/target_signal.h > linux-user/microblaze/target_signal.h > linux-user/mips64/target_signal.h > linux-user/mips/target_signal.h > linux-user/mips/target_syscall.h > linux-user/mips/termbits.h > linux-user/ppc/target_signal.h > linux-user/sh4/target_signal.h > linux-user/sh4/termbits.h > linux-user/sparc64/target_syscall.h > linux-user/sparc/target_signal.h > linux-user/x86_64/target_signal.h > linux-user/x86_64/termbits.h > pc-bios/optionrom/optionrom.h > slirp/mbuf.h > slirp/misc.h > slirp/sbuf.h > slirp/tcp.h > slirp/tcp_timer.h > slirp/tcp_var.h > target/i386/svm.h > target/sparc/asi.h > target/xtensa/core-dc232b/xtensa-modules.inc.c > target/xtensa/core-dc233c/xtensa-modules.inc.c > target/xtensa/core-de212/core-isa.h > target/xtensa/core-de212/xtensa-modules.inc.c > target/xtensa/core-fsf/xtensa-modules.inc.c > target/xtensa/core-sample_controller/core-isa.h > target/xtensa/core-sample_controller/xtensa-modules.inc.c > target/xtensa/core-test_kc705_be/core-isa.h > target/xtensa/core-test_kc705_be/xtensa-modules.inc.c > tests/tcg/cris/check_abs.c > tests/tcg/cris/check_addc.c > tests/tcg/cris/check_addcm.c > tests/tcg/cris/check_addoq.c > tests/tcg/cris/check_bound.c > tests/tcg/cris/check_ftag.c > tests/tcg/cris/check_int64.c > tests/tcg/cris/check_lz.c > tests/tcg/cris/check_openpf5.c > tests/tcg/cris/check_sigalrm.c > tests/tcg/cris/crisutils.h > tests/tcg/cris/sys.c > tests/tcg/i386/test-i386-ssse3.c > ui/vgafont.h > > Signed-off-by: Paolo Bonzini Seems sane: Reviewed-by: Michael S. Tsirkin From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: [PATCH 2/2] avoid TABs in files that only contain a few Date: Fri, 14 Dec 2018 10:17:56 -0500 Message-ID: <20181214101725-mutt-send-email-mst@kernel.org> References: <20181213223737.11793-1-pbonzini@redhat.com> <20181213223737.11793-3-pbonzini@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from all-amaz-eas1.inumbo.com ([34.197.232.57] helo=us1-amaz-eas2.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.89) (envelope-from ) id 1gXpEg-0005rr-U1 for xen-devel@lists.xenproject.org; Fri, 14 Dec 2018 15:18:42 +0000 Content-Disposition: inline In-Reply-To: <20181213223737.11793-3-pbonzini@redhat.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Sender: "Xen-devel" To: Paolo Bonzini Cc: Peter Maydell , Dmitry Fleytman , Jason Wang , qemu-devel@nongnu.org, Gerd Hoffmann , "Edgar E. Iglesias" , Fam Zheng , Eric Blake , Stefano Stabellini , "open list:bochs" , Stefan Markovic , Alex =?iso-8859-1?Q?Benn=E9e?= , Magnus Damm , =?iso-8859-1?Q?Herv=E9?= Poussineau , Marcel Apfelbaum , Anthony Perard , =?iso-8859-1?Q?Marc-Andr=E9?= Lureau , David Gibson , Andrzej Zaborowski , Thomas Huth , Jiri Pirko , Edu List-Id: xen-devel@lists.xenproject.org T24gVGh1LCBEZWMgMTMsIDIwMTggYXQgMTE6Mzc6MzdQTSArMDEwMCwgUGFvbG8gQm9uemluaSB3 cm90ZToKPiBNb3N0IGZpbGVzIHRoYXQgaGF2ZSBUQUJzIG9ubHkgY29udGFpbiBhIGhhbmRmdWwg b2YgdGhlbS4gIENoYW5nZQo+IHRoZW0gdG8gc3BhY2VzIHNvIHRoYXQgd2UgZG9uJ3QgY29uZnVz ZSBwZW9wbGUuCj4gCj4gZGlzYXMsIHN0YW5kYXJkLWhlYWRlcnMsIGxpbnV4LWhlYWRlcnMgYW5k IGxpYmRlY251bWJlciBhcmUgaW1wb3J0ZWQKPiBmcm9tIG90aGVyIHByb2plY3RzIGFuZCBwcm9i YWJseSBzaG91bGQgYmUgZXhlbXB0ZWQgZnJvbSB0aGUgY2hlY2suCj4gT3V0c2lkZSB0aG9zZSwg YWZ0ZXIgdGhpcyBwYXRjaCB0aGUgZm9sbG93aW5nIGZpbGVzIHN0aWxsIGNvbnRhaW4gYm90aAo+ IDgtc3BhY2UgYW5kIFRBQiBzZXF1ZW5jZXMgYXQgdGhlIGJlZ2lubmluZyBvZiB0aGUgbGluZS4g IE1hbnkgb2YgdGhlbQo+IGhhdmUgYSBtYWpvcml0eSBvZiBUQUJzLCBvciB3ZXJlIGluaXRpYWxs eSBjb21taXR0ZWQgd2l0aCBhbGwgdGFicy4KPiAKPiAgICAgYnNkLXVzZXIvaTM4Ni90YXJnZXRf c3lzY2FsbC5oCj4gICAgIGJzZC11c2VyL3g4Nl82NC90YXJnZXRfc3lzY2FsbC5oCj4gICAgIGNy eXB0by9hZXMuYwo+ICAgICBody9hdWRpby9mbW9wbC5jCj4gICAgIGh3L2F1ZGlvL2Ztb3BsLmgK PiAgICAgaHcvYmxvY2svdGM1ODEyOC5jCj4gICAgIGh3L2Rpc3BsYXkvY2lycnVzX3ZnYS5jCj4g ICAgIGh3L2Rpc3BsYXkveGVuZmIuYwo+ICAgICBody9kbWEvZXRyYXhmc19kbWEuYwo+ICAgICBo dy9pbnRjL3NoX2ludGMuYwo+ICAgICBody9taXNjL21zdF9mcGdhLmMKPiAgICAgaHcvbmV0L3Bj bmV0LmMKPiAgICAgaHcvc2g0L3NoNzc1MC5jCj4gICAgIGh3L3RpbWVyL200OHQ1OS5jCj4gICAg IGh3L3RpbWVyL3NoX3RpbWVyLmMKPiAgICAgaW5jbHVkZS9jcnlwdG8vYWVzLmgKPiAgICAgaW5j bHVkZS9kaXNhcy9iZmQuaAo+ICAgICBpbmNsdWRlL2h3L3NoNC9zaC5oCj4gICAgIGxpYmRlY251 bWJlci9kZWNOdW1iZXIuYwo+ICAgICBsaW51eC1oZWFkZXJzL2FzbS1nZW5lcmljL3VuaXN0ZC5o Cj4gICAgIGxpbnV4LWhlYWRlcnMvbGludXgva3ZtLmgKPiAgICAgbGludXgtdXNlci9hbHBoYS90 YXJnZXRfc3lzY2FsbC5oCj4gICAgIGxpbnV4LXVzZXIvYXJtL253ZnBlL2RvdWJsZV9jcGRvLmMK PiAgICAgbGludXgtdXNlci9hcm0vbndmcGUvZnBhMTFfY3BkdC5jCj4gICAgIGxpbnV4LXVzZXIv YXJtL253ZnBlL2ZwYTExX2NwcnQuYwo+ICAgICBsaW51eC11c2VyL2FybS9ud2ZwZS9mcGExMS5o Cj4gICAgIGxpbnV4LXVzZXIvZmxhdC5oCj4gICAgIGxpbnV4LXVzZXIvZmxhdGxvYWQuYwo+ICAg ICBsaW51eC11c2VyL2kzODYvdGFyZ2V0X3N5c2NhbGwuaAo+ICAgICBsaW51eC11c2VyL3BwYy90 YXJnZXRfc3lzY2FsbC5oCj4gICAgIGxpbnV4LXVzZXIvc3BhcmMvdGFyZ2V0X3N5c2NhbGwuaAo+ ICAgICBsaW51eC11c2VyL3N5c2NhbGwuYwo+ICAgICBsaW51eC11c2VyL3N5c2NhbGxfZGVmcy5o Cj4gICAgIGxpbnV4LXVzZXIveDg2XzY0L3RhcmdldF9zeXNjYWxsLmgKPiAgICAgc2xpcnAvY2tz dW0uYwo+ICAgICBzbGlycC9pZi5jCj4gICAgIHNsaXJwL2lwLmgKPiAgICAgc2xpcnAvaXBfaWNt cC5jCj4gICAgIHNsaXJwL2lwX2ljbXAuaAo+ICAgICBzbGlycC9pcF9pbnB1dC5jCj4gICAgIHNs aXJwL2lwX291dHB1dC5jCj4gICAgIHNsaXJwL21idWYuYwo+ICAgICBzbGlycC9taXNjLmMKPiAg ICAgc2xpcnAvc2J1Zi5jCj4gICAgIHNsaXJwL3NvY2tldC5jCj4gICAgIHNsaXJwL3NvY2tldC5o Cj4gICAgIHNsaXJwL3RjcF9pbnB1dC5jCj4gICAgIHNsaXJwL3RjcGlwLmgKPiAgICAgc2xpcnAv dGNwX291dHB1dC5jCj4gICAgIHNsaXJwL3RjcF9zdWJyLmMKPiAgICAgc2xpcnAvdGNwX3RpbWVy LmMKPiAgICAgc2xpcnAvdGZ0cC5jCj4gICAgIHNsaXJwL3VkcC5jCj4gICAgIHNsaXJwL3VkcC5o Cj4gICAgIHRhcmdldC9jcmlzL2NwdS5oCj4gICAgIHRhcmdldC9jcmlzL21tdS5jCj4gICAgIHRh cmdldC9jcmlzL29wX2hlbHBlci5jCj4gICAgIHRhcmdldC9zaDQvaGVscGVyLmMKPiAgICAgdGFy Z2V0L3NoNC9vcF9oZWxwZXIuYwo+ICAgICB0YXJnZXQvc2g0L3RyYW5zbGF0ZS5jCj4gICAgIHRj Zy9zcGFyYy90Y2ctdGFyZ2V0LmluYy5jCj4gICAgIHRlc3RzL3RjZy9jcmlzL2NoZWNrX2FkZG8u Ywo+ICAgICB0ZXN0cy90Y2cvY3Jpcy9jaGVja19tb3ZlcS5jCj4gICAgIHRlc3RzL3RjZy9jcmlz L2NoZWNrX3N3YXAuYwo+ICAgICB0ZXN0cy90Y2cvbXVsdGlhcmNoL3Rlc3QtbW1hcC5jCj4gICAg IHVpL3ZuYy1lbmMtaGV4dGlsZS10ZW1wbGF0ZS5oCj4gICAgIHVpL3ZuYy1lbmMtenl3cmxlLmgK PiAgICAgdXRpbC9lbnZsaXN0LmMKPiAgICAgdXRpbC9yZWFkbGluZS5jCj4gCj4gVGhlIGZvbGxv d2luZyBoYXZlIG9ubHkgVEFCczoKPiAKPiAgICAgYnNkLXVzZXIvaTM4Ni90YXJnZXRfc2lnbmFs LmgKPiAgICAgYnNkLXVzZXIvc3BhcmM2NC90YXJnZXRfc2lnbmFsLmgKPiAgICAgYnNkLXVzZXIv c3BhcmM2NC90YXJnZXRfc3lzY2FsbC5oCj4gICAgIGJzZC11c2VyL3NwYXJjL3RhcmdldF9zaWdu YWwuaAo+ICAgICBic2QtdXNlci9zcGFyYy90YXJnZXRfc3lzY2FsbC5oCj4gICAgIGJzZC11c2Vy L3g4Nl82NC90YXJnZXRfc2lnbmFsLmgKPiAgICAgY3J5cHRvL2Rlc3JmYi5jCj4gICAgIGh3L2F1 ZGlvL2ludGVsLWhkYS1kZWZzLmgKPiAgICAgaHcvY29yZS91Ym9vdF9pbWFnZS5oCj4gICAgIGh3 L3NoNC9zaDc3NTBfcmVnbmFtZXMuYwo+ICAgICBody9zaDQvc2g3NzUwX3JlZ3MuaAo+ICAgICBp bmNsdWRlL2h3L2NyaXMvZXRyYXhmc19kbWEuaAo+ICAgICBsaW51eC11c2VyL2FscGhhL3Rlcm1i aXRzLmgKPiAgICAgbGludXgtdXNlci9hcm0vbndmcGUvZnBvcGNvZGUuaAo+ICAgICBsaW51eC11 c2VyL2FybS9ud2ZwZS9mcHNyLmgKPiAgICAgbGludXgtdXNlci9hcm0vc3lzY2FsbF9uci5oCj4g ICAgIGxpbnV4LXVzZXIvYXJtL3RhcmdldF9zaWduYWwuaAo+ICAgICBsaW51eC11c2VyL2NyaXMv dGFyZ2V0X3NpZ25hbC5oCj4gICAgIGxpbnV4LXVzZXIvaTM4Ni90YXJnZXRfc2lnbmFsLmgKPiAg ICAgbGludXgtdXNlci9saW51eF9sb29wLmgKPiAgICAgbGludXgtdXNlci9tNjhrL3RhcmdldF9z aWduYWwuaAo+ICAgICBsaW51eC11c2VyL21pY3JvYmxhemUvdGFyZ2V0X3NpZ25hbC5oCj4gICAg IGxpbnV4LXVzZXIvbWlwczY0L3RhcmdldF9zaWduYWwuaAo+ICAgICBsaW51eC11c2VyL21pcHMv dGFyZ2V0X3NpZ25hbC5oCj4gICAgIGxpbnV4LXVzZXIvbWlwcy90YXJnZXRfc3lzY2FsbC5oCj4g ICAgIGxpbnV4LXVzZXIvbWlwcy90ZXJtYml0cy5oCj4gICAgIGxpbnV4LXVzZXIvcHBjL3Rhcmdl dF9zaWduYWwuaAo+ICAgICBsaW51eC11c2VyL3NoNC90YXJnZXRfc2lnbmFsLmgKPiAgICAgbGlu dXgtdXNlci9zaDQvdGVybWJpdHMuaAo+ICAgICBsaW51eC11c2VyL3NwYXJjNjQvdGFyZ2V0X3N5 c2NhbGwuaAo+ICAgICBsaW51eC11c2VyL3NwYXJjL3RhcmdldF9zaWduYWwuaAo+ICAgICBsaW51 eC11c2VyL3g4Nl82NC90YXJnZXRfc2lnbmFsLmgKPiAgICAgbGludXgtdXNlci94ODZfNjQvdGVy bWJpdHMuaAo+ICAgICBwYy1iaW9zL29wdGlvbnJvbS9vcHRpb25yb20uaAo+ICAgICBzbGlycC9t YnVmLmgKPiAgICAgc2xpcnAvbWlzYy5oCj4gICAgIHNsaXJwL3NidWYuaAo+ICAgICBzbGlycC90 Y3AuaAo+ICAgICBzbGlycC90Y3BfdGltZXIuaAo+ICAgICBzbGlycC90Y3BfdmFyLmgKPiAgICAg dGFyZ2V0L2kzODYvc3ZtLmgKPiAgICAgdGFyZ2V0L3NwYXJjL2FzaS5oCj4gICAgIHRhcmdldC94 dGVuc2EvY29yZS1kYzIzMmIveHRlbnNhLW1vZHVsZXMuaW5jLmMKPiAgICAgdGFyZ2V0L3h0ZW5z YS9jb3JlLWRjMjMzYy94dGVuc2EtbW9kdWxlcy5pbmMuYwo+ICAgICB0YXJnZXQveHRlbnNhL2Nv cmUtZGUyMTIvY29yZS1pc2EuaAo+ICAgICB0YXJnZXQveHRlbnNhL2NvcmUtZGUyMTIveHRlbnNh LW1vZHVsZXMuaW5jLmMKPiAgICAgdGFyZ2V0L3h0ZW5zYS9jb3JlLWZzZi94dGVuc2EtbW9kdWxl cy5pbmMuYwo+ICAgICB0YXJnZXQveHRlbnNhL2NvcmUtc2FtcGxlX2NvbnRyb2xsZXIvY29yZS1p c2EuaAo+ICAgICB0YXJnZXQveHRlbnNhL2NvcmUtc2FtcGxlX2NvbnRyb2xsZXIveHRlbnNhLW1v ZHVsZXMuaW5jLmMKPiAgICAgdGFyZ2V0L3h0ZW5zYS9jb3JlLXRlc3Rfa2M3MDVfYmUvY29yZS1p c2EuaAo+ICAgICB0YXJnZXQveHRlbnNhL2NvcmUtdGVzdF9rYzcwNV9iZS94dGVuc2EtbW9kdWxl cy5pbmMuYwo+ICAgICB0ZXN0cy90Y2cvY3Jpcy9jaGVja19hYnMuYwo+ICAgICB0ZXN0cy90Y2cv Y3Jpcy9jaGVja19hZGRjLmMKPiAgICAgdGVzdHMvdGNnL2NyaXMvY2hlY2tfYWRkY20uYwo+ICAg ICB0ZXN0cy90Y2cvY3Jpcy9jaGVja19hZGRvcS5jCj4gICAgIHRlc3RzL3RjZy9jcmlzL2NoZWNr X2JvdW5kLmMKPiAgICAgdGVzdHMvdGNnL2NyaXMvY2hlY2tfZnRhZy5jCj4gICAgIHRlc3RzL3Rj Zy9jcmlzL2NoZWNrX2ludDY0LmMKPiAgICAgdGVzdHMvdGNnL2NyaXMvY2hlY2tfbHouYwo+ICAg ICB0ZXN0cy90Y2cvY3Jpcy9jaGVja19vcGVucGY1LmMKPiAgICAgdGVzdHMvdGNnL2NyaXMvY2hl Y2tfc2lnYWxybS5jCj4gICAgIHRlc3RzL3RjZy9jcmlzL2NyaXN1dGlscy5oCj4gICAgIHRlc3Rz L3RjZy9jcmlzL3N5cy5jCj4gICAgIHRlc3RzL3RjZy9pMzg2L3Rlc3QtaTM4Ni1zc3NlMy5jCj4g ICAgIHVpL3ZnYWZvbnQuaAo+IAo+IFNpZ25lZC1vZmYtYnk6IFBhb2xvIEJvbnppbmkgPHBib256 aW5pQHJlZGhhdC5jb20+CgpTZWVtcyBzYW5lOgpSZXZpZXdlZC1ieTogTWljaGFlbCBTLiBUc2ly a2luIDxtc3RAcmVkaGF0LmNvbT4KCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fClhlbi1kZXZlbCBtYWlsaW5nIGxpc3QKWGVuLWRldmVsQGxpc3RzLnhlbnBy b2plY3Qub3JnCmh0dHBzOi8vbGlzdHMueGVucHJvamVjdC5vcmcvbWFpbG1hbi9saXN0aW5mby94 ZW4tZGV2ZWw= From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51401) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gXpEv-0004rV-9b for qemu-devel@nongnu.org; Fri, 14 Dec 2018 10:18:58 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gXpEr-00033k-3U for qemu-devel@nongnu.org; Fri, 14 Dec 2018 10:18:57 -0500 Date: Fri, 14 Dec 2018 10:17:56 -0500 From: "Michael S. Tsirkin" Message-ID: <20181214101725-mutt-send-email-mst@kernel.org> References: <20181213223737.11793-1-pbonzini@redhat.com> <20181213223737.11793-3-pbonzini@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20181213223737.11793-3-pbonzini@redhat.com> Subject: Re: [Qemu-devel] [PATCH 2/2] avoid TABs in files that only contain a few List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: qemu-devel@nongnu.org, Stefan Hajnoczi , Kevin Wolf , Max Reitz , Stefan Weil , Viktor Prutyanov , Richard Henderson , Peter Maydell , =?iso-8859-1?Q?Marc-Andr=E9?= Lureau , Amit Shah , Stefano Stabellini , Anthony Perard , Andrzej Zaborowski , Gerd Hoffmann , Eduardo Habkost , Marcel Apfelbaum , John Snow , Aurelien Jarno , Aleksandar Markovic , Stefan Markovic , Jason Wang , Jiri Pirko , Dmitry Fleytman , David Gibson , =?iso-8859-1?Q?Herv=E9?= Poussineau , Fam Zheng , Magnus Damm , Igor Mammedov , Riku Voipio , Laurent Vivier , Eric Blake , Thomas Huth , "Edgar E. Iglesias" , Alex =?iso-8859-1?Q?Benn=E9e?= , Daniel =?iso-8859-1?Q?P=2E_Berrang=E9?= , "open list:bochs" , "open list:Stellaris" , "open list:X86" , "open list:ppc4xx" On Thu, Dec 13, 2018 at 11:37:37PM +0100, Paolo Bonzini wrote: > Most files that have TABs only contain a handful of them. Change > them to spaces so that we don't confuse people. > > disas, standard-headers, linux-headers and libdecnumber are imported > from other projects and probably should be exempted from the check. > Outside those, after this patch the following files still contain both > 8-space and TAB sequences at the beginning of the line. Many of them > have a majority of TABs, or were initially committed with all tabs. > > bsd-user/i386/target_syscall.h > bsd-user/x86_64/target_syscall.h > crypto/aes.c > hw/audio/fmopl.c > hw/audio/fmopl.h > hw/block/tc58128.c > hw/display/cirrus_vga.c > hw/display/xenfb.c > hw/dma/etraxfs_dma.c > hw/intc/sh_intc.c > hw/misc/mst_fpga.c > hw/net/pcnet.c > hw/sh4/sh7750.c > hw/timer/m48t59.c > hw/timer/sh_timer.c > include/crypto/aes.h > include/disas/bfd.h > include/hw/sh4/sh.h > libdecnumber/decNumber.c > linux-headers/asm-generic/unistd.h > linux-headers/linux/kvm.h > linux-user/alpha/target_syscall.h > linux-user/arm/nwfpe/double_cpdo.c > linux-user/arm/nwfpe/fpa11_cpdt.c > linux-user/arm/nwfpe/fpa11_cprt.c > linux-user/arm/nwfpe/fpa11.h > linux-user/flat.h > linux-user/flatload.c > linux-user/i386/target_syscall.h > linux-user/ppc/target_syscall.h > linux-user/sparc/target_syscall.h > linux-user/syscall.c > linux-user/syscall_defs.h > linux-user/x86_64/target_syscall.h > slirp/cksum.c > slirp/if.c > slirp/ip.h > slirp/ip_icmp.c > slirp/ip_icmp.h > slirp/ip_input.c > slirp/ip_output.c > slirp/mbuf.c > slirp/misc.c > slirp/sbuf.c > slirp/socket.c > slirp/socket.h > slirp/tcp_input.c > slirp/tcpip.h > slirp/tcp_output.c > slirp/tcp_subr.c > slirp/tcp_timer.c > slirp/tftp.c > slirp/udp.c > slirp/udp.h > target/cris/cpu.h > target/cris/mmu.c > target/cris/op_helper.c > target/sh4/helper.c > target/sh4/op_helper.c > target/sh4/translate.c > tcg/sparc/tcg-target.inc.c > tests/tcg/cris/check_addo.c > tests/tcg/cris/check_moveq.c > tests/tcg/cris/check_swap.c > tests/tcg/multiarch/test-mmap.c > ui/vnc-enc-hextile-template.h > ui/vnc-enc-zywrle.h > util/envlist.c > util/readline.c > > The following have only TABs: > > bsd-user/i386/target_signal.h > bsd-user/sparc64/target_signal.h > bsd-user/sparc64/target_syscall.h > bsd-user/sparc/target_signal.h > bsd-user/sparc/target_syscall.h > bsd-user/x86_64/target_signal.h > crypto/desrfb.c > hw/audio/intel-hda-defs.h > hw/core/uboot_image.h > hw/sh4/sh7750_regnames.c > hw/sh4/sh7750_regs.h > include/hw/cris/etraxfs_dma.h > linux-user/alpha/termbits.h > linux-user/arm/nwfpe/fpopcode.h > linux-user/arm/nwfpe/fpsr.h > linux-user/arm/syscall_nr.h > linux-user/arm/target_signal.h > linux-user/cris/target_signal.h > linux-user/i386/target_signal.h > linux-user/linux_loop.h > linux-user/m68k/target_signal.h > linux-user/microblaze/target_signal.h > linux-user/mips64/target_signal.h > linux-user/mips/target_signal.h > linux-user/mips/target_syscall.h > linux-user/mips/termbits.h > linux-user/ppc/target_signal.h > linux-user/sh4/target_signal.h > linux-user/sh4/termbits.h > linux-user/sparc64/target_syscall.h > linux-user/sparc/target_signal.h > linux-user/x86_64/target_signal.h > linux-user/x86_64/termbits.h > pc-bios/optionrom/optionrom.h > slirp/mbuf.h > slirp/misc.h > slirp/sbuf.h > slirp/tcp.h > slirp/tcp_timer.h > slirp/tcp_var.h > target/i386/svm.h > target/sparc/asi.h > target/xtensa/core-dc232b/xtensa-modules.inc.c > target/xtensa/core-dc233c/xtensa-modules.inc.c > target/xtensa/core-de212/core-isa.h > target/xtensa/core-de212/xtensa-modules.inc.c > target/xtensa/core-fsf/xtensa-modules.inc.c > target/xtensa/core-sample_controller/core-isa.h > target/xtensa/core-sample_controller/xtensa-modules.inc.c > target/xtensa/core-test_kc705_be/core-isa.h > target/xtensa/core-test_kc705_be/xtensa-modules.inc.c > tests/tcg/cris/check_abs.c > tests/tcg/cris/check_addc.c > tests/tcg/cris/check_addcm.c > tests/tcg/cris/check_addoq.c > tests/tcg/cris/check_bound.c > tests/tcg/cris/check_ftag.c > tests/tcg/cris/check_int64.c > tests/tcg/cris/check_lz.c > tests/tcg/cris/check_openpf5.c > tests/tcg/cris/check_sigalrm.c > tests/tcg/cris/crisutils.h > tests/tcg/cris/sys.c > tests/tcg/i386/test-i386-ssse3.c > ui/vgafont.h > > Signed-off-by: Paolo Bonzini Seems sane: Reviewed-by: Michael S. Tsirkin