[Insight-developers] Testing philosophy question

Bill Lorensen bill.lorensen at gmail.com
Tue Dec 23 11:29:03 EST 2008


Dan,

Maybe they should be signed, but they don't have to be. I believe the
user would still get a valid result with signed input. I would fix the
warning.

Bill

On Tue, Dec 23, 2008 at 11:04 AM, Blezek, Daniel J.
<Blezek.Daniel at mayo.edu> wrote:
> Hi,
>
>   I found a warning in itkConnectComponentsImageFilter when the output type
> is signed.  There is a comparison on line 608 of the .txx file between the
> backgroundvalue (Output type) an unsigned long int.  The code is correct,
> but gcc warns of a comparison between signed and unsigned.
>
>   Now I realize that my label maps should be unsigned, which would fix the
> warning.  Should the filter warn that the output type should be unsigned? or
> should code be fixed to accommodate signed and unsigned types?  I'm happy to
> do the fix and add a new test.
>
> -dan
>
> Daniel Blezek, PhD
> Medical Imaging Informatics Innovation Center
>
> P 127 or (77) 8 8886
> T 507 538 8886
> E blezek.daniel at mayo.edu
>
> Mayo Clinic
> 200 First St. S.W.
> Harwick SL-44
> Rochester, MN 55905
> mayoclinic.org
>
> _______________________________________________
> Insight-developers mailing list
> Insight-developers at itk.org
> http://www.itk.org/mailman/listinfo/insight-developers
>
>


More information about the Insight-developers mailing list