From 0f28c8612f4269cec95688b53d182c7c0169236d Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Tue, 23 Jan 2018 16:45:23 +0200 Subject: loplugin:unused-returns in basegfx..cppcanvas Change-Id: I32dc8c92871c8349651d2f4204a332d387e6e1b2 Reviewed-on: https://gerrit.libreoffice.org/48428 Tested-by: Jenkins Reviewed-by: Noel Grandin --- basegfx/source/matrix/b3dhommatrix.cxx | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) (limited to 'basegfx/source') diff --git a/basegfx/source/matrix/b3dhommatrix.cxx b/basegfx/source/matrix/b3dhommatrix.cxx index e6a2bd3b4f88..6cfd054992ec 100644 --- a/basegfx/source/matrix/b3dhommatrix.cxx +++ b/basegfx/source/matrix/b3dhommatrix.cxx @@ -85,7 +85,7 @@ namespace basegfx *mpImpl = Impl3DHomMatrix(); } - bool B3DHomMatrix::invert() + void B3DHomMatrix::invert() { Impl3DHomMatrix aWork(*mpImpl); std::unique_ptr pIndex( new sal_uInt16[Impl3DHomMatrix_Base::getEdgeLength()] ); @@ -94,10 +94,7 @@ namespace basegfx if(aWork.ludcmp(pIndex.get(), nParity)) { mpImpl->doInvert(aWork, pIndex.get()); - return true; } - - return false; } double B3DHomMatrix::determinant() const @@ -376,15 +373,15 @@ namespace basegfx mpImpl->doMulMatrix(aOrientationMat); } - bool B3DHomMatrix::decompose(B3DTuple& rScale, B3DTuple& rTranslate, B3DTuple& rRotate, B3DTuple& rShear) const + void B3DHomMatrix::decompose(B3DTuple& rScale, B3DTuple& rTranslate, B3DTuple& rRotate, B3DTuple& rShear) const { // when perspective is used, decompose is not made here if(!mpImpl->isLastLineDefault()) - return false; + return; // If determinant is zero, decomposition is not possible if(determinant() == 0.0) - return false; + return; // isolate translation rTranslate.setX(mpImpl->get(0, 3)); @@ -536,8 +533,6 @@ namespace basegfx // correct rotate values rRotate.correctValues(); } - - return true; } } // end of namespace basegfx -- cgit