From ca9d48641b0e34bc486cba2dfd0c24cd2ecc9b4e Mon Sep 17 00:00:00 2001
From: Ayush Agarwal <ayush@fastmail.in>
Date: Sun, 16 Jan 2022 18:51:40 +0530
Subject: refactor: change the find_backend function

Recently, I realized that bash local variables aren't actually local and
use dynamic scoping. Renaming `backend` to `dmbd` is just a precaution.
---
 tessen | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

(limited to 'tessen')

diff --git a/tessen b/tessen
index 1e08a19..d92d5c6 100755
--- a/tessen
+++ b/tessen
@@ -380,14 +380,15 @@ get_default_opts() {
 }
 
 find_backend() {
-  local backend
-  for backend in "${tsn_known_backends[@]}"; do
-    if is_installed "${backend}"; then
-      printf "%s" "${backend}"
+  local dmbd
+
+  for dmbd in "${tsn_known_backends[@]}"; do
+    if is_installed "$dmbd"; then
+      printf "%s" "$dmbd"
       return 0
     fi
   done
-  _die "Please use '--backend' option or 'TESSEN_BACKEND' environment variable to configure installed 'dmenu' compatible application to use ${0##*/}"
+  _die "%s\n" "unable to find a 'dmenu' compatible application"
 }
 
 validate_cliptime() {
@@ -471,7 +472,7 @@ main() {
   unset -v _opt
 
   if [[ -z "${tsn_backend}" ]]; then
-    tsn_backend=$(find_backend)
+    tsn_backend="$(find_backend)"
   fi
 
   setup_backend "${tsn_backend}"
-- 
cgit v1.2.3