about summary refs log tree commit homepage
path: root/lib/PublicInbox/Spawn.pm
diff options
context:
space:
mode:
authorŠtěpán Němec <stepnem@smrk.net>2023-08-28 12:42:42 +0200
committerEric Wong <e@80x24.org>2023-08-28 21:32:27 +0000
commit376d71e65320b50a81093fa6047b475efc7c2395 (patch)
tree2b3f5fcf37077c3ba5b1b2799febb17bcfc7a67c /lib/PublicInbox/Spawn.pm
parenta036772bdf732d4779f44974095bfdd0b30271ef (diff)
downloadpublic-inbox-376d71e65320b50a81093fa6047b475efc7c2395.tar.gz
'-' could never match, remove that alternative (it might have been a
typo of '--', but that is already covered by '*--|--*' ('*' matches
the null string)).

Replace '*--*' with the equivalent '*' ('--' is always present).

It would seem clearer to just replace the whole case command with
something like '[ "$ID" -a "$VERSION_ID" ] && break' (or the
POSIX-non-deprecated equivalent '[ "$ID" ] && [ "$VERSION_ID" ]' ); I
assume a preference of using case here (e.g., to avoid syscall
overhead in case [ is not implemented as a shell builtin (which seems
far-fetched given the context, though)).
Diffstat (limited to 'lib/PublicInbox/Spawn.pm')
0 files changed, 0 insertions, 0 deletions