summaryrefslogtreecommitdiff
path: root/lotuswordpro
diff options
context:
space:
mode:
authorTamás Zolnai <tamas.zolnai@collabora.com>2019-04-09 16:43:39 +0200
committerTamás Zolnai <tamas.zolnai@collabora.com>2019-04-10 12:18:24 +0200
commitadfba503c792fdbd4748d6680c2dd8d8d5bb0d69 (patch)
tree88fe08b0bd11b07641022f8aae90b3a3106021e8 /lotuswordpro
parent835ddd98d72617c2ed6ae6950f1fec0cc21b3ba2 (diff)
clang-tidy: Silence warnings from WIP unhandled-self-assignment check
Where it can be done by removing useless / duplicate code. For XFListStyle I removed the copy operator entirely, because it was bugous and it seems not to be used anyway. Change-Id: Iba0eb0d5c45b42f0e78be466c617acdc1216eb22 Reviewed-on: https://gerrit.libreoffice.org/70482 Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com> Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Diffstat (limited to 'lotuswordpro')
-rw-r--r--lotuswordpro/inc/xfilter/xfliststyle.hxx2
-rw-r--r--lotuswordpro/source/filter/xfilter/xfliststyle.cxx21
2 files changed, 1 insertions, 22 deletions
diff --git a/lotuswordpro/inc/xfilter/xfliststyle.hxx b/lotuswordpro/inc/xfilter/xfliststyle.hxx
index f4abb93a5b10..3cde9644a612 100644
--- a/lotuswordpro/inc/xfilter/xfliststyle.hxx
+++ b/lotuswordpro/inc/xfilter/xfliststyle.hxx
@@ -190,7 +190,7 @@ public:
XFListStyle(const XFListStyle& other);
- XFListStyle& operator=(const XFListStyle& other);
+ XFListStyle& operator=(const XFListStyle& other) = delete;
virtual ~XFListStyle() override;
diff --git a/lotuswordpro/source/filter/xfilter/xfliststyle.cxx b/lotuswordpro/source/filter/xfilter/xfliststyle.cxx
index 85db3c77a973..5205020784d4 100644
--- a/lotuswordpro/source/filter/xfilter/xfliststyle.cxx
+++ b/lotuswordpro/source/filter/xfilter/xfliststyle.cxx
@@ -185,27 +185,6 @@ XFListStyle::XFListStyle(const XFListStyle& other):XFStyle(other)
}
}
-XFListStyle& XFListStyle::operator=(const XFListStyle& other)
-{
- for( int i=0; i<10; i++ )
- {
- const enumXFListLevel type = other.m_pListLevels[i]->m_eListType;
- if( type == enumXFListLevelNumber )
- {
- XFListlevelNumber *pNum = static_cast<XFListlevelNumber*>(m_pListLevels[i].get());
- m_pListLevels[i].reset(new XFListlevelNumber(*pNum));
- }
- else if( type == enumXFListLevelBullet )
- {
- XFListLevelBullet *pBullet = static_cast<XFListLevelBullet*>(m_pListLevels[i].get());
- m_pListLevels[i].reset(new XFListLevelBullet(*pBullet));
- }
- else
- m_pListLevels[i].reset();
- }
- return *this;
-}
-
XFListStyle::~XFListStyle()
{
}