On Fri, Oct 23, 2020 at 10:53 AM Neal Gompa <ngompa13@xxxxxxxxx> wrote: > > On Wed, Oct 21, 2020 at 11:45 AM Michal Privoznik <mprivozn@xxxxxxxxxx> wrote: > > > > The more I look into the code the more things to fix I find. Well, here > > are some fixes. > > > > Michal Prívozník (4): > > hyperv: Don't overwrite errors from hypervCreateInvokeParamsList() > > hyperv: Use hypervRequestStateChange() in hypervDomainSuspend() > > hyperv: Use two empty lines between functions > > hyperv: Reformat > > > > src/hyperv/hyperv_driver.c | 186 +++++++++--------------- > > src/hyperv/hyperv_util.c | 1 - > > src/hyperv/hyperv_wmi.c | 288 +++++++++++++++++++------------------ > > src/hyperv/hyperv_wmi.h | 15 +- > > 4 files changed, 229 insertions(+), 261 deletions(-) > > > > -- > > 2.26.2 > > > > Series LGTM. > > Reviewed-by: Neal Gompa <ngompa13@xxxxxxxxx> > > But... uhh... there's also this series from Matt Coleman, which I just > reviewed: https://www.redhat.com/archives/libvir-list/2020-October/msg01282.html > > Can we put the two together somehow? > Ignore me, apparently this was seen by him already (his review was in spam...) -- 真実はいつも一つ！/ Always, there's only one truth!
