Merge pull request #2 from billyzkid/billyzkid-patch-2

Fix default argument logic
billyzkid · Aug 24, 2023 · c4cffc7 · c4cffc7
2 parents 4af757a + d3d705e
commit c4cffc7
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/lib/getopts_long.bash b/lib/getopts_long.bash
@@ -9,10 +9,10 @@ getopts_long() {
     shift 2
 
     if [[ "${#}" == 0 ]]; then
-        local args=()
-        while [[ ${#BASH_ARGV[@]} -gt ${#args[@]} ]]; do
-            local index=$(( ${#BASH_ARGV[@]} - ${#args[@]} - 1 ))
-            args[${#args[@]}]="${BASH_ARGV[${index}]}"
+        local -i i
+        local -a args=()
+        for (( i = BASH_ARGC[0] + BASH_ARGC[1] - 1; i >= BASH_ARGC[0]; i-- )); do
+            args+=("${BASH_ARGV[i]}")
         done
         set -- "${args[@]}"
     fi