[Libreoffice-commits] core.git: Branch 'distro/collabora/co-2021' - include/oox

[Mike Kaganski (via logerrit)]

 include/oox/helper/graphichelper.hxx |    8 ++++++++
 1 file changed, 8 insertions(+)

New commits:
commit a062fb0447dc461d2b2702aa08a769dc638e37d9
Author:     Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Mon Jun 21 13:50:48 2021 +0300
Commit:     Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Fri Aug 27 07:14:47 2021 +0200

    Delete copy ctors/assignments for DLLEXPORT class
    
    Their default definitions may create/destroy temporary OutputDevice objects,
    which requires that the class is defined at the site of instantiation of the
    implicit definition.
    
    The solution is to delete these unused methods. If they would happen to be
    needed, then move the default definitions to the correct place.
    
    This reverts commit 30a17b05fb8a860c1e53cb7cfc405b1cea0a9d15.
    
    Change-Id: I43f4e76763ff4409e9527f44cd2d8e227982257c
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117574
    Tested-by: Jenkins
    Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/121078
    Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice at gmail.com>

diff --git a/include/oox/helper/graphichelper.hxx b/include/oox/helper/graphichelper.hxx
index 6e6a255d9bd6..fac83230a6d7 100644
--- a/include/oox/helper/graphichelper.hxx
+++ b/include/oox/helper/graphichelper.hxx
@@ -35,6 +35,7 @@
 #include <vcl/vclptr.hxx>
 
 struct WmfExternal;
+class OutputDevice;
 
 namespace com::sun::star {
     namespace awt { struct Point; }
@@ -64,6 +65,13 @@ public:
                             const StorageRef& rxStorage );
     virtual             ~GraphicHelper();
 
+    // Avoid implicitly defined copy constructors/assignments for the DLLPUBLIC class (they may
+    // require forward-declared classes used internally to be defined in places using GraphicHelper)
+    GraphicHelper(const GraphicHelper&) = delete;
+    GraphicHelper(GraphicHelper&&) = delete;
+    GraphicHelper& operator=(const GraphicHelper&) = delete;
+    GraphicHelper& operator=(GraphicHelper&&) = delete;
+
     // System colors and predefined colors ------------------------------------
 
     /** Returns a system color specified by the passed XML token identifier. */