On Nov 29, 2023 / 10:05, Bart Van Assche wrote: > On 11/29/23 02:31, Shin'ichiro Kawasaki wrote: > > + if [[ -r $TEST_DEV_SYSFS/device/state ]]; then > > + state=$(cat "$TEST_DEV_SYSFS/device/state") > > Why the separate -r test and cat instead of this? > > state=$({ cat "$TEST_DEV_SYSFS/device/state"; } 2>/dev/null) Thanks Bart, this looks the better. Will reflect this change.