don't use echo -e

[muttjump] / muttjump

diff --git a/muttjump b/muttjump
index 28fb3a79579315c8cf3f031d60fae3d6a4433142..5c7e15f122b25599540caba5f302796ec7becc44 100755 (executable)
--- a/muttjump
+++ b/muttjump
@@ -170,25 +170,24 @@ fi
 count=$(echo "$orig_msgfiles" | wc -l)
 if [ $count -gt 1 -a -n "$DIALOG_PROG" ] ; then
     choices=$(echo "$orig_msgfiles" | while read line ;
-        do echo -e "$(dirname "$(dirname "$line")")\n." ; done)
+        do dirname "$(dirname "$line")" ; echo . ; done)
     maxwidth=$(echo "$choices" | wc -L)
     orig_maildir=$($DIALOG_PROG --clear --title "More than one mailbox found" \
         --menu "Select mailbox:" $((count+8)) $((maxwidth+16)) \
         $count $choices 3>&2 2>&1 1>&3-)
 elif [ $count -gt 1 ] ; then
-    dirs=()
-    for f in $orig_msgfiles ; do
-        dirs=(${dirs[@]} "$(dirname "$(dirname "$f")")")
-    done
+    choices=()
+    while read line ; do
+        choices=("${choices[@]}" "$(dirname "$(dirname "$line")")")
+    done <<<"$orig_msgfiles"
     echo "More than one mailbox found"
     echo "Select mailbox:"
-    select md in ${dirs[@]} ; do
-        orig_maildir="$md"
-        orig_msgfile="${orig_msgfiles[$REPLY]}"
+    select md in "${choices[@]}" ; do
+        orig_maildir=$md
         break
     done <> /dev/tty
 else
-    orig_msgfile="$orig_msgfiles"
+    orig_msgfile=$orig_msgfiles
     # get containing maildir of $orig_msgfile
     orig_maildir=$(dirname "$(dirname "$orig_msgfile")")
 fi