Re: [Qemu-devel] [PATCH 0/2] Style and bug on linux-user/signal.c

2012-03-15 Thread Alex Barcelo
Thanks and sorry! I forgot to add a CC when doing the git send-email. And I thought that I saw some pulls to PPC, so I assumed that the patch was incomplete (I didn't prepare it very well), and thought that better to do a style patch next to it. And comment it better. On Thu, Mar 15, 2012 at 12:4

Re: [Qemu-devel] [PATCH 0/2] Style and bug on linux-user/signal.c

2012-03-15 Thread Alexander Graf
On 15.03.2012, at 12:41, Stefan Hajnoczi wrote: > On Thu, Mar 15, 2012 at 8:52 AM, Alex Barcelo wrote: >> I tried to send a trivial patch and a v2, but I did it horribly wrong >> and seems that have not yet been updated. I thought that maybe >> it was a good moment to do things "betterly". > >

Re: [Qemu-devel] [PATCH 0/2] Style and bug on linux-user/signal.c

2012-03-15 Thread Stefan Hajnoczi
On Thu, Mar 15, 2012 at 8:52 AM, Alex Barcelo wrote: > I tried to send a trivial patch and a v2, but I did it horribly wrong > and seems that have not yet been updated. I thought that maybe > it was a good moment to do things "betterly". Alex Graf mentioned he'd apply it to ppc-next, I have CCed

[Qemu-devel] [PATCH 0/2] Style and bug on linux-user/signal.c

2012-03-15 Thread Alex Barcelo
I tried to send a trivial patch and a v2, but I did it horribly wrong and seems that have not yet been updated. I thought that maybe it was a good moment to do things "betterly". The bug is really a typo, because everywhere there is the same 0 comparison. But each comparison is different one from