| Message ID | 20200912074922.26103-1-thuth@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | tests/qtest/qmp-cmd-test: Use inclusive language | expand |
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 <thuth@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@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; > } > >
On 12/09/20 09:49, 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 <thuth@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[] = { Just "ignored", it's obviously a list. Paolo > /* 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; > } > >
Paolo Bonzini <pbonzini@redhat.com> writes: > On 12/09/20 09:49, 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 <thuth@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) Okay, although I'd prefer query_is_ignored(), to signal "this is a predicate", whereas ignore_query() suggests the function ignores queries. >> { >> - const char *blacklist[] = { >> + const char *ignorelist[] = { > > Just "ignored", it's obviously a list. Yes, that's better. "blacklist" contains "list" not because "list" is important, but because it's an actual word that makes the variable's purpose immediately obvious. "ignored" is an acceptable replacement, and is less clumsy than "ignorelist". > > Paolo > >> /* 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; >> } >> >>
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; }
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 <thuth@redhat.com> --- tests/qtest/qmp-cmd-test.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-)