selftests: forwarding: lib: rewrite processing of command line arguments

The piece of code which processes the command line arguments and
populates NETIFS based on them is really unobvious. Rewrite it so that
the intention is clear and the code is easy to follow.

Suggested-by: Petr Machata <petrm@nvidia.com>
Signed-off-by: Ioana Ciornei <ioana.ciornei@nxp.com>
Reviewed-by: Petr Machata <petrm@nvidia.com>
Link: https://patch.msgid.link/20260407102058.867279-1-ioana.ciornei@nxp.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
Ioana Ciornei 2026-04-07 13:20:58 +03:00 committed by Jakub Kicinski
parent 686a7587bd
commit fff75dba79

View File

@ -467,17 +467,18 @@ if [ "${DRIVER_TEST_CONFORMANT}" = "yes" ]; then
TARGETS[$remote_netif]="$REMOTE_TYPE:$REMOTE_ARGS"
else
count=0
# Prime NETIFS from the command line, but retain if none given.
if [[ $# -gt 0 ]]; then
unset NETIFS
declare -A NETIFS
while [[ $# -gt 0 ]]; do
if [[ "$count" -eq "0" ]]; then
unset NETIFS
declare -A NETIFS
fi
count=$((count + 1))
NETIFS[p$count]="$1"
TARGETS[$1]="local:"
shift
done
while [[ $# -gt 0 ]]; do
count=$((count + 1))
NETIFS[p$count]="$1"
TARGETS[$1]="local:"
shift
done
fi
fi
##############################################################################