Dmitry Gutov<dmitry@gutov.dev> writes:
Have you tried going in the other direction? Meaning, killing all locals
by default but keeping the ones that are really needed.
This is less backward compatible, but ultimately a more predictable and
common design. This seems to work okay here:
Stefan Monnier writes:
Looking at the patch from a distance it makes me wonder whether we
shouldn't call `kill-all-local-variables` instead?
IOW instead of listing all the vars that need to be killed, change to
code to preserve the few vars that should not be killed.
SGTM, that works too and it's definitely a simpler approach.