From d38eb710d92864b0b1f7cd36f17e273e3d8c735c Mon Sep 17 00:00:00 2001 From: Lars Hjemli Date: Mon, 24 Sep 2007 00:51:42 +0200 Subject: [PATCH] git-merge: refactor option parsing Move the option parsing into a separate function as preparation for reuse by the next commit. Signed-off-by: Lars Hjemli Signed-off-by: Junio C Hamano --- git-merge.sh | 20 ++++++++++++++------ 1 file changed, 14 insertions(+), 6 deletions(-) diff --git a/git-merge.sh b/git-merge.sh index 919e6be4b..49185eb5d 100755 --- a/git-merge.sh +++ b/git-merge.sh @@ -119,11 +119,7 @@ merge_name () { fi } -case "$#" in 0) usage ;; esac - -have_message= -while test $# != 0 -do +parse_option () { case "$1" in -n|--n|--no|--no-|--no-s|--no-su|--no-sum|--no-summ|\ --no-summa|--no-summar|--no-summary) @@ -166,9 +162,21 @@ do have_message=t ;; -*) usage ;; - *) break ;; + *) return 1 ;; esac shift + args_left=$# +} + +test $# != 0 || usage + +have_message= +while parse_option "$@" +do + while test $args_left -lt $# + do + shift + done done if test -z "$show_diffstat"; then -- 2.26.2