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.

Sign up for GitHub

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

Jump to bottom

Does el-patch byte compile the function that it patches? #20

Closed
rgrinberg opened this issue Feb 23, 2018 · 1 comment
Closed

Does el-patch byte compile the function that it patches? #20

rgrinberg opened this issue Feb 23, 2018 · 1 comment
Labels
question

Comments

@rgrinberg
Copy link

rgrinberg commented Feb 23, 2018 •
edited
Loading

Sorry if this question is silly, but I'm trying to using el-patch to patch a function for performance reasons. In this case, it seems like it might be important that the patched function is also byte compiled. Does el-patch guarantee this?
@raxod502
Copy link
Member

I don't byte-compile my init file, so I don't know the answer to this. It seems possible, since el-patch resolves patches at macroexpansion time, and macroexpanded code should be subject to byte-compilation. Why don't you look at the compiled code and see?

@raxod502 raxod502 closed this as completed May 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
question
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rgrinberg @raxod502