Re: [libvirt] [PATCH] virQEMUDriverPtr parameters clean up in function qemuExtTPMStart() in /src/qemu/qemu_tpm.c

On Mon, Mar 25, 2019 at 10:03:31AM +0100, Peter Krempa wrote: > 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 For cleaning things up I think this makes sense, and I understood the above as Daniel not being convinced because there was no reasoning behind that at all (no commit message, etc.), hopefully I am not mistaken.