From 3872be543252a30d020e35233415c5fccc0ea806 Mon Sep 17 00:00:00 2001 From: Noel Grandin Date: Fri, 26 Apr 2013 13:41:29 +0200 Subject: Clang cleanup, unused mutex variables Some of these were doing nothing useful at all. I removed those. A couple were protecting static data, and I fixed those to use Mutex and MutexGuard properly. Change-Id: I444d13f8b6c50e526cce8d69d09aeaf240e5c6ca Reviewed-on: https://gerrit.libreoffice.org/3678 Reviewed-by: Tor Lillqvist Tested-by: Tor Lillqvist --- drawinglayer/source/geometry/viewinformation2d.cxx | 7 ------- drawinglayer/source/geometry/viewinformation3d.cxx | 3 --- drawinglayer/source/primitive3d/polygontubeprimitive3d.cxx | 3 ++- drawinglayer/source/primitive3d/sdrextrudeprimitive3d.cxx | 2 -- drawinglayer/source/primitive3d/sdrlatheprimitive3d.cxx | 2 -- 5 files changed, 2 insertions(+), 15 deletions(-) (limited to 'drawinglayer/source') diff --git a/drawinglayer/source/geometry/viewinformation2d.cxx b/drawinglayer/source/geometry/viewinformation2d.cxx index f7d0de8f0b85..8d7bc4c7811d 100644 --- a/drawinglayer/source/geometry/viewinformation2d.cxx +++ b/drawinglayer/source/geometry/viewinformation2d.cxx @@ -20,7 +20,6 @@ #include #include #include -#include #include #include #include @@ -322,8 +321,6 @@ namespace drawinglayer const basegfx::B2DRange& getDiscreteViewport() const { - ::osl::Mutex m_mutex; - if(maDiscreteViewport.isEmpty() && !maViewport.isEmpty()) { basegfx::B2DRange aDiscreteViewport(maViewport); @@ -336,8 +333,6 @@ namespace drawinglayer const basegfx::B2DHomMatrix& getObjectToViewTransformation() const { - ::osl::Mutex m_mutex; - if(maObjectToViewTransformation.isIdentity() && (!maObjectTransformation.isIdentity() || !maViewTransformation.isIdentity())) { @@ -350,8 +345,6 @@ namespace drawinglayer const basegfx::B2DHomMatrix& getInverseObjectToViewTransformation() const { - ::osl::Mutex m_mutex; - if(maInverseObjectToViewTransformation.isIdentity() && (!maObjectTransformation.isIdentity() || !maViewTransformation.isIdentity())) { diff --git a/drawinglayer/source/geometry/viewinformation3d.cxx b/drawinglayer/source/geometry/viewinformation3d.cxx index 7593dc0045f0..4707be1c6924 100644 --- a/drawinglayer/source/geometry/viewinformation3d.cxx +++ b/drawinglayer/source/geometry/viewinformation3d.cxx @@ -392,7 +392,6 @@ namespace drawinglayer const basegfx::B3DHomMatrix& getObjectToView() const { // on demand WorldToView creation - ::osl::Mutex m_mutex; if(maObjectToView.isIdentity()) { @@ -404,8 +403,6 @@ namespace drawinglayer const uno::Sequence< beans::PropertyValue >& getViewInformationSequence() const { - ::osl::Mutex m_mutex; - if(!mxViewInformation.hasElements()) { const_cast< ImpViewInformation3D* >(this)->impFillViewInformationFromContent(); diff --git a/drawinglayer/source/primitive3d/polygontubeprimitive3d.cxx b/drawinglayer/source/primitive3d/polygontubeprimitive3d.cxx index 77487db04db8..495c06f303dd 100644 --- a/drawinglayer/source/primitive3d/polygontubeprimitive3d.cxx +++ b/drawinglayer/source/primitive3d/polygontubeprimitive3d.cxx @@ -42,9 +42,10 @@ namespace drawinglayer static Primitive3DSequence aLineTubeList; static sal_uInt32 nLineTubeSegments(0L); static attribute::MaterialAttribute3D aLineMaterial; + static ::osl::Mutex aMutex; // may exclusively change static data, use mutex - ::osl::Mutex m_mutex; + ::osl::MutexGuard aGuard(aMutex); if(nSegments != nLineTubeSegments || !(rMaterial == aLineMaterial)) { diff --git a/drawinglayer/source/primitive3d/sdrextrudeprimitive3d.cxx b/drawinglayer/source/primitive3d/sdrextrudeprimitive3d.cxx index a488ad06dc48..198710299dcd 100644 --- a/drawinglayer/source/primitive3d/sdrextrudeprimitive3d.cxx +++ b/drawinglayer/source/primitive3d/sdrextrudeprimitive3d.cxx @@ -377,7 +377,6 @@ namespace drawinglayer // again when no longer geometry is needed for non-visible 3D objects as it is now for chart if(getPolyPolygon().count() && !maSlices.size()) { - ::osl::Mutex m_mutex; const_cast< SdrExtrudePrimitive3D& >(*this).impCreateSlices(); } @@ -493,7 +492,6 @@ namespace drawinglayer { // conditions of last local decomposition with reduced lines have changed. Remember // new one and clear current decompositiopn - ::osl::Mutex m_mutex; SdrExtrudePrimitive3D* pThat = const_cast< SdrExtrudePrimitive3D* >(this); pThat->setBuffered3DDecomposition(Primitive3DSequence()); delete pThat->mpLastRLGViewInformation; diff --git a/drawinglayer/source/primitive3d/sdrlatheprimitive3d.cxx b/drawinglayer/source/primitive3d/sdrlatheprimitive3d.cxx index 78825bdf35d7..33e39b148e6a 100644 --- a/drawinglayer/source/primitive3d/sdrlatheprimitive3d.cxx +++ b/drawinglayer/source/primitive3d/sdrlatheprimitive3d.cxx @@ -233,7 +233,6 @@ namespace drawinglayer // again when no longer geometry is needed for non-visible 3D objects as it is now for chart if(getPolyPolygon().count() && !maSlices.size()) { - ::osl::Mutex m_mutex; const_cast< SdrLathePrimitive3D& >(*this).impCreateSlices(); } @@ -355,7 +354,6 @@ namespace drawinglayer { // conditions of last local decomposition with reduced lines have changed. Remember // new one and clear current decompositiopn - ::osl::Mutex m_mutex; SdrLathePrimitive3D* pThat = const_cast< SdrLathePrimitive3D* >(this); pThat->setBuffered3DDecomposition(Primitive3DSequence()); delete pThat->mpLastRLGViewInformation; -- cgit