From: Nicolas Aupetit <[email protected]>

The `repeat` function calls the provided command one time too many:

    repeat 10 echo "foo" | wc -l ⇒ 11

Also, the provided `seq` function shadows `/usr/bin/seq`.

The patch to fix these issues has originally been provided by Nicolas
Auperit in <https://bugs.debian.org/423488>.
---
 examples/startup-files/Bash_aliases | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/examples/startup-files/Bash_aliases 
b/examples/startup-files/Bash_aliases
index 2abb93ee..4bd8e66f 100644
--- a/examples/startup-files/Bash_aliases
+++ b/examples/startup-files/Bash_aliases
@@ -41,20 +41,20 @@ repeat ()
 { 
     local count="$1" i;
     shift;
-    for i in $(seq 1 "$count");
+    for i in $(_seq 1 "$count");
     do
         eval "$@";
     done
 }
 
 # Subfunction needed by `repeat'.
-seq ()
+_seq ()
 { 
     local lower upper output;
     lower=$1 upper=$2;
 
     if [ $lower -ge $upper ]; then return; fi
-    while [ $lower -le $upper ];
+    while [ $lower -lt $upper ];
     do
        echo -n "$lower "
         lower=$(($lower + 1))
-- 
2.51.0


Reply via email to