Messages in this thread Patch in this message | | | From | Olof Johansson <> | Subject | [PATCH 03/10] merge_config.sh: minor argument parsing refactoring | Date | Wed, 20 May 2015 15:00:53 -0700 |
| |
Every case that continues iterating needs a shift, so move it to common location. Also, the continues are redundant.
Signed-off-by: Olof Johansson <olof@lixom.net> --- scripts/kconfig/merge_config.sh | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-)
diff --git a/scripts/kconfig/merge_config.sh b/scripts/kconfig/merge_config.sh index 5a01aa6..fb0c463 100755 --- a/scripts/kconfig/merge_config.sh +++ b/scripts/kconfig/merge_config.sh @@ -48,13 +48,9 @@ while true; do case $1 in "-n") ALLTARGET=allnoconfig - shift - continue ;; "-m") RUNMAKE=false - shift - continue ;; "-h") usage @@ -62,8 +58,6 @@ while true; do ;; "-r") WARNREDUN=true - shift - continue ;; "-O") if [ -d $2 ];then @@ -72,13 +66,14 @@ while true; do echo "output directory $2 does not exist" 1>&2 exit 1 fi - shift 2 - continue + shift ;; *) break ;; esac + + shift done if [ "$#" -lt 2 ] ; then -- 1.7.10.4
| |