<span class="Apple-style-span" style="border-collapse:collapse;color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px">I would fully support removing the environment variable test for deciding whether to fail or not here.<div>
<br></div><div>There's no need to pass the tests when there are memory leaks. That only encourages memory leaks, and enables people to ignore them.</div><div><br></div><div>Get rid of them (mem leaks that is). If you don't like it, then build with mem leak detection turned off entirely.</div>
<div><br></div><div><br></div><div>Just my opinion, of course.</div><div>David C.</div><div><br></div></span><br><div class="gmail_quote">On Mon, Feb 13, 2012 at 11:46 AM, Marcus D. Hanwell <span dir="ltr"><<a href="mailto:marcus.hanwell@kitware.com">marcus.hanwell@kitware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I would like to change the default behavior of debug leaks. Currently<br>
when you turn it on, if you run tests locally and there are leaks a<br>
test will pass. You will only spot the failure if you run the tests<br>
with -V and scan the output. See,<br>
<br>
976: vtkDebugLeaks has detected LEAKS!<br>
976: Class "vtkBlueObeliskData" has 1 instance still around.<br>
976: Class "vtkUnsignedShortArray" has 2 instances still around.<br>
976: Class "vtkFloatArray" has 10 instances still around.<br>
976: Class "vtkStringArray" has 7 instances still around.<br>
976:<br>
1/1 Test #976: Chemistry-TestBallAndStick .......   Passed    0.37 sec<br>
<br>
This has caught me out on more than one occasion, including the recent<br>
merge of the chemistry branch into VTK. We will work on getting these<br>
fixed up, but I would like to make the test fail by default locally.<br>
Does anyone remember the reason for doing this, it seems to go back<br>
quite a way as it also looks for DART_TEST_FROM_DART.<br>
<br>
Exporting either of those variables will then cause the test to fail<br>
locally. If there is a good reason to do this, wouldn't it be better<br>
to pass if the user explicitly exported a variable,<br>
I_KNOW_I_TURNED_ON_DEBUG_LEAKS_BUT_I_STILL_WANT_TESTS_TO_PASS or<br>
similar?<br>
<br>
Thanks,<br>
<br>
Marcus<br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br>
</blockquote></div><br>