Luis,

In short no reason. The patch should just be taken as a list of 14 tests, which 
currently are not being run. Arguments need to be determined for them.

If I recall correctly, my first goal was to determine the list of test which 
were being compiled but not being run. So I commented out all that appears to 
not be running, and tested. Then as I added tests, I un-commented the test file 
line. I was stashing and add via a patch, so I clearly made a mistake during 
the tedious process.

I hope this list is useful.

Brad



On Feb 14, 2014, at 7:32 AM, Luis Ibanez <[email protected]> wrote:

> Brad,
> 
> Why are you commenting out the .cxx files in lines 42-45 ?
> 
> http://review.source.kitware.com/#/c/13646/2/Modules/Filtering/LabelMap/test/CMakeLists.txt
> 
> Wouldn't that reduce the code coverage instead of increasing it ?
> 
> 
>     Luis
> 

_______________________________________________
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://www.itk.org/mailman/listinfo/insight-developers

Reply via email to