Re: M-x elp-instrument-package RET RET makes emacs unusable

[Simon Josefsson]

"Eli Zaretskii" <eliz@is.elta.co.il> writes:

> > I wanted to instrument a package for profiling, and typed M-x
> > elp-instrument-packe RET and accidently hit one RET extra, thus
> > presumably trying to instrument all elisp functions.  After this emacs
> > is completely unusable, I can't even kill the window:
> > 
> > Lisp nesting exceeds max-lisp-eval-depth
> > 
> > I'm not sure if it's possible to prevent this properly, but it's quite
> > annoying if you had lots of work in progress.
> > 
> > (Happens in both 20.7 and 21.0 pretest)
> 
> Does the change below look okay?

Yup, works fine.  It was the quick hack I had in mind.

> (It's possible that a better way would be to count the number of
> instrumented functions and bail out if it's too large, but I don't
> know how to define ``too large''.)

Yes, it is difficult.

And I'm not even sure it's only due to instrumenting too many
functions, it might be a problem instrumenting some single function --
I mean, C-h f ? only list about 8000 functions, while C-h f gnus ?
list about 2000, and I often profile entire gnus without any problems.
So raising max-lisp-eval-depth and max-specdl-size by about 4 times
should help, but it doesn't. (I've tried raising it 100 times without
any improvement.)