cobra icon indicating copy to clipboard operation
cobra copied to clipboard

fix(bash-v2): activeHelp length check syntax

Open scop opened this issue 2 years ago • 3 comments

$ set -u
$ foo=()
$ echo ${#foo}
bash: foo: unbound variable
echo ${#foo[*]}
0

scop avatar Jul 22 '22 14:07 scop

#1702 requires a rebase after this is in, will take care of it.

scop avatar Jul 22 '22 14:07 scop

Nice catch! Thanks @scop. I'll run some tests on this next week and get it merged. I assume this was the only unbound variable problem with ActiveHelp?

marckhouzam avatar Jul 22 '22 14:07 marckhouzam

The only one I happened to hit with my normal usage patterns, yes. I didn't analyze beyond that.

scop avatar Jul 22 '22 19:07 scop