GNU bug report logs - #26187
Bug in indent-sexp

Previous Next

Full log

View this message in rfc822 format

From: npostavs <at> users.sourceforge.net
To: Gdobbins <gdobbins <at> protonmail.com>
Cc: 26187 <at> debbugs.gnu.org
Subject: bug#26187: Bug in indent-sexp
Date: Mon, 20 Mar 2017 09:20:25 -0400

Gdobbins <gdobbins <at> protonmail.com> writes:

> The refactor of indent-sexp in commit 3ee3995d105ff02f0fac540757431d36cb45c6c7 broke the assumption
>
> ;; If ENDPOS is nil, it is safe not to scan before point
> ;; since every line we indent is more deeply nested than point is.
>
> thereby breaking functionality in paredit. The attached patch fixes
> this.

Could you post the scenario which is broken?  I have a another pending
change to indent-sexp for Bug#25122 [1] which actually removes the
rescanning completely, but I'm not sure if that would rebreak this
scenario or fix it...

[1]: https://debbugs.gnu.org/cgi/bugreport.cgi?att=1;bug=25122;msg=74;filename=v2-0001-Don-t-reparse-the-sexp-in-indent-sexp-Bug-25122.patch

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.