I posted a patch in BerliOS a few days ago [...]
We are not that fast - there are patches much older than that. What came in mind when I saw this particular one: It's missing a few words of explanation. I had to look for myself where the new[] stuff was applied and whether this is correct at all. Please add an explanation next time which will help to easily understand what your patch actually fixes. For example:
"Memory leak detected. You are creating a char buffer array with
buf = new char[len]; in
bool Manager::LoadResource(const wxString& file) but do not release the memory by
delete[] buf;, instead you are only using
delete buf;. This patch fixes the issue."
Please understand that this is much easier than reading the raw patches.
With regards, Morten.
Edit: To avoid any misunderstanding: This is a *good* patch, anyway.