diff options
author | Daniel Shahaf <d.s@daniel.shahaf.name> | 2021-03-07 17:07:06 +0000 |
---|---|---|
committer | Daniel Shahaf <d.s@daniel.shahaf.name> | 2021-03-07 21:58:13 +0000 |
commit | f87b73e677739970c61eab55d4a3276beba1bd43 (patch) | |
tree | 452a62974d359972dd048a2e41c52738f3ef9f2f /Functions/Math | |
parent | 6a2a8acf09bc39795f18cc60e90a0d4427d36edd (diff) | |
download | zsh-f87b73e677739970c61eab55d4a3276beba1bd43.tar.gz zsh-f87b73e677739970c61eab55d4a3276beba1bd43.tar.xz zsh-f87b73e677739970c61eab55d4a3276beba1bd43.zip |
48147/0002: zmathfunc: Fix bug where the exit code would be non-zero if the expression evaluted to zero.
Diffstat (limited to 'Functions/Math')
-rw-r--r-- | Functions/Math/zmathfunc | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/Functions/Math/zmathfunc b/Functions/Math/zmathfunc index 4ff40700d..8e4b78549 100644 --- a/Functions/Math/zmathfunc +++ b/Functions/Math/zmathfunc @@ -1,34 +1,40 @@ #autoload zsh_math_func_min() { + emulate -L zsh local result=$1 shift local arg for arg ; do (( $arg < result )) && result=$arg done - (( result )) # return + (( result )) + true # Careful here: `return 0` evaluates an arithmetic expression } functions -M min 1 -1 zsh_math_func_min # at least one argument zsh_math_func_max() { + emulate -L zsh local result=$1 shift local arg for arg ; do (( $arg > result )) && result=$arg done - (( result )) # return + (( result )) + true # Careful here: `return 0` evaluates an arithmetic expression } functions -M max 1 -1 zsh_math_func_max # at least one argument zsh_math_func_sum() { + emulate -L zsh local sum local arg for arg ; do (( sum += $arg )) done (( sum )) + true # Careful here: `return 0` evaluates an arithmetic expression } functions -M sum 0 -1 zsh_math_func_sum |