From d25aaac9ad9a366c05f0b67af9cf9c8e3b35479c Mon Sep 17 00:00:00 2001 From: Leah Rowe Date: Sun, 6 Jul 2025 03:25:35 +0100 Subject: lib.sh: remove erroneous break from fx_ it means nothing here. in context, if a non-zero return is observed, we should not do anything here, which is already the behaviour anyway, except that "break" means nothing since we're not in a loop here. where an error exit should be observed, x_ is used inside the command given for fx_ Signed-off-by: Leah Rowe --- include/lib.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'include') diff --git a/include/lib.sh b/include/lib.sh index b40772fa..911d9c4d 100644 --- a/include/lib.sh +++ b/include/lib.sh @@ -100,7 +100,7 @@ fx_() fd="`mktemp`" && x_ rm -f "$fd" && x_ touch "$fd" xx="$1" && shift 1 "$@" 2>/dev/null | sort 1>"$fd" 2>/dev/null || err "FATAL: !sort fx_" - dx_ "$xx" "$fd" || break + dx_ "$xx" "$fd" || : x_ rm -f "$fd" } -- cgit v1.2.1