Blitz logo

Blitz Bugs :

From: Matthew K Grimes (mkg_at_[hidden])
Date: 2004-05-17 18:18:30


I've seen 2 documentation bugs:

1) Misleading "column array" entry.

Description:
In the framed HTML documentation, the index sidebar has a link for
"column major", which takes you to a part of the documentation that says
that the FortranArray<int> memory layout is column major. This implies
that if you want to make a column-major array, you should use
FortranArray. What it doesn't mention, at least in that section, is that
FortranArray isn't simply a column-major array, but an array laid out in
memory exactly as FORTRAN would have it.

Symptom:
I wanted to wrap a double[] array "dblArr" with a column major matrix
"cmArr", so I used the constructor:

    Array<double,2> cmArr( dblArr, Shape(numRows, numColumns),
neverDeleteData, fortranArray);

When I subsequently indexed into cmArr, the reported values were corrupted.
 

Proposed solution:
Make the "column major" link point instead to section 2.9.1, where
ColumnArray<int> is mentioned.

2) Outdated name "ColumnArray"

Description:
Section 2.9.1 mentions the array layout "ColumnArray<int>", but the
template has since been renamed to ColumnMajorArray<int>.

Best,
-- Matt