[Libreoffice-commits] core.git: comphelper/source connectivity/source cppcanvas/source

Jochen Nitschke j.nitschke+logerrit at ok.de
Fri Apr 8 09:59:00 UTC 2016


 comphelper/source/container/enumerablemap.cxx            |    8 ++--
 comphelper/source/misc/servicedecl.cxx                   |    7 ++-
 comphelper/source/property/opropertybag.hxx              |    6 ++-
 comphelper/source/streaming/seqinputstreamserv.cxx       |    8 ++--
 comphelper/source/streaming/seqoutputstreamserv.cxx      |    8 ++--
 connectivity/source/commontools/TTableHelper.cxx         |    7 ++-
 cppcanvas/source/mtfrenderer/bitmapaction.cxx            |    1 
 cppcanvas/source/mtfrenderer/cachedprimitivebase.hxx     |    7 ++-
 cppcanvas/source/mtfrenderer/lineaction.cxx              |    7 ++-
 cppcanvas/source/mtfrenderer/pointaction.cxx             |    7 ++-
 cppcanvas/source/mtfrenderer/polypolyaction.cxx          |    2 -
 cppcanvas/source/mtfrenderer/textaction.cxx              |   29 ++++++++++-----
 cppcanvas/source/mtfrenderer/transparencygroupaction.cxx |    7 ++-
 13 files changed, 63 insertions(+), 41 deletions(-)

New commits:
commit f781997ee1dcb61b01b04cc050001e2f46b12dfe
Author: Jochen Nitschke <j.nitschke+logerrit at ok.de>
Date:   Thu Apr 7 20:31:52 2016 +0200

    tdf#94306 replace boost::noncopyable in c...
    
    comphelper, connectivity and cppcanvas.
    
    Replace with C++11 delete copy-constructur
    and copy-assignment.
    Removed unused boost/noncopyable.hpp includes from
    some source files in cppcanvas.
    
    Change-Id: I90780820e21fbfd291ac10c266e7d16616e3a81b
    Reviewed-on: https://gerrit.libreoffice.org/23905
    Tested-by: Jenkins <ci at libreoffice.org>
    Reviewed-by: Michael Stahl <mstahl at redhat.com>

diff --git a/comphelper/source/container/enumerablemap.cxx b/comphelper/source/container/enumerablemap.cxx
index fc255f0..afbfb29 100644
--- a/comphelper/source/container/enumerablemap.cxx
+++ b/comphelper/source/container/enumerablemap.cxx
@@ -42,8 +42,6 @@
 #include <memory>
 #include <utility>
 
-#include <boost/noncopyable.hpp>
-
 namespace comphelper
 {
 
@@ -212,7 +210,7 @@ namespace comphelper
     };
 
 
-    class MapEnumerator: private boost::noncopyable
+    class MapEnumerator
     {
     public:
         MapEnumerator( ::cppu::OWeakObject& _rParent, MapData& _mapData, const EnumerationType _type )
@@ -239,6 +237,10 @@ namespace comphelper
             }
         }
 
+        // noncopyable
+        MapEnumerator(const MapEnumerator&) = delete;
+        const MapEnumerator& operator=(const MapEnumerator&) = delete;
+
         // XEnumeration equivalents
         bool hasMoreElements();
         Any nextElement();
diff --git a/comphelper/source/misc/servicedecl.cxx b/comphelper/source/misc/servicedecl.cxx
index 78bd811..7744d3c 100644
--- a/comphelper/source/misc/servicedecl.cxx
+++ b/comphelper/source/misc/servicedecl.cxx
@@ -27,7 +27,6 @@
 #include <com/sun/star/lang/XSingleComponentFactory.hpp>
 #include <cassert>
 #include <vector>
-#include <boost/noncopyable.hpp>
 
 using namespace com::sun::star;
 
@@ -36,12 +35,14 @@ namespace service_decl {
 
 class ServiceDecl::Factory :
         public cppu::WeakImplHelper<lang::XSingleComponentFactory,
-                                     lang::XServiceInfo>,
-        private boost::noncopyable
+                                     lang::XServiceInfo>
 {
 public:
     explicit Factory( ServiceDecl const& rServiceDecl )
         : m_rServiceDecl(rServiceDecl) {}
+    // noncopyable
+    Factory(const Factory&) = delete;
+    const Factory& operator=(const Factory&) = delete;
 
     // XServiceInfo:
     virtual OUString SAL_CALL getImplementationName()
diff --git a/comphelper/source/property/opropertybag.hxx b/comphelper/source/property/opropertybag.hxx
index e2246f4..214a6b8 100644
--- a/comphelper/source/property/opropertybag.hxx
+++ b/comphelper/source/property/opropertybag.hxx
@@ -28,7 +28,6 @@
 #include <com/sun/star/uno/XComponentContext.hpp>
 #include <com/sun/star/container/XSet.hpp>
 
-#include <boost/noncopyable.hpp>
 #include <cppuhelper/implbase5.hxx>
 #include <comphelper/interfacecontainer2.hxx>
 #include <comphelper/propstate.hxx>
@@ -71,7 +70,6 @@ namespace comphelper
                         ,public OPropertyBag_PBase
                         ,public OPropertyBag_Base
                         ,public ::cppu::IEventNotificationHook
-                        ,private boost::noncopyable
     {
     private:
         /// our IPropertyArrayHelper implementation
@@ -90,6 +88,10 @@ namespace comphelper
         bool            m_isModified;
 
     public:
+        //noncopyable
+        OPropertyBag(const OPropertyBag&) = delete;
+        const OPropertyBag& operator=(const OPropertyBag&) = delete;
+
         // XServiceInfo - static versions
         static css::uno::Sequence< OUString > getSupportedServiceNames_static() throw( css::uno::RuntimeException );
         static OUString getImplementationName_static() throw( css::uno::RuntimeException );
diff --git a/comphelper/source/streaming/seqinputstreamserv.cxx b/comphelper/source/streaming/seqinputstreamserv.cxx
index 9cd3d62..6cf44bc 100644
--- a/comphelper/source/streaming/seqinputstreamserv.cxx
+++ b/comphelper/source/streaming/seqinputstreamserv.cxx
@@ -22,7 +22,6 @@
 #include "comphelper_module.hxx"
 #include "comphelper_services.hxx"
 
-#include <boost/noncopyable.hpp>
 #include <osl/mutex.hxx>
 #include <cppuhelper/factory.hxx>
 #include <cppuhelper/implementationentry.hxx>
@@ -44,12 +43,15 @@ class SequenceInputStreamService:
     public ::cppu::WeakImplHelper<
         lang::XServiceInfo,
         io::XSeekableInputStream,
-        lang::XInitialization>,
-    private boost::noncopyable
+        lang::XInitialization>
 {
 public:
     explicit SequenceInputStreamService();
 
+    // noncopyable
+    SequenceInputStreamService(const SequenceInputStreamService&) = delete;
+    const SequenceInputStreamService& operator=(const SequenceInputStreamService&) = delete;
+
     // css::lang::XServiceInfo:
     virtual OUString SAL_CALL getImplementationName() throw ( uno::RuntimeException, std::exception ) override;
     virtual sal_Bool SAL_CALL supportsService( const OUString & ServiceName ) throw ( uno::RuntimeException, std::exception ) override;
diff --git a/comphelper/source/streaming/seqoutputstreamserv.cxx b/comphelper/source/streaming/seqoutputstreamserv.cxx
index cacc29a..58b3235 100644
--- a/comphelper/source/streaming/seqoutputstreamserv.cxx
+++ b/comphelper/source/streaming/seqoutputstreamserv.cxx
@@ -22,7 +22,6 @@
 #include "comphelper_module.hxx"
 #include "comphelper_services.hxx"
 
-#include <boost/noncopyable.hpp>
 #include <osl/mutex.hxx>
 #include <cppuhelper/factory.hxx>
 #include <cppuhelper/implementationentry.hxx>
@@ -39,12 +38,15 @@ using namespace ::com::sun::star;
 namespace {
 
 class SequenceOutputStreamService:
-    public cppu::WeakImplHelper<lang::XServiceInfo, io::XSequenceOutputStream>,
-    private boost::noncopyable
+    public cppu::WeakImplHelper<lang::XServiceInfo, io::XSequenceOutputStream>
 {
 public:
     explicit SequenceOutputStreamService();
 
+    // noncopyable
+    SequenceOutputStreamService(const SequenceOutputStreamService&) = delete;
+    const SequenceOutputStreamService& operator=(const SequenceOutputStreamService&) = delete;
+
     // css::lang::XServiceInfo:
     virtual OUString SAL_CALL getImplementationName() throw ( uno::RuntimeException, std::exception ) override;
     virtual sal_Bool SAL_CALL supportsService( const OUString & ServiceName ) throw ( uno::RuntimeException, std::exception ) override;
diff --git a/connectivity/source/commontools/TTableHelper.cxx b/connectivity/source/commontools/TTableHelper.cxx
index 875af21..63a1c6d 100644
--- a/connectivity/source/commontools/TTableHelper.cxx
+++ b/connectivity/source/commontools/TTableHelper.cxx
@@ -19,7 +19,6 @@
 
 #include <sal/config.h>
 
-#include <boost/noncopyable.hpp>
 #include <connectivity/TTableHelper.hxx>
 #include <com/sun/star/sdbc/XRow.hpp>
 #include <com/sun/star/sdbc/XResultSet.hpp>
@@ -55,8 +54,7 @@ namespace
 {
     /// helper class for column property change events which holds the OComponentDefinition weak
 class OTableContainerListener:
-    public ::cppu::WeakImplHelper< XContainerListener >,
-    private boost::noncopyable
+    public ::cppu::WeakImplHelper< XContainerListener >
 {
     OTableHelper* m_pComponent;
     ::std::map< OUString,bool> m_aRefNames;
@@ -65,6 +63,9 @@ protected:
     virtual ~OTableContainerListener(){}
 public:
     explicit OTableContainerListener(OTableHelper* _pComponent) : m_pComponent(_pComponent){}
+    // noncopyable
+    OTableContainerListener(const OTableContainerListener&) = delete;
+    const OTableContainerListener& operator=(const OTableContainerListener&) = delete;
     virtual void SAL_CALL elementInserted( const ::com::sun::star::container::ContainerEvent& /*Event*/ ) throw (RuntimeException, std::exception) override
     {
     }
diff --git a/cppcanvas/source/mtfrenderer/bitmapaction.cxx b/cppcanvas/source/mtfrenderer/bitmapaction.cxx
index 941690c..2777abb 100644
--- a/cppcanvas/source/mtfrenderer/bitmapaction.cxx
+++ b/cppcanvas/source/mtfrenderer/bitmapaction.cxx
@@ -30,7 +30,6 @@
 #include <basegfx/point/b2dpoint.hxx>
 #include <basegfx/range/b2drange.hxx>
 #include <basegfx/tools/canvastools.hxx>
-#include <boost/noncopyable.hpp>
 #include "cachedprimitivebase.hxx"
 #include "bitmapaction.hxx"
 #include "outdevstate.hxx"
diff --git a/cppcanvas/source/mtfrenderer/cachedprimitivebase.hxx b/cppcanvas/source/mtfrenderer/cachedprimitivebase.hxx
index fb69778..5fc7e0a 100644
--- a/cppcanvas/source/mtfrenderer/cachedprimitivebase.hxx
+++ b/cppcanvas/source/mtfrenderer/cachedprimitivebase.hxx
@@ -24,7 +24,6 @@
 #include <com/sun/star/rendering/XCanvas.hpp>
 
 #include <cppcanvas/canvas.hxx>
-#include <boost/noncopyable.hpp>
 
 #include "action.hxx"
 
@@ -46,8 +45,7 @@ namespace cppcanvas
             class' public render() method gets called, the cached
             representation is taken.
          */
-        class CachedPrimitiveBase : public Action,
-                                    private ::boost::noncopyable
+        class CachedPrimitiveBase : public Action
         {
         public:
             /** Constructor
@@ -65,6 +63,9 @@ namespace cppcanvas
                                  bool                   bOnlyRedrawWithSameTransform );
             virtual ~CachedPrimitiveBase() {}
 
+            CachedPrimitiveBase(const CachedPrimitiveBase&) = delete;
+            const CachedPrimitiveBase& operator=(const CachedPrimitiveBase&) = delete;
+
             virtual bool render( const ::basegfx::B2DHomMatrix& rTransformation ) const override;
 
         protected:
diff --git a/cppcanvas/source/mtfrenderer/lineaction.cxx b/cppcanvas/source/mtfrenderer/lineaction.cxx
index abda71d..5d9acd2 100644
--- a/cppcanvas/source/mtfrenderer/lineaction.cxx
+++ b/cppcanvas/source/mtfrenderer/lineaction.cxx
@@ -31,8 +31,6 @@
 #include <basegfx/tools/canvastools.hxx>
 #include <canvas/canvastools.hxx>
 
-#include <boost/noncopyable.hpp>
-
 #include <cppcanvas/canvas.hxx>
 
 #include <mtftools.hxx>
@@ -46,7 +44,7 @@ namespace cppcanvas
     {
         namespace
         {
-            class LineAction : public Action, private ::boost::noncopyable
+            class LineAction : public Action
             {
             public:
                 LineAction( const ::basegfx::B2DPoint&,
@@ -54,6 +52,9 @@ namespace cppcanvas
                             const CanvasSharedPtr&,
                             const OutDevState& );
 
+                LineAction(const LineAction&) = delete;
+                const LineAction& operator=(const LineAction&) = delete;
+
                 virtual bool render( const ::basegfx::B2DHomMatrix& rTransformation ) const override;
                 virtual bool renderSubset( const ::basegfx::B2DHomMatrix& rTransformation,
                                            const Subset&                  rSubset ) const override;
diff --git a/cppcanvas/source/mtfrenderer/pointaction.cxx b/cppcanvas/source/mtfrenderer/pointaction.cxx
index b8e3788..65cd910 100644
--- a/cppcanvas/source/mtfrenderer/pointaction.cxx
+++ b/cppcanvas/source/mtfrenderer/pointaction.cxx
@@ -28,8 +28,6 @@
 #include <basegfx/tools/canvastools.hxx>
 #include <canvas/canvastools.hxx>
 
-#include <boost/noncopyable.hpp>
-
 #include "pointaction.hxx"
 #include "outdevstate.hxx"
 #include "cppcanvas/canvas.hxx"
@@ -44,7 +42,7 @@ namespace cppcanvas
     {
         namespace
         {
-            class PointAction : public Action, private ::boost::noncopyable
+            class PointAction : public Action
             {
             public:
                 PointAction( const ::basegfx::B2DPoint&,
@@ -55,6 +53,9 @@ namespace cppcanvas
                              const OutDevState&,
                              const ::Color&     );
 
+                PointAction(const PointAction&) = delete;
+                const PointAction& operator=(const PointAction&) = delete;
+
                 virtual bool render( const ::basegfx::B2DHomMatrix& rTransformation ) const override;
                 virtual bool renderSubset( const ::basegfx::B2DHomMatrix& rTransformation,
                                            const Subset&                  rSubset ) const override;
diff --git a/cppcanvas/source/mtfrenderer/polypolyaction.cxx b/cppcanvas/source/mtfrenderer/polypolyaction.cxx
index 5c94513..e5363a6 100644
--- a/cppcanvas/source/mtfrenderer/polypolyaction.cxx
+++ b/cppcanvas/source/mtfrenderer/polypolyaction.cxx
@@ -31,8 +31,6 @@
 #include <basegfx/matrix/b2dhommatrix.hxx>
 #include <canvas/canvastools.hxx>
 
-#include <boost/noncopyable.hpp>
-
 #include "cachedprimitivebase.hxx"
 #include "polypolyaction.hxx"
 #include "outdevstate.hxx"
diff --git a/cppcanvas/source/mtfrenderer/textaction.cxx b/cppcanvas/source/mtfrenderer/textaction.cxx
index b29e7de..14588d6 100644
--- a/cppcanvas/source/mtfrenderer/textaction.cxx
+++ b/cppcanvas/source/mtfrenderer/textaction.cxx
@@ -41,7 +41,6 @@
 #include <canvas/canvastools.hxx>
 
 #include <memory>
-#include <boost/noncopyable.hpp>
 
 #include "textaction.hxx"
 #include "outdevstate.hxx"
@@ -606,7 +605,7 @@ namespace cppcanvas
             }
 
 
-            class TextAction : public Action, private ::boost::noncopyable
+            class TextAction : public Action
             {
             public:
                 TextAction( const ::basegfx::B2DPoint&  rStartPoint,
@@ -624,6 +623,9 @@ namespace cppcanvas
                             const OutDevState&              rState,
                             const ::basegfx::B2DHomMatrix&  rTextTransform );
 
+                TextAction(const TextAction&) = delete;
+                const TextAction& operator=(const TextAction&) = delete;
+
                 virtual bool render( const ::basegfx::B2DHomMatrix& rTransformation ) const override;
                 virtual bool renderSubset( const ::basegfx::B2DHomMatrix& rTransformation,
                                            const Subset&                  rSubset ) const override;
@@ -759,8 +761,7 @@ namespace cppcanvas
 
             class EffectTextAction :
                 public Action,
-                public TextRenderer,
-                private ::boost::noncopyable
+                public TextRenderer
             {
             public:
                 EffectTextAction( const ::basegfx::B2DPoint& rStartPoint,
@@ -788,6 +789,9 @@ namespace cppcanvas
                                   const OutDevState&                rState,
                                   const ::basegfx::B2DHomMatrix&    rTextTransform );
 
+                EffectTextAction(const EffectTextAction&) = delete;
+                const EffectTextAction& operator=(const EffectTextAction&) = delete;
+
                 virtual bool render( const ::basegfx::B2DHomMatrix& rTransformation ) const override;
                 virtual bool renderSubset( const ::basegfx::B2DHomMatrix& rTransformation,
                                            const Subset&                  rSubset ) const override;
@@ -996,7 +1000,7 @@ namespace cppcanvas
             }
 
 
-            class TextArrayAction : public Action, private ::boost::noncopyable
+            class TextArrayAction : public Action
             {
             public:
                 TextArrayAction( const ::basegfx::B2DPoint&     rStartPoint,
@@ -1016,6 +1020,9 @@ namespace cppcanvas
                                  const OutDevState&             rState,
                                  const ::basegfx::B2DHomMatrix& rTextTransform );
 
+                TextArrayAction(const TextArrayAction&) = delete;
+                const TextArrayAction& operator=(const TextArrayAction&) = delete;
+
                 virtual bool render( const ::basegfx::B2DHomMatrix& rTransformation ) const override;
                 virtual bool renderSubset( const ::basegfx::B2DHomMatrix& rTransformation,
                                            const Subset&                  rSubset ) const override;
@@ -1174,8 +1181,7 @@ namespace cppcanvas
 
             class EffectTextArrayAction :
                 public Action,
-                public TextRenderer,
-                private ::boost::noncopyable
+                public TextRenderer
             {
             public:
                 EffectTextArrayAction( const ::basegfx::B2DPoint&       rStartPoint,
@@ -1204,6 +1210,9 @@ namespace cppcanvas
                                        const OutDevState&               rState,
                                        const ::basegfx::B2DHomMatrix&   rTextTransform );
 
+                EffectTextArrayAction(const EffectTextArrayAction&) = delete;
+                const EffectTextArrayAction& operator=(const EffectTextArrayAction&);
+
                 virtual bool render( const ::basegfx::B2DHomMatrix& rTransformation ) const override;
                 virtual bool renderSubset( const ::basegfx::B2DHomMatrix& rTransformation,
                                            const Subset&                  rSubset ) const override;
@@ -1510,8 +1519,7 @@ namespace cppcanvas
 
             class OutlineAction :
                 public Action,
-                public TextRenderer,
-                private ::boost::noncopyable
+                public TextRenderer
             {
             public:
                 OutlineAction( const ::basegfx::B2DPoint&                           rStartPoint,
@@ -1538,6 +1546,9 @@ namespace cppcanvas
                                const OutDevState&                                   rState,
                                const ::basegfx::B2DHomMatrix&                       rTextTransform );
 
+                OutlineAction(const OutlineAction&) = delete;
+                const OutlineAction& operator=(const OutlineAction&);
+
                 virtual bool render( const ::basegfx::B2DHomMatrix& rTransformation ) const override;
                 virtual bool renderSubset( const ::basegfx::B2DHomMatrix& rTransformation,
                                            const Subset&                  rSubset ) const override;
diff --git a/cppcanvas/source/mtfrenderer/transparencygroupaction.cxx b/cppcanvas/source/mtfrenderer/transparencygroupaction.cxx
index a8f61f2..474d22b 100644
--- a/cppcanvas/source/mtfrenderer/transparencygroupaction.cxx
+++ b/cppcanvas/source/mtfrenderer/transparencygroupaction.cxx
@@ -48,8 +48,6 @@
 #include <basegfx/tools/canvastools.hxx>
 #include <basegfx/matrix/b2dhommatrixtools.hxx>
 
-#include <boost/noncopyable.hpp>
-
 #include "transparencygroupaction.hxx"
 #include "outdevstate.hxx"
 #include "mtftools.hxx"
@@ -66,7 +64,7 @@ namespace cppcanvas
         // ======================
         namespace
         {
-            class TransparencyGroupAction : public Action, private ::boost::noncopyable
+            class TransparencyGroupAction : public Action
             {
             public:
                 /** Create new transparency group action.
@@ -94,6 +92,9 @@ namespace cppcanvas
                                          const CanvasSharedPtr&         rCanvas,
                                          const OutDevState&             rState );
 
+                TransparencyGroupAction(const TransparencyGroupAction&) = delete;
+                const TransparencyGroupAction& operator=(const TransparencyGroupAction&) = delete;
+
                 virtual bool render( const ::basegfx::B2DHomMatrix& rTransformation ) const override;
                 virtual bool renderSubset( const ::basegfx::B2DHomMatrix& rTransformation,
                                            const Subset&                  rSubset ) const override;


More information about the Libreoffice-commits mailing list