bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#25122: 24.5; function describe-variable hangs on large variables


From: npostavs
Subject: bug#25122: 24.5; function describe-variable hangs on large variables
Date: Mon, 17 Apr 2017 23:53:30 -0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux)

npostavs@users.sourceforge.net writes:

>>> npostavs@users.sourceforge.net writes:
>>>
>>> Okay, I think I found the real fix now:
>
> Same issue with indent-region.  The gains are not so dramatic for
> typical lisp code that has normal size sexps, but C-x h C-M-\ on
> subr.el's text runs twice as fast for me with the new lisp-indent-region
> function.

Had some test failures due to some corner cases, should all be fixed
now.  I think this is the final patch set.  This also fixes
lisp-indent-region and lisp-indent-line to not indent string literal
contents.

I intend to close this bug as fixed after merging these, as this does
fix the performance bug.  I will probably pursue the alternate pretty
printer separately; it doesn't help performance (after the indent-sexp
performance is fixed), but gives prettier results in some cases.  The
threading idea may also be worth looking at, but can also be considered
separately, as this bug report is already long enough.

Attachment: v3-0001-Don-t-reparse-the-sexp-in-indent-sexp-Bug-25122.patch
Description: patch

Attachment: v3-0002-lisp-emacs-lisp-lisp-mode.el-indent-sexp-Clean-up.patch
Description: patch

Attachment: v3-0003-Remove-ignored-argument-from-lisp-indent-line.patch
Description: patch

Attachment: v3-0004-Add-new-lisp-indent-region-that-doesn-t-reparse-t.patch
Description: patch


reply via email to

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