[Insight-developers] Re: STEP 1: itpack.cxx: 3999 lines non covered

[Luis Ibanez]

Hi Brian,

Thanks for looking into this.

Utilities is build before the rest of the code.
So, it should be safe to move it there.


It happens that

           Utilities/vxl/v3p/

has already "netlib" code from which this file
seems to be originated.

    http://netlib . caspur . it/itpack/

The suggestion will be to move the itpack file to

           Utilities/vxl/v3p/netlib

with its original name            "dsrc2c.c"
along with its fortran version    "dsrc2c.f"

The C functions will then be available for you.



Would you like us to move the file ?

Could you point us to the original source fo the
fortran file ? or should we just take it from the
URL:  http://netlib . caspur . it/itpack/  ?

Please let us know,
We are quite anxious for taking this 3999 lines out
of the red coverage list.

These 20 most wanted files are standing between us
and our beers   :-)


Thanks


   Luis



-----------------------
Brian B Avants wrote:
> 
> Luis -
> 
> I believe the only issues would involve linking -> changing the cmake
> files etc.  There's nothing special going on there ... we'd just have to
> make sure that it's compiled before the FEM stuff and gets linked in where
> needed.
> 
> correct me if i'm wrong ...
> 
> Brian
> 
> 
> On Thu, 7 Aug 2003, Luis Ibanez wrote:
> 
> 
>>Tessa, Aljaz, Brian, Jim
>>
>>Looking into the coverage report, we found that
>>itpack.cxx has 3999 lines of uncovered code.
>>
>>It seems that this class is actually ported from
>>netlib/itpack and hence only a subsete of its
>>functionality is tested in ITK.
>>
>>The file says:
>>
>>"  ITPACK was converted with the following command line options:
>>    *    f2c -C++P dsrc2c.f"
>>
>>This puts this file in the same position as VNL,
>>that it, it should be in the Utilities directory
>>since it is a third party library for which we
>>shouldn't be responsible of the testing.
>>
>>Do you anticipate any issues if we move this file
>>into Utilities ?
>>
>>
>>Please let us know,
>>
>>
>>    Thanks
>>
>>
>>      Luis
>>
> 
> 
> _______________________________________________
> Insight-developers mailing list
> Insight-developers at itk . org
> http://www . itk . org/mailman/listinfo/insight-developers
>