lewang / flx

Fuzzy matching for Emacs ... a la Sublime Text.
GNU General Public License v3.0
518 stars 37 forks source link

No longer bind obsolete max-specpdl-size #114

Closed tarsius closed 1 year ago

tarsius commented 1 year ago

It is obsolete since Emacs 29 and its docstring suggests binding max-lisp-eval-depth instead (which we already do):

Former limit on specbindings, now without effect. This variable used to limit the size of the specpdl stack which, among other things, holds dynamic variable bindings and unwind-protect activations. To prevent runaway recursion, use max-lisp-eval-depth instead; it will indirectly limit the specpdl stack size as well.

To play it safe, we could instead wrap the use of the obsolete variable with (with-suppressed-warnings ((obsolete max-specpdl-size)) ...).


The second commit enforces the use of spaces for indentation.