From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56861) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZQAPT-0007w7-82 for qemu-devel@nongnu.org; Fri, 14 Aug 2015 04:32:20 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZQAPP-0002lR-Ie for qemu-devel@nongnu.org; Fri, 14 Aug 2015 04:32:19 -0400 Received: from mx1.redhat.com ([209.132.183.28]:50617) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZQAPP-0002kk-Dw for qemu-devel@nongnu.org; Fri, 14 Aug 2015 04:32:15 -0400 Date: Fri, 14 Aug 2015 09:32:09 +0100 From: "Daniel P. Berrange" Message-ID: <20150814083209.GA7776@redhat.com> References: <1438360263-25445-1-git-send-email-berrange@redhat.com> <1438360263-25445-3-git-send-email-berrange@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [PATCH RFC 02/10] maint: remove double semicolons in many files Reply-To: "Daniel P. Berrange" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: QEMU Developers On Thu, Aug 13, 2015 at 06:57:55PM +0100, Peter Maydell wrote: > On 31 July 2015 at 17:30, Daniel P. Berrange wrote: > > A number of source files have statements accidentally > > terminated by a double semicolon - eg 'foo = bar;;'. > > This is harmless but a mistake none the less. > > > > The tcg/ia64/tcg-target.c file is whitelisted because > > it has valid use of ';;' in a comment containing assembly > > code. > > > > Signed-off-by: Daniel P. Berrange > > --- > > block/vhdx.c | 2 +- > > cfg.mk | 5 ++++- > > hw/arm/vexpress.c | 4 ++-- > > hw/intc/arm_gic.c | 2 +- > > numa.c | 2 +- > > qga/commands-win32.c | 2 +- > > 6 files changed, 10 insertions(+), 7 deletions(-) > > If you kept the "enable the check in cfg.mk" change out of > the patches like these, we could review and commit them > without them being tangled up or waiting on the review of > the syntax-checking infrastructure itself... Sure, I just really wanted to illustrate the use of the syntax check infrastructure with this series of fixes. Now I've done that I'll resubmit just the fixes, while debate over the checking infrastructure continues. Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|