[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