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.

1 files changed, 3 insertions, 3 deletions

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() {