[v3,03/10] check_maintainers: change variable names
diff mbox series

Message ID 20200613000055.7909-4-stephen@networkplumber.org
State Superseded
Delegated to: Thomas Monjalon
Headers show
Series
  • rename blacklist/whitelist to block/allow
Related show

Checks

Context Check Description
ci/Intel-compilation success Compilation OK
ci/checkpatch success coding style OK

Commit Message

Stephen Hemminger June 13, 2020, midnight UTC
Use blocklist where blacklist was used and allowlist
where whitelist was used.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Acked-by: Luca Boccassi <bluca@debian.org>
---
 devtools/check-maintainers.sh | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Comments

Thomas Monjalon July 10, 2020, 5:01 p.m. UTC | #1
13/06/2020 02:00, Stephen Hemminger:
> Use blocklist where blacklist was used and allowlist
> where whitelist was used.

This is a mechanical, not accurate, replacement.

> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> Acked-by: Luca Boccassi <bluca@debian.org>
> ---
> --- a/devtools/check-maintainers.sh
> +++ b/devtools/check-maintainers.sh
>  			# empty line delimit end of section
> -			whitelist=$(files $flines)
> -			blacklist=$(files $xlines)
> -			match=$(aminusb "$whitelist" "$blacklist")
> -			if [ -n "$whitelist" ] ; then
> +			allowlist=$(files $flines)
> +			blocklist=$(files $xlines)
> +			match=$(aminusb "$allowlist" "$blocklist")
> +			if [ -n "$allowlist" ] ; then
>  				printf "# $title "

It is listing files with exclusions.
The concept allow/block does not match here.
I would prefer "files" and "xfiles" as variable names.

Patch
diff mbox series

diff --git a/devtools/check-maintainers.sh b/devtools/check-maintainers.sh
index 85a300f0a029..edb452b55833 100755
--- a/devtools/check-maintainers.sh
+++ b/devtools/check-maintainers.sh
@@ -42,10 +42,10 @@  parse_fx () # <index file>
 	for line in $( (sed '/^-\+$/d' $1 ; echo) | sed 's,^$,§,') ; do
 		if echo "$line" | grep -q '^§$' ; then
 			# empty line delimit end of section
-			whitelist=$(files $flines)
-			blacklist=$(files $xlines)
-			match=$(aminusb "$whitelist" "$blacklist")
-			if [ -n "$whitelist" ] ; then
+			allowlist=$(files $flines)
+			blocklist=$(files $xlines)
+			match=$(aminusb "$allowlist" "$blocklist")
+			if [ -n "$allowlist" ] ; then
 				printf "# $title "
 				maintainers=$(echo "$maintainers" | sed -r 's,.*<(.*)>.*,\1,')
 				maintainers=$(printf "$maintainers" | sed -e 's,^,<,' -e 's,$,>,')