🐛 :: Append closure to observers before executing #5
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.
When subscribing to a
Variable<T>
the initial value triggers the provided closure. If the closure makes changes to the underlying value of the sameVariable<T>
the closure does not re-trigger on its first run.Sample Code Showcasing Issue
Expected Output
InitialValue
SecondaryValue
Actual Output
InitialValue
Solution
The proposed changes make it so the closure is added to the observer list before it's executed for the first time