From 7c17c8aec3b5cb2fa65e202fea22cc2b8e8f1118 Mon Sep 17 00:00:00 2001
From: Matthias Radestock <matthias.radestock@gmail.com>
Date: Sun, 19 Jul 2015 21:56:04 +0100
Subject: [PATCH] refactor: simplify router arg parsing

using the same "always shift one" approach as in proxy_args
---
 weave | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/weave b/weave
index 1733b4f7..262b5cc8 100755
--- a/weave
+++ b/weave
@@ -1047,38 +1047,35 @@ launch_router() {
                 [ $# -gt 1 ] || usage
                 WEAVE_PASSWORD="$2"
                 export WEAVE_PASSWORD
-                shift 2
+                shift
                 ;;
             --password=*)
                 WEAVE_PASSWORD="${1#*=}"
                 export WEAVE_PASSWORD
-                shift
                 ;;
             -port|--port)
                 [ $# -gt 1 ] || usage
                 CONTAINER_PORT="$2"
-                shift 2
+                shift
                 ;;
             --port=*)
                 CONTAINER_PORT="${1#*=}"
-                shift
                 ;;
             -iprange|--iprange|--ipalloc-range)
                 [ $# -gt 1 ] || usage
                 IPRANGE="$2"
                 IPRANGE_SPECIFIED=1
-                shift 2
+                shift
                 ;;
             --ipalloc-range=*)
                 IPRANGE="${1#*=}"
                 IPRANGE_SPECIFIED=1
-                shift
                 ;;
             *)
                 ARGS="$ARGS '$(echo "$1" | sed "s|'|'\"'\"'|g")'"
-                shift
                 ;;
         esac
+        shift
     done
     eval "set -- $ARGS"
     if [ -z "$IPRANGE_SPECIFIED" ] ; then