On 9/12/20 9:49 AM, Thomas Huth wrote: > We simply want to ignore certain queries here, so let's rather > use the term 'ignore' to express this intention. > > Signed-off-by: Thomas Huth <th...@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com> > --- > tests/qtest/qmp-cmd-test.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/tests/qtest/qmp-cmd-test.c b/tests/qtest/qmp-cmd-test.c > index 3109a9fe96..18069a82fa 100644 > --- a/tests/qtest/qmp-cmd-test.c > +++ b/tests/qtest/qmp-cmd-test.c > @@ -82,9 +82,9 @@ static void test_query(const void *data) > qtest_quit(qts); > } > > -static bool query_is_blacklisted(const char *cmd) > +static bool ignore_query(const char *cmd) > { > - const char *blacklist[] = { > + const char *ignorelist[] = { > /* Not actually queries: */ > "add-fd", > /* Success depends on target arch: */ > @@ -101,8 +101,8 @@ static bool query_is_blacklisted(const char *cmd) > }; > int i; > > - for (i = 0; blacklist[i]; i++) { > - if (!strcmp(cmd, blacklist[i])) { > + for (i = 0; ignorelist[i]; i++) { > + if (!strcmp(cmd, ignorelist[i])) { > return true; > } > } > @@ -179,7 +179,7 @@ static void add_query_tests(QmpSchema *schema) > continue; > } > > - if (query_is_blacklisted(si->name)) { > + if (ignore_query(si->name)) { > continue; > } > >