On Mon, Mar 25, 2019 at 09:15:23 +0100, Erik Skultety wrote:
On Mon, Mar 25, 2019 at 09:14:38AM +0100, Erik Skultety wrote:
> You're missing:
> - commit message explaining the change
> - Your full name as author
> - compliance with developer certificate of origin, see [1]
https://www.redhat.com/archives/libvir-list/2019-March/msg01148.html
>
> On Sat, Mar 23, 2019 at 08:34:42PM +0400, Humaid wrote:
> > ---
> > src/qemu/qemu_tpm.c | 6 +++---
> > 1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/src/qemu/qemu_tpm.c b/src/qemu/qemu_tpm.c
> > index 835a9caf46..b60e443f14 100644
> > --- a/src/qemu/qemu_tpm.c
> > +++ b/src/qemu/qemu_tpm.c
> > @@ -834,16 +834,16 @@ qemuExtTPMStartEmulator(virQEMUDriverPtr driver,
> >
> >
> > int
> > -qemuExtTPMStart(virQEMUDriverPtr driver,
> > - virDomainObjPtr vm,
> > +qemuExtTPMStart(virDomainObjPtr vm,
> > qemuDomainLogContextPtr logCtxt)
> > {
> > int ret = 0;
> > + qemuDomainObjPrivatePtr priv = vm->privateData;
>
> A quick grep through the code base shows that we could do this at many more
> places actually.
Daniel pointed out that it's not actually worth doing as a separate
cleanup:
https://www.redhat.com/archives/libvir-list/2019-March/msg01147.html