Skip to content

Conversation

jakobjpeters
Copy link

Closes #50.

@timholy
Copy link
Owner

timholy commented Jul 7, 2025

Sorry I didn't see this earlier. It's a nice idea, but I do worry about the unrestricted signature for x: this replaces the default "do-nothing" fallback with a method that tries to do something.

I wonder if rather than relying on dispatching the callee, what if we put a check for !isempty(methods(x)) && _visit(...) in the caller of the method you've written?

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.

Incorrect dispatch on Base.Callable

2 participants