[ITK] [ITK-dev] Recent changes have introduced valgrind defects

[Matt McCormick]

On Wed, Aug 19, 2015 at 11:57 AM, Luc Hermitte <luc.hermitte at c-s.fr> wrote:
>>> Recent changes have caused valgrid defects:
>>> https://open.cdash.org/viewDynamicAnalysis.php?buildid=3962722
>> These are probably related to this change:
>>
>>   http://review.source.kitware.com/#/c/19962/
>>

> The tests on lines 160, 166, and 172 are incorrect. After the SetSize,
> the value x[0] is left uninitialized on purpose. It could be equal to
> ref[0].
> Instead, we should test that x[0] is uninitialized. But I don't know any
> way to do it.

In that case, please remove them.


> I'll comment these lines for my next commit (on C++ rvalue references)
> if it's OK with you.

Please keep the patches divided by subject if possible.


Is there a different issue causing the defect in
itkImageToVectorImageFilterTest?

Thanks,
Matt
_______________________________________________
Powered by www.kitware.com

Visit other Kitware open-source projects at
http://www.kitware.com/opensource/opensource.html

Kitware offers ITK Training Courses, for more information visit:
http://kitware.com/products/protraining.php

Please keep messages on-topic and check the ITK FAQ at:
http://www.itk.org/Wiki/ITK_FAQ

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/insight-developers