[pacman-dev] [PATCH] remove unneeded break argument from option parsing
Rafael Ascensão
rafa.almas at gmail.com
Tue May 1 16:54:04 UTC 2018
Some scripts are using `break 2` to break out of the option parsing
loop.
Since a single `break` is sufficient in these cases, remove the extra
argument.
Signed-off-by: Rafael Ascensão <rafa.almas at gmail.com>
---
Greetings,
This is just a small patch meant to remove some unnecessary noise from
the option parsing loop.
The lines are not technically wrong but may cause some confusion or
wonder why they exist in the first place.
Thanks for spending your time on my first contribution to an official
Arch Linux project!
scripts/libmakepkg/source.sh.in | 2 +-
scripts/makepkg.sh.in | 2 +-
scripts/pacman-db-upgrade.sh.in | 2 +-
scripts/pacman-key.sh.in | 2 +-
scripts/pkgdelta.sh.in | 2 +-
5 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/scripts/libmakepkg/source.sh.in b/scripts/libmakepkg/source.sh.in
index 02bb16f5..5d7df732 100644
--- a/scripts/libmakepkg/source.sh.in
+++ b/scripts/libmakepkg/source.sh.in
@@ -48,7 +48,7 @@ download_sources() {
get_vcs=0
;;
*)
- break 2
+ break
;;
esac
shift
diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in
index eee8a56c..be80cc0e 100644
--- a/scripts/makepkg.sh.in
+++ b/scripts/makepkg.sh.in
@@ -1277,7 +1277,7 @@ while true; do
-h|--help) usage; exit $E_OK ;;
-V|--version) version; exit $E_OK ;;
- --) OPT_IND=0; shift; break 2;;
+ --) OPT_IND=0; shift; break ;;
esac
shift
done
diff --git a/scripts/pacman-db-upgrade.sh.in b/scripts/pacman-db-upgrade.sh.in
index 9d447cbc..a529dadd 100644
--- a/scripts/pacman-db-upgrade.sh.in
+++ b/scripts/pacman-db-upgrade.sh.in
@@ -103,7 +103,7 @@ while true; do
-h|--help) usage; exit 0 ;;
--nocolor) USE_COLOR='n' ;;
-V|--version) version; exit 0 ;;
- -- ) shift; break 2 ;;
+ --) shift; break ;;
esac
shift
done
diff --git a/scripts/pacman-key.sh.in b/scripts/pacman-key.sh.in
index 060d8369..01e0bd5e 100644
--- a/scripts/pacman-key.sh.in
+++ b/scripts/pacman-key.sh.in
@@ -557,7 +557,7 @@ while (( $# )); do
-h|--help) usage; exit 0 ;;
-V|--version) version; exit 0 ;;
- --) shift; break 2 ;;
+ --) shift; break ;;
esac
shift
done
diff --git a/scripts/pkgdelta.sh.in b/scripts/pkgdelta.sh.in
index 4500dd75..316d6395 100644
--- a/scripts/pkgdelta.sh.in
+++ b/scripts/pkgdelta.sh.in
@@ -202,7 +202,7 @@ while :; do
shift ;;
--)
shift
- break 2 ;;
+ break ;;
esac
shift
done
--
2.17.0
More information about the pacman-dev
mailing list