Merge pull request #919 from Samueru-sama/dev

use while instead of for loop sandboxes.am

modules/sandboxes.am

@@ -123,7 +123,7 @@ _generate_sandbox_script() {
 	APPCONF=$( ls "$CONFIGDIR" | grep -i "$APPNAME" | head -1 ) 
 	# Disable sandbox
 	if [ "$1" = "--disable-sandbox" ]; then
-	  APPIMAGEDIR="$(echo ${APPEXEC%/*})"
+	  APPIMAGEDIR="${APPEXEC%/*}"
 	  echo ""
 	  echo " Giving exec permissions back to $APPEXEC..."
 	  chmod a+x "$APPEXEC" || exit 1
@@ -244,7 +244,7 @@ _install_sandbox_script() {
 case "$1" in
 	'--sandbox')
 		shift
-		for arg in ${@}; do
+		while [ "$#" -gt 0 ]; do
 			_check_appimage "${@}" && _check_aisap "${@}" \
 			&& _generate_sandbox_script "${@}" \
 			&& _configure_dirs_access "${@}" \
@@ -255,23 +255,23 @@ case "$1" in
 
 	'--disable-sandbox')
 		shift
-		for arg in ${@}; do
+		while [ "$#" -gt 0 ]; do
 			_disable_sandbox "${@}"
 			shift
 		done
 		;;
 
 	'-H'|'--home')
 		shift
-		for arg in ${@}; do
+		while [ "$#" -gt 0 ]; do
 			_check_appimage "${@}" && _home "${@}"
 			shift
 		done
 		;;
 
 	'-C'|'--config')
 		shift
-		for arg in ${@}; do
+		while [ "$#" -gt 0 ]; do
 			_check_appimage "${@}" && _config "${@}"
 			shift
 		done