utils-code/tj.git-extras
1
0
Fork 0
mirror of https://github.com/tj/git-extras.git synced 2026-09-11 07:56:18 -04:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Guard against 'seq 0'

Browse source 
Most implementations of seq yields no output for 'seq 0',
but some yields '1(newline)0'.
This commit is contained in:
Nicolai Skogheim 2015-07-28 21:00:27 +02:00
parent cdb773c065
commit 1956d385bd
1 changed files with 29 additions and 25 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

54
bin/git-effort
View file

@ -119,33 +119,37 @@ above_index=0
has_above=false
next_is_above=false
num_files=0
for i in `seq ${#@}`
do
cur="${!i}"
if "$next_is_above" ; then
above="$cur"
next_is_above=false
continue
fi
# Some implementations of `seq` gives "1\n0" for seq 0
if [ $# -gt 0 ] ; then
for i in `seq ${#@}`
do
cur="${!i}"
case "$cur" in
--above)
if "$has_above" ; then
echo "error: --above can only be specified one time" 1>&2
exit 1
fi
next_is_above=true
has_above=true
above_index=$(( i - 1 ))
;;
--*)
;;
*)
num_files=$(( num_files + 1 ))
;;
esac
done
if "$next_is_above" ; then
above="$cur"
next_is_above=false
continue
fi
case "$cur" in
--above)
if "$has_above" ; then
echo "error: --above can only be specified one time" 1>&2
exit 1
fi
next_is_above=true
has_above=true
above_index=$(( i - 1 ))
;;
--*)
;;
*)
num_files=$(( num_files + 1 ))
;;
esac
done
fi
# Exit if above-value is not an int
if [ -z "${above##*[!0-9]*}" ] ; then