[Libreoffice-commits] core.git: external/skia

Stephan Bergmann (via logerrit) logerrit at kemper.freedesktop.org
Wed Dec 4 17:40:32 UTC 2019


 external/skia/UnpackedTarball_skia.mk       |    1 +
 external/skia/Wdeprecated-copy-dtor.patch.0 |   11 +++++++++++
 2 files changed, 12 insertions(+)

New commits:
commit 2914bde7e1d4a588b597e91b59b3870c26c1b3a4
Author:     Stephan Bergmann <sbergman at redhat.com>
AuthorDate: Wed Dec 4 17:06:00 2019 +0100
Commit:     Stephan Bergmann <sbergman at redhat.com>
CommitDate: Wed Dec 4 18:39:36 2019 +0100

    external/skia: Fix -Werror=deprecated-copy-dtor
    
    ...happening when LO code includes skia files:
    
    > vcl/skia/gdiimpl.cxx: In member function ‘void SkiaSalGraphicsImpl::createOffscreenSurface()’:
    > vcl/skia/gdiimpl.cxx:246:83: error: implicitly-declared ‘sk_app::VulkanWindowContext::SharedGrContext& sk_app::VulkanWindowContext::SharedGrContext::operator=(const sk_app::VulkanWindowContext::SharedGrContext&)’ is deprecated [-Werror=deprecated-copy-dtor]
    >   246 |             mOffscreenGrContext = sk_app::VulkanWindowContext::getSharedGrContext();
    >       |                                                                                   ^
    > In file included from vcl/inc/skia/gdiimpl.hxx:29,
    >                  from vcl/skia/gdiimpl.cxx:20:
    > workdir/UnpackedTarball/skia/tools/sk_app/VulkanWindowContext.h:35:9: note: because ‘sk_app::VulkanWindowContext::SharedGrContext’ has user-provided ‘sk_app::VulkanWindowContext::SharedGrContext::~SharedGrContext()’
    >    35 |         ~SharedGrContext() { shared.reset(); checkDestroyShared(); }
    >       |         ^
    
    See e9e4eb0736d5582fa37dcad20bf5826c50029249 "Fix some
    -Werror=deprecated-copy-dtor" for details about -Wdeprecated-copy-dtor.
    
    Change-Id: Ic393acff5c4d8e3eaa78b4dfed51626f73e95ed4
    Reviewed-on: https://gerrit.libreoffice.org/84425
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sbergman at redhat.com>

diff --git a/external/skia/UnpackedTarball_skia.mk b/external/skia/UnpackedTarball_skia.mk
index acbd68d87c9e..84178749caed 100644
--- a/external/skia/UnpackedTarball_skia.mk
+++ b/external/skia/UnpackedTarball_skia.mk
@@ -24,6 +24,7 @@ skia_patches := \
     share-grcontext.patch.1 \
     c++20-comparison.patch.0 \
     Wdeprecated-copy.patch.0 \
+    Wdeprecated-copy-dtor.patch.0 \
 
 $(eval $(call gb_UnpackedTarball_set_patchlevel,skia,1))
 
diff --git a/external/skia/Wdeprecated-copy-dtor.patch.0 b/external/skia/Wdeprecated-copy-dtor.patch.0
new file mode 100644
index 000000000000..335ad9b84b94
--- /dev/null
+++ b/external/skia/Wdeprecated-copy-dtor.patch.0
@@ -0,0 +1,11 @@
+--- tools/sk_app/VulkanWindowContext.h
++++ tools/sk_app/VulkanWindowContext.h
+@@ -33,6 +33,8 @@
+         SharedGrContext() {}
+         GrContext* getGrContext() { return shared ? shared->fContext.get() : nullptr; }
+         ~SharedGrContext() { shared.reset(); checkDestroyShared(); }
++        SharedGrContext(SharedGrContext const &) = default;
++        SharedGrContext & operator =(SharedGrContext const &) = default;
+         bool operator!() const { return !shared; }
+         void reset() { shared.reset(); }
+     private:


More information about the Libreoffice-commits mailing list