From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH] libkvm: fix mungled default case for switch in kvm_arch_run Date: Sun, 30 Dec 2007 14:29:49 +0200 Message-ID: <47778F3D.5030100@qumranet.com> References: <20071228042930.GA22639@tapir> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org To: Carlo Marcelo Arenas Belon Return-path: In-Reply-To: <20071228042930.GA22639@tapir> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: kvm-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Errors-To: kvm-devel-bounces-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Id: kvm.vger.kernel.org Carlo Marcelo Arenas Belon wrote: > This patch corrects 03f1b5e137e7e4c8dd51dbaea6779be853f4fde0, that > modified the logic behind kvm_arch_run because of a mangled endif > which accidentally included the default entry for the switch. > > kvm_arch_run as defined in 4f19bdea03330641fd48514ea84d1ed1bf431507 > was affected by returning 0 by default instead of 1 and reporting > the following warning at compile time : > > libkvm-x86.c:201:9: warning: extra tokens at end of #endif directive > > Applied, thanks. -- error compiling committee.c: too many arguments to function ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2005. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/