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

memleaks => using (var x = y()) {...} #122

Closed
GerHobbelt opened this issue Oct 27, 2019 · 1 comment
Closed

memleaks => using (var x = y()) {...} #122

GerHobbelt opened this issue Oct 27, 2019 · 1 comment
Labels
🐛bug Something isn't working 🕵investigate Needs further analysis to find the root cause.
Projects
TODO list v82pre4
Milestone
v82

Comments

@GerHobbelt
Copy link
Collaborator

DevStudio Code Analysis does not report IDisposable objects: it has previously missed quite a few Stream / StreamReader / WebRequest instantiations which were lacking any kind of Dispose invocation.
@GerHobbelt GerHobbelt created this issue from a note in TODO list (In progress) Oct 27, 2019
@GerHobbelt GerHobbelt changed the title **memleaks => using (var x = y()) {...}** memleaks => using (var x = y()) {...} Oct 27, 2019
@GerHobbelt GerHobbelt added 🐛bug Something isn't working 🕵investigate Needs further analysis to find the root cause. labels Oct 27, 2019
@GerHobbelt GerHobbelt added this to the v82 milestone Oct 27, 2019
GerHobbelt added a commit to GerHobbelt/qiqqa-open-source that referenced this issue Oct 27, 2019
@GerHobbelt
more work done on jimmejardine#112 / jimmejardine#122 :
90364ad 
- memleak prevention via `using(){...}` of IDisposable
@GerHobbelt
Copy link
Collaborator Author

Fixed as per today (marker commit: 95dff9b), will be available in upcoming pre-release v82pre4.

TODO list automation moved this from In progress to Done Nov 1, 2019
@GerHobbelt GerHobbelt added this to Done in v82pre4 Nov 3, 2019
GerHobbelt added a commit to GerHobbelt/qiqqa-open-source that referenced this issue Nov 5, 2019
@GerHobbelt
one more for jimmejardine#122 after the fact.
75d02e8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🐛bug Something isn't working 🕵investigate Needs further analysis to find the root cause.
Projects
TODO list
  
Done
v82pre4
  
Done
Milestone
v82
Development

No branches or pull requests
1 participant
@GerHobbelt