Question

After reading about Control.Clear(); causing memory leaks (verified at MSDN), I was wondering if:

while (Controls.Count > 0) Controls[0].Dispose();

will be enough, or do I have to iterate recursively through all controls within controls?

Also, is there any reason to do a Control.Clear(); after that? (as I saw someone saying somewhere)?

Thanks.

Answer 1

The posted snippet is correct. No Clear() is required, the Control.Dispose() method already removes the control from the Controls collection. Which is why it works.

The less alarming version of the loop is:

 for (int ix = Controls.Count-1; ix >= 0; --ix) Controls[ix].Dispose();

No need to iterate through the children of the control and dispose them, that already happens automatically.