[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: moving some indentation tests
From: |
Noam Postavsky |
Subject: |
Re: moving some indentation tests |
Date: |
Fri, 7 Apr 2017 12:06:08 -0400 |
On Fri, Apr 7, 2017 at 11:51 AM, Phillip Lord
<address@hidden> wrote:
> Noam Postavsky <address@hidden> writes:
>> Ah, because lisp-indent-line doesn't check the parse context.
>> `indent-sexp' does the right thing, if called outside the string (and
>> also inside since #21343 was fixed).
>>
>> I believe my patches at
>> https://debbugs.gnu.org/cgi/bugreport.cgi?bug=25122#77 will fix this
>> for `indent-region' (since I happened to copy the syntax-ppss fix from
>> #21343). Probably `lisp-indent-line' should also use syntax-ppss to
>> get the correct parse context.
>
> So, in this case, assess is working? lisp-indent-line is, effectively,
> broken?
I think both assess and lisp-indent-line are broken for this case, in
different ways. js-indent-line seems to handle this correctly though.
So you should be able to test assess with js-mode.
- Re: moving some indentation tests, (continued)
- Re: moving some indentation tests, Eli Zaretskii, 2017/04/02
- Re: moving some indentation tests, Dmitry Gutov, 2017/04/02
- Re: moving some indentation tests, Phillip Lord, 2017/04/02
- Re: moving some indentation tests, Noam Postavsky, 2017/04/02
- Re: moving some indentation tests, Phillip Lord, 2017/04/03
- Re: moving some indentation tests, Noam Postavsky, 2017/04/03
- Re: moving some indentation tests, Phillip Lord, 2017/04/03
- Re: moving some indentation tests, Noam Postavsky, 2017/04/03
- Re: moving some indentation tests, Phillip Lord, 2017/04/07
- Re: moving some indentation tests,
Noam Postavsky <=
Re: moving some indentation tests, Tom Tromey, 2017/04/05