Re: [Qemu-devel] [PATCH v2 3/3] WHPX: fix some compiler warnings

[Lucian Petrut]

Oh, right, got it. Sorry for the confusion, I’ll take that out of the patch.



Thanks,

Lucian Petrut



From: Paolo Bonzini<mailto:address@hidden>
Sent: Wednesday, May 16, 2018 1:42 PM
To: Lucian Petrut<mailto:address@hidden>; address@hidden<mailto:address@hidden>
Cc: Alessandro Pilotti<mailto:address@hidden>; Justin 
Terry<mailto:address@hidden>; Richard Henderson<mailto:address@hidden>; Eduardo 
Habkost<mailto:address@hidden>; open list:All patches CC 
here<mailto:address@hidden>
Subject: Re: [PATCH v2 3/3] WHPX: fix some compiler warnings



On 16/05/2018 12:21, Lucian Petrut wrote:
> Hi,
>
> Thanks for reviewing those changes.
>
> /“can you send a patch with the bugfix only”/
>
> By “the bugfix only” you mean the previous commit, the one that
> dynamically loads the DLLs?
>
> FWIW, without this patch, i386 targets won’t compile, so I’d consider
> that an improvement. Probably I should’ve explicitly mentioned this in
> the commit message.

I mean this patch, but without the switch from {0} initializers to memset.

I've already queued patch 2 independently, so you are set with that one!

Thanks,

Paolo