oox: avoid -Werror=deprecated-copy (GCC trunk towards GCC 9)

...by explicitly defaulting the copy/move functions (and, where needed in turn,
also a default ctor) for classes that have a user-declared dtor that does
nothing other than an implicitly-defined one would do, but needs to be user-
declared because it is virtual and potentially serves as a key function to
emit the vtable, or is non-public, etc.; and by removing explicitly user-
provided functions that do the same as their implicitly-defined counterparts,
but may prevent implicitly declared copy functions from being defined as non-
deleted in the future.  (Even if such a user-provided function was declared
non-inline in an include file, the apparently-used implicitly-defined copy
functions are already include, so why bother with non-inline functions.)

Change-Id: Id05e85ac8ac4155e6345b6f96ddf9449f640dc98
Reviewed-on: https://gerrit.libreoffice.org/58083
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>

8 files changed, 5 insertions, 20 deletions

diff --git a/oox/inc/drawingml/chart/converterbase.hxx b/oox/inc/drawingml/chart/converterbase.hxx
index 11e14ebaf596..afd0f097f63c 100644
--- a/oox/inc/drawingml/chart/converterbase.hxx
+++ b/oox/inc/drawingml/chart/converterbase.hxx
@@ -64,6 +64,11 @@ public:
                             const css::awt::Size& rChartSize );
     virtual             ~ConverterRoot();
 
+    ConverterRoot(ConverterRoot const &) = default;
+    ConverterRoot(ConverterRoot &&) = default;
+    ConverterRoot & operator =(ConverterRoot const &) = default;
+    ConverterRoot & operator =(ConverterRoot &&) = default;
+
     /** Creates an instance for the passed service name, using the process service factory. */
     css::uno::Reference< css::uno::XInterface >
                         createInstance( const OUString& rServiceName ) const;
diff --git a/oox/inc/drawingml/table/tableproperties.hxx b/oox/inc/drawingml/table/tableproperties.hxx
index 578230daa876..0cd500279974 100644
--- a/oox/inc/drawingml/table/tableproperties.hxx
+++ b/oox/inc/drawingml/table/tableproperties.hxx
@@ -36,7 +36,6 @@ class TableProperties
 public:
 
     TableProperties();
-    ~TableProperties();
 
     std::vector< sal_Int32 >&           getTableGrid() { return mvTableGrid; };
     std::vector< TableRow >&            getTableRows() { return mvTableRows; };
diff --git a/oox/inc/drawingml/table/tablestylepart.hxx b/oox/inc/drawingml/table/tablestylepart.hxx
index 43759ae96672..2a03339962ca 100644
--- a/oox/inc/drawingml/table/tablestylepart.hxx
+++ b/oox/inc/drawingml/table/tablestylepart.hxx
@@ -38,7 +38,6 @@ class TableStylePart
 public:
 
     TableStylePart();
-    ~TableStylePart();
 
     ::oox::drawingml::Color&            getTextColor(){ return maTextColor; }
     ::boost::optional< bool >&          getTextBoldStyle(){ return maTextBoldStyle; }
diff --git a/oox/inc/drawingml/textparagraphproperties.hxx b/oox/inc/drawingml/textparagraphproperties.hxx
index 2710dcb484b2..5b8093bab391 100644
--- a/oox/inc/drawingml/textparagraphproperties.hxx
+++ b/oox/inc/drawingml/textparagraphproperties.hxx
@@ -80,7 +80,6 @@ class TextParagraphProperties
 public:
 
     TextParagraphProperties();
-    ~TextParagraphProperties();
 
     void                                setLevel( sal_Int16 nLevel ) { mnLevel = nLevel; }
     sal_Int16                           getLevel( ) const { return mnLevel; }
diff --git a/oox/source/drawingml/table/tableproperties.cxx b/oox/source/drawingml/table/tableproperties.cxx
index 2c92390772e8..3be685a954d3 100644
--- a/oox/source/drawingml/table/tableproperties.cxx
+++ b/oox/source/drawingml/table/tableproperties.cxx
@@ -48,9 +48,6 @@ TableProperties::TableProperties()
 , mbBandCol( false )
 {
 }
-TableProperties::~TableProperties()
-{
-}
 
 void CreateTableRows( const uno::Reference< XTableRows >& xTableRows, const std::vector< TableRow >& rvTableRows )
 {
diff --git a/oox/source/drawingml/table/tablestylepart.cxx b/oox/source/drawingml/table/tablestylepart.cxx
index 7e3e9eb0702c..322021e1e6ef 100644
--- a/oox/source/drawingml/table/tablestylepart.cxx
+++ b/oox/source/drawingml/table/tablestylepart.cxx
@@ -31,10 +31,6 @@ TableStylePart::TableStylePart()
 {
 }
 
-TableStylePart::~TableStylePart()
-{
-}
-
 } } }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/oox/source/drawingml/textparagraphproperties.cxx b/oox/source/drawingml/textparagraphproperties.cxx
index 8731047cd4b6..c4ad9e7493c5 100644
--- a/oox/source/drawingml/textparagraphproperties.cxx
+++ b/oox/source/drawingml/textparagraphproperties.cxx
@@ -366,10 +366,6 @@ TextParagraphProperties::TextParagraphProperties()
 {
 }
 
-TextParagraphProperties::~TextParagraphProperties()
-{
-}
-
 void TextParagraphProperties::apply( const TextParagraphProperties& rSourceProps )
 {
     maTextParagraphPropertyMap.assignAll( rSourceProps.maTextParagraphPropertyMap );
diff --git a/oox/source/dump/dumperbase.cxx b/oox/source/dump/dumperbase.cxx
index e68db52a2015..b38e5f54fbe9 100644
--- a/oox/source/dump/dumperbase.cxx
+++ b/oox/source/dump/dumperbase.cxx
@@ -1385,12 +1385,6 @@ void SharedConfigData::createUnitConverter( const OUString& rData )
     }
 }
 
-Config::Config( const Config& rParent ) :
-    Base()  // c'tor needs to be called explicitly to avoid compiler warning
-{
-    *this = rParent;
-}
-
 Config::Config( const sal_Char* pcEnvVar, const FilterBase& rFilter )
 {
     construct( pcEnvVar, rFilter );