[PATCH v7 04/10] qemu_tpm, security, tests: change 'switch' clauses for 'if'

Ján Tomko jtomko at redhat.com
Tue Jun 23 13:36:00 UTC 2020


On a Wednesday in 2020, Daniel Henrique Barboza wrote:
>This trivial rework is aimed to reduce the amount of line changes
>made by the next patch, when 'def->tpm' will become a 'def->tpms'
>array.
>
>Instead of using a 'switch' where only the VIR_DOMAIN_TPM_TYPE_EMULATOR
>label does something, use an 'if' clause instead.
>
>Tested-by: Satheesh Rajendran <sathnaga at linux.vnet.ibm.com>
>Reviewed-by: Stefan Berger <stefanb at linux.ibm.com>
>Signed-off-by: Daniel Henrique Barboza <danielhb413 at gmail.com>
>---
> src/qemu/qemu_tpm.c             | 44 +++++----------------------------
> src/security/security_selinux.c | 16 ++----------
> src/security/virt-aa-helper.c   |  7 +-----
> tests/qemuxml2argvtest.c        |  7 +-----
> 4 files changed, 10 insertions(+), 64 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko at redhat.com>

Jano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20200623/659f8762/attachment-0001.sig>


More information about the libvir-list mailing list