[Libreoffice-commits] core.git: 2 commits - external/skia include/vcl
Mike Kaganski (via logerrit)
logerrit at kemper.freedesktop.org
Fri Aug 7 08:07:22 UTC 2020
external/skia/UnpackedTarball_skia.mk | 1 +
external/skia/missing-include.patch.1 | 11 +++++++++++
include/vcl/mnemonic.hxx | 2 +-
3 files changed, 13 insertions(+), 1 deletion(-)
New commits:
commit 071c044352d24bb0c8cad0d239c8b1fc644c1823
Author: Mike Kaganski <mike.kaganski at collabora.com>
AuthorDate: Thu Aug 6 22:49:41 2020 +0300
Commit: Mike Kaganski <mike.kaganski at collabora.com>
CommitDate: Fri Aug 7 10:06:33 2020 +0200
Fix Skia build using VS 2019 v.16.7.0 with --disable-pch
No idea if updated VS makes a difference, or some recent Skia update...
In file included from C:/lo/src/build/workdir/UnpackedTarball/skia/src/core/SkBitmapDevice.cpp:11:
In file included from C:/lo/src/build/workdir/UnpackedTarball/skia\include/core/SkPath.h:13:
C:/lo/src/build/workdir/UnpackedTarball/skia\include/private/SkPathRef.h(113,41): error: implicit instantiation of undefined template
'std::tuple<SkPoint *, float *>'
std::tuple<SkPoint*, SkScalar*> growForVerbsInPath(const SkPathRef& path) {
^
C:/PROGRA~2/MICROS~1/2019/COMMUN~1/VC/Tools/MSVC/1427~1.291/Include\utility(126,7): note: template is declared here
class tuple;
^
In file included from C:/lo/src/build/workdir/UnpackedTarball/skia/src/core/SkBitmapDevice.cpp:11:
In file included from C:/lo/src/build/workdir/UnpackedTarball/skia\include/core/SkPath.h:13:
C:/lo/src/build/workdir/UnpackedTarball/skia\include/private/SkPathRef.h(114,30): error: implicit instantiation of undefined template
'std::tuple<SkPoint *, float *>'
return fPathRef->growForVerbsInPath(path);
^
C:/PROGRA~2/MICROS~1/2019/COMMUN~1/VC/Tools/MSVC/1427~1.291/Include\utility(126,7): note: template is declared here
class tuple;
^
In file included from C:/lo/src/build/workdir/UnpackedTarball/skia/src/core/SkBitmapDevice.cpp:11:
C:/lo/src/build/workdir/UnpackedTarball/skia\include/core/SkPath.h(1543,65): error: implicit instantiation of undefined template 'std::tuple<SkPathVerb, const
SkPoint *, const float *>'
std::tuple<SkPathVerb, const SkPoint*, const SkScalar*> operator*() const {
^
C:/PROGRA~2/MICROS~1/2019/COMMUN~1/VC/Tools/MSVC/1427~1.291/Include\utility(126,7): note: template is declared here
class tuple;
^
In file included from C:/lo/src/build/workdir/UnpackedTarball/skia/src/core/SkBitmapDevice.cpp:11:
C:/lo/src/build/workdir/UnpackedTarball/skia\include/core/SkPath.h(1550,20): error: implicit instantiation of undefined template 'std::tuple<SkPathVerb, const
SkPoint *, const float *>'
return {verb, fPoints + backset, fWeights};
^
C:/PROGRA~2/MICROS~1/2019/COMMUN~1/VC/Tools/MSVC/1427~1.291/Include\utility(126,7): note: template is declared here
class tuple;
^
In file included from C:/lo/src/build/workdir/UnpackedTarball/skia/src/core/SkBitmapDevice.cpp:11:
C:/lo/src/build/workdir/UnpackedTarball/skia\include/core/SkPath.h(1625,68): error: implicit instantiation of undefined template 'std::tuple<SkPathVerb, const
SkPoint *, const float *>'
return (fIter != fEnd) ? static_cast<Verb>(std::get<0>(*fIter)) : kDone_Verb;
^
C:/PROGRA~2/MICROS~1/2019/COMMUN~1/VC/Tools/MSVC/1427~1.291/Include\utility(126,7): note: template is declared here
class tuple;
^
5 errors generated.
make[1]: *** [C:/lo/src/core/solenv/gbuild/LinkTarget.mk:351: C:/lo/src/build/workdir/GenCxxObject/UnpackedTarball/skia/src/core/SkBitmapDevice.o] Error 1
Change-Id: Ica85829cf61d86e486cf4b2a730f50e06e3fb337
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100271
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski at collabora.com>
diff --git a/external/skia/UnpackedTarball_skia.mk b/external/skia/UnpackedTarball_skia.mk
index 23e864e3f206..f95d0d5f7118 100644
--- a/external/skia/UnpackedTarball_skia.mk
+++ b/external/skia/UnpackedTarball_skia.mk
@@ -39,6 +39,7 @@ skia_patches := \
fix-windows-dwrite.patch.1 \
public-make-from-backend-texture.patch.1 \
c++20.patch.0 \
+ missing-include.patch.1 \
$(eval $(call gb_UnpackedTarball_set_patchlevel,skia,1))
diff --git a/external/skia/missing-include.patch.1 b/external/skia/missing-include.patch.1
new file mode 100644
index 000000000000..91bc4d977416
--- /dev/null
+++ b/external/skia/missing-include.patch.1
@@ -0,0 +1,11 @@
+diff --git a/include/private/SkPathRef.h b/include/private/SkPathRef.h
+--- a/include/private/SkPathRef.h
++++ b/include/private/SkPathRef.h
+@@ -21,6 +21,7 @@
+
+ #include <atomic>
+ #include <limits>
++#include <tuple>
+
+ class SkRBuffer;
+ class SkWBuffer;
commit d5f66e070ca27f054f9cd2ebe3427146b6c2ac3a
Author: Caolán McNamara <caolanm at redhat.com>
AuthorDate: Thu Aug 6 17:12:21 2020 +0100
Commit: Caolán McNamara <caolanm at redhat.com>
CommitDate: Fri Aug 7 10:06:13 2020 +0200
add SAL_WARN_UNUSED_RESULT to MnemonicGenerator::CreateMnemonic
Change-Id: I159826406806166991adffb31ec04dc83d6d7504
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100264
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm at redhat.com>
diff --git a/include/vcl/mnemonic.hxx b/include/vcl/mnemonic.hxx
index 9d144c24fd90..0d62a95eb053 100644
--- a/include/vcl/mnemonic.hxx
+++ b/include/vcl/mnemonic.hxx
@@ -63,7 +63,7 @@ public:
MnemonicGenerator(sal_Unicode cMnemonic = MNEMONIC_CHAR);
void RegisterMnemonic( const OUString& rKey );
- OUString CreateMnemonic( const OUString& rKey );
+ SAL_WARN_UNUSED_RESULT OUString CreateMnemonic(const OUString& rKey);
css::uno::Reference< css::i18n::XCharacterClassification > const & GetCharClass();
// returns a string where all '~'-characters and CJK mnemonics of the form (~A) are completely removed
More information about the Libreoffice-commits
mailing list