[Top][All Lists]

[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.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]