Oh, right, got it. Sorry for the confusion, I’ll take that out of the patch.
Thanks, Lucian Petrut From: Paolo Bonzini<mailto:pbonz...@redhat.com> Sent: Wednesday, May 16, 2018 1:42 PM To: Lucian Petrut<mailto:lpet...@cloudbasesolutions.com>; petrutlucia...@gmail.com<mailto:petrutlucia...@gmail.com> Cc: Alessandro Pilotti<mailto:apilo...@cloudbasesolutions.com>; Justin Terry<mailto:jute...@microsoft.com>; Richard Henderson<mailto:r...@twiddle.net>; Eduardo Habkost<mailto:ehabk...@redhat.com>; open list:All patches CC here<mailto:qemu-devel@nongnu.org> 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
