[PATCH] lxcxml2xmltest: Substitute 'inactive' variable with 'active'
Kristina Hanicova
khanicov at redhat.com
Tue Aug 24 16:43:47 UTC 2021
On Wed, Jul 21, 2021 at 10:33 AM Kristina Hanicova <khanicov at redhat.com>
wrote:
> I removed negation from the name of a variable to make the code
> more readable.
>
> Signed-off-by: Kristina Hanicova <khanicov at redhat.com>
> ---
> tests/lxcxml2xmltest.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/tests/lxcxml2xmltest.c b/tests/lxcxml2xmltest.c
> index 00341ad695..69c333b9b5 100644
> --- a/tests/lxcxml2xmltest.c
> +++ b/tests/lxcxml2xmltest.c
> @@ -21,7 +21,7 @@ static virLXCDriver *driver;
> struct testInfo {
> const char *name;
> int different;
> - bool inactive_only;
> + bool active_only;
> unsigned int parse_flags;
> };
>
> @@ -40,7 +40,7 @@ testCompareXMLToXMLHelper(const void *data)
>
> ret = testCompareDomXML2XMLFiles(driver->caps, driver->xmlopt, xml_in,
> info->different ? xml_out : xml_in,
> - !info->inactive_only,
> + info->active_only,
> info->parse_flags,
>
> TEST_COMPARE_DOM_XML2XML_RESULT_SUCCESS);
> VIR_FREE(xml_in);
> @@ -57,9 +57,9 @@ mymain(void)
> if (!(driver = testLXCDriverInit()))
> return EXIT_FAILURE;
>
> -# define DO_TEST_FULL(name, is_different, inactive, parse_flags) \
> +# define DO_TEST_FULL(name, is_different, active, parse_flags) \
> do { \
> - const struct testInfo info = {name, is_different, inactive, \
> + const struct testInfo info = {name, is_different, active, \
> parse_flags}; \
> if (virTestRun("LXC XML-2-XML " name, \
> testCompareXMLToXMLHelper, &info) < 0) \
> @@ -67,10 +67,10 @@ mymain(void)
> } while (0)
>
> # define DO_TEST(name) \
> - DO_TEST_FULL(name, 0, false, 0)
> + DO_TEST_FULL(name, 0, true, 0)
>
> # define DO_TEST_DIFFERENT(name) \
> - DO_TEST_FULL(name, 1, false, 0)
> + DO_TEST_FULL(name, 1, true, 0)
>
> /* Unset or set all envvars here that are copied in
> lxcdBuildCommandLine
> * using ADD_ENV_COPY, otherwise these tests may fail due to
> unexpected
> --
> 2.31.1
>
>
Polite ping
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20210824/ae7fbbab/attachment-0001.htm>
More information about the libvir-list
mailing list