expand: simplify ValidName arithmetic loop

syntax.ValidName now returns false on empty strings, as it should - so the logic here can be cleaned up.
ScrambleShell · Jan 7, 2019 · 23a244e · 23a244e
1 parent e3c5ce0
commit 23a244e
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/expand/arith.go b/expand/arith.go
@@ -19,7 +19,7 @@ func Arithm(cfg *Config, expr syntax.ArithmExpr) (int, error) {
 		}
 		// recursively fetch vars
 		i := 0
-		for str != "" && syntax.ValidName(str) {
+		for syntax.ValidName(str) {
 			val := cfg.envGet(str)
 			if val == "" {
 				break