[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#25122: 24.5; function describe-variable hangs on large variables
From: |
Stefan Monnier |
Subject: |
bug#25122: 24.5; function describe-variable hangs on large variables |
Date: |
Sat, 11 Mar 2017 10:33:44 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.0.50 (gnu/linux) |
> Actually, it's the indent-sexp on the whole object that takes time.
> Possibly we could sacrifice some indentation correctness if the printed
> representation is big.
Actually, I think that trying to optimize this is bound to fail: we want
*Help* to show up "instantly", so we're off by a factor of more
than 100, which seems way out of reach (unless there's really an
algorithmic problem in our code, admittedly).
I think the better way out is to do less work. E.g. bound the total
amount printed (and replace the rest with "..." that can be expanded on
demand).
Stefan
- bug#25122: 24.5; function describe-variable hangs on large variables, npostavs, 2017/03/11
- bug#25122: 24.5; function describe-variable hangs on large variables,
Stefan Monnier <=
- bug#25122: 24.5; function describe-variable hangs on large variables, Thierry Volpiatto, 2017/03/11
- bug#25122: 24.5; function describe-variable hangs on large variables, npostavs, 2017/03/11
- bug#25122: 24.5; function describe-variable hangs on large variables, Drew Adams, 2017/03/11
- bug#25122: 24.5; function describe-variable hangs on large variables, Thierry Volpiatto, 2017/03/12
- bug#25122: 24.5; function describe-variable hangs on large variables, Stefan Monnier, 2017/03/12
- bug#25122: 24.5; function describe-variable hangs on large variables, npostavs, 2017/03/12
- bug#25122: 24.5; function describe-variable hangs on large variables, Drew Adams, 2017/03/12
- bug#25122: 24.5; function describe-variable hangs on large variables, Stefan Monnier, 2017/03/12
- bug#25122: 24.5; function describe-variable hangs on large variables, npostavs, 2017/03/12
- bug#25122: 24.5; function describe-variable hangs on large variables, npostavs, 2017/03/13