[vtkusers] Minor problem in vtkVolumeRayCastMIPFunction.cxx

Mathieu Malaterre mathieu.malaterre at kitware.com
Wed Apr 14 10:37:58 EDT 2004


Thanks Tuhin,

	this has been checked in the CVS. But I think you can use it safely as 
I don't know any implementation that has a different size for float and int.

Mathieu

Tuhin Sinha wrote:
> I noticed a mis-typed line when examining the code in
> vtkVolumeRayCastMIPFunction.cxx.
> 
> Variables voxel and prev_voxel are declared as int[3] arrays.  Yet there
> is a memcpy on line 170 (v1.30) that copies 3 floats from voxel to
> prev_voxel.
> 
> I have appended a patch to the latest CVS version (v1.30) of
> vtkVolumeRayCastMIPFunction.cxx that corrects this line.
> 
> Best Regards,
> 
> Tuhin Sinha
> 
> ###PATCH FOLLOWS###
> 170c170
> <         memcpy( prev_voxel, voxel, 3*sizeof(float) );
> ---
> 
>>        memcpy( prev_voxel, voxel, 3*sizeof(int) );
>>
>>
>>------------------------------------------------------------------------
>>
>>170c170
>><         memcpy( prev_voxel, voxel, 3*sizeof(float) );
>>---
>>
>>>        memcpy( prev_voxel, voxel, 3*sizeof(int) );






More information about the vtkusers mailing list