|
|
Blitz Support : |
From: Julian Cummings (cummings_at_[hidden])
Date: 2004-09-17 12:38:12
I have committed some changes to the blitz cvs repository last night to
correct this failure in the interlace testsuite code. The problem was
related to confusion between the MemoryBlock data_ pointer and the
MemoryBlockReference data_ pointer, which have very different uses. All
of the testsuite codes are now passing once again.
Regards, Julian C.
Patrick Guio wrote:
>
>
> Hi Julian,
>
> I ran the testsuite recently and found that the
> testsuite/interlace.cpp is broken.
> I tracked back the problem and it looks like the
> "subarray.reference(tmp)" is not done correctly in the
> makeInterlacedArray() at line 81 in blitz/array/interlace.cc
> Reintroducing the commented line (at line 213) in blitz/array/methods.cc:
> data_ = const_cast<P_numtype*>(array.data_);
> fixes the problem for interlace.cpp.
> but as your comment suggests:
> // JCC: changeBlock() already resets the data_ pointer to zeroth element
> it should already be done already in the call to changeBlock() at line
> 332 in blitz/memblock.h:
> data_ = block_->data() + offset;
> What do you think?
> Cheers, Patrick
>
>
> _______________________________________________
> Blitz-support mailing list
> Blitz-support_at_[hidden]
> http://www.oonumerics.org/mailman/listinfo.cgi/blitz-support
-- Dr. Julian C. Cummings E-mail: cummings_at_[hidden] California Institute of Technology Phone: 626-395-2543 1200 E. California Blvd., Mail Code 158-79 Fax: 626-584-5917 Pasadena, CA 91125