Re: [Qemu-devel] [PATCH] Fix a number of unused-but-set-variable warnings (new with gcc-4.6)

2011-06-08 Thread Christophe Fergeau
On Wed, Jun 08, 2011 at 11:22:06AM +0200, Kevin Wolf wrote: > Am 08.06.2011 10:28, schrieb Hans de Goede: > > --- > > hw/lsi53c895a.c |2 -- > > target-i386/kvm.c |4 ++-- > > 2 files changed, 2 insertions(+), 4 deletions(-) > > SoB is missing. And don't we have multiple patches to fix

Re: [Qemu-devel] [PATCH] Fix a number of unused-but-set-variable warnings (new with gcc-4.6)

2011-06-08 Thread Kevin Wolf
Am 08.06.2011 10:28, schrieb Hans de Goede: > --- > hw/lsi53c895a.c |2 -- > target-i386/kvm.c |4 ++-- > 2 files changed, 2 insertions(+), 4 deletions(-) SoB is missing. And don't we have multiple patches to fix the same thing already? Someone should just merge them... Kevin

[Qemu-devel] [PATCH] Fix a number of unused-but-set-variable warnings (new with gcc-4.6)

2011-06-08 Thread Hans de Goede
--- hw/lsi53c895a.c |2 -- target-i386/kvm.c |4 ++-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/hw/lsi53c895a.c b/hw/lsi53c895a.c index 83084b6..90c6cbc 100644 --- a/hw/lsi53c895a.c +++ b/hw/lsi53c895a.c @@ -889,7 +889,6 @@ static void lsi_do_msgout(LSIState *s)

Re: [Qemu-devel] [PATCH] Fix a number of unused-but-set-variable warnings (new with gcc-4.6)

2011-05-27 Thread Amit Shah
On (Tue) 03 May 2011 [13:03:40], Hans de Goede wrote: > --- > target-i386/kvm.c |4 ++-- > tcg/tcg.c |4 ++-- > 2 files changed, 4 insertions(+), 4 deletions(-) Thanks; just got hit by this. However, there are a couple of whitespace issues: > --- a/tcg/tcg.c > +++ b/tcg/tcg.c >

Re: [Qemu-devel] [PATCH] Fix a number of unused-but-set-variable warnings (new with gcc-4.6)

2011-05-17 Thread Aurelien Jarno
On Tue, May 17, 2011 at 01:25:10PM +0200, Christophe Fergeau wrote: > Hi Hans, > > On Tue, May 03, 2011 at 01:03:40PM +0200, Hans de Goede wrote: > > diff --git a/target-i386/kvm.c b/target-i386/kvm.c > > index a13599d..e9e8d54 100644 > > --- a/target-i386/kvm.c > > +++ b/target-i386/kvm.c > > @@

Re: [Qemu-devel] [PATCH] Fix a number of unused-but-set-variable warnings (new with gcc-4.6)

2011-05-17 Thread Christophe Fergeau
Hi Hans, On Tue, May 03, 2011 at 01:03:40PM +0200, Hans de Goede wrote: > diff --git a/target-i386/kvm.c b/target-i386/kvm.c > index a13599d..e9e8d54 100644 > --- a/target-i386/kvm.c > +++ b/target-i386/kvm.c > @@ -950,7 +950,7 @@ static int kvm_get_xsave(CPUState *env) > @@ -966,7 +966,7 @@ stati

[Qemu-devel] [PATCH] Fix a number of unused-but-set-variable warnings (new with gcc-4.6)

2011-05-03 Thread Hans de Goede
--- target-i386/kvm.c |4 ++-- tcg/tcg.c |4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/target-i386/kvm.c b/target-i386/kvm.c index a13599d..e9e8d54 100644 --- a/target-i386/kvm.c +++ b/target-i386/kvm.c @@ -950,7 +950,7 @@ static int kvm_get_xsave(CPUState