On 12/21/2012 03:51 AM, Kinkie wrote: >> Please do not check for nil before deleting a pointer. Delete does that >> for you. > > Hi all, > revised patch, addressing this for the whole destructor. > It passes build and unit tests.
Looks good to me, but I did not check whether there are some hidden dependencies among the objects we are destroying that require us to set their pointers to NULL and/or destroy them in a certain order. However, I do not object to this being committed if nobody can check this. Thank you, Alex.
