[libvirt] [PATCH 06/12] tests: qemuxml2argv: Rename testInfo* to testQemuInfo*
Andrea Bolognani
abologna at redhat.com
Wed Apr 10 16:19:09 UTC 2019
On Mon, 2019-04-01 at 12:47 -0400, Cole Robinson wrote:
> In preparation for moving these bits to a shared place, rename them
> to match one of the testqemuutils.c function prefixes. Rename
s/testqemuutils/testutilsqemu/
[...]
> static int
> -testCheckExclusiveFlags(int flags)
> +testQemuInfoCheckExclusiveFlags(int flags)
According to the logic used in the commit message to justify not
renaming testInfoSetPaths(), this one should either keep its name as
well or be moved to testutilsqemu. I would go for the former right
now, but it would be neat if you explored whether calling it from
xml2xml too would make sense, for a follow-up patch.
Everything else looks good, so assuming that you don't rename the
function
Reviewed-by: Andrea Bolognani <abologna at redhat.com>
--
Andrea Bolognani / Red Hat / Virtualization
More information about the libvir-list
mailing list