Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Evaluate using lexical-binding if the file is lexically scoped #91

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

phikal
Copy link

@phikal phikal commented May 28, 2021

I have code in my init.el that requires lexical scoping, and it could not be evaluated correctly without passing setting lexical when calling eval.

This patch uses the buffer-local value of lexical-binding to invoke eval correctly.

@phikal
Copy link
Author

phikal commented May 28, 2021

And as a side note, while debugging I ended up simplifying the code a bit, but there too many changes to make a clean sequence of commits. Ultimately, I just grouped them all into one big dummy commit, and if you are interested, I could extract certain changes and propose them as separate push requests.

@phikal
Copy link
Author

phikal commented May 6, 2022

Ping?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant