codereckons/rotgen
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[ARITHMETIC][FUNCTIONS] modified teh transpose and adjoint implementations to use the in place version and avoid having a redundant copy

Browse source
This commit is contained in:
kallore 2025-05-20 15:22:55 +02:00
parent 2cb2724bcd
commit b206f40372
1 changed files with 5 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

8
src/matrix_impl64.cpp
View file

@ -79,8 +79,9 @@ namespace rotgen
matrix_impl64 matrix_impl64::transpose() const matrix_impl64 matrix_impl64::transpose() const
{ {
matrix_impl64 result(*this); matrix_impl64 result(*this);
result.storage_->data = storage_->data.transpose(); result.storage_->data.transposeInPlace();
return result; return result;
} }
matrix_impl64 matrix_impl64::conjugate() const matrix_impl64 matrix_impl64::conjugate() const
@ -93,7 +94,7 @@ namespace rotgen
matrix_impl64 matrix_impl64::adjoint() const matrix_impl64 matrix_impl64::adjoint() const
{ {
matrix_impl64 result(*this); matrix_impl64 result(*this);
result.storage_->data = storage_->data.adjoint(); result.storage_->data.adjointInPlace();
return result; return result;
} }
@ -144,7 +145,8 @@ namespace rotgen
return *this; return *this;
} }
matrix_impl64 matrix_impl64::operator-() const { matrix_impl64 matrix_impl64::operator-() const
{
matrix_impl64 result(*this); matrix_impl64 result(*this);
result.storage_->data = -result.storage_->data; result.storage_->data = -result.storage_->data;
return result; return result;