[kvm-unit-tests RFC PATCH 07/17] shellcheck: Fix SC2235
Nicholas Piggin
npiggin at gmail.com
Fri Apr 5 02:00:39 PDT 2024
SC2235 (style): Use { ..; } instead of (..) to avoid subshell
overhead.
No bug identified. Overhead is pretty irrelevant.
Signed-off-by: Nicholas Piggin <npiggin at gmail.com>
---
scripts/arch-run.bash | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/scripts/arch-run.bash b/scripts/arch-run.bash
index ae4b06679..d1edd1d69 100644
--- a/scripts/arch-run.bash
+++ b/scripts/arch-run.bash
@@ -580,15 +580,15 @@ kvm_available ()
return 1
[ "$HOST" = "$ARCH_NAME" ] ||
- ( [ "$HOST" = aarch64 ] && [ "$ARCH" = arm ] ) ||
- ( [ "$HOST" = x86_64 ] && [ "$ARCH" = i386 ] )
+ { [ "$HOST" = aarch64 ] && [ "$ARCH" = arm ] ; } ||
+ { [ "$HOST" = x86_64 ] && [ "$ARCH" = i386 ] ; }
}
hvf_available ()
{
[ "$(sysctl -n kern.hv_support 2>/dev/null)" = "1" ] || return 1
[ "$HOST" = "$ARCH_NAME" ] ||
- ( [ "$HOST" = x86_64 ] && [ "$ARCH" = i386 ] )
+ { [ "$HOST" = x86_64 ] && [ "$ARCH" = i386 ] ; }
}
set_qemu_accelerator ()
--
2.43.0
More information about the kvm-riscv
mailing list