Oh, right, got it. Sorry for the confusion, I’ll take that out of the patch.
Thanks, Lucian Petrut From: Paolo Bonzini<mailto:[email protected]> Sent: Wednesday, May 16, 2018 1:42 PM To: Lucian Petrut<mailto:[email protected]>; [email protected]<mailto:[email protected]> Cc: Alessandro Pilotti<mailto:[email protected]>; Justin Terry<mailto:[email protected]>; Richard Henderson<mailto:[email protected]>; Eduardo Habkost<mailto:[email protected]>; open list:All patches CC here<mailto:[email protected]> 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
