Skip to content

Remove invoke#348

Open
ptiede wants to merge 1 commit intoJuliaArrays:masterfrom
ptiede:ptiede-invoke
Open

Remove invoke#348
ptiede wants to merge 1 commit intoJuliaArrays:masterfrom
ptiede:ptiede-invoke

Conversation

@ptiede
Copy link

@ptiede ptiede commented Mar 24, 2026

Using invoke in the SA extension is a little strange and is creating a StackOverflow in Reactant land. This removes the invoke and tries to be a little more explicit about what we are calling here.

@ptiede
Copy link
Author

ptiede commented Mar 24, 2026

pre error appears to be independent from this PR

@aplavin
Copy link
Member

aplavin commented Mar 25, 2026

invoke support seems a Reactant-specific limitation, but I don't have anything against this PR – will merge unless anyone objects
code readable both before and after

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.

2 participants