Re: [PATCH] fix "Illegal number" on FreeBSD & macOS for x=; echo $((x))

[Martijn Dekker <martijn@xxxxxxxx>]

I had totally forgotten this bug had come up before in January 2017:

https://www.spinics.net/lists/dash/msg01271.html

The original reporter, Michael Greenberg, didn't specify the bug only
occurs on macOS and FreeBSD but it does seem to be the same bug.

He submitted a different patch to fix it:

diff --git a/src/var.c b/src/var.c
index cc6f7f2..e34f9cf 100644
--- a/src/var.c
+++ b/src/var.c
@@ -353,7 +353,7 @@ lookupvar(const char *name)

 intmax_t lookupvarint(const char *name)
 {
-	return atomax(lookupvar(name) ?: nullstr, 0);
+	return atomax(lookupvar(name) ?: "0", 0);
 }

Which is better?

- M.
--
To unsubscribe from this list: send the line "unsubscribe dash" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html