[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[pooma-dev] small bugs
From: |
ron hylton |
Subject: |
[pooma-dev] small bugs |
Date: |
Mon, 24 May 2004 10:59:26 -0400 |
Here are some small bugs that perhaps should be fixed in CVS.
in Array.h
Array::physicalDomain() should return an object rather than a reference
because view engines return a temporary layout and the current behavior is
sometimes returning a reference to the interior of a temporary. Fix:
inline const Domain_t physicalDomain() const
in IndexFunctionEngine.h
Engine::setDomain() should set firsts_m to be consistent with the
constructors. Fix:
void setDomain(const Domain_t &dom)
{
domain_m = dom;
for (int d = 0; d < Dim; ++d)
firsts_m[d] = domain_m[d].first();
}
I think IndexFunctionEngine also should have an Engine::layout() member to
be consistent with other Engines. The simplest possibility is:
inline Layout_t layout() const { return Layout_t(domain_m); }
in ForwardingEngine.h
in struct NewEngine
typedef Engine<Dim, T, CompFwd<NewEngine_t, Components> > Type_t;
should be
typedef Engine<NewEngine_t::dimensions, T, CompFwd<NewEngine_t,
Components> > Type_t;
Ron Hylton
_________________________________________________________________
Stop worrying about overloading your inbox - get MSN Hotmail Extra Storage!
http://join.msn.click-url.com/go/onm00200362ave/direct/01/
- [pooma-dev] small bugs,
ron hylton <=