[mgunit] Object leak in mgunit.pro in trunk

[Michael Galloy]

Ah, thanks! I made the runner own the test suites so that they would be freed in the same manner for the GUI runner and the others.

-Mike

On Feb 9, 2010, at 9:49 AM, Symons, Greg - GS wrote:

> The patch below fixes an object leak in mgunit.pro. I didn't really look to see if this was necessarily the best place to fix it; should the testRunner take ownership?
> 
> Index: src/mgunit.pro
> ===================================================================
> --- src/mgunit.pro      (revision 90)
> +++ src/mgunit.pro      (working copy)
> @@ -59,6 +59,6 @@
>     testsuite->run
>     testsuite->getProperty, npass=npass, nfail=nfail, ntests=ntests
> 
> -    if (~keyword_set(gui)) then obj_destroy, testRunner
> +    if (~keyword_set(gui)) then obj_destroy, [testRunner, testSuite]
>   endif
> end
> 
> --
> Gregory Symons
> Software Engineer
> ITT Corporation
> Geospatial Systems
> 
> 1919 West Cook Rd
> PO Box 3700
> Fort Wayne, IN 46801
> M/S 604
> 
> greg.symons at itt.com
> tel: (260) 451-6402
> 
> 
> This e-mail and any files transmitted with it may be proprietary and are intended solely for the use of the individual or entity to whom they are addressed. If you have received this e-mail in error please notify the sender.
> Please note that any views or opinions presented in this e-mail are solely those of the author and do not necessarily represent those of ITT Corporation. The recipient should check this e-mail and any attachments for the presence of viruses. ITT accepts no liability for any damage caused by any virus transmitted by this e-mail.
> _______________________________________________
> mgunit mailing list
> mgunit at lists.idldev.com
> http://lists.idldev.com/listinfo.cgi/mgunit-idldev.com