Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Handle not imported ServiceRefs #82

Open
junha1 opened this issue Jul 23, 2020 · 1 comment
Open

Handle not imported ServiceRefs #82

junha1 opened this issue Jul 23, 2020 · 1 comment
Milestone
0.4.1

Comments

@junha1
Copy link
Contributor

junha1 commented Jul 23, 2020

It should always be imported.

The policy to handle a not imported case is not decided
@junha1 junha1 added this to the 0.3 milestone Jul 23, 2020
@junha1
Copy link
Contributor Author

junha1 commented Jul 23, 2020 •
edited
Loading

not imported ServiceRef is problematic since it's a memory leak.
However not exported ServiceRef is not that problematic, but highly likely to be a mistake anyway, so worth it to warn that.

@junha1 junha1 modified the milestones: 0.3, 0.3.1, 0.4 Jul 29, 2020
@junha1 junha1 modified the milestones: 0.4, 0.4.1 Aug 6, 2020
@junha1 junha1 changed the title Handle a not imported ServiceRef Handle not imported ServiceRefs Sep 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.4.1
Development

No branches or pull requests
1 participant
@junha1