Skip to content
This repository has been archived by the owner on Sep 28, 2024. It is now read-only.

The di() delegate is no longer caching value from dicontainer #837

Closed
gtnarg opened this issue Nov 7, 2018 · 1 comment
Closed

The di() delegate is no longer caching value from dicontainer #837

gtnarg opened this issue Nov 7, 2018 · 1 comment

Comments

@gtnarg
Copy link
Collaborator

gtnarg commented Nov 7, 2018

It seems in the last refactoring the semantics of this delegate changed. It should not be querying the dicontainer on every access. I will submit a PR to fix.
edvin pushed a commit that referenced this issue Nov 8, 2018
Merge pull request #838 from gtnarg/di_issue
c86d7cb 
Pull request for Issue #837
@edvin
Copy link
Owner

edvin commented Nov 8, 2018

Merged, thank you Grant!

@edvin edvin closed this as completed Nov 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@edvin @gtnarg