Here are a couple of discussions explaining why Dispose is not necessary for a DataSet. To Dispose or Not to Dispose ?: The Dispose method in DataSet exists ONLY because of side effect of inheritance– in other words, it doesn’t actually do anything useful in the finalization. Should Dispose be called on DataTable and DataSet … Read more
The general consensus is that you do not (should not) need to dispose of HttpClient. Many people who are intimately involved in the way it works have stated this. See Darrel Miller’s blog post and a related SO post: HttpClient crawling results in memory leak for reference. I’d also strongly suggest that you read the … Read more
The recommended IDisposable pattern is here. When programming a class that uses IDisposable, generally you should use two patterns: When implementing a sealed class that doesn’t use unmanaged resources, you simply implement a Dispose method as with normal interface implementations: public sealed class A : IDisposable { public void Dispose() { // get rid of … Read more
The point of Dispose is to free unmanaged resources. It needs to be done at some point, otherwise they will never be cleaned up. The garbage collector doesn’t know how to call DeleteHandle() on a variable of type IntPtr, it doesn’t know whether or not it needs to call DeleteHandle(). Note: What is an unmanaged … Read more