remove container ref from signal #333
wmertens
started this conversation in
Proposals For Qwik
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
What is it about?
memory optimization
What's the motivation for this proposal?
Each signal stores the container so that effects can be scheduled. But the scheduling is only needed for vnode subscribers, so then the container can be retrieved by walking up the vnode tree.
The container will also often already be in the context.
Proposed Solution / Feature
Therefore, we don't need to store the container on the signal, and in doing so each signal is 8 bytes smaller. That matters when you have large DOM structures.
Links / References
No response
Beta Was this translation helpful? Give feedback.
All reactions