clang-tidy modernize-use-equals-default in svl

SfxPoolItem has a copy constructor and its copy assignment
operator is deleted. Derived classes have a implicit defined
copy constructor too, if all members are copy constructible.

This patch removes default able copy constructors on such items.

Note: clang-tidy only finds functions defined in source, inlined
functions in headers are picked manualy.

Change-Id: I7e642a2ecf139420c53f6fcb39208918dd1a8d52
Reviewed-on: https://gerrit.libreoffice.org/43003
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>

1 files changed, 1 insertions, 9 deletions

diff --git a/include/svl/eitem.hxx b/include/svl/eitem.hxx
index cbfc1a4930a0..eb488f65b251 100644
--- a/include/svl/eitem.hxx
+++ b/include/svl/eitem.hxx
@@ -36,10 +36,7 @@ protected:
         , m_nValue(nValue)
     { }
 
-    SfxEnumItem(const SfxEnumItem & rItem)
-        : SfxEnumItemInterface(rItem)
-        , m_nValue(rItem.m_nValue)
-    { }
+    SfxEnumItem(const SfxEnumItem &) = default;
 
     SfxEnumItem(sal_uInt16 const nWhich, SvStream & rStream)
         : SfxEnumItemInterface(nWhich)
@@ -95,11 +92,6 @@ public:
         , m_bValue(bValue)
     { }
 
-    SfxBoolItem(SfxBoolItem const& rItem)
-        : SfxPoolItem(rItem)
-        , m_bValue(rItem.m_bValue)
-    { }
-
     SfxBoolItem(sal_uInt16 nWhich, SvStream & rStream);
 
     bool GetValue() const { return m_bValue; }