-
Notifications
You must be signed in to change notification settings - Fork 3
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
macro to set all the def-ed vars to nil #13
Comments
Thanks for this library, it looks fun & useful! 😄 About this issue: How were you thinking that it would work? Say that you're Would keeping the list of def-ed vars be the way to go? In say Let me know, thanks |
hey, @dotemacs I hope you found it helpful. Initially, I thought I could write a macro that would set the def-ed vars to nil when the function was called, but now that I think of it, your idea is better. we could keep a list of vars that the defined function has. and call a function like Is this something you'd like to work on ? |
Sure :)On 6 Oct 2022, at 10:04 am, Abhinav Omprakash ***@***.***> wrote:
hey, @dotemacs I hope you found it helpful.
Initially, I thought I could write a macro that would set the def-ed vars to nil when the function was called, but now that I think of it, your idea is better.
we could keep a list of vars that the defined function has. and call a function like foo-nil! (or a smaller name, the fewer characters the better. easier to remember, and more ergonomic)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Great, thanks! do you have an approach in mind? |
I'd like to suggest using My reasoning is that while using snitch, once vars have been def'd, sometimes after making changes, I don't realize that the code where that var was bound is not being hit any more (it was def'd from an earlier evaluation). When the var cannot be resolved, I know that the code was not hit. If the value were set to |
No description provided.
The text was updated successfully, but these errors were encountered: