[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#753: [Fwd: sh-script.el: indentation of ( )]
From: |
Andrew Hyatt |
Subject: |
bug#753: [Fwd: sh-script.el: indentation of ( )] |
Date: |
Tue, 29 Dec 2015 19:45:28 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.4 (darwin) |
+occitan (the original reported)
Stefan Monnier <monnier@IRO.UMontreal.CA> writes:
>>> It doesn't matter. "done)" is not an entity in `sh', so we should not
>>> treat it as one.
>> Yes, but it is a useful shortcut to treat it as one IMO, since it
>> gets the indentation "mostly right" for little effort.
>
> It still gets it wrong for "done )". And it introduces bugs elsewhere.
> I'm pretty sure it was not done on purpose: it's basically a bug that
> accidentally does the right thing in some corner cases.
>
>> I'm afraid I don't know how to fix it (indentation) properly, so I
>> reopened this bug.
>
> We know how to fix the OP's bug without re-introducing the bug that my
> misguided patch tried to fix. So we should do that and close this bug.
> Then we can open another bug about the "done )" case pattern since it's
> a different bug.
>
>
> Stefan
I can't reproduce the original bug on Emacs 25. Can anyone reproduce
it? If not, or I don't hear anything in a few weeks, I'll close this.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- bug#753: [Fwd: sh-script.el: indentation of ( )],
Andrew Hyatt <=