[libvirt PATCH 10/23] build: remove use of the term 'blacklist' from helper files
Peter Krempa
pkrempa at redhat.com
Fri Jun 19 10:18:23 UTC 2020
On Fri, Jun 19, 2020 at 10:32:47 +0100, Daniel Berrange wrote:
> The term is redundant or easily substituted with 'prohibit'.
>
> Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
> ---
> build-aux/syntax-check.mk | 2 +-
> m4/virt-compile-warnings.m4 | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/build-aux/syntax-check.mk b/build-aux/syntax-check.mk
> index d47a92b530..2e49b5172e 100644
> --- a/build-aux/syntax-check.mk
> +++ b/build-aux/syntax-check.mk
> @@ -1155,7 +1155,7 @@ sc_prohibit_class:
> # exists some filesystems will only ever return DT_UNKNOWN.
> # This field should only be used by code which is exclusively
> # run platforms supporting "d_type" and must expect DT_UNKNOWN.
> -# We blacklist it to discourage accidental usage which has
> +# We prohibit it to discourage accidental usage which has
This may or may not make sence depending on how syntax check is ported
to meson. You can use my R-b for this hunk if this survives the rewrite.
> # happened many times. Add an exclude rule if it is genuinely
> # needed and the above restrictions are acceptable.
> sc_prohibit_dirent_d_type:
> diff --git a/m4/virt-compile-warnings.m4 b/m4/virt-compile-warnings.m4
> index d3538d59f8..4ebc25302c 100644
> --- a/m4/virt-compile-warnings.m4
> +++ b/m4/virt-compile-warnings.m4
> @@ -123,7 +123,7 @@ AC_DEFUN([LIBVIRT_COMPILE_WARNINGS],[
> # Get all possible GCC warnings
> gl_MANYWARN_ALL_GCC([maybewarn])
>
> - # Remove the ones we don't want, blacklisted earlier
> + # Remove the ones we don't want
> gl_MANYWARN_COMPLEMENT([wantwarn], [$maybewarn], [$dontwarn])
IMO this change doesn't make sense in the light of the meson rewrite.
> # -Wunused-functin is implied by -Wall we must turn it
> --
> 2.24.1
>
More information about the libvir-list
mailing list