template trouble

I'm working on a little class that runs marginally faster than vector< vector<> >.

The compiler doesn't like my syntax for specifying a type defined within the class template.
You'll get the error, then the code, then the class.

Code:

/home/david/Documents/c++/bitmap/bitmap/array2d.hpp|46|error: expected unqualified-id before ‘&’ token|
/home/david/Documents/c++/bitmap/bitmap/array2d.hpp|46|error: expected ‘,’ or ‘...’ before ‘&’ token|
/home/david/Documents/c++/bitmap/bitmap/array2d.hpp||In function ‘void for_each(array2d<T, uint>&)’:|
/home/david/Documents/c++/bitmap/bitmap/array2d.hpp|49|error: ‘top_left_point’ was not declared in this scope|
/home/david/Documents/c++/bitmap/bitmap/array2d.hpp|49|error: ‘bottom_right_point’ was not declared in this scope|

Code:

template<typename T, typename uint, class ternary_func_t>
inline void for_each(array2d<T,uint> & a, const array2d<T,uint>::point2d & top_left_point,
		const array2d<T,uint>::point2d & bottom_right_point, ternary_func_t & f)
{
	for(uint i = top_left_point.first; i < bottom_right_point.first; ++i)
	{
		for(uint j = top_left_point.second; j < bottom_right_point.second; ++j)
			f( a, i, j );
	}
}

Code:

template<typename T, typename uint = std::size_t>
class array2d
{
	T** data;
	const uint sz;
	const uint w;
	const uint h;
	array2d(array2d<T> & other);		//I want to disable
	array2d(const array2d<T> & other);	//copy behavior for now.
	array2d & operator = (array2d<T> & other);
	array2d & operator = (const array2d<T> & other);
  public:
       typedef std::pair<uint, uint> point2d;
	array2d(uint, uint);
	~array2d();
	const uint size() const { return sz; }
	const uint width() const { return w; }
	const uint height() const { return h; }
	T & operator()(uint x, uint y) { return data[x][y]; }
	T & operator()(point2d p) { return (*this)(p.first, p.second); }
};

Thanks.

It would be better if you provided the smallest and simplest program that demonstrates the error. My guess is that array2d<T,uint>::point2d should be typename array2d<T,uint>::point2d.

That did it.

Thanks, and sorry for the dump.

No problem

By the way, to disable copying you just need to make these private:

Code:

array2d(const array2d<T> & other);
array2d & operator = (const array2d<T> & other);

The versions that have a non-const parameter are not const correct anyway.

There's Boost.Multi_Array, if you just need a multi-dimensional array.