On Fri, Apr 05, 2024 at 05:36:40PM +0200, Matthieu Baerts (NGI0) wrote: > From: Geliang Tang <tanggeliang@xxxxxxxxxx> > > This patch uses addition assignment operator (+=) to append strings > instead of duplicating the variable name in mptcp_connect.sh and > mptcp_join.sh. > > This can make the statements shorter. > > Note: in mptcp_connect.sh, add a local variable extra in do_transfer to > save the various extra warning logs, using += to append it. And add a > new variable tc_info to save various tc info, also using += to append it. > This can make the code more readable and prepare for the next commit. > > Signed-off-by: Geliang Tang <tanggeliang@xxxxxxxxxx> > Reviewed-by: Matthieu Baerts (NGI0) <matttbe@xxxxxxxxxx> > Signed-off-by: Matthieu Baerts (NGI0) <matttbe@xxxxxxxxxx> > Link: https://lore.kernel.org/r/20240308-upstream-net-next-20240308-selftests-mptcp-unification-v1-8-4f42c347b653@xxxxxxxxxx > Signed-off-by: Jakub Kicinski <kuba@xxxxxxxxxx> > (cherry picked from commit e7c42bf4d320affe37337aa83ae0347832b3f568) > Signed-off-by: Matthieu Baerts (NGI0) <matttbe@xxxxxxxxxx> > --- > .../selftests/net/mptcp/mptcp_connect.sh | 53 ++++++++++--------- > .../testing/selftests/net/mptcp/mptcp_join.sh | 30 +++++------ > 2 files changed, 43 insertions(+), 40 deletions(-) Odd, this one did not apply. All others did, thanks! greg k-h
