* lisp/progmodes/sh-script.el (sh-smie-sh-rules): For { after &&, don't

align with the surrounding parent.

Fixes: debbugs:17721

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index b7cd062d15c350531c52dd18bac996bb36108612..81a8945be11603764c595dd2872fb058951b0c3f 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2014-06-20  Stefan Monnier  <monnier@iro.umontreal.ca>
+
+       * progmodes/sh-script.el (sh-smie-sh-rules): For { after &&, don't
+       align with the surrounding parent (bug#17721).
+
 2014-06-20  Eli Zaretskii  <eliz@gnu.org>
 
        * textmodes/texinfo.el (texinfo-mode): Set skeleton-end-newline

diff --git a/lisp/progmodes/sh-script.el b/lisp/progmodes/sh-script.el
index f248494eed1befac1db2653b7c6f155d339f2481..4db462594dc9d5c5dd924c429baf7c81aca5642f 100644 (file)
--- a/lisp/progmodes/sh-script.el
+++ b/lisp/progmodes/sh-script.el
@@ -1952,7 +1952,11 @@ May return nil if the line should not be treated as continued."
                   (<= indent initial))
          `(column . ,(+ initial sh-indentation)))))
     (`(:before . ,(or `"(" `"{" `"["))
-     (if (smie-rule-hanging-p) (smie-rule-parent)))
+     (when (smie-rule-hanging-p)
+       (if (not (smie-rule-prev-p "&&" "||" "|"))
+          (smie-rule-parent)
+        (smie-backward-sexp 'halfexp)
+        `(column . ,(smie-indent-virtual)))))
     ;; FIXME: Maybe this handling of ;; should be made into
     ;; a smie-rule-terminator function that takes the substitute ";" as arg.
     (`(:before . ,(or `";;" `";&" `";;&"))

diff --git a/test/indent/shell.sh b/test/indent/shell.sh
index ed6bcf8da3a1a55daa4d965312023dc452ed455b..8e831bb8f110b9d17704320c0bb7713b5274789b 100755 (executable)
--- a/test/indent/shell.sh
+++ b/test/indent/shell.sh
@@ -33,6 +33,14 @@ esac
         bar
 }
 
+for foo in bar; do              #  bug#17721
+    [ -e $foo ] && {
+        echo t
+    } && {
+       echo r
+    }
+done
+
 echo -n $(( 5 << 2 ))
 # This should not be treated as a heredoc (bug#12770).
 2