diff options
| author |  Benoit Jacob <jacob.benoit.1@gmail.com> | 2008-05-28 05:14:16 +0000 |
|---|---|---|
| committer | Benoit Jacob <jacob.benoit.1@gmail.com> | 2008-05-28 05:14:16 +0000 |
| commit | f54760c889efcfabfdb8be594abee0647ed0c8da (patch) | |
| tree | 0e20a6997401343ac1f64139c0b7098f5b237c7d /Eigen/src/Core/util/Constants.h | |
| parent | aebecae510dd29f5573d3f86dfed526e6d8be9a8 (diff) | |
hehe, the complicated nesting scheme in Flagged in the previous commit
was a sign that we were doing something wrong. In fact, having
NestByValue as a special case of Flagged was wrong, and the previous
commit, while not buggy, was inefficient because then when the resulting
NestByValue xpr was nested -- hence copied -- the original xpr which was
already nested by value was copied again; hence instead of 1 copy we got
3 copies.
The solution was to ressuscitate the old Temporary.h (renamed
NestByValue.h) as it was the right approach.
Diffstat (limited to 'Eigen/src/Core/util/Constants.h')
| -rw-r--r-- | Eigen/src/Core/util/Constants.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/Eigen/src/Core/util/Constants.h b/Eigen/src/Core/util/Constants.h index 0e6ed4b21..1a84e64c6 100644 --- a/Eigen/src/Core/util/Constants.h +++ b/Eigen/src/Core/util/Constants.h @@ -45,7 +45,6 @@ const unsigned int SelfAdjointBit = 0x100; ///< means the matrix is selfadjoint const unsigned int UpperTriangularBit = 0x200; ///< means the strictly triangular lower part is 0 const unsigned int LowerTriangularBit = 0x400; ///< means the strictly triangular upper part is 0 const unsigned int DirectAccessBit = 0x800; ///< means the underlying matrix data can be direclty accessed -const unsigned int NestByValueBit = 0x1000; ///< means the expression should be copied by value when nested // list of flags that are inherited by default const unsigned int HereditaryBits = RowMajorBit |