Types that own disposable fields should be disposable

Hello everyone,


For this best practices rule,

http://msdn.microsoft.com/en-us/ms182172.aspx

What are the benefits of following this rule even if I am not wrapping an unmanaged resource, like FileStream in the sample in the page.

I think it is fine if the class does not implement IDisposable, since FileStream could be disposed (to release the native file handle) since it has its own Dispose method. Any ideas?


thanks in advance,
George

It doesn't matter if the class you wrap has any handles internally. For all you know, the class you wrap needs to be disposed. If your class is disposed, the members of your class should be disposed, too. And if the class you wrap is disposable, you should not hide this with your wrapper class.