[Insight-users] Resampling 64-bit crash
Luis Ibanez
luis.ibanez at kitware.com
Thu Feb 15 13:38:28 EST 2007
Hi Peter,
Yes, it seems that we should review many classes in order to
make a widespread use of the types that will be appropriate
for representing offsets in 64-bits.
During the release of ITK-3.0 we already did a good number of
updates, that were pointed out by warnings. Many of them were
related to size_t and size_type as they are used by std::
containers. We probably should take this review one step
further for ITK 3.2.
Regards,
Luis
----------------
Peter Cech wrote:
> On Thu, Feb 15, 2007 at 14:21:32 +0000, Florian Pierron wrote:
>
>>Hi Peter,
>>
>>For me as well, it's 8 for both.
>>
>>I think after recompiling the project properly, the resampling test is
>>working now. But I don't know how many itk files I should apply the
>>modification: long to long long and which itk filters will work and
>>which won't... So telling the compiler to code long on 8 byte seems the
>>easiest thing to do (if possible) but even so I'm not sure this would be
>>enough to make ITK handle 64-bit addressing properly...
>
>
> My guess is that with 8 byte longs, most of the filters will behave
> correctly (maybe try looking for 'memory model' or 'code model' to
> change it). I have looked into MetaIO code and it is using int in
> several places, so this will need to be fixed to handle loading/saving
> of big images. There might be several other places like that.
>
> @ ITK developers: What do you think about using size_t and ptrdiff_t for
> all size and offset variables in ITK?
>
> Regards,
> Peter Cech
>
>
>
>>Thanks for your help,
>>Florian
>>
>>On 15/02/2007 11:37, Atwood, Robert C wrote:
>>
>>>>What is the result of sizeof(size_t) and sizeof(ptrdiff_t)?
>>>>(Both types are from <cstddef> header.)
>>>>
>>>
>>>My results: 8 for both. Some details of compiler/system below.
>>>Hope it helps. Allocation appears to work but file i/o (.mhd or .hdr so
>>>far tested) does not, for such large files.
>>>Hope it helps find the problems.
>>>
>>>Many thanks,
>>>Robert
>
> _______________________________________________
> Insight-users mailing list
> Insight-users at itk.org
> http://www.itk.org/mailman/listinfo/insight-users
>
More information about the Insight-users
mailing list