[vtkusers] Bug in vtkDataArray.cxx

Berk Geveci berk.geveci at kitware.com
Tue Mar 19 11:50:21 EST 2002


This is not really a bug. vtkTimeStamp has a cast
operator and it can be used as if it was unsigned int.
It's poor style however.
-Berk

On Tue, 2002-03-19 at 11:19, Sander Niemeijer wrote:
> While browsing the code I think I stumbled onto a possible bug in 
> Common/vtkDataArray.cxx  (VTK 4.0 official release) at:
> ---
> void vtkDataArray::ComputeRange(int comp)
> {
>    float s;
>    vtkIdType numTuples;
> 
>    if ( (this->GetMTime() > this->ComputeTimeForLastRange) ||
>         (comp != this->ComponentForLastRange))
>      {
>      numTuples=this->GetNumberOfTuples();
>      this->Range[0] =  VTK_LARGE_FLOAT;
> ---
> 
> There is a check between a GetMTime() return value (which is an unsigned 
> long) and an object pointer (this->ComputeTimeForLastRange). I presume
> ---
>    if ( (this->GetMTime() > this->ComputeTimeForLastRange.GetMTime()) ||
> ---
> was meant.
> 
> By the way, I was wondering... Is this list the appropriate place to 
> post bugreports?
> I have posted another bugreport to this list a month ago about a bug in 
> vtkWin32OpenGLRenderWindow but I still haven't received any response on 
> that e-mail.
> 
> Regards,
> Sander Niemeijer
> 
> _______________________________________________
> This is the private VTK discussion list. 
> Please keep messages on-topic. Check the FAQ at: <http://public.kitware.com/cgi-bin/vtkfaq>
> Follow this link to subscribe/unsubscribe:
> http://public.kitware.com/mailman/listinfo/vtkusers





More information about the vtkusers mailing list