From daae3d6eb38b970e6e7330cf86ebf7fa8a78eb00 Mon Sep 17 00:00:00 2001
From: Wynn Wolf Arbor
Date: Thu, 4 Jun 2020 11:25:43 +0200
Subject: skein-infra: Use positional parameter instead of variable

Instead of relying on positional parameters, bind_umount() still
referenced "$i" from the function it initially came from, teardown().
This worked because the variable was still in scope, but is arguably
unclean and confusing.

Use positional parameters for bind_umount() instead, like bind_mount()
does.
---
 skein-infra | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

(limited to 'skein-infra')

diff --git a/skein-infra b/skein-infra
index 850b980..ac8084d 100755
--- a/skein-infra
+++ b/skein-infra
@@ -34,9 +34,9 @@ bind_mount() {
 }
 
 bind_umount() {
-	umount "$i"
-	rmdir "$i"
-	ldel "$(remove_prefix "$i")"
+	umount "$1"
+	rmdir "$1"
+	ldel "$(remove_prefix "$1")"
 }
 
 setup() {
-- 
cgit v1.2.3-2-gb3c3