Generic WeakDom using AsRef<Instance> #488
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hey I had this idea for allowing WeakDom to be usable with a generic type parameter. The problem I am trying to solve is that additional metadata about instances (such as
RBXScriptSignal
) cannot be directly owned by the WeakDom. Each additional piece of metadata has to live in aHashMap<Ref,_>
somewhere else and be managed every time the dom is updated, so this essentially becomes a garbage collection problem. Let me know your thoughts!Here's a code example of how I imagine this being used:
Of course
MyInstance
would have some sort of collection ofRBXScriptSignal
instead of every instance having arender_stepped
property, but this is just an example.