diff options
author | Štěpán Němec <stepnem@smrk.net> | 2023-08-28 12:42:42 +0200 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2023-08-28 21:32:27 +0000 |
commit | 376d71e65320b50a81093fa6047b475efc7c2395 (patch) | |
tree | 2b3f5fcf37077c3ba5b1b2799febb17bcfc7a67c /lib/PublicInbox/Spawn.pm | |
parent | a036772bdf732d4779f44974095bfdd0b30271ef (diff) | |
download | public-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