[Libreoffice-commits] core.git: ucb/source

2020-08-10 Thread Stephan Bergmann (via logerrit)
 ucb/source/core/ucbprops.cxx |   18 ++
 ucb/source/core/ucbprops.hxx |   11 +++
 2 files changed, 5 insertions(+), 24 deletions(-)

New commits:
commit 0de191e1d201d691c2196cf1aef412a98affb66f
Author: Stephan Bergmann 
AuthorDate: Tue Aug 11 06:52:25 2020 +0200
Commit: Stephan Bergmann 
CommitDate: Tue Aug 11 08:32:16 2020 +0200

Heap use-after-free

...as seen during UITest_writer_tests2:

> ==2548829==ERROR: AddressSanitizer: heap-use-after-free on address 
0x60b0002be9d0 at pc 0x7f42be5ddc7f bp 0x7ffe2d26b090 sp 0x7ffe2d26b088
> READ of size 1 at 0x60b0002be9d0 thread T0
>  #0 in cppu::WeakComponentImplHelperBase::release() at 
cppuhelper/source/implbase.cxx:84:9
>  #1 in 
cppu::PartialWeakComponentImplHelper::release() at 
include/cppuhelper/compbase.hxx:86:36
>  #2 in rtl::Reference::~Reference() at 
include/rtl/ref.hxx:113:22
>  #3 in __run_exit_handlers at 
/usr/src/debug/glibc-2.31-48-g64246fccaf/stdlib/exit.c:108:8
>  #4 in exit at 
/usr/src/debug/glibc-2.31-48-g64246fccaf/stdlib/exit.c:139:3
>  #5 in __libc_start_main at 
/usr/src/debug/glibc-2.31-48-g64246fccaf/csu/../csu/libc-start.c:342:3
> 0x60b0002be9d0 is located 64 bytes inside of 112-byte region 
[0x60b0002be990,0x60b0002bea00)
> freed by thread T0 here:
>  #0 in free at 
/home/sbergman/github.com/llvm/llvm-project/compiler-rt/lib/asan/asan_malloc_linux.cpp:123:3
>  #1 in rtl_freeMemory at sal/rtl/alloc_global.cxx:51:5
>  #2 in cppu::WeakComponentImplHelperBase::operator delete(void*) at 
include/cppuhelper/compbase_ex.hxx:66:11
>  #3 in UcbPropertiesManager::~UcbPropertiesManager() at 
ucb/source/core/ucbprops.cxx:197:1
>  #4 in cppu::OWeakObject::release() at cppuhelper/source/weak.cxx:233:9
>  #5 in cppu::WeakComponentImplHelperBase::release() at 
cppuhelper/source/implbase.cxx:86:18
>  #6 in 
cppu::PartialWeakComponentImplHelper::release() at 
include/cppuhelper/compbase.hxx:86:36
>  #7 in rtl::Reference::clear() at 
include/rtl/ref.hxx:180:19
>  #8 in UcbPropertiesManager::dispose() at 
ucb/source/core/ucbprops.cxx:205:16
>  #9 in cppu::WeakComponentImplHelperBase::release() at 
cppuhelper/source/implbase.cxx:79:13
>  #10 in 
cppu::PartialWeakComponentImplHelper::release() at 
include/cppuhelper/compbase.hxx:86:36
>  #11 in rtl::Reference::~Reference() at 
include/rtl/ref.hxx:113:22
>  #12 in __run_exit_handlers at 
/usr/src/debug/glibc-2.31-48-g64246fccaf/stdlib/exit.c:108:8

The elaborate g_Instance disposal scheme had been introduced with
3d44c6a49b20415616dab7a2de2820da5efab309 "ucb/core: create instances with 
uno
constructors", but it is unclear to me for what reason.

Change-Id: I768bc3a8674e0e81cf89adae58e4a67d14509985
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100456
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann 

diff --git a/ucb/source/core/ucbprops.cxx b/ucb/source/core/ucbprops.cxx
index 19a42e653883..fe64bfad6cfe 100644
--- a/ucb/source/core/ucbprops.cxx
+++ b/ucb/source/core/ucbprops.cxx
@@ -45,12 +45,8 @@ using namespace com::sun::star::uno;
 
 #define ATTR_DEFAULT ( PropertyAttribute::BOUND | PropertyAttribute::MAYBEVOID 
| PropertyAttribute::MAYBEDEFAULT )
 
-static osl::Mutex g_InstanceGuard;
-static rtl::Reference g_Instance;
-
 UcbPropertiesManager::UcbPropertiesManager()
-: UcbPropertiesManager_Base(m_aMutex),
-  m_pProps({
+: m_pProps({
 { "Account", -1, cppu::UnoType::get(), ATTR_DEFAULT },
 { "AutoUpdateInterval", -1, cppu::UnoType::get(), ATTR_DEFAULT 
},
 { "ConfirmEmpty", -1, cppu::UnoType::get(), ATTR_DEFAULT },
@@ -197,14 +193,6 @@ UcbPropertiesManager::~UcbPropertiesManager()
 {
 }
 
-// XComponent
-void SAL_CALL UcbPropertiesManager::dispose()
-{
-UcbPropertiesManager_Base::dispose();
-osl::MutexGuard aGuard(g_InstanceGuard);
-g_Instance.clear();
-}
-
 // XServiceInfo methods.
 
 OUString SAL_CALL UcbPropertiesManager::getImplementationName()
@@ -228,9 +216,7 @@ extern "C" SAL_DLLPUBLIC_EXPORT css::uno::XInterface*
 ucb_UcbPropertiesManager_get_implementation(
 css::uno::XComponentContext* , css::uno::Sequence const&)
 {
-osl::MutexGuard aGuard(g_InstanceGuard);
-if (!g_Instance)
-g_Instance.set(new UcbPropertiesManager());
+static rtl::Reference g_Instance(new 
UcbPropertiesManager());
 g_Instance->acquire();
 return static_cast(g_Instance.get());
 }
diff --git a/ucb/source/core/ucbprops.hxx b/ucb/source/core/ucbprops.hxx
index 6ef4e5a5bfcb..89e31a389e9f 100644
--- a/ucb/source/core/ucbprops.hxx
+++ b/ucb/source/core/ucbprops.hxx
@@ -24,15 +24,13 @@
 #include 
 #include 
 #include 
-#include 
-#include 
+#include 
 
-
-using UcbPropertiesManager_Base = cppu::WeakComponentImplHelper <
+using UcbPropertiesManager_Base = cppu::WeakImplHelper <
 css::lang::XServiceInfo,
 css::beans::XP

[Libreoffice-commits] online.git: Branch 'distro/collabora/co-4-2' - loleaflet/src

2020-08-10 Thread Szymon Kłos (via logerrit)
 loleaflet/src/core/Socket.js |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 6b05c81ffed4496586705c8c825d93b45d0f2257
Author: Szymon Kłos 
AuthorDate: Tue Aug 4 09:27:00 2020 +0200
Commit: Aron Budea 
CommitDate: Tue Aug 11 05:43:15 2020 +0200

Don't show [object Window] in about dialog

Change-Id: I4ba1ab2fa443a46dd205411da558980ddee44865
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/100043
Tested-by: Jenkins CollaboraOffice 
Tested-by: Jenkins
Reviewed-by: Szymon Kłos 
(cherry picked from commit d6bfb8867495f34010620b68a8d298e96cb51d56)
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/100359
Reviewed-by: Aron Budea 

diff --git a/loleaflet/src/core/Socket.js b/loleaflet/src/core/Socket.js
index 41b173209..a5f8ee5ed 100644
--- a/loleaflet/src/core/Socket.js
+++ b/loleaflet/src/core/Socket.js
@@ -283,7 +283,7 @@ L.Socket = L.Class.extend({
this.WSDServer = 
JSON.parse(textMsg.substring(textMsg.indexOf('{')));
var h = this.WSDServer.Hash;
if (parseInt(h,16).toString(16) === 
h.toLowerCase().replace(/^0+/, '')) {
-   h = 'https://hub.libreoffice.org/git-online/' + h + 
'\');">' + h + '';
+   h = 'https://hub.libreoffice.org/git-online/' + 
h + '\'));">' + h + '';

$('#loolwsd-version').html(this.WSDServer.Version + ' (git hash: ' + h + ')');
}
else {
@@ -306,7 +306,7 @@ L.Socket = L.Class.extend({
var lokitVersionObj = 
JSON.parse(textMsg.substring(textMsg.indexOf('{')));
h = lokitVersionObj.BuildId.substring(0, 7);
if (parseInt(h,16).toString(16) === 
h.toLowerCase().replace(/^0+/, '')) {
-   h = 'https://hub.libreoffice.org/git-core/' + h + 
'\');">' + h + '';
+   h = 'https://hub.libreoffice.org/git-core/' + h 
+ '\'));">' + h + '';
}
$('#lokit-version').html(lokitVersionObj.ProductName + 
' ' +
 lokitVersionObj.ProductVersion 
+ lokitVersionObj.ProductExtension.replace('.10.','-') +
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-7-0' - framework/source icon-themes/breeze icon-themes/breeze_dark icon-themes/breeze_dark_svg icon-themes/breeze_svg icon-themes/colibre icon-theme

2020-08-10 Thread Maxim Monastirsky (via logerrit)
 framework/source/uielement/popuptoolbarcontroller.cxx  |2 
 icon-themes/breeze/cmd/lc_newhtmldoc.png   |binary
 icon-themes/breeze/cmd/sc_newhtmldoc.png   |binary
 icon-themes/breeze/cmd/sc_questionanswers.png  |binary
 icon-themes/breeze/links.txt   |4 +
 icon-themes/breeze/res/lx03245.png |binary
 icon-themes/breeze/res/lx03245_32.png  |binary
 icon-themes/breeze/res/lx03246.png |binary
 icon-themes/breeze/res/lx03246_32.png  |binary
 icon-themes/breeze/res/lx03247.png |binary
 icon-themes/breeze/res/lx03247_32.png  |binary
 icon-themes/breeze/res/lx03248.png |binary
 icon-themes/breeze/res/lx03248_32.png  |binary
 icon-themes/breeze/res/lx03249.png |binary
 icon-themes/breeze/res/lx03249_32.png  |binary
 icon-themes/breeze/res/lx03250.png |binary
 icon-themes/breeze/res/lx03250_32.png  |binary
 icon-themes/breeze/res/lx03251.png |binary
 icon-themes/breeze/res/lx03251_32.png  |binary
 icon-themes/breeze/res/lx03255.png |binary
 icon-themes/breeze/res/lx03255_32.png  |binary
 icon-themes/breeze_dark/cmd/lc_newhtmldoc.png  |binary
 icon-themes/breeze_dark/cmd/sc_newhtmldoc.png  |binary
 icon-themes/breeze_dark/cmd/sc_questionanswers.png |binary
 icon-themes/breeze_dark/links.txt  |4 +
 icon-themes/breeze_dark/res/lx03245.png|binary
 icon-themes/breeze_dark/res/lx03245_32.png |binary
 icon-themes/breeze_dark/res/lx03246.png|binary
 icon-themes/breeze_dark/res/lx03246_32.png |binary
 icon-themes/breeze_dark/res/lx03247.png|binary
 icon-themes/breeze_dark/res/lx03247_32.png |binary
 icon-themes/breeze_dark/res/lx03248.png|binary
 icon-themes/breeze_dark/res/lx03248_32.png |binary
 icon-themes/breeze_dark/res/lx03249.png|binary
 icon-themes/breeze_dark/res/lx03249_32.png |binary
 icon-themes/breeze_dark/res/lx03250.png|binary
 icon-themes/breeze_dark/res/lx03250_32.png |binary
 icon-themes/breeze_dark/res/lx03251.png|binary
 icon-themes/breeze_dark/res/lx03251_32.png |binary
 icon-themes/breeze_dark/res/lx03255.png|binary
 icon-themes/breeze_dark/res/lx03255_32.png |binary
 icon-themes/breeze_dark_svg/cmd/lc_newhtmldoc.svg  |1 
 icon-themes/breeze_dark_svg/cmd/sc_newhtmldoc.svg  |1 
 icon-themes/breeze_dark_svg/cmd/sc_questionanswers.svg |2 
 icon-themes/breeze_dark_svg/res/lx03245.svg|2 
 icon-themes/breeze_dark_svg/res/lx03245_32.svg |1 
 icon-themes/breeze_dark_svg/res/lx03246.svg|2 
 icon-themes/breeze_dark_svg/res/lx03246_32.svg |1 
 icon-themes/breeze_dark_svg/res/lx03247.svg|2 
 icon-themes/breeze_dark_svg/res/lx03247_32.svg |1 
 icon-themes/breeze_dark_svg/res/lx03248.svg|2 
 icon-themes/breeze_dark_svg/res/lx03248_32.svg |1 
 icon-themes/breeze_dark_svg/res/lx03249.svg|2 
 icon-themes/breeze_dark_svg/res/lx03249_32.svg |1 
 icon-themes/breeze_dark_svg/res/lx03250.svg|2 
 icon-themes/breeze_dark_svg/res/lx03250_32.svg |1 
 icon-themes/breeze_dark_svg/res/lx03251.svg|2 
 icon-themes/breeze_dark_svg/res/lx03251_32.svg |1 
 icon-themes/breeze_dark_svg/res/lx03255.svg|2 
 icon-themes/breeze_dark_svg/res/lx03255_32.svg |1 
 icon-themes/breeze_svg/cmd/lc_newhtmldoc.svg   |1 
 icon-themes/breeze_svg/cmd/sc_newhtmldoc.svg   |1 
 icon-themes/breeze_svg/cmd/sc_questionanswers.svg  |2 
 icon-themes/breeze_svg/res/lx03245.svg |2 
 icon-themes/breeze_svg/res/lx03245_32.svg  |1 
 icon-themes/breeze_svg/res/lx03246.svg |2 
 icon-themes/breeze_svg/res/lx03246_32.svg  |1 
 icon-themes/breeze_svg/res/lx03247.svg |2 
 icon-themes/breeze_svg/res/lx03247_32.svg  |1 
 icon-themes/breeze_svg/res/lx03248.svg |2 
 icon-themes/breeze_svg/res/lx03248_32.svg  |1 
 icon-themes/breeze_svg/res/lx03249.svg |2 
 icon-themes/breeze_svg/res/lx03249_32.svg  |1 
 icon-themes/breeze_svg/res/lx03250.svg |2 
 icon-themes/breeze_svg/res/lx03250_32.svg  |1 
 icon-themes/breeze_svg/res/lx03251.svg |2 
 icon-themes/breeze_svg/res/lx03251_32.svg  |1 
 icon-themes/breeze_svg/res/lx03255.svg |2 
 icon

[Libreoffice-commits] core.git: Branch 'feature/windows-cross-build' - 278 commits - accessibility/inc basctl/source basegfx/source basic/qa basic/source bin/get_config_variables bridges/Library_cpp_u

2020-08-10 Thread Jan-Marek Glogowski (via logerrit)
Rebased ref, commits from common ancestor:
commit 8acc3b1727f1615a0b7eed08917383b61578ce71
Author: Jan-Marek Glogowski 
AuthorDate: Wed Jul 29 12:04:03 2020 +0200
Commit: Jan-Marek Glogowski 
CommitDate: Tue Aug 11 01:46:31 2020 +0200

cross-build: fix Java NI linking

LibreOffice has a JNI component on Windows and Linux, the
officebean. Therefore we need a host JDK for linkage to the
jawt, and a build JDK to compile the Java code.

Change-Id: I4138628ab3ea2ef5900a5b4e9281050ae84e4eb5

diff --git a/config_host.mk.in b/config_host.mk.in
index d3d269394380..ce4e70a9bc5e 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -266,7 +266,6 @@ export HARFBUZZ_LIBS=$(gb_SPACE)@HARFBUZZ_LIBS@
 export GSSAPI_LIBS=@GSSAPI_LIBS@
 export GSTREAMER_1_0_CFLAGS=$(gb_SPACE)@GSTREAMER_1_0_CFLAGS@
 export GSTREAMER_1_0_LIBS=$(gb_SPACE)@GSTREAMER_1_0_LIBS@
-export GTHREAD_CFLAGS=$(gb_SPACE)@GTHREAD_CFLAGS@
 export GTK3_CFLAGS=$(gb_SPACE)@GTK3_CFLAGS@
 export GTK3_LIBS=$(gb_SPACE)@GTK3_LIBS@
 export USING_X11=@USING_X11@
@@ -319,15 +318,18 @@ export IWYU_PATH=@IWYU_PATH@
 export JAVACOMPILER=@JAVACOMPILER@
 export JAVADOC=@JAVADOC@
 export JAVADOCISGJDOC=@JAVADOCISGJDOC@
-export JAVAFLAGS=@JAVAFLAGS@
+export JAVACFLAGS=@JAVACFLAGS@
 export JAVAIFLAGS=@JAVAIFLAGS@
+export JAVAIFLAGS_FOR_BUILD=@JAVAIFLAGS@
 export JAVA_CLASSPATH_NOT_SET=@JAVA_CLASSPATH_NOT_SET@
 export JAVAINTERPRETER=@JAVAINTERPRETER@
 export JAVA_HOME=@JAVA_HOME@
+export JAVA_HOME_FOR_BUILD=@JAVA_HOME_FOR_BUILD@
 export JAVA_SOURCE_VER=@JAVA_SOURCE_VER@
 export JAVA_TARGET_VER=@JAVA_TARGET_VER@
 export JAWTLIB=@JAWTLIB@
 export JDK=@JDK@
+export JDK_FOR_BUILD=@JDK_FOR_BUILD@
 export JFREEREPORT_JAR=@JFREEREPORT_JAR@
 export JITC_PROCESSOR_TYPE=@JITC_PROCESSOR_TYPE@
 export JVM_ONE_PATH_CHECK=@JVM_ONE_PATH_CHECK@
diff --git a/configure.ac b/configure.ac
index a636a396f3f5..008096708247 100644
--- a/configure.ac
+++ b/configure.ac
@@ -4900,7 +4900,18 @@ if test "$cross_compiling" = "yes"; then
 sub_conf_opts=""
 test -n "$enable_ccache" && sub_conf_opts="$sub_conf_opts 
--enable-ccache=$enable_ccache"
 test -n "$with_ant_home" && sub_conf_opts="$sub_conf_opts 
--with-ant-home=$with_ant_home"
-test $with_junit = no && sub_conf_opts="$sub_conf_opts --without-junit"
+test "$with_junit" = "no" && sub_conf_opts="$sub_conf_opts --without-junit"
+if test -n "$ENABLE_JAVA"; then
+if test "$_os" != "iOS" -a "$_os" != "Android"; then
+if ! echo "$with_build_platform_configure_options" | grep -q -- 
'--with-jdk-home='; then
+AC_MSG_ERROR([Missing build JDK (see 
--with-build-platform-configure-options, --with-jdk-home and use 'cygpath -ms' 
on Windows)!])
+fi
+else
+test -n "$with_jdk_home" && sub_conf_opts="$sub_conf_opts 
--with-jdk-home=$with_jdk_home"
+fi
+else
+sub_conf_opts="$sub_conf_opts --without-java"
+fi
 test -n "$TARFILE_LOCATION" && sub_conf_opts="$sub_conf_opts 
--with-external-tar=$TARFILE_LOCATION"
 test "$with_system_icu_for_build" = "yes" -o "$with_system_icu_for_build" 
= "force" && sub_conf_opts="$sub_conf_opts --with-system-icu"
 sub_conf_opts="$sub_conf_opts $with_build_platform_configure_options"
@@ -4920,7 +4931,6 @@ if test "$cross_compiling" = "yes"; then
 --disable-skia \
 --enable-icecream="$enable_icecream" \
 --without-doxygen \
---without-java \
 --without-webdav \
 --with-parallelism="$with_parallelism" \
 --with-theme="$with_theme" \
@@ -4972,15 +4982,53 @@ if test "$cross_compiling" = "yes"; then
 mkdir -p ../config_build
 mv config_host/*.h ../config_build
 
+# all these will get a _FOR_BUILD postfix
+DIRECT_FOR_BUILD_SETTINGS="
+CC
+CXX
+ILIB
+JAVA_HOME
+JAVAIFLAGS
+JDK
+LIBO_BIN_FOLDER
+LIBO_LIB_FOLDER
+LIBO_URE_LIB_FOLDER
+LIBO_URE_MISC_FOLDER
+OS
+SDKDIRNAME
+SYSTEM_LIBXML
+SYSTEM_LIBXSLT
+"
+# these overwrite host config with build config
+OVERWRITING_SETTINGS="
+ANT
+ANT_HOME
+ANT_LIB
+HSQLDB_USE_JDBC_4_1
+JAVA_CLASSPATH_NOT_SET
+JAVA_SOURCE_VER
+JAVA_TARGET_VER
+JAVACFLAGS
+JAVACOMPILER
+JAVADOC
+JAVADOCISGJDOC
+"
+# these need some special handling
+EXTRA_HANDLED_SETTINGS="
+INSTDIR
+INSTROOT
+PATH
+WORKDIR
+"
 OLD_PATH=$PATH
-. ./bin/get_config_variables CC CXX ILIB INSTDIR INSTROOT LIBO_BIN_FOLDER 
LIBO_LIB_FOLDER LIBO_URE_LIB_FOLDER LIBO_URE_MISC_FOLDER OS PATH SDKDIRNAME 
SYSTEM_LIBXML SYSTEM_LIBXSLT WORKDIR
+. ./bin/get_config_variables $DIRECT_FOR_BUILD_SETTINGS 
$OVERWRITING_SETTINGS $EXTRA_HANDLED_SETTINGS
 BUILD_PATH=$PATH
 PATH=$OLD_PATH
 
 line=`echo "LO_PATH_FOR_BUILD='${BUILD_PATH}'" | sed -e 
's,/CONF-FOR-BUILD,,

[Libreoffice-commits] core.git: Branch 'private/jmux/win-arm64' - 296 commits - accessibility/inc basctl/source basegfx/source basic/qa basic/source bin/get_config_variables bridges/inc bridges/Librar

2020-08-10 Thread Jan-Marek Glogowski (via logerrit)
Rebased ref, commits from common ancestor:
commit bec5262785e530db0d9a1e582f6eccb39053bdfe
Author: Jan-Marek Glogowski 
AuthorDate: Sat Jul 18 04:01:32 2020 +0200
Commit: Jan-Marek Glogowski 
CommitDate: Tue Aug 11 01:51:17 2020 +0200

Add my current Windows Arm64 build config

Change-Id: I762049d34e6a00fec0960395f94d6537047f6061

diff --git a/distro-configs/LibreOfficeWinArm64.conf 
b/distro-configs/LibreOfficeWinArm64.conf
new file mode 100644
index ..05a526a8786f
--- /dev/null
+++ b/distro-configs/LibreOfficeWinArm64.conf
@@ -0,0 +1,25 @@
+--host=aarch64-pc-cygwin
+--disable-ccache
+--disable-coinmp
+--disable-ext-nlpsolver
+--disable-ext-wiki-publisher
+--disable-firebird-sdbc
+--disable-gpgmepp
+--disable-odk
+--disable-online-update
+--disable-skia
+--disable-werror
+--enable-assert-always-abort
+--enable-debug
+--enable-extension-integration
+--enable-pch
+--enable-scripting-beanshell
+--enable-scripting-javascript
+--with-ant-home=/cygdrive/c/lode/packages/apache-ant-1.9.4
+--with-jdk-home=/cygdrive/c/Program Files/Java/jdk-14.0.1 
+--with-junit=/cygdrive/c/lode/opt/share/java/junit.jar
+--with-theme=colibre colibre_svg
+--without-doxygen
+--without-help
+--without-helppack-integration
+--without-java
commit e0e60c79194790ee75a6fffab6700498cdfb3d8e
Author: Jan-Marek Glogowski 
AuthorDate: Mon Aug 3 07:08:56 2020 +0200
Commit: Jan-Marek Glogowski 
CommitDate: Tue Aug 11 01:51:17 2020 +0200

jvmfwk: add vendorbase entry for Windows Arm64

Change-Id: Iea89befded02ecfd7513cc3d8f116dd6ac2913be

diff --git a/jvmfwk/inc/vendorbase.hxx b/jvmfwk/inc/vendorbase.hxx
index df536bc3477e..7e98a2409c35 100644
--- a/jvmfwk/inc/vendorbase.hxx
+++ b/jvmfwk/inc/vendorbase.hxx
@@ -39,6 +39,8 @@ namespace jfw_plugin
 #define JFW_PLUGIN_ARCH "sparc"
 #elif defined X86_64
 #define JFW_PLUGIN_ARCH "amd64"
+#elif defined ARM64
+#define JFW_PLUGIN_ARCH "arm64"
 #elif defined INTEL
 #define JFW_PLUGIN_ARCH "i386"
 #elif defined POWERPC64
commit a531cd42dcf22ea9dd94f225293d83c4b74392c0
Author: Jan-Marek Glogowski 
AuthorDate: Mon Aug 3 07:06:36 2020 +0200
Commit: Jan-Marek Glogowski 
CommitDate: Tue Aug 11 01:51:17 2020 +0200

Disable .NET support for Windows Arm64

The current .NET 5.0 Arm64 preview doesn't have a mscoree.lib,
so linking the climaker isn't possible.

Change-Id: Ibbac88aa465a9ca2eb8fb0efaad91d20f358229b

diff --git a/cli_ure/Module_cli_ure.mk b/cli_ure/Module_cli_ure.mk
index d1ff09073a18..91863abb59c9 100644
--- a/cli_ure/Module_cli_ure.mk
+++ b/cli_ure/Module_cli_ure.mk
@@ -10,6 +10,7 @@
 $(eval $(call gb_Module_Module,cli_ure))
 
 ifeq ($(COM),MSC)
+ifneq ($(CPUNAME),ARM64)
 $(eval $(call gb_Module_add_targets,cli_ure,\
CliLibrary_cli_basetypes \
CliLibrary_cli_ure \
@@ -22,5 +23,6 @@ $(eval $(call gb_Module_add_targets,cli_ure,\
Package_cli_basetypes_copy \
 ))
 endif
+endif
 
 # vim: set noet sw=4 ts=4:
diff --git a/unoil/Module_unoil.mk b/unoil/Module_unoil.mk
index 5977368f2dab..6b1cb5fd064f 100644
--- a/unoil/Module_unoil.mk
+++ b/unoil/Module_unoil.mk
@@ -17,9 +17,11 @@ $(eval $(call gb_Module_add_targets,unoil,\
 endif
 
 ifeq ($(COM),MSC)
+ifneq ($(CPUNAME),ARM64)
 $(eval $(call gb_Module_add_targets,unoil,\
 CliUnoApi_oootypes \
 ))
 endif
+endif
 
 # vim:set noet sw=4 ts=4:
commit 95c1bfb3149d577fa34bf28197032074224d61d6
Author: Jan-Marek Glogowski 
AuthorDate: Mon Aug 3 07:04:28 2020 +0200
Commit: Jan-Marek Glogowski 
CommitDate: Tue Aug 11 01:51:17 2020 +0200

libcurl: fix Windows Arm64 build

Change-Id: I22a203cb56e4b6dd8f80249cd30cf9a4efb676eb

diff --git a/RepositoryExternal.mk b/RepositoryExternal.mk
index fbb38e457459..5205d071301d 100644
--- a/RepositoryExternal.mk
+++ b/RepositoryExternal.mk
@@ -2748,7 +2748,7 @@ $(call gb_LinkTarget_set_include,$(1),\
 
 ifeq ($(COM),MSC)
 $(call gb_LinkTarget_add_libs,$(1),\
-   $(call gb_UnpackedTarball_get_dir,curl)/builds/libcurl-vc12-$(if 
$(filter X86_64,$(CPUNAME)),x64,x86)-$(if 
$(MSVC_USE_DEBUG_RUNTIME),debug,release)-dll-ipv6-sspi-winssl/lib/libcurl$(if 
$(MSVC_USE_DEBUG_RUNTIME),_debug).lib \
+   $(call 
gb_UnpackedTarball_get_dir,curl)/builds/libcurl-vc12-$(gb_MSBUILD_PLATFORM)-$(gb_MSBUILD_CONFIG)-dll-ipv6-sspi-winssl/lib/libcurl$(if
 $(MSVC_USE_DEBUG_RUNTIME),_debug).lib \
 )
 else
 $(call gb_LinkTarget_add_libs,$(1),\
commit 802dd347c9db550098de4332ec3ec240ab23eea6
Author: Jan-Marek Glogowski 
AuthorDate: Mon Aug 3 06:49:32 2020 +0200
Commit: Jan-Marek Glogowski 
CommitDate: Tue Aug 11 01:51:16 2020 +0200

cppunit: fix Windows Arm64 build

Change-Id: I75b169362ed703bcae0720aeac0602f389435317

diff --git a/external/cppunit/ExternalProject_cppunit.mk 
b/external/cppunit/ExternalProject_cppunit.mk
index 9b4a69c627e1..1c5ac725af59 100644
--- a/external/cppunit/ExternalProject_cppunit.mk
+++ b/external/cppunit/ExternalProject_cppunit.mk
@@ -17,8 +17,8 @@ ifeq ($(OS),WNT)
 $(call gb_Exte

[Libreoffice-commits] core.git: Branch 'private/quwex/gsoc-box2d-experimental' - 3 commits - offapi/com slideshow/source

2020-08-10 Thread Sarper Akdemir (via logerrit)
Rebased ref, commits from common ancestor:
commit 786cfdcf2ac6ce0e3d8a0cc48b5110b685db2aae
Author: Sarper Akdemir 
AuthorDate: Mon Aug 10 15:30:26 2020 +0300
Commit: Sarper Akdemir 
CommitDate: Tue Aug 11 01:38:59 2020 +0300

add version tag to ANIMATEPHYSICS

Change-Id: Ia6db8ca10a0311ae8492cdc5ab518efaba611cb2

diff --git a/offapi/com/sun/star/animations/AnimationNodeType.idl 
b/offapi/com/sun/star/animations/AnimationNodeType.idl
index d0cd6e268fd6..7c6abb105947 100644
--- a/offapi/com/sun/star/animations/AnimationNodeType.idl
+++ b/offapi/com/sun/star/animations/AnimationNodeType.idl
@@ -68,7 +68,10 @@ constants AnimationNodeType
 /** Defines a command effect. */
 const short COMMAND = 11;
 
-/** Defines a physics animation */
+/** Defines a physics animation
+
+@since LibreOffice 7.1
+*/
 const short ANIMATEPHYSICS = 12;
 
 };
commit a5392206b824075e930880b5b379813982abb72f
Author: Sarper Akdemir 
AuthorDate: Thu Aug 6 10:32:55 2020 +0300
Commit: Sarper Akdemir 
CommitDate: Tue Aug 11 01:38:59 2020 +0300

make physics based animation effects always processed last

Change-Id: I92d436aced6ef3ee2c8b0bf0167c1f7e642ba3b5

diff --git a/slideshow/source/engine/activitiesqueue.cxx 
b/slideshow/source/engine/activitiesqueue.cxx
index ba982385356e..38e79d1e5677 100644
--- a/slideshow/source/engine/activitiesqueue.cxx
+++ b/slideshow/source/engine/activitiesqueue.cxx
@@ -50,6 +50,8 @@ namespace slideshow::internal
 {
 for( const auto& pActivity : maCurrentActivitiesWaiting )
 pActivity->dispose();
+for( const auto& pActivity : 
maCurrentActivitiesToBeProcessedLast )
+pActivity->dispose();
 for( const auto& pActivity : maCurrentActivitiesReinsert )
 pActivity->dispose();
 }
@@ -59,7 +61,7 @@ namespace slideshow::internal
 }
 }
 
-bool ActivitiesQueue::addActivity( const ActivitySharedPtr& pActivity )
+bool ActivitiesQueue::addActivity( const ActivitySharedPtr& pActivity, 
const bool bProcessLast )
 {
 OSL_ENSURE( pActivity, "ActivitiesQueue::addActivity: activity ptr 
NULL" );
 
@@ -67,7 +69,17 @@ namespace slideshow::internal
 return false;
 
 // add entry to waiting list
-maCurrentActivitiesWaiting.push_back( pActivity );
+if( !bProcessLast )
+{
+maCurrentActivitiesWaiting.push_back( pActivity );
+}
+else
+{
+// Activities that should be processed last is kept in a 
different
+// ActivityQueue, and later added to the end of the 
maCurrentActivitiesWaiting
+// at the start of ActivitiesQueue::process()
+maCurrentActivitiesToBeProcessedLast.push_back( pActivity );
+}
 
 return true;
 }
@@ -76,6 +88,12 @@ namespace slideshow::internal
 {
 SAL_INFO("slideshow.verbose", "ActivitiesQueue: outer loop 
heartbeat" );
 
+// If there are activities to be processed last add them to the 
end of the ActivitiesQueue
+maCurrentActivitiesWaiting.insert( 
maCurrentActivitiesWaiting.end(),
+   
maCurrentActivitiesToBeProcessedLast.begin(),
+   
maCurrentActivitiesToBeProcessedLast.end() );
+maCurrentActivitiesToBeProcessedLast.clear();
+
 // accumulate time lag for all activities, and lag time
 // base if necessary:
 double fLag = 0.0;
diff --git a/slideshow/source/engine/animationnodes/animationbasenode.cxx 
b/slideshow/source/engine/animationnodes/animationbasenode.cxx
index 4dcb640795aa..7999b5a7654a 100644
--- a/slideshow/source/engine/animationnodes/animationbasenode.cxx
+++ b/slideshow/source/engine/animationnodes/animationbasenode.cxx
@@ -23,6 +23,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include "nodetools.hxx"
 #include 
@@ -294,7 +295,10 @@ void AnimationBaseNode::activate_st()
 mpActivity->setTargets( getShape(), maAttributeLayerHolder.get() );
 
 // add to activities queue
-getContext().mrActivitiesQueue.addActivity( mpActivity );
+if( mxAnimateNode->getType() == 
css::animations::AnimationNodeType::ANIMATEPHYSICS )
+getContext().mrActivitiesQueue.addActivity(mpActivity, true);
+else
+getContext().mrActivitiesQueue.addActivity( mpActivity );
 }
 else {
 // Actually, DO generate the event for empty activity,
diff --git a/slideshow/source/inc/activitiesqueue.hxx 
b/slideshow/source/inc/activitiesqueue.hxx
index b4f88b1b39d1..76dc981f8f65 100644
--- a/slideshow/source/inc/activitiesqueue.hxx
+++ b/slideshow/source/inc/activitiesqueue.hxx
@@ -57,7 +57,7 @@ namespace slideshow
 
 /*

[Libreoffice-commits] core.git: oox/source

2020-08-10 Thread Gülşah Köse (via logerrit)
 oox/source/drawingml/table/tableproperties.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 99dff69b561a8fe2d9437e6aa67a9581af41
Author: Gülşah Köse 
AuthorDate: Mon Aug 10 10:04:51 2020 +0300
Commit: Gülşah Köse 
CommitDate: Mon Aug 10 23:51:39 2020 +0200

tdf#133015 Fix table position during import multicol textbox.

Change-Id: Ied1a03ff9f4556a551738b698ccb284fe74299da
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100414
Tested-by: Jenkins
Reviewed-by: Gülşah Köse 

diff --git a/oox/source/drawingml/table/tableproperties.cxx 
b/oox/source/drawingml/table/tableproperties.cxx
index bf7d0dcfe7a2..1c12c10eda47 100644
--- a/oox/source/drawingml/table/tableproperties.cxx
+++ b/oox/source/drawingml/table/tableproperties.cxx
@@ -145,7 +145,7 @@ void TableProperties::pushToPropSet(const 
::oox::core::XmlFilterBase& rFilterBas
 for (auto& tableRow : mvTableRows)
 {
 sal_Int32 nColumn = 0;
-sal_Int32 nColumnSize = tableRow.getTableCells().size();
+sal_Int32 nColumnSize = mvTableGrid.size();
 sal_Int32 nRemovedColumn = 0; //
 
 for (sal_Int32 nColIndex = 0; nColIndex < nColumnSize; nColIndex++)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-6.4' - 2 commits - desktop/qa desktop/source sd/qa

2020-08-10 Thread Tomaž Vajngerl (via logerrit)
 desktop/qa/data/BlankDrawDocument.odg|binary
 desktop/qa/desktop_lib/test_desktop_lib.cxx  |   68 +++
 desktop/source/lib/init.cxx  |   52 
 sd/qa/unit/tiledrendering/data/dummy.odg |binary
 sd/qa/unit/tiledrendering/tiledrendering.cxx |   56 +-
 5 files changed, 172 insertions(+), 4 deletions(-)

New commits:
commit 68e5ff123e7bca884fcf23cf9f69422eb41fa0f7
Author: Tomaž Vajngerl 
AuthorDate: Sun Jul 26 21:58:33 2020 +0200
Commit: Tomaž Vajngerl 
CommitDate: Mon Aug 10 23:08:14 2020 +0200

allow to .uno:Save a PDF doc - divert to SaveAs to the doc. file

To save PDF annotations, we need to allow a .uno:Save if the
document was opened from a PDF file. When we get a .uno:Save
command, we need to divert that to SaveAs into the same file as
the current document was opened with.

Change-Id: I0c511c4e5501de03ea8b0efb5aaf37cd09936e6e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99463
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl 
(cherry picked from commit 6f55a64f002d80a19201e2a9b0171725fb71a7fb)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99883
Tested-by: Jenkins CollaboraOffice 

diff --git a/desktop/source/lib/init.cxx b/desktop/source/lib/init.cxx
index 906d3c4c6baa..fdd92d2d3430 100644
--- a/desktop/source/lib/init.cxx
+++ b/desktop/source/lib/init.cxx
@@ -105,6 +105,8 @@
 #include 
 #include 
 #include 
+#include 
+#include 
 #include 
 #include 
 #include 
@@ -721,6 +723,28 @@ std::string extractPrivateKey(const std::string & 
privateKey)
 return privateKey.substr(pos1, pos2);
 }
 
+OUString lcl_getCurrentDocumentMimeType(LibLODocument_Impl* pDocument)
+{
+OUString aMimeType;
+SfxBaseModel* pBaseModel = 
dynamic_cast(pDocument->mxComponent.get());
+if (!pBaseModel)
+return aMimeType;
+
+SfxObjectShell* pObjectShell = pBaseModel->GetObjectShell();
+if (!pObjectShell)
+return aMimeType;
+
+SfxMedium* pMedium = pObjectShell->GetMedium();
+if (!pMedium)
+return aMimeType;
+
+auto pFilter = pMedium->GetFilter();
+if (!pFilter)
+return aMimeType;
+
+return pFilter->GetMimeType();
+}
+
 // Gets an undo manager to enter and exit undo context. Needed by 
ToggleOrientation
 css::uno::Reference< css::document::XUndoManager > getUndoManager( const 
css::uno::Reference< css::frame::XFrame >& rxFrame )
 {
@@ -2463,6 +2487,9 @@ static int doc_saveAs(LibreOfficeKitDocument* pThis, 
const char* sUrl, const cha
 
 OUString sFormat = getUString(pFormat);
 OUString aURL(getAbsoluteURL(sUrl));
+
+uno::Reference xStorable(pDocument->mxComponent, 
uno::UNO_QUERY_THROW);
+
 if (aURL.isEmpty())
 {
 SetLastExceptionMsg("Filename to save to was not provided.");
@@ -2603,7 +2630,6 @@ static int doc_saveAs(LibreOfficeKitDocument* pThis, 
const char* sUrl, const cha
 aSaveMediaDescriptor[MediaDescriptor::PROP_INTERACTIONHANDLER()] 
<<= xInteraction;
 }
 
-uno::Reference xStorable(pDocument->mxComponent, 
uno::UNO_QUERY_THROW);
 
 if (bTakeOwnership)
 xStorable->storeAsURL(aURL, 
aSaveMediaDescriptor.getAsConstPropertyValueList());
@@ -3785,6 +3811,30 @@ static void doc_postUnoCommand(LibreOfficeKitDocument* 
pThis, const char* pComma
 // handle potential interaction
 if (gImpl && aCommand == ".uno:Save")
 {
+// Check if saving a PDF file
+OUString aMimeType = lcl_getCurrentDocumentMimeType(pDocument);
+if (aMimeType == "application/pdf")
+{
+// If we have a PDF file (for saving annotations for example), we 
need
+// to run save-as to the same file as the opened document. Plain 
save
+// doesn't work as the PDF is not a "native" format.
+uno::Reference xStorable(pDocument->mxComponent, 
uno::UNO_QUERY_THROW);
+OUString aURL = xStorable->getLocation();
+OString aURLUtf8 = OUStringToOString(aURL, RTL_TEXTENCODING_UTF8);
+bool bResult = doc_saveAs(pThis, aURLUtf8.getStr(), "pdf", 
nullptr);
+
+// Send the result of save
+boost::property_tree::ptree aTree;
+aTree.put("commandName", pCommand);
+aTree.put("success", bResult);
+std::stringstream aStream;
+boost::property_tree::write_json(aStream, aTree);
+OString aPayload = aStream.str().c_str();
+
pDocument->mpCallbackFlushHandlers[nView]->queue(LOK_CALLBACK_UNO_COMMAND_RESULT,
 aPayload.getStr());
+return;
+}
+
+
 rtl::Reference const pInteraction(
 new LOKInteractionHandler("save", gImpl, pDocument));
 uno::Reference const 
xInteraction(pInteraction.get());
commit b4d0b45ba6bce22501fbd623ac2ebf5da29ee356
Author: Tomaž Vajngerl 
AuthorDate: Sat Jul 11 21:18:34 2020 +0200
Commit: Tomaž Vajngerl 

[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-6.4' - 2 commits - include/tools tools/qa tools/source

2020-08-10 Thread Noel Grandin (via logerrit)
 include/tools/json_writer.hxx |   66 +---
 tools/qa/cppunit/test_json_writer.cxx |   38 ++-
 tools/source/misc/json_writer.cxx |  112 ++
 3 files changed, 190 insertions(+), 26 deletions(-)

New commits:
commit f271eca1eb24ed28658afe4c73d71cbf77a78dec
Author: Noel Grandin 
AuthorDate: Wed Jun 24 18:24:37 2020 +0200
Commit: Tomaž Vajngerl 
CommitDate: Mon Aug 10 23:07:32 2020 +0200

fix JsonWriter::reallocBuffer

was not updating mSpaceAllocated

Change-Id: Ie5404e58c6520d32b72c19ddfa58b2ab2b895199
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/97049
Tested-by: Jenkins
Reviewed-by: Noel Grandin 
(cherry picked from commit 3fb5c11ac69e6687e579d4129cb892c5ae746a5e)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100445
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Tomaž Vajngerl 

diff --git a/tools/source/misc/json_writer.cxx 
b/tools/source/misc/json_writer.cxx
index 251c44c0246f..f2b008ea6fd2 100644
--- a/tools/source/misc/json_writer.cxx
+++ b/tools/source/misc/json_writer.cxx
@@ -320,6 +320,7 @@ void JsonWriter::reallocBuffer(int noMoreBytesRequired)
 free(mpBuffer);
 mpBuffer = pNew;
 mPos = mpBuffer + currentUsed;
+mSpaceAllocated = newSize;
 }
 
 /** Hands ownership of the underlying storage buffer to the caller,
commit 29954dbeb16ef206940a8fe7ed94e487f842d17a
Author: Noel Grandin 
AuthorDate: Thu Jun 18 21:39:30 2020 +0200
Commit: Tomaž Vajngerl 
CommitDate: Mon Aug 10 23:07:20 2020 +0200

improvements to JSON Writer

part of the master commit cb95276e6e6bf12a1c06d5c252551e55c788fcb2

Change-Id: Icf18fb4b3ebced375196a8cdbd2a543acf4e43c5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100444
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Tomaž Vajngerl 

diff --git a/include/tools/json_writer.hxx b/include/tools/json_writer.hxx
index 588577d41303..bf07aa0aaa35 100644
--- a/include/tools/json_writer.hxx
+++ b/include/tools/json_writer.hxx
@@ -13,6 +13,11 @@
 #include 
 #include 
 
+namespace rtl
+{
+class OStringBuffer;
+}
+
 /** Simple JSON encoder designed specifically for LibreOfficeKit purposes.
  *
  * (1) Minimal allocations/re-allocations/copying
@@ -22,10 +27,14 @@
 namespace tools
 {
 class ScopedJsonWriterNode;
+class ScopedJsonWriterArray;
+class ScopedJsonWriterStruct;
 
 class TOOLS_DLLPUBLIC JsonWriter
 {
 friend class ScopedJsonWriterNode;
+friend class ScopedJsonWriterArray;
+friend class ScopedJsonWriterStruct;
 
 int mSpaceAllocated;
 char* mpBuffer;
@@ -38,33 +47,36 @@ public:
 ~JsonWriter();
 
 [[nodiscard]] ScopedJsonWriterNode startNode(const char*);
+[[nodiscard]] ScopedJsonWriterArray startArray(const char*);
+[[nodiscard]] ScopedJsonWriterStruct startStruct();
 
 void put(const char* pPropName, const OUString& rPropValue);
 void put(const char* pPropName, const OString& rPropValue);
 void put(const char* pPropName, const char* pPropVal);
 void put(const char*, int);
 
-/** Hands ownership of the the underlying storage buffer to the caller,
+/// This assumes that this data belongs at this point in the stream, and 
is valid, and properly encoded
+void putRaw(const rtl::OStringBuffer&);
+
+/** Hands ownership of the underlying storage buffer to the caller,
  * after this no more document modifications may be written. */
 char* extractData();
 OString extractAsOString();
 
 private:
 void endNode();
+void endArray();
+void endStruct();
 void addCommaBeforeField();
+void reallocBuffer(int noMoreBytesRequired);
 
+// this part inline to speed up the fast path
 inline void ensureSpace(int noMoreBytesRequired)
 {
+assert(mpBuffer && "already extracted data");
 int currentUsed = mPos - mpBuffer;
 if (currentUsed + noMoreBytesRequired >= mSpaceAllocated)
-{
-auto newSize = std::max(mSpaceAllocated * 2, (currentUsed + 
noMoreBytesRequired) * 2);
-char* pNew = static_cast(malloc(newSize));
-memcpy(pNew, mpBuffer, currentUsed);
-free(mpBuffer);
-mpBuffer = pNew;
-mPos = mpBuffer;
-}
+reallocBuffer(noMoreBytesRequired);
 }
 };
 
@@ -85,5 +97,41 @@ class ScopedJsonWriterNode
 public:
 ~ScopedJsonWriterNode() { mrWriter.endNode(); }
 };
+
+/**
+ * Auto-closes the node.
+ */
+class ScopedJsonWriterArray
+{
+friend class JsonWriter;
+
+JsonWriter& mrWriter;
+
+ScopedJsonWriterArray(JsonWriter& rWriter)
+: mrWriter(rWriter)
+{
+}
+
+public:
+~ScopedJsonWriterArray() { mrWriter.endArray(); }
+};
+
+/**
+ * Auto-closes the node.
+ */
+class ScopedJsonWriterStruct
+{
+friend class JsonWriter;
+
+JsonWriter& mrWriter;
+
+ScopedJsonWriterStruct(JsonWriter& rWriter)
+: mrWriter(rWriter)
+{
+}
+
+public:
+ 

[Libreoffice-commits] online.git: Branch 'distro/collabora/co-4-2' - loleaflet/src

2020-08-10 Thread Szymon Kłos (via logerrit)
 loleaflet/src/control/Control.Menubar.js |7 ++-
 1 file changed, 6 insertions(+), 1 deletion(-)

New commits:
commit 22649095ea1f0bace04c78766052915242097001
Author: Szymon Kłos 
AuthorDate: Thu Aug 6 11:42:35 2020 +0200
Commit: Jan Holesovsky 
CommitDate: Mon Aug 10 21:15:48 2020 +0200

Add missing table operations to menu in Impress

Change-Id: I1f644b0a69ee2eb17edf4c64f7de3e13b39e6192
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/100207
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Jan Holesovsky 

diff --git a/loleaflet/src/control/Control.Menubar.js 
b/loleaflet/src/control/Control.Menubar.js
index d888a0740..f08d5d966 100644
--- a/loleaflet/src/control/Control.Menubar.js
+++ b/loleaflet/src/control/Control.Menubar.js
@@ -338,7 +338,12 @@ L.Control.Menubar = L.Control.extend({
{uno: '.uno:InsertColumnsAfter'}]},
{name: _UNO('.uno:TableDeleteMenu', 
'text'/*HACK should be 'presentation', but not in xcu*/), type: 'menu', menu: [
{uno: '.uno:DeleteRows'},
-   {uno: '.uno:DeleteColumns'}]},
+   {uno: '.uno:DeleteColumns'},
+   {uno: '.uno:DeleteTable'}]},
+   {name: _UNO('.uno:TableSelectMenu', 'text'), 
type: 'menu', menu: [
+   {uno: '.uno:SelectTable'},
+   {uno: '.uno:EntireRow'},
+   {uno: '.uno:EntireColumn'}]},
{uno: '.uno:MergeCells'}]
},
{name: _UNO('.uno:SlideMenu', 'presentation'), type: 
'menu', menu: [
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-6.2' - include/svx officecfg/registry sd/sdi sd/source svx/sdi svx/source

2020-08-10 Thread Justin Luth (via logerrit)
 include/svx/svxids.hrc   |
3 +
 officecfg/registry/data/org/openoffice/Office/UI/DrawImpressCommands.xcu |
8 
 sd/sdi/tables.sdi|
5 ++
 sd/source/ui/table/tableobjectbar.cxx|
1 
 svx/sdi/svx.sdi  |   
17 ++
 svx/source/table/tablecontroller.cxx |   
16 -
 6 files changed, 47 insertions(+), 3 deletions(-)

New commits:
commit df4c1556478bf0cbe1f6c754e22d137a79a9b0d5
Author: Justin Luth 
AuthorDate: Fri Jul 3 11:45:24 2020 +0300
Commit: Jan Holesovsky 
CommitDate: Mon Aug 10 21:15:04 2020 +0200

tdf#100772 sd: add uno:DeleteTable to non-NB menus

Delete Row and Delete Column were there, but no option to delete table,
so that major omission was fixed for Draw and Impress.

The notebookbar looks very incomplete. Also, I didn't really
understand the pop-out menu configuration, and at least in the
one case there didn't seem to be a good space to add delete table.
So I left the notebookbar completely alone.

Change-Id: I5d6c98e3238bc545a02325edfd62f5d937ac6371
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/97821
Tested-by: Jenkins
Tested-by: Maxim Monastirsky 
Reviewed-by: Justin Luth 
Reviewed-by: Maxim Monastirsky 
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100205
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Jan Holesovsky 

diff --git a/include/svx/svxids.hrc b/include/svx/svxids.hrc
index a9960d216b4e..0ffe72e3bb30 100644
--- a/include/svx/svxids.hrc
+++ b/include/svx/svxids.hrc
@@ -985,7 +985,8 @@ class SfxStringItem;
 #define SID_EDIT_SIGNATURELINE  ( SID_SVX_START + 1174 
)
 #define SID_SIGN_SIGNATURELINE  ( SID_SVX_START + 1175 
)
 #define SID_MEASURE_DLG ( SID_SVX_START + 1176 
)
-
+// free slots - available for use
+#define SID_TABLE_DELETE_TABLE  ( SID_SVX_START + 1184 
)
 #define SID_TABLE_MINIMAL_COLUMN_WIDTH  ( SID_SVX_START + 1185 
)
 #define SID_TABLE_MINIMAL_ROW_HEIGHT( SID_SVX_START + 1186 
)
 #define SID_TABLE_OPTIMAL_COLUMN_WIDTH  ( SID_SVX_START + 1187 
)
diff --git 
a/officecfg/registry/data/org/openoffice/Office/UI/DrawImpressCommands.xcu 
b/officecfg/registry/data/org/openoffice/Office/UI/DrawImpressCommands.xcu
index 36a275b6268c..6390f895c2d9 100644
--- a/officecfg/registry/data/org/openoffice/Office/UI/DrawImpressCommands.xcu
+++ b/officecfg/registry/data/org/openoffice/Office/UI/DrawImpressCommands.xcu
@@ -1986,6 +1986,14 @@
   1
 
   
+  
+
+  Delete Table
+
+
+  1
+
+  
   
 
   Select Table
diff --git a/sd/sdi/tables.sdi b/sd/sdi/tables.sdi
index a5918af490db..9943ce719e29 100644
--- a/sd/sdi/tables.sdi
+++ b/sd/sdi/tables.sdi
@@ -113,6 +113,11 @@ shell TableObjectBar
 ExecMethod = Execute;
 StateMethod = GetState;
 ]
+SID_TABLE_DELETE_TABLE
+[
+ExecMethod = Execute;
+StateMethod = GetState;
+]
 SID_TABLE_SELECT_ALL
 [
 ExecMethod = Execute;
diff --git a/sd/source/ui/table/tableobjectbar.cxx 
b/sd/source/ui/table/tableobjectbar.cxx
index 9b01ee63a290..54687d02d174 100644
--- a/sd/source/ui/table/tableobjectbar.cxx
+++ b/sd/source/ui/table/tableobjectbar.cxx
@@ -191,6 +191,7 @@ void TableObjectBar::Execute( SfxRequest& rReq )
 case SID_OPTIMIZE_TABLE:
 case SID_TABLE_DELETE_ROW:
 case SID_TABLE_DELETE_COL:
+case SID_TABLE_DELETE_TABLE:
 case SID_FORMAT_TABLE_DLG:
 case SID_TABLE_INSERT_ROW:
 case SID_TABLE_INSERT_COL:
diff --git a/svx/sdi/svx.sdi b/svx/sdi/svx.sdi
index f077e5abaddb..36fe1d2e528d 100644
--- a/svx/sdi/svx.sdi
+++ b/svx/sdi/svx.sdi
@@ -11108,6 +11108,23 @@ SfxVoidItem DeleteColumns SID_TABLE_DELETE_COL
 GroupId = SfxGroupId::Table;
 ]
 
+SfxVoidItem DeleteTable SID_TABLE_DELETE_TABLE
+()
+[
+AutoUpdate = FALSE,
+FastCall = TRUE,
+ReadOnlyDoc = FALSE,
+Toggle = FALSE,
+Container = FALSE,
+RecordAbsolute = FALSE,
+RecordPerSet;
+
+AccelConfig = TRUE,
+MenuConfig = TRUE,
+ToolBoxConfig = TRUE,
+GroupId = SfxGroupId::Table;
+]
+
 SfxVoidItem SelectTable SID_TABLE_SELECT_ALL
 ()
 [
diff --git a/svx/source/table/tablecontroller.cxx 
b/svx/source/table/tablecontroller.cxx
index d119367632a6..43c74993961b 100644
--- a/svx/source/table/tablecontroller.cxx
+++ b/svx/source/table/tablecontroller.cxx
@@ -468,6 +468,10 @@ void SvxTableController::GetState( SfxItemSet& rSet )
 if( !mxTable.is() || !hasSelectedCells() || 
(!comphelper::LibreOfficeKit::isActive() && mxTable->getColumnCount()

[Libreoffice-commits] online.git: Branch 'distro/collabora/co-4-2' - loleaflet/css loleaflet/src

2020-08-10 Thread Szymon Kłos (via logerrit)
 loleaflet/css/mobilewizard.css   |4 +
 loleaflet/src/control/Control.JSDialogBuilder.js |   64 ++-
 2 files changed, 66 insertions(+), 2 deletions(-)

New commits:
commit 402a42d3db9e5d3b1836c5fd5010c5a5f7054b02
Author: Szymon Kłos 
AuthorDate: Mon Aug 10 12:58:53 2020 +0200
Commit: Jan Holesovsky 
CommitDate: Mon Aug 10 21:12:12 2020 +0200

mobile-wizard: update selected cell border style

Change-Id: Id2f98c0824e589d63d658f00b384fa0abfb48f2f
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/100425
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Jan Holesovsky 

diff --git a/loleaflet/css/mobilewizard.css b/loleaflet/css/mobilewizard.css
index 0d9ef8a62..8cba28d8e 100644
--- a/loleaflet/css/mobilewizard.css
+++ b/loleaflet/css/mobilewizard.css
@@ -299,6 +299,10 @@ p.mobile-wizard.ui-combobox-text.selected {
-webkit-box-shadow: inset 0px 0px 0px 2px #fff;
box-shadow: inset 0px 0px 0px 2px #fff;
 }
+#mobile-wizard-content > #ScCellAppearancePropertyPanel > 
.ui-content.level-0.mobile-wizard .borderbutton.selected {
+   -webkit-box-shadow: inset 0px 0px 0px 2px #0867af !important;
+   box-shadow: inset 0px 0px 0px 2px #0867af !important;
+}
 .ui-header.mobile-wizard {
width: 100%;
height: 56px !important;
diff --git a/loleaflet/src/control/Control.JSDialogBuilder.js 
b/loleaflet/src/control/Control.JSDialogBuilder.js
index 101e3d3bb..6a2f8e0aa 100644
--- a/loleaflet/src/control/Control.JSDialogBuilder.js
+++ b/loleaflet/src/control/Control.JSDialogBuilder.js
@@ -1846,7 +1846,48 @@ L.Control.JSDialogBuilder = L.Control.extend({
return false;
},
 
-   _borderControlItem: function(parentContainer, data, builder, i) {
+   _getCurrentBorderNumber: function(builder) {
+   var outer = 
builder.map['stateChangeHandler'].getItemValue('.uno:BorderOuter');
+   var inner = 
builder.map['stateChangeHandler'].getItemValue('.uno:BorderInner');
+
+   if (!outer || !inner)
+   return 0;
+
+   var left = outer.left === 'true';
+   var right = outer.right === 'true';
+   var bottom = outer.bottom === 'true';
+   var top = outer.top === 'true';
+   var horiz = inner.horizontal === 'true';
+   var vert = inner.vertical === 'true';
+
+   if (left && !right && !bottom && !top && !horiz && !vert) {
+   return 2;
+   } else if (!left && right && !bottom && !top && !horiz && 
!vert) {
+   return 3;
+   } else if (left && right && !bottom && !top && !horiz && !vert) 
{
+   return 4;
+   } else if (!left && !right && !bottom && top && !horiz && 
!vert) {
+   return 5;
+   } else if (!left && !right && bottom && !top && !horiz && 
!vert) {
+   return 6;
+   } else if (!left && !right && bottom && top && !horiz && !vert) 
{
+   return 7;
+   } else if (left && right && bottom && top && !horiz && !vert) {
+   return 8;
+   } else if (!left && !right && bottom && top && horiz && !vert) {
+   return 9;
+   } else if (left && right && bottom && top && horiz && !vert) {
+   return 10;
+   } else if (left && right && bottom && top && !horiz && vert) {
+   return 11;
+   } else if (left && right && bottom && top && horiz && vert) {
+   return 12;
+   }
+
+   return 1;
+   },
+
+   _borderControlItem: function(parentContainer, data, builder, i, 
selected) {
var button = null;
 
var div = this._createIdentifiable('div', 'ui-content unospan', 
parentContainer, data);
@@ -1856,6 +1897,9 @@ L.Control.JSDialogBuilder = L.Control.extend({
button.src = L.LOUtil.getImageURL('fr0' + i + '.svg');
button.id = buttonId;
 
+   if (selected)
+   $(button).addClass('selected');
+
$(div).click(function () {
var color = 0;
// Find our associated color picker
@@ -1870,8 +1914,24 @@ L.Control.JSDialogBuilder = L.Control.extend({
var bordercontrollabel = L.DomUtil.create('p', 
builder.options.cssClass + ' ui-text', parentContainer);
bordercontrollabel.innerHTML = _('Cell borders');
bordercontrollabel.id = data.id + 'label';
+   var current = builder._getCurrentBorderNumber(builder);
for (var i = 1; i < 13; ++i)
-   builder._borderControlItem(parentContainer, data, 
builder, i);
+   builder._borderControlItem(parentContainer, data, 
builder, i, i

[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-6.2' - desktop/source editeng/source include/editeng

2020-08-10 Thread Szymon Kłos (via logerrit)
 desktop/source/lib/init.cxx   |4 +++-
 editeng/source/items/frmitems.cxx |   31 +++
 include/editeng/boxitem.hxx   |4 
 3 files changed, 38 insertions(+), 1 deletion(-)

New commits:
commit 035d850179688137dcf145cf3e779b6e05779239
Author: Szymon Kłos 
AuthorDate: Mon Aug 10 12:57:00 2020 +0200
Commit: Jan Holesovsky 
CommitDate: Mon Aug 10 21:09:01 2020 +0200

lok: send cell border state updates

Change-Id: I400ee3cb9f0a98804d98e25d0164fa5148b79191
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100424
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Jan Holesovsky 

diff --git a/desktop/source/lib/init.cxx b/desktop/source/lib/init.cxx
index 7e6b71c77c1e..d33454297cf8 100644
--- a/desktop/source/lib/init.cxx
+++ b/desktop/source/lib/init.cxx
@@ -2756,7 +2756,9 @@ static void doc_iniUnoCommands ()
 OUString(".uno:ParaspaceIncrease"),
 OUString(".uno:ParaspaceDecrease"),
 OUString(".uno:AcceptTrackedChange"),
-OUString(".uno:RejectTrackedChange")
+OUString(".uno:RejectTrackedChange"),
+OUString(".uno:BorderInner"),
+OUString(".uno:BorderOuter")
 };
 
 util::URL aCommandURL;
diff --git a/editeng/source/items/frmitems.cxx 
b/editeng/source/items/frmitems.cxx
index 0e1c22731ad7..d21e3ce75591 100644
--- a/editeng/source/items/frmitems.cxx
+++ b/editeng/source/items/frmitems.cxx
@@ -1440,6 +1440,23 @@ SvxBoxItem& SvxBoxItem::operator=( const SvxBoxItem& 
rBox )
 }
 
 
+boost::property_tree::ptree SvxBoxItem::dumpAsJSON() const
+{
+boost::property_tree::ptree aTree;
+
+boost::property_tree::ptree aState;
+aState.put("top", GetTop() && !GetTop()->isEmpty());
+aState.put("bottom", GetBottom() && !GetBottom()->isEmpty());
+aState.put("left", GetLeft() && !GetLeft()->isEmpty());
+aState.put("right", GetRight() && !GetRight()->isEmpty());
+
+aTree.push_back(std::make_pair("state", aState));
+aTree.put("commandName", ".uno:BorderOuter");
+
+return aTree;
+}
+
+
 static bool CmpBrdLn( const std::unique_ptr & pBrd1, const 
SvxBorderLine* pBrd2 )
 {
 if( pBrd1.get() == pBrd2 )
@@ -2318,6 +2335,20 @@ SvxBoxInfoItem::~SvxBoxInfoItem()
 {
 }
 
+boost::property_tree::ptree SvxBoxInfoItem::dumpAsJSON() const
+{
+boost::property_tree::ptree aTree;
+
+boost::property_tree::ptree aState;
+aState.put("vertical", GetVert() && !GetVert()->isEmpty());
+aState.put("horizontal", GetHori() && !GetHori()->isEmpty());
+
+aTree.push_back(std::make_pair("state", aState));
+aTree.put("commandName", ".uno:BorderInner");
+
+return aTree;
+}
+
 SvxBoxInfoItem &SvxBoxInfoItem::operator=( const SvxBoxInfoItem& rCpy )
 {
 if (this != &rCpy)
diff --git a/include/editeng/boxitem.hxx b/include/editeng/boxitem.hxx
index 103967578f20..8cf1c2d60545 100644
--- a/include/editeng/boxitem.hxx
+++ b/include/editeng/boxitem.hxx
@@ -118,6 +118,8 @@ public:
 static css::table::BorderLine2 SvxLineToLine( const 
editeng::SvxBorderLine* pLine, bool bConvert );
 static bool LineToSvxLine(const css::table::BorderLine& rLine, 
editeng::SvxBorderLine& rSvxLine, bool bConvert);
 static bool LineToSvxLine(const css::table::BorderLine2& rLine, 
editeng::SvxBorderLine& rSvxLine, bool bConvert);
+
+virtual boost::property_tree::ptree dumpAsJSON() const override;
 };
 
 inline void SvxBoxItem::SetAllDistances(sal_uInt16 const nNew)
@@ -234,6 +236,8 @@ public:
 nValidFlags &= ~nValid;
 }
 voidResetFlags();
+
+virtual boost::property_tree::ptree dumpAsJSON() const override;
 };
 
 namespace editeng
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: chart2/uiconfig

2020-08-10 Thread Caolán McNamara (via logerrit)
 chart2/uiconfig/ui/tp_3D_SceneAppearance.ui |   33 ++--
 1 file changed, 8 insertions(+), 25 deletions(-)

New commits:
commit 386add495b3d8b7816a521bb28c5f48fc243bc99
Author: Caolán McNamara 
AuthorDate: Mon Aug 10 15:37:29 2020 +0100
Commit: Caolán McNamara 
CommitDate: Mon Aug 10 20:17:37 2020 +0200

tdf#135568 an unexpected model for a GtkComboBox

Change-Id: I5d4ec47fe6dcce4c4f0891fae22ed4b44796a270
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100437
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 

diff --git a/chart2/uiconfig/ui/tp_3D_SceneAppearance.ui 
b/chart2/uiconfig/ui/tp_3D_SceneAppearance.ui
index a214ebe21247..6cf5681f3771 100644
--- a/chart2/uiconfig/ui/tp_3D_SceneAppearance.ui
+++ b/chart2/uiconfig/ui/tp_3D_SceneAppearance.ui
@@ -1,28 +1,7 @@
 
+
 
   
-  
-
-  
-  
-  
-  
-
-
-  
-Simple
-0
-  
-  
-Realistic
-1
-  
-  
-Custom
-2
-  
-
-  
   
 True
 False
@@ -38,10 +17,10 @@
   
 True
 False
-0
 Sche_me
 True
 LB_SCHEME
+0
   
   
 False
@@ -50,10 +29,14 @@
   
 
 
-  
+  
 True
 False
-liststoreSCHEME
+
+  Simple
+  Realistic
+  Custom
+
   
   
 False
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: vcl/source

2020-08-10 Thread Caolán McNamara (via logerrit)
 vcl/source/window/builder.cxx |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit f9eea764389eae7e6b77076c1a80a8e6f280f9c4
Author: Caolán McNamara 
AuthorDate: Mon Aug 10 15:55:50 2020 +0100
Commit: Caolán McNamara 
CommitDate: Mon Aug 10 18:35:12 2020 +0200

tdf#135495 builder file format has annoyingly escaped into user config

Change-Id: I38882e2c6e1815ac5fd747642c8934e3e4ca3ffb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100438
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 

diff --git a/vcl/source/window/builder.cxx b/vcl/source/window/builder.cxx
index 2cdc33966ff6..2d27f27e9f1a 100644
--- a/vcl/source/window/builder.cxx
+++ b/vcl/source/window/builder.cxx
@@ -1910,8 +1910,9 @@ VclPtr VclBuilder::makeObject(vcl::Window 
*pParent, const OString &
 xWindow = xListBox;
 }
 }
-else if (name == "VclOptionalBox")
+else if (name == "VclOptionalBox" || name == "sfxlo-OptionalBox")
 {
+// tdf#135495 fallback sfxlo-OptionalBox to VclOptionalBox as a stopgap
 xWindow = VclPtr::Create(pParent);
 }
 else if (name == "GtkIconView")
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: solenv/sanitizers

2020-08-10 Thread Caolán McNamara (via logerrit)
 solenv/sanitizers/ui/modules/schart.suppr |1 -
 1 file changed, 1 deletion(-)

New commits:
commit 54ffd565154bd1ad890bbb8e8a252ba14d85d4b2
Author: Caolán McNamara 
AuthorDate: Mon Aug 10 15:33:30 2020 +0100
Commit: Caolán McNamara 
CommitDate: Mon Aug 10 17:52:39 2020 +0200

unused suppression

Change-Id: I6c28703a577c63a410cadbd4f08fd9c03bc3229f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100436
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 

diff --git a/solenv/sanitizers/ui/modules/schart.suppr 
b/solenv/sanitizers/ui/modules/schart.suppr
index f68f0c120cb4..03a355609666 100644
--- a/solenv/sanitizers/ui/modules/schart.suppr
+++ b/solenv/sanitizers/ui/modules/schart.suppr
@@ -9,7 +9,6 @@ 
chart2/uiconfig/ui/dlg_InsertErrorBars.ui://GtkEntry[@id='ED_RANGE_NEGATIVE'] no
 
chart2/uiconfig/ui/dlg_InsertErrorBars.ui://GtkLabel[@id='STR_DATA_SELECT_RANGE_FOR_POSITIVE_ERRORBARS']
 orphan-label
 
chart2/uiconfig/ui/dlg_InsertErrorBars.ui://GtkLabel[@id='STR_DATA_SELECT_RANGE_FOR_NEGATIVE_ERRORBARS']
 orphan-label
 
chart2/uiconfig/ui/dlg_InsertErrorBars.ui://GtkLabel[@id='STR_CONTROLTEXT_ERROR_BARS_FROM_DATA']
 orphan-label
-chart2/uiconfig/ui/sidebarelements.ui://GtkCheckButton[@id='checkbutton_legend']
 button-no-label
 chart2/uiconfig/ui/sidebarelements.ui://GtkLabel[@id='placement_label'] 
orphan-label
 
chart2/uiconfig/ui/sidebarelements.ui://GtkComboBoxText[@id='comboboxtext_legend']
 no-labelled-by
 chart2/uiconfig/ui/sidebarelements.ui://GtkLabel[@id='text_title'] orphan-label
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: bridges/Library_cpp_uno.mk bridges/source cppuhelper/source sal/osl

2020-08-10 Thread Tor Lillqvist (via logerrit)
 bridges/Library_cpp_uno.mk|   16 +-
 bridges/source/cpp_uno/gcc3_linux_aarch64/abi.cxx |   62 
+-
 bridges/source/cpp_uno/gcc3_linux_aarch64/abi.hxx |3 
 bridges/source/cpp_uno/gcc3_linux_aarch64/callvirtualfunction.cxx |2 
 bridges/source/cpp_uno/gcc3_linux_aarch64/cpp2uno.cxx |   10 +
 bridges/source/cpp_uno/gcc3_linux_aarch64/vtableslotcall.s|   11 +
 bridges/source/cpp_uno/shared/vtablefactory.cxx   |   11 +
 cppuhelper/source/exc_thrower.cxx |6 
 sal/osl/unx/salinit.cxx   |8 -
 9 files changed, 115 insertions(+), 14 deletions(-)

New commits:
commit e2bd5afd726abd5df438b6b821416bd7cf496e4d
Author: Tor Lillqvist 
AuthorDate: Sun Aug 9 10:51:22 2020 +0100
Commit: Tor Lillqvist 
CommitDate: Mon Aug 10 17:51:27 2020 +0200

Make the C++/UNO bridge work to some extent on macOS on arm64

Use the same code as for Linux on aarch64, with minor
additional hacks. But that will not actually work in all cases, as
there are slight differences in the ABI. See

https://developer.apple.com/library/archive/documentation/Xcode/Conceptual/iPhoneOSABIReference/Articles/ARM64FunctionCallingConventions.html

Thus we can drop the use of the lo_mobile_throwException() hack that
was very temporarily used.

The run-time code generation requires use of a new API on macOS to
work: See the use of pthread_jit_write_protect_np() in
bridges/source/cpp_uno/shared/vtablefactory.cxx.

For some reason, with the Xcode 12 betas, when compiling for
arm64-apple-macos, the symbols for the type_infos for the UNO
exception types (_ZTIN3com3sun4star3uno16RuntimeExceptionE etc) end up
as "weak private external" in the object file, as displayed by "nm -f
darwin". We try to look them up with dlsym(), but that then fails. So
use a gross hack: Introduce separate real variables that point to
these typeinfos, and look up and dereference them instead. If this
hack ends up needing to be permanent, instead of having a manually
edited set of such pointer variables, we should teach codemaker to
generate corresponding functions, and look up and invoke them to get
the std::type_info pointer.

When compiling for x86_64-apple-macos, the type_info symbols end up as
"weak external" which is fine.

With this, LibreOffice starts and seems to work to some extent, and
many unit tests succeed.

Change-Id: I05f46a122a51ade1ac7dccd57cb90e594547740e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100408
Tested-by: Jenkins
Reviewed-by: Tor Lillqvist 

diff --git a/bridges/Library_cpp_uno.mk b/bridges/Library_cpp_uno.mk
index cf6fb489c3a1..9adc891f4e3d 100644
--- a/bridges/Library_cpp_uno.mk
+++ b/bridges/Library_cpp_uno.mk
@@ -23,7 +23,7 @@ endif
 
 else ifeq ($(CPUNAME),AARCH64)
 
-ifneq ($(filter ANDROID DRAGONFLY FREEBSD LINUX NETBSD OPENBSD,$(OS)),)
+ifneq ($(filter ANDROID DRAGONFLY FREEBSD LINUX MACOSX NETBSD OPENBSD,$(OS)),)
 bridges_SELECTED_BRIDGE := gcc3_linux_aarch64
 bridge_asm_objects := vtableslotcall
 bridge_exception_objects := abi cpp2uno uno2cpp
@@ -32,10 +32,8 @@ $(eval $(call 
gb_Library_add_exception_objects,$(gb_CPPU_ENV)_uno, \
 bridges/source/cpp_uno/$(bridges_SELECTED_BRIDGE)/callvirtualfunction, \
 $(if $(HAVE_GCC_STACK_CLASH_PROTECTION),-fno-stack-clash-protection) \
 ))
-else ifneq ($(filter iOS MACOSX,$(OS)),)
-# For now, use the same bridge for macOS on arm64 as for iOS. But we
-# will eventually obviously want one that does generate code
-# dynamically on macOS.
+
+else ifeq ($(OS),iOS)
 bridges_SELECTED_BRIDGE := gcc3_ios
 bridge_noopt_objects := cpp2uno except uno2cpp
 bridge_asm_objects := ios64_helper
@@ -184,6 +182,14 @@ $(eval $(call 
gb_Library_use_internal_comprehensive_api,$(gb_CPPU_ENV)_uno,\
udkapi \
 ))
 
+ifeq ($(OS),MACOSX)
+ifeq ($(CPUNAME),AARCH64)
+$(eval $(call gb_Library_use_internal_comprehensive_api,$(gb_CPPU_ENV)_uno,\
+   offapi \
+))
+endif
+endif
+
 $(eval $(call gb_Library_set_include,$(gb_CPPU_ENV)_uno,\
-I$(SRCDIR)/bridges/inc \
$$(INCLUDE) \
diff --git a/bridges/source/cpp_uno/gcc3_linux_aarch64/abi.cxx 
b/bridges/source/cpp_uno/gcc3_linux_aarch64/abi.cxx
index 4a5a1d1b662d..b0a35996784e 100644
--- a/bridges/source/cpp_uno/gcc3_linux_aarch64/abi.cxx
+++ b/bridges/source/cpp_uno/gcc3_linux_aarch64/abi.cxx
@@ -1,4 +1,4 @@
-/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4; 
fill-column: 100 -*- */
 /*
  * This file is part of the LibreOffice project.
  *
@@ -31,6 +31,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -106,6 +107,31 @@ std::type_info * Rtti::getRtti(typelib_TypeDescription 

[Libreoffice-commits] core.git: Branch 'libreoffice-7-0' - chart2/uiconfig

2020-08-10 Thread Caolán McNamara (via logerrit)
 chart2/uiconfig/ui/tp_3D_SceneAppearance.ui |   33 ++--
 1 file changed, 8 insertions(+), 25 deletions(-)

New commits:
commit e61e4ec53ad54a17a1726220f1dbe12ee3307e7b
Author: Caolán McNamara 
AuthorDate: Mon Aug 10 15:37:29 2020 +0100
Commit: Caolán McNamara 
CommitDate: Mon Aug 10 17:40:28 2020 +0200

tdf#135568 an unexpected model for a GtkComboBox

Change-Id: I5d4ec47fe6dcce4c4f0891fae22ed4b44796a270
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100360
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 

diff --git a/chart2/uiconfig/ui/tp_3D_SceneAppearance.ui 
b/chart2/uiconfig/ui/tp_3D_SceneAppearance.ui
index a214ebe21247..6cf5681f3771 100644
--- a/chart2/uiconfig/ui/tp_3D_SceneAppearance.ui
+++ b/chart2/uiconfig/ui/tp_3D_SceneAppearance.ui
@@ -1,28 +1,7 @@
 
+
 
   
-  
-
-  
-  
-  
-  
-
-
-  
-Simple
-0
-  
-  
-Realistic
-1
-  
-  
-Custom
-2
-  
-
-  
   
 True
 False
@@ -38,10 +17,10 @@
   
 True
 False
-0
 Sche_me
 True
 LB_SCHEME
+0
   
   
 False
@@ -50,10 +29,14 @@
   
 
 
-  
+  
 True
 False
-liststoreSCHEME
+
+  Simple
+  Realistic
+  Custom
+
   
   
 False
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: wsd/TileCache.cpp wsd/TileDesc.hpp

2020-08-10 Thread Ashod Nakashian (via logerrit)
 wsd/TileCache.cpp |2 +-
 wsd/TileDesc.hpp  |   12 ++--
 2 files changed, 7 insertions(+), 7 deletions(-)

New commits:
commit 999df3fffb9c100a86acd200fdc9066203a13553
Author: Ashod Nakashian 
AuthorDate: Sun Aug 9 17:55:56 2020 -0400
Commit: Michael Meeks 
CommitDate: Mon Aug 10 16:00:27 2020 +0200

wsd: hashmaps have better data locality

They are especially efficient for small lookups.

Change-Id: Ia005f40127cf222debe185515fc45cd92b8ae752
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/100413
Tested-by: Jenkins CollaboraOffice 
Tested-by: Jenkins
Reviewed-by: Michael Meeks 

diff --git a/wsd/TileCache.cpp b/wsd/TileCache.cpp
index 982cd985b..d36ca1aa6 100644
--- a/wsd/TileCache.cpp
+++ b/wsd/TileCache.cpp
@@ -552,7 +552,7 @@ void TileCache::saveDataToCache(const TileDesc &desc, const 
char *data, const si
 
 TileCache::Tile tile = std::make_shared>(size);
 std::memcpy(tile->data(), data, size);
-auto res = _cache.insert(std::make_pair(desc, tile));
+auto res = _cache.emplace(desc, tile);
 if (!res.second)
 {
 _cacheSize -= itemCacheSize(res.first->second);
diff --git a/wsd/TileDesc.hpp b/wsd/TileDesc.hpp
index 28ecc4256..e60984359 100644
--- a/wsd/TileDesc.hpp
+++ b/wsd/TileDesc.hpp
@@ -10,7 +10,7 @@
 #pragma once
 
 #include 
-#include 
+#include 
 #include 
 #include 
 
@@ -24,7 +24,7 @@ using TileBinaryHash = uint64_t;
 
 /// Tile Descriptor
 /// Represents a tile's coordinates and dimensions.
-class TileDesc
+class TileDesc final
 {
 public:
 TileDesc(int normalizedViewId, int part, int width, int height, int 
tilePosX, int tilePosY, int tileWidth,
@@ -206,7 +206,7 @@ public:
 {
 // We don't expect undocumented fields and
 // assume all values to be int.
-std::map pairs;
+std::unordered_map pairs(16);
 
 // Optional.
 pairs["ver"] = -1;
@@ -261,7 +261,7 @@ public:
 return tileID.str();
 }
 
-protected:
+private:
 int _normalizedViewId;
 int _part;
 int _width;
@@ -281,7 +281,7 @@ protected:
 /// One or more tile header.
 /// Used to request the rendering of multiple
 /// tiles as well as the header of the response.
-class TileCombined
+class TileCombined final
 {
 private:
 TileCombined(int normalizedViewId, int part, int width, int height,
@@ -465,7 +465,7 @@ public:
 {
 // We don't expect undocumented fields and
 // assume all values to be int.
-std::map pairs;
+std::unordered_map pairs(16);
 
 std::string tilePositionsX;
 std::string tilePositionsY;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: loleaflet/admin

2020-08-10 Thread gokaysatir (via logerrit)
 loleaflet/admin/admin.strings.js   |1 +
 loleaflet/admin/src/AdminSocketOverview.js |1 +
 2 files changed, 2 insertions(+)

New commits:
commit 758b82ee19cf7cc63f3e28aae2d377fe8b18051a
Author: gokaysatir 
AuthorDate: Mon Aug 10 10:43:03 2020 +0300
Commit: Michael Meeks 
CommitDate: Mon Aug 10 15:44:34 2020 +0200

Admin Console: Add tooltip to kill session button.

Change-Id: I5a6117741c9635c52dc3768cebec66214e1fcbe7
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/100415
Tested-by: Jenkins
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Michael Meeks 

diff --git a/loleaflet/admin/admin.strings.js b/loleaflet/admin/admin.strings.js
index dc48930d3..9ac1e2fb1 100644
--- a/loleaflet/admin/admin.strings.js
+++ b/loleaflet/admin/admin.strings.js
@@ -36,6 +36,7 @@ l10nstrings.strMemoryStatsCachesize = _('Cache size of memory 
statistics');
 l10nstrings.strMemoryStatsInterval = _('Time interval of memory statistics (in 
ms)');
 l10nstrings.strCpuStatsCachesize = _('Cache size of CPU statistics');
 l10nstrings.strCpuStatsInterval = _('Time interval of CPU statistics (in ms)');
+l10nstrings.strKillSessionToolTip = _('Kill session.');
 l10nstrings.strLimitVirtMemMb = _('Maximum Document process virtual memory (in 
MB) - reduce only');
 l10nstrings.strLimitStackMemKb = _('Maximum Document process stack memory (in 
KB) - reduce only');
 l10nstrings.strLimitFileSizeMb = _('Maximum file size allowed to write to disk 
(in MB) - reduce only');
diff --git a/loleaflet/admin/src/AdminSocketOverview.js 
b/loleaflet/admin/src/AdminSocketOverview.js
index abca74c1d..cb94193f4 100644
--- a/loleaflet/admin/src/AdminSocketOverview.js
+++ b/loleaflet/admin/src/AdminSocketOverview.js
@@ -88,6 +88,7 @@ function upsertDocsTable(doc, sName, socket) {
 
var sessionCloseCell = document.createElement('td'); // This cell will 
open "Do you want to kill this session?" dialog.
sessionCloseCell.innerText = '✖';
+   sessionCloseCell.title = _('Kill session.');
sessionCloseCell.className = 'has-text-centered';
sessionCloseCell.style.cursor = 'pointer';
if (add === true) { row.appendChild(sessionCloseCell); } else { 
row.cells[0] = sessionCloseCell; }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: 2 commits - cypress_test/Makefile.am cypress_test/package.json

2020-08-10 Thread Tamás Zolnai (via logerrit)
 cypress_test/Makefile.am  |   10 +++---
 cypress_test/package.json |2 +-
 2 files changed, 4 insertions(+), 8 deletions(-)

New commits:
commit cfc62f864dfbea8e39ac053f37489b3e745485a2
Author: Tamás Zolnai 
AuthorDate: Mon Aug 10 14:51:53 2020 +0200
Commit: Tamás Zolnai 
CommitDate: Mon Aug 10 15:37:46 2020 +0200

cypress: use actual LOOLWSD version hash.

The related issue was fixed upstream.

Change-Id: I8b1191d3c0d5543233ace9322a7d768b3424d62d
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/100435
Tested-by: Jenkins
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Tamás Zolnai 

diff --git a/cypress_test/Makefile.am b/cypress_test/Makefile.am
index 0c343e9eb..230f8dc7d 100644
--- a/cypress_test/Makefile.am
+++ b/cypress_test/Makefile.am
@@ -52,10 +52,6 @@ FILTER_DEBUG=cypress:electron,cypress:launcher
 export DEBUG=$(if $(ENABLE_LOGGING),$(FILTER_DEBUG),)
 endif
 
-# We use a hard coded hash here, because of this issue:
-# https://github.com/cypress-io/cypress/issues/6891
-WSD_VERSION_HASH := ""
-
 if HAVE_LO_PATH
 
 MOBILE_TEST_FILES=$(subst $(MOBILE_TEST_FOLDER)/,,$(wildcard 
$(MOBILE_TEST_FOLDER)/*_spec.js) $(wildcard $(MOBILE_TEST_FOLDER)/*/*_spec.js))
@@ -249,19 +245,19 @@ DESKTOP_CONFIG = \

integrationFolder=$(DESKTOP_TEST_FOLDER),supportFile=$(SUPPORT_FILE),userAgent=$(DESKTOP_USER_AGENT)
 
 DESKTOP_ENV = \
-   
DATA_FOLDER=$(DESKTOP_DATA_FOLDER),WORKDIR=$(DESKTOP_WORKDIR),WSD_VERSION_HASH=$(WSD_VERSION_HASH),SERVER_PORT=$(FREE_PORT),LO_CORE_VERSION="$(CORE_VERSION)"
+   
DATA_FOLDER=$(DESKTOP_DATA_FOLDER),WORKDIR=$(DESKTOP_WORKDIR),WSD_VERSION_HASH=$(LOOLWSD_VERSION_HASH),SERVER_PORT=$(FREE_PORT),LO_CORE_VERSION="$(CORE_VERSION)"
 
 MOBILE_CONFIG = \

integrationFolder=$(MOBILE_TEST_FOLDER),supportFile=$(SUPPORT_FILE),userAgent=$(MOBILE_USER_AGENT)
 
 MOBILE_ENV = \
-   
DATA_FOLDER=$(MOBILE_DATA_FOLDER),WORKDIR=$(MOBILE_WORKDIR),WSD_VERSION_HASH=$(WSD_VERSION_HASH),SERVER_PORT=$(FREE_PORT),LO_CORE_VERSION="$(CORE_VERSION)"
+   
DATA_FOLDER=$(MOBILE_DATA_FOLDER),WORKDIR=$(MOBILE_WORKDIR),WSD_VERSION_HASH=$(LOOLWSD_VERSION_HASH),SERVER_PORT=$(FREE_PORT),LO_CORE_VERSION="$(CORE_VERSION)"
 
 MULTIUSER_CONFIG = \

integrationFolder=$(MULTIUSER_TEST_FOLDER),supportFile=$(SUPPORT_FILE),userAgent=$(DESKTOP_USER_AGENT),defaultCommandTimeout=3
 
 MULTIUSER_ENV = \
-   
DATA_FOLDER=$(MULTIUSER_DATA_FOLDER),WORKDIR=$(MULTIUSER_WORKDIR),WSD_VERSION_HASH=$(WSD_VERSION_HASH),SERVER_PORT=$(FREE_PORT),LO_CORE_VERSION="$(CORE_VERSION)"
+   
DATA_FOLDER=$(MULTIUSER_DATA_FOLDER),WORKDIR=$(MULTIUSER_WORKDIR),WSD_VERSION_HASH=$(LOOLWSD_VERSION_HASH),SERVER_PORT=$(FREE_PORT),LO_CORE_VERSION="$(CORE_VERSION)"
 
 define run_interactive_desktop
$(if $(1),\
commit 76c2481190df40d040107f30a0f5b2b45aa8ce7d
Author: Tamás Zolnai 
AuthorDate: Mon Aug 10 14:10:33 2020 +0200
Commit: Tamás Zolnai 
CommitDate: Mon Aug 10 15:37:37 2020 +0200

update cypress: 4.9.0 -> 4.12.1.

Change-Id: I97e94f32697cac9002aa34632b6a80f8c6a0e127
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/100434
Tested-by: Jenkins
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Tamás Zolnai 

diff --git a/cypress_test/package.json b/cypress_test/package.json
index 176b2ea0d..1e29462ed 100644
--- a/cypress_test/package.json
+++ b/cypress_test/package.json
@@ -4,7 +4,7 @@
   "description": "Cypress integration test suit",
   "license": "MPL-2.0",
   "dependencies": {
-"cypress": "4.9.0",
+"cypress": "4.12.1",
 "cypress-failed-log": "2.7.0",
 "cypress-file-upload": "4.0.7",
 "cypress-log-to-output": "1.0.8",
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: Branch 'distro/collabora/co-4-2' - cypress_test/integration_tests

2020-08-10 Thread Tamás Zolnai (via logerrit)
 cypress_test/integration_tests/mobile/calc/number_format_spec.js |8 

 1 file changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 17c813c48859da2d9d086fb7089b4ede64d02f81
Author: Tamás Zolnai 
AuthorDate: Mon Aug 10 15:08:35 2020 +0200
Commit: Tamás Zolnai 
CommitDate: Mon Aug 10 15:27:18 2020 +0200

cypress: disable randomly failing checks on co-4-2.

Change-Id: Id9d8a95c09fec6ad9aa3b8a998591728943eb411
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/100433
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Tamás Zolnai 

diff --git a/cypress_test/integration_tests/mobile/calc/number_format_spec.js 
b/cypress_test/integration_tests/mobile/calc/number_format_spec.js
index a80bf4716..2c8034338 100644
--- a/cypress_test/integration_tests/mobile/calc/number_format_spec.js
+++ b/cypress_test/integration_tests/mobile/calc/number_format_spec.js
@@ -291,11 +291,11 @@ describe('Apply number formatting.', function() {
selectFormatting('Fraction');
 
// Decimal and leading zeros are changed.
-   cy.get('#decimalplaces input')
-   .should('have.attr', 'value', '1');
+   //cy.get('#decimalplaces input')
+   //  .should('have.attr', 'value', '1');
 
-   cy.get('#leadingzeroes input')
-   .should('have.attr', 'value', '0');
+   //cy.get('#leadingzeroes input')
+   //  .should('have.attr', 'value', '0');
 
calcHelper.selectAllMobile();
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: external/mariadb-connector-c

2020-08-10 Thread Stephan Bergmann (via logerrit)
 external/mariadb-connector-c/StaticLibrary_mariadb-connector-c.mk |1 -
 1 file changed, 1 deletion(-)

New commits:
commit 301b56f92b9058731f76e32604a771cac460693d
Author: Stephan Bergmann 
AuthorDate: Mon Aug 10 13:06:02 2020 +0200
Commit: Stephan Bergmann 
CommitDate: Mon Aug 10 15:18:20 2020 +0200

external/mariadb-connector-c: ma_bmove_upp is defined twice

...in UnpackedTarball/mariadb-connector-c/libmariadb/bmove_upp and in
workdir/UnpackedTarball/mariadb-connector-c/libmariadb/ma_stmt_codec.c.  
Given
that the first of the two contains nothing but that redundant declaration, 
lets
drop it from the (hand-curated?) list of included source files.

(I came across this when experimenting with --enable-lto on Linux and
temporarily including static libraries as --whole-archive, which thus 
caused a
"multiple definition" error when linking StaticLibrary_mariadb-connector-c 
into
Library_mysqlc.)

Change-Id: I8c5f4de476a4bbd036fd25940cdb44d11954ecc8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100427
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann 

diff --git a/external/mariadb-connector-c/StaticLibrary_mariadb-connector-c.mk 
b/external/mariadb-connector-c/StaticLibrary_mariadb-connector-c.mk
index b0c62e1b160e..fee019afa4f8 100644
--- a/external/mariadb-connector-c/StaticLibrary_mariadb-connector-c.mk
+++ b/external/mariadb-connector-c/StaticLibrary_mariadb-connector-c.mk
@@ -34,7 +34,6 @@ $(eval $(call 
gb_StaticLibrary_set_include,mariadb-connector-c,\
 endif
 
 $(eval $(call gb_StaticLibrary_add_generated_cobjects,mariadb-connector-c,\
-   UnpackedTarball/mariadb-connector-c/libmariadb/bmove_upp \
UnpackedTarball/mariadb-connector-c/libmariadb/get_password \
UnpackedTarball/mariadb-connector-c/libmariadb/ma_alloc \
UnpackedTarball/mariadb-connector-c/libmariadb/ma_array \
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: scp2/source solenv/bin

2020-08-10 Thread Mike Kaganski (via logerrit)
 scp2/source/ooo/vc_redist.scp   |4 +
 solenv/bin/modules/installer/windows/mergemodule.pm |   43 +---
 2 files changed, 34 insertions(+), 13 deletions(-)

New commits:
commit 0872e8cc87d753e6bdda9fad510a6b71cf96565f
Author: Mike Kaganski 
AuthorDate: Mon Aug 10 13:58:57 2020 +0300
Commit: Mike Kaganski 
CommitDate: Mon Aug 10 15:06:14 2020 +0200

tdf#135579: Don't uninstall vc_redist: make it permanent

Redist is a system component, that includes a varying set of DLLs,
and those DLLs are ref-counted. Installing a newer redist - i.e.
updating and increasing refcount of existing DLLs - may add new
DLLs (with initial refcount 1) in addition to the updated old DLLs
that start depending on the newly added ones; at uninstall, the
newly added DLLs may get removed because their refcount gets 0,
while other redist DLLs are kept at the updated levels - so their
dependencies now are not met, and redist gets broken.

Just mark the redist components permanent, which, according to [1],
"registers an extra system client for the component in the Windows
Installer registry settings".

A downside is that uninstall doesn't restore the original system
state ideally.

[1] https://docs.microsoft.com/en-us/windows/win32/msi/component-table

Change-Id: I3fe82bcb5844f826f5b1df622273b4e3a1e3c436
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100426
Tested-by: Mike Kaganski 
Reviewed-by: Mike Kaganski 

diff --git a/scp2/source/ooo/vc_redist.scp b/scp2/source/ooo/vc_redist.scp
index d32758312766..75ccf2b8d09d 100644
--- a/scp2/source/ooo/vc_redist.scp
+++ b/scp2/source/ooo/vc_redist.scp
@@ -32,22 +32,26 @@
 
 #if defined(WINDOWS_X86_MERGEMODULE)
 
+/* Attributes: msidbComponentAttributesPermanent = 0x10 */
 MergeModule WINDOWS_X86_MERGEMODULE
 Feature = gm_Root;
 Name = WINDOWS_X86_MERGEMODULE_FILE;
 RootDir = "TARGETDIR";
 ComponentCondition = "VC_REDIST=1";
+Attributes_Add = "0x10";
 End
 
 #endif
 
 #if defined(WINDOWS_X64) && defined(WINDOWS_X64_MERGEMODULE)
 
+/* Attributes: msidbComponentAttributesPermanent = 0x10 */
 MergeModule WINDOWS_X64_MERGEMODULE
 Feature = gm_Root;
 Name = WINDOWS_X64_MERGEMODULE_FILE;
 RootDir = "TARGETDIR";
 ComponentCondition = "VC_REDIST=1";
+Attributes_Add = "0x10";
 End
 
 #endif
diff --git a/solenv/bin/modules/installer/windows/mergemodule.pm 
b/solenv/bin/modules/installer/windows/mergemodule.pm
index 68bb203f1053..defd59588c95 100644
--- a/solenv/bin/modules/installer/windows/mergemodule.pm
+++ b/solenv/bin/modules/installer/windows/mergemodule.pm
@@ -294,6 +294,7 @@ sub merge_mergemodules_into_msi_database
 $onemergemodulehash{'filenumber'} = $filecounter;
 $onemergemodulehash{'componentnames'} = \%componentnames;
 $onemergemodulehash{'componentcondition'} = 
$mergemodule->{'ComponentCondition'};
+$onemergemodulehash{'attributes_add'} = 
$mergemodule->{'Attributes_Add'};
 $onemergemodulehash{'cabfilename'} = $cabfilename;
 $onemergemodulehash{'feature'} = $mergemodule->{'Feature'};
 $onemergemodulehash{'rootdir'} = $mergemodule->{'RootDir'};
@@ -405,7 +406,7 @@ sub merge_mergemodules_into_msi_database
 my $workingtables = "File Media Directory FeatureComponents"; # 
required tables
 # Optional tables can be added now
 if ( $mergemodulehash->{'hasmsiassemblies'} ) { $workingtables = 
$workingtables . " MsiAssembly"; }
-if ( $mergemodulehash->{'componentcondition'} ) { $workingtables = 
$workingtables . " Component"; }
+if ( ( $mergemodulehash->{'componentcondition'} ) || ( 
$mergemodulehash->{'attributes_add'} ) ) { $workingtables = $workingtables . " 
Component"; }
 
 # Table "Feature" has to be exported, but it is not necessary to 
import it.
 if ( $^O =~ /cygwin/i ) {
@@ -462,7 +463,7 @@ sub merge_mergemodules_into_msi_database
 change_msiassembly_table($mergemodulehash, $workdir);
 }
 
-if ( $mergemodulehash->{'componentcondition'} )
+if ( ( $mergemodulehash->{'componentcondition'} ) || ( 
$mergemodulehash->{'attributes_add'} ) )
 {
 
installer::logger::include_timestamp_into_logfile("\nPerformance Info: Changing 
Component table");
 change_component_table($mergemodulehash, $workdir);
@@ -1368,7 +1369,7 @@ sub change_featurecomponent_table
 }
 
 ###
-# In the components table, the conditions of merge modules should be updated
+# In the components table, the conditions or attributes of merge modules 
should be updated
 ###
 
 sub change_component_table
@@ -1388,25 +13

[Libreoffice-commits] core.git: Branch 'private/quwex/gsoc-box2d-experimental' - 3 commits - offapi/com slideshow/source

2020-08-10 Thread Sarper Akdemir (via logerrit)
Rebased ref, commits from common ancestor:
commit 5d2cbc804325571dd22c70f6e0e97a43268b2f59
Author: Sarper Akdemir 
AuthorDate: Mon Aug 10 15:30:26 2020 +0300
Commit: Sarper Akdemir 
CommitDate: Mon Aug 10 15:44:53 2020 +0300

add version tag to ANIMATEPHYSICS

Change-Id: Ia6db8ca10a0311ae8492cdc5ab518efaba611cb2

diff --git a/offapi/com/sun/star/animations/AnimationNodeType.idl 
b/offapi/com/sun/star/animations/AnimationNodeType.idl
index d0cd6e268fd6..7c6abb105947 100644
--- a/offapi/com/sun/star/animations/AnimationNodeType.idl
+++ b/offapi/com/sun/star/animations/AnimationNodeType.idl
@@ -68,7 +68,10 @@ constants AnimationNodeType
 /** Defines a command effect. */
 const short COMMAND = 11;
 
-/** Defines a physics animation */
+/** Defines a physics animation
+
+@since LibreOffice 7.1
+*/
 const short ANIMATEPHYSICS = 12;
 
 };
commit cf744b80d177ba98a5f9c8ae45f7a77328eb2394
Author: Sarper Akdemir 
AuthorDate: Thu Aug 6 10:32:55 2020 +0300
Commit: Sarper Akdemir 
CommitDate: Mon Aug 10 15:44:53 2020 +0300

make physics based animation effects always processed last

Change-Id: I92d436aced6ef3ee2c8b0bf0167c1f7e642ba3b5

diff --git a/slideshow/source/engine/activitiesqueue.cxx 
b/slideshow/source/engine/activitiesqueue.cxx
index ba982385356e..38e79d1e5677 100644
--- a/slideshow/source/engine/activitiesqueue.cxx
+++ b/slideshow/source/engine/activitiesqueue.cxx
@@ -50,6 +50,8 @@ namespace slideshow::internal
 {
 for( const auto& pActivity : maCurrentActivitiesWaiting )
 pActivity->dispose();
+for( const auto& pActivity : 
maCurrentActivitiesToBeProcessedLast )
+pActivity->dispose();
 for( const auto& pActivity : maCurrentActivitiesReinsert )
 pActivity->dispose();
 }
@@ -59,7 +61,7 @@ namespace slideshow::internal
 }
 }
 
-bool ActivitiesQueue::addActivity( const ActivitySharedPtr& pActivity )
+bool ActivitiesQueue::addActivity( const ActivitySharedPtr& pActivity, 
const bool bProcessLast )
 {
 OSL_ENSURE( pActivity, "ActivitiesQueue::addActivity: activity ptr 
NULL" );
 
@@ -67,7 +69,17 @@ namespace slideshow::internal
 return false;
 
 // add entry to waiting list
-maCurrentActivitiesWaiting.push_back( pActivity );
+if( !bProcessLast )
+{
+maCurrentActivitiesWaiting.push_back( pActivity );
+}
+else
+{
+// Activities that should be processed last is kept in a 
different
+// ActivityQueue, and later added to the end of the 
maCurrentActivitiesWaiting
+// at the start of ActivitiesQueue::process()
+maCurrentActivitiesToBeProcessedLast.push_back( pActivity );
+}
 
 return true;
 }
@@ -76,6 +88,12 @@ namespace slideshow::internal
 {
 SAL_INFO("slideshow.verbose", "ActivitiesQueue: outer loop 
heartbeat" );
 
+// If there are activities to be processed last add them to the 
end of the ActivitiesQueue
+maCurrentActivitiesWaiting.insert( 
maCurrentActivitiesWaiting.end(),
+   
maCurrentActivitiesToBeProcessedLast.begin(),
+   
maCurrentActivitiesToBeProcessedLast.end() );
+maCurrentActivitiesToBeProcessedLast.clear();
+
 // accumulate time lag for all activities, and lag time
 // base if necessary:
 double fLag = 0.0;
diff --git a/slideshow/source/engine/animationnodes/animationbasenode.cxx 
b/slideshow/source/engine/animationnodes/animationbasenode.cxx
index 4dcb640795aa..7999b5a7654a 100644
--- a/slideshow/source/engine/animationnodes/animationbasenode.cxx
+++ b/slideshow/source/engine/animationnodes/animationbasenode.cxx
@@ -23,6 +23,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include "nodetools.hxx"
 #include 
@@ -294,7 +295,10 @@ void AnimationBaseNode::activate_st()
 mpActivity->setTargets( getShape(), maAttributeLayerHolder.get() );
 
 // add to activities queue
-getContext().mrActivitiesQueue.addActivity( mpActivity );
+if( mxAnimateNode->getType() == 
css::animations::AnimationNodeType::ANIMATEPHYSICS )
+getContext().mrActivitiesQueue.addActivity(mpActivity, true);
+else
+getContext().mrActivitiesQueue.addActivity( mpActivity );
 }
 else {
 // Actually, DO generate the event for empty activity,
diff --git a/slideshow/source/inc/activitiesqueue.hxx 
b/slideshow/source/inc/activitiesqueue.hxx
index b4f88b1b39d1..76dc981f8f65 100644
--- a/slideshow/source/inc/activitiesqueue.hxx
+++ b/slideshow/source/inc/activitiesqueue.hxx
@@ -57,7 +57,7 @@ namespace slideshow
 
 /*

[Libreoffice-commits] core.git: cui/source

2020-08-10 Thread Caolán McNamara (via logerrit)
 cui/source/customize/SvxMenuConfigPage.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit f6e9feb5bf7d227b33fe9cd4e61720d6205e5143
Author: Caolán McNamara 
AuthorDate: Mon Aug 10 12:15:45 2020 +0100
Commit: Caolán McNamara 
CommitDate: Mon Aug 10 14:44:31 2020 +0200

Resolves: tdf#135603 crash on rename item in customize dialog

Change-Id: Ib5b04ec8ce974c87646b6bf4989094c8334a0006
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100428
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 

diff --git a/cui/source/customize/SvxMenuConfigPage.cxx 
b/cui/source/customize/SvxMenuConfigPage.cxx
index c73502c573c7..a811264d2ca0 100644
--- a/cui/source/customize/SvxMenuConfigPage.cxx
+++ b/cui/source/customize/SvxMenuConfigPage.cxx
@@ -445,7 +445,7 @@ IMPL_LINK(SvxMenuConfigPage, ModifyItemHdl, const OString&, 
rIdent, void)
 aNewName = aNameDialog.GetName();
 
 pEntry->SetName( aNewName );
-m_xContentsListBox->set_text(nActEntry, aNewName, 1);
+m_xContentsListBox->set_text(nActEntry, aNewName, 0);
 
 GetSaveInData()->SetModified();
 GetTopLevelSelection()->SetModified();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-7-0' - oox/source sw/qa

2020-08-10 Thread Michael Stahl (via logerrit)
 oox/source/export/vmlexport.cxx|   41 +
 sw/qa/extras/ooxmlexport/ooxmlexport15.cxx |4 ++
 2 files changed, 44 insertions(+), 1 deletion(-)

New commits:
commit ad3813c0ab5303625cda7d04ebe6a6b48712255d
Author: Michael Stahl 
AuthorDate: Fri Aug 7 15:50:12 2020 +0200
Commit: Xisco Fauli 
CommitDate: Mon Aug 10 13:29:18 2020 +0200

oox: VML export: convert ESCHER_Prop_AnchorText to v-text-anchor

Change-Id: I903cac8d7b02138680613b5a1b6dab4b1c448158
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100333
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
(cherry picked from commit 8d83c29905ca6c4067ae0330d3544ddb983cafbc)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100343
Reviewed-by: Xisco Fauli 

diff --git a/oox/source/export/vmlexport.cxx b/oox/source/export/vmlexport.cxx
index 964cf27c6921..79733db5ced0 100644
--- a/oox/source/export/vmlexport.cxx
+++ b/oox/source/export/vmlexport.cxx
@@ -430,6 +430,47 @@ void VMLExport::Commit( EscherPropertyContainer& rProps, 
const tools::Rectangle&
 bAlreadyWritten[ ESCHER_Prop_WrapText ] = true;
 break;
 
+case ESCHER_Prop_AnchorText: // 135
+{
+char const* pValue(nullptr);
+switch (opt.nPropValue)
+{
+case ESCHER_AnchorTop:
+pValue = "top";
+break;
+case ESCHER_AnchorMiddle:
+pValue = "middle";
+break;
+case ESCHER_AnchorBottom:
+pValue = "bottom";
+break;
+case ESCHER_AnchorTopCentered:
+pValue = "top-center";
+break;
+case ESCHER_AnchorMiddleCentered:
+pValue = "middle-center";
+break;
+case ESCHER_AnchorBottomCentered:
+pValue = "bottom-center";
+break;
+case ESCHER_AnchorTopBaseline:
+pValue = "top-baseline";
+break;
+case ESCHER_AnchorBottomBaseline:
+pValue = "bottom-baseline";
+break;
+case ESCHER_AnchorTopCenteredBaseline:
+pValue = "top-center-baseline";
+break;
+case ESCHER_AnchorBottomCenteredBaseline:
+pValue = "bottom-center-baseline";
+break;
+}
+m_ShapeStyle.append(";v-text-anchor:");
+m_ShapeStyle.append(pValue);
+}
+break;
+
 case ESCHER_Prop_txflTextFlow: // 136
 {
 // at least "bottom-to-top" only has an effect when it's 
on the v:textbox element, not on v:shape
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport15.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport15.cxx
index 67184ba57636..8dd905a6405b 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport15.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport15.cxx
@@ -84,7 +84,9 @@ DECLARE_OOXMLEXPORT_TEST(testAtPageShapeRelOrientation, 
"rotated_shape.fodt")
 assertXPath(pXmlDocument, 
"/w:document/w:body/w:p/w:r/mc:AlternateContent[1]/mc:Fallback/w:pict/v:shape/v:textbox",
 "style", "mso-layout-flow-alt:bottom-to-top");
 // text wrap -> VML
 assertXPath(pXmlDocument, 
"/w:document/w:body/w:p/w:r/mc:AlternateContent[1]/mc:Fallback/w:pict/v:shape/w10:wrap",
 "type", "none");
-
+// vertical alignment -> VML
+OUString const style = getXPath(pXmlDocument, 
"/w:document/w:body/w:p/w:r/mc:AlternateContent[1]/mc:Fallback/w:pict/v:shape", 
"style");
+CPPUNIT_ASSERT(style.indexOf("v-text-anchor:middle") != -1);
 }
 
 DECLARE_OOXMLEXPORT_TEST(testRelativeAnchorHeightFromBottomMarginHasFooter,
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-7-0' - include/oox oox/source

2020-08-10 Thread Michael Stahl (via logerrit)
 include/oox/export/vmlexport.hxx |3 +++
 oox/source/export/vmlexport.cxx  |   36 ++--
 2 files changed, 29 insertions(+), 10 deletions(-)

New commits:
commit d108af8aff3a5a03b8a3b68c69fb477c18c8cc1f
Author: Michael Stahl 
AuthorDate: Fri Aug 7 15:45:25 2020 +0200
Commit: Xisco Fauli 
CommitDate: Mon Aug 10 13:28:28 2020 +0200

oox: VML export: produce bottom-to-top in a better way

Replace code added in 090c61eb93db4302d4565d5f11f7673190835fdb
with something that uses the already generated ESCHER property; this
lets a warning about the unhandled property disappear too.

Change-Id: Ieed83dd8e17e92eea9901124fce5e6da2a17196a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100332
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
(cherry picked from commit 2af2b9be05a4733c691db7201e76b4058516c47b)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100354
Reviewed-by: Xisco Fauli 

diff --git a/include/oox/export/vmlexport.hxx b/include/oox/export/vmlexport.hxx
index 61185e57d5de..71d3441fdaaa 100644
--- a/include/oox/export/vmlexport.hxx
+++ b/include/oox/export/vmlexport.hxx
@@ -99,6 +99,9 @@ class OOX_DLLPUBLIC VMLExport : public EscherEx
 /// Remember style, the most important shape attribute ;-)
 OStringBuffer m_ShapeStyle;
 
+/// style for textbox
+OStringBuffer m_TextboxStyle;
+
 /// Remember the generated shape id.
 OString m_sShapeId;
 
diff --git a/oox/source/export/vmlexport.cxx b/oox/source/export/vmlexport.cxx
index 950b0afd240d..964cf27c6921 100644
--- a/oox/source/export/vmlexport.cxx
+++ b/oox/source/export/vmlexport.cxx
@@ -430,6 +430,28 @@ void VMLExport::Commit( EscherPropertyContainer& rProps, 
const tools::Rectangle&
 bAlreadyWritten[ ESCHER_Prop_WrapText ] = true;
 break;
 
+case ESCHER_Prop_txflTextFlow: // 136
+{
+// at least "bottom-to-top" only has an effect when it's 
on the v:textbox element, not on v:shape
+assert(m_TextboxStyle.isEmpty());
+switch (opt.nPropValue)
+{
+case ESCHER_txflHorzN:
+m_TextboxStyle.append("layout-flow:horizontal");
+break;
+case ESCHER_txflTtoBA:
+m_TextboxStyle.append("layout-flow:vertical");
+break;
+case ESCHER_txflBtoT:
+
m_TextboxStyle.append("mso-layout-flow-alt:bottom-to-top");
+break;
+default:
+assert(false); // unimplemented in escher export
+break;
+}
+}
+break;
+
 // coordorigin
 case ESCHER_Prop_geoLeft: // 320
 case ESCHER_Prop_geoTop: // 321
@@ -1345,6 +1367,8 @@ sal_Int32 VMLExport::StartShape()
 // start of the shape
 m_pSerializer->startElementNS( XML_v, nShapeElement, 
XFastAttributeListRef( m_pShapeAttrList ) );
 
+OString const textboxStyle(m_TextboxStyle.makeStringAndClear());
+
 // now check if we have some editeng text (not associated textbox) and we 
have a text exporter registered
 const SdrTextObj* pTxtObj = dynamic_cast( m_pSdrObject 
);
 if (pTxtObj && m_pTextExport && 
msfilter::util::HasTextBoxContent(m_nShapeType) && 
!IsWaterMarkShape(m_pSdrObject->GetName()) && !lcl_isTextBox(m_pSdrObject))
@@ -1369,19 +1393,11 @@ sal_Int32 VMLExport::StartShape()
 
 if( pParaObj )
 {
-uno::Reference 
xPropertySet(const_cast(m_pSdrObject)->getUnoShape(), 
uno::UNO_QUERY);
 sax_fastparser::FastAttributeList* pTextboxAttrList = 
FastSerializerHelper::createAttrList();
 sax_fastparser::XFastAttributeListRef 
xTextboxAttrList(pTextboxAttrList);
-if 
(xPropertySet->getPropertySetInfo()->hasPropertyByName("RotateAngle"))
+if (!textboxStyle.isEmpty())
 {
-sal_Int32 nTextRotateAngle = sal_Int32();
-if (xPropertySet->getPropertyValue("RotateAngle") >>= 
nTextRotateAngle)
-{
-if (nTextRotateAngle == 9000)
-{
-pTextboxAttrList->add(XML_style, 
"mso-layout-flow-alt:bottom-to-top");
-}
-}
+pTextboxAttrList->add(XML_style, textboxStyle);
 }
 
 // this is reached only in case some text is attached to the shape
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-7-0' - filter/source sw/qa

2020-08-10 Thread Michael Stahl (via logerrit)
 filter/source/msfilter/escherex.cxx|   20 
 sw/qa/extras/ooxmlexport/ooxmlexport15.cxx |2 ++
 2 files changed, 22 insertions(+)

New commits:
commit 2eb4fe77fa7d8e7e2154f9807e1eed0076e1d7e2
Author: Michael Stahl 
AuthorDate: Fri Aug 7 15:26:48 2020 +0200
Commit: Xisco Fauli 
CommitDate: Mon Aug 10 13:27:35 2020 +0200

filter: MSO export: convert TextWrap property to Escher_Wrap*

There's a paucity of working wrapping modes in Escher unfortunately.

Change-Id: Ibaf99c3249a6492dc129f9c9b5707778038f9a4c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100331
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
(cherry picked from commit 2cb90a5c87fe46737c8d840967d8836284f92ffd)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100353
Reviewed-by: Xisco Fauli 

diff --git a/filter/source/msfilter/escherex.cxx 
b/filter/source/msfilter/escherex.cxx
index 443d533ecadb..8bf8c779af64 100644
--- a/filter/source/msfilter/escherex.cxx
+++ b/filter/source/msfilter/escherex.cxx
@@ -61,6 +61,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -692,6 +693,10 @@ void EscherPropertyContainer::CreateTextProperties(
 bool bWordWrap  ( false );
 bool bAutoGrowSize  ( false );
 
+uno::Any aTextWrap;
+
+EscherPropertyValueHelper::GetPropertyValue(aTextWrap, rXPropSet, 
"TextWrap", true);
+
 if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, 
"TextWritingMode", true ) )
 aAny >>= eWM;
 if ( EscherPropertyValueHelper::GetPropertyValue( aAny, rXPropSet, 
"TextVerticalAdjust", true ) )
@@ -830,6 +835,21 @@ void EscherPropertyContainer::CreateTextProperties(
 nTextAttr |= 0x20002;
 }
 }
+
+if (aTextWrap.hasValue())
+{   // explicit text wrap overrides whatever was inferred previously
+switch (aTextWrap.get())
+{
+case text::WrapTextMode_THROUGH:
+eWrapMode = ESCHER_WrapNone;
+break;
+// in theory there are 3 more Escher_Wrap, but [MS-ODRAW] says 
they are useless
+default:
+eWrapMode = ESCHER_WrapSquare;
+break;
+}
+}
+
 AddOpt( ESCHER_Prop_dxTextLeft, nLeft * 360 );
 AddOpt( ESCHER_Prop_dxTextRight, nRight * 360 );
 AddOpt( ESCHER_Prop_dyTextTop, nTop * 360 );
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport15.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport15.cxx
index ad019ecd0504..67184ba57636 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport15.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport15.cxx
@@ -82,6 +82,8 @@ DECLARE_OOXMLEXPORT_TEST(testAtPageShapeRelOrientation, 
"rotated_shape.fodt")
 
 // now test text rotation -> VML writing direction
 assertXPath(pXmlDocument, 
"/w:document/w:body/w:p/w:r/mc:AlternateContent[1]/mc:Fallback/w:pict/v:shape/v:textbox",
 "style", "mso-layout-flow-alt:bottom-to-top");
+// text wrap -> VML
+assertXPath(pXmlDocument, 
"/w:document/w:body/w:p/w:r/mc:AlternateContent[1]/mc:Fallback/w:pict/v:shape/w10:wrap",
 "type", "none");
 
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-7-0' - dbaccess/source

2020-08-10 Thread Caolán McNamara (via logerrit)
 dbaccess/source/ui/dlg/adtabdlg.cxx |8 
 1 file changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 09be57965fbdc291a25c4f48550f467b43a7177c
Author: Caolán McNamara 
AuthorDate: Tue Aug 4 14:19:01 2020 +0100
Commit: Xisco Fauli 
CommitDate: Mon Aug 10 13:18:17 2020 +0200

explicit text column to avoid dummy nodes for non-toggle case

Change-Id: If216705929454146d39b291b9a6483c68c3eac89
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100061
Tested-by: Jenkins
Reviewed-by: Xisco Fauli 

diff --git a/dbaccess/source/ui/dlg/adtabdlg.cxx 
b/dbaccess/source/ui/dlg/adtabdlg.cxx
index 4ad01555d226..35e29ed17f31 100644
--- a/dbaccess/source/ui/dlg/adtabdlg.cxx
+++ b/dbaccess/source/ui/dlg/adtabdlg.cxx
@@ -106,12 +106,12 @@ OUString TableListFacade::getSelectedName( OUString& 
_out_rAliasName ) const
 if (rTableList.iter_parent(*xCatalog))
 {
 if (!xAll || !xCatalog->equal(*xAll))
-aCatalog = rTableList.get_text(*xCatalog);
+aCatalog = rTableList.get_text(*xCatalog, 0);
 }
-aSchema = rTableList.get_text(*xSchema);
+aSchema = rTableList.get_text(*xSchema, 0);
 }
 }
-aTableName = rTableList.get_text(*xEntry);
+aTableName = rTableList.get_text(*xEntry, 0);
 
 OUString aComposedName;
 try
@@ -318,7 +318,7 @@ OUString QueryListFacade::getSelectedName( OUString& 
_out_rAliasName ) const
 std::unique_ptr xEntry(m_rQueryList.make_iterator());
 const bool bEntry = m_rQueryList.get_selected(xEntry.get());
 if (bEntry)
-sSelected = _out_rAliasName = m_rQueryList.get_text(*xEntry);
+sSelected = _out_rAliasName = m_rQueryList.get_text(*xEntry, 0);
 return sSelected;
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] online.git: wsd/LOOLWSD.cpp

2020-08-10 Thread Gökhan Karabulut (via logerrit)
 wsd/LOOLWSD.cpp |8 
 1 file changed, 8 insertions(+)

New commits:
commit e2e638c468f35322fc123664fd749b199f531780
Author: Gökhan Karabulut 
AuthorDate: Sat Aug 8 23:05:17 2020 +0300
Commit: Michael Meeks 
CommitDate: Mon Aug 10 12:20:08 2020 +0200

tdf#124478: Log maximum file descriptor related limits

The number of available file descriptors in a system limits the number
of documents we can open. We use an fd for client connection, another fd
for communication with a kit process and a wakeup pipe with 2 fds.
Therefore, we are left with the maximum number of fds divided by 4
documents. Out of these documents, reserve 8 (i.e., 32 fds) and log the
remaining number of documents allowed by the system. Note that Online
instance can further configure a limit for the maximum number of open
documents, which is also logged.

Also log the maximum file descriptor allowed by the system, which is the
number of available file descriptors - 1.

Change-Id: I3972690a6c9995e8d74dcfe25fe87b1ef4c33d4b
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/100393
Tested-by: Jenkins
Tested-by: Jenkins CollaboraOffice 
Tested-by: Michael Meeks 
Reviewed-by: Michael Meeks 

diff --git a/wsd/LOOLWSD.cpp b/wsd/LOOLWSD.cpp
index 02feb23b7..b8c75e5b5 100644
--- a/wsd/LOOLWSD.cpp
+++ b/wsd/LOOLWSD.cpp
@@ -41,6 +41,7 @@
 
 #include 
 #include 
+#include 
 
 #include 
 #include 
@@ -1327,6 +1328,13 @@ void LOOLWSD::initialize(Application& self)
 LOOLWSD::MaxDocuments = LOOLWSD::MaxConnections;
 }
 
+struct rlimit rlim;
+::getrlimit(RLIMIT_NOFILE, &rlim);
+LOG_INF("Maximum file descriptor supported by the system: " << 
rlim.rlim_cur - 1);
+// 4 fds per document are used for client connection, Kit process 
communication, and
+// a wakeup pipe with 2 fds. 32 fds (i.e. 8 documents) are reserved.
+LOG_INF("Maximum number of open documents supported by the system: " << 
rlim.rlim_cur / 4 - 8);
+
 LOG_INF("Maximum concurrent open Documents limit: " << 
LOOLWSD::MaxDocuments);
 LOG_INF("Maximum concurrent client Connections limit: " << 
LOOLWSD::MaxConnections);
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-7-0' - svx/inc svx/source

2020-08-10 Thread Caolán McNamara (via logerrit)
 svx/inc/CommonStylePreviewRenderer.hxx   |2 ++
 svx/source/styles/CommonStylePreviewRenderer.cxx |   12 
 2 files changed, 14 insertions(+)

New commits:
commit d67cd0efbc08c92492f3b5fd7ffe35556052d52f
Author: Caolán McNamara 
AuthorDate: Wed Aug 5 15:50:31 2020 +0100
Commit: Xisco Fauli 
CommitDate: Mon Aug 10 12:00:25 2020 +0200

tdf#135438 Paragraph styles preview in sidebar are clipped

since...

commit fe9a13dc0e6d1384416c2a2343223b33925fc925
Author: Caolán McNamara 
Date:   Sun Apr 26 15:43:25 2020 +0100

weld SfxTemplatePanelControl

getRenderSize used to be called after recalculate and before
render to change maSizePixel

Change-Id: Ifc1b6b0b3399c7de61d5f4d46f8346d13e72a4d7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100069
Tested-by: Jenkins
Reviewed-by: Xisco Fauli 

diff --git a/svx/inc/CommonStylePreviewRenderer.hxx 
b/svx/inc/CommonStylePreviewRenderer.hxx
index 55391327533e..5dfd41faa39a 100644
--- a/svx/inc/CommonStylePreviewRenderer.hxx
+++ b/svx/inc/CommonStylePreviewRenderer.hxx
@@ -31,6 +31,8 @@ class CommonStylePreviewRenderer final : public 
sfx2::StylePreviewRenderer
 Size maPixelSize;
 OUString maStyleName;
 
+Size getRenderSize() const;
+
 public:
 CommonStylePreviewRenderer(const SfxObjectShell& rShell, OutputDevice& 
rOutputDev,
SfxStyleSheetBase* pStyle, long nMaxHeight);
diff --git a/svx/source/styles/CommonStylePreviewRenderer.cxx 
b/svx/source/styles/CommonStylePreviewRenderer.cxx
index d677a135461e..9920903df469 100644
--- a/svx/source/styles/CommonStylePreviewRenderer.cxx
+++ b/svx/source/styles/CommonStylePreviewRenderer.cxx
@@ -168,9 +168,21 @@ bool CommonStylePreviewRenderer::recalculate()
 }
 
 m_pFont = std::move(pFont);
+maPixelSize = getRenderSize();
 return true;
 }
 
+Size CommonStylePreviewRenderer::getRenderSize() const
+{
+assert(m_pFont);
+Size aPixelSize = m_pFont->GetTextSize(&mrOutputDev, maStyleName);
+
+if (aPixelSize.Height() > mnMaxHeight)
+aPixelSize.setHeight( mnMaxHeight );
+
+return aPixelSize;
+}
+
 bool CommonStylePreviewRenderer::render(const tools::Rectangle& aRectangle, 
RenderAlign eRenderAlign)
 {
 const OUString& rText = maStyleName;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-6-4' - sw/source

2020-08-10 Thread Caolán McNamara (via logerrit)
 sw/source/core/docnode/ndtbl.cxx |9 +
 1 file changed, 9 insertions(+)

New commits:
commit 0d5affd7a25396dbff866adae524a4c01329801f
Author: Caolán McNamara 
AuthorDate: Wed Aug 5 21:08:04 2020 +0100
Commit: Xisco Fauli 
CommitDate: Mon Aug 10 11:56:32 2020 +0200

tdf#135098 update SwTableCursor m_SelectedBoxes before merge

so it does not contain the soon to-be-deleted SwTableBox so if the rPam is
queried via a11y it doesn't claim the deleted cell still exists.

tdf#122844 may be the same issue

Change-Id: I1ac3752676162ba5a29c0916039b2b467e2ac41a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100214
Tested-by: Jenkins
Reviewed-by: Xisco Fauli 

diff --git a/sw/source/core/docnode/ndtbl.cxx b/sw/source/core/docnode/ndtbl.cxx
index 00e249b9e0fb..15a49729ce51 100644
--- a/sw/source/core/docnode/ndtbl.cxx
+++ b/sw/source/core/docnode/ndtbl.cxx
@@ -2280,6 +2280,15 @@ TableMergeErr SwDoc::MergeTable( SwPaM& rPam )
 while( &rPam != ( pTmp = pTmp->GetNext() ))
 for( int i = 0; i < 2; ++i )
 pTmp->GetBound( static_cast(i) ) = *rPam.GetPoint();
+
+if (SwTableCursor* pTableCursor = 
dynamic_cast(&rPam))
+{
+// tdf#135098 update selection so rPam's m_SelectedBoxes is 
updated
+// to not contain the soon to-be-deleted SwTableBox so if the 
rPam
+// is queried via a11y it doesn't claim the deleted cell still
+// exists
+pTableCursor->NewTableSelection();
+}
 }
 
 // Merge them
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-7-0' - sw/source

2020-08-10 Thread Caolán McNamara (via logerrit)
 sw/source/core/docnode/ndtbl.cxx |9 +
 1 file changed, 9 insertions(+)

New commits:
commit 1bef764ac84d113a0bcb237a0634579ee8dccec0
Author: Caolán McNamara 
AuthorDate: Wed Aug 5 21:08:04 2020 +0100
Commit: Xisco Fauli 
CommitDate: Mon Aug 10 11:56:07 2020 +0200

tdf#135098 update SwTableCursor m_SelectedBoxes before merge

so it does not contain the soon to-be-deleted SwTableBox so if the rPam is
queried via a11y it doesn't claim the deleted cell still exists.

tdf#122844 may be the same issue

Change-Id: I1ac3752676162ba5a29c0916039b2b467e2ac41a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100213
Tested-by: Jenkins
Reviewed-by: Xisco Fauli 

diff --git a/sw/source/core/docnode/ndtbl.cxx b/sw/source/core/docnode/ndtbl.cxx
index 4d5469e8fdac..5dcbdf86575f 100644
--- a/sw/source/core/docnode/ndtbl.cxx
+++ b/sw/source/core/docnode/ndtbl.cxx
@@ -2285,6 +2285,15 @@ TableMergeErr SwDoc::MergeTable( SwPaM& rPam )
 while( &rPam != ( pTmp = pTmp->GetNext() ))
 for( int i = 0; i < 2; ++i )
 pTmp->GetBound( static_cast(i) ) = *rPam.GetPoint();
+
+if (SwTableCursor* pTableCursor = 
dynamic_cast(&rPam))
+{
+// tdf#135098 update selection so rPam's m_SelectedBoxes is 
updated
+// to not contain the soon to-be-deleted SwTableBox so if the 
rPam
+// is queried via a11y it doesn't claim the deleted cell still
+// exists
+pTableCursor->NewTableSelection();
+}
 }
 
 // Merge them
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: download.lst external/nss

2020-08-10 Thread Michael Stahl (via logerrit)
 download.lst |4 -
 external/nss/UnpackedTarball_nss.mk  |1 
 external/nss/macos-dlopen.patch.0|   18 
 external/nss/nss.nspr-parallel-win-debug_build.patch |   40 ---
 4 files changed, 11 insertions(+), 52 deletions(-)

New commits:
commit 495a5944a3d442cfe748a3bb0dcef76f6a961d30
Author: Michael Stahl 
AuthorDate: Fri Aug 7 18:57:00 2020 +0200
Commit: Michael Stahl 
CommitDate: Mon Aug 10 11:39:38 2020 +0200

nss: upgrade to release 3.55.0

Fixes CVE-2020-6829, CVE-2020-12400 CVE-2020-12401 CVE-2020-12403.
(also CVE-2020-12402 CVE-2020-12399 in older releases since 3.47)

* external/nss/nss.nspr-parallel-win-debug_build.patch:
  remove, merged upstream

Change-Id: I8b48e25ce68a2327cde1420abdaea8f9e51a7888
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100345
Tested-by: Jenkins
Reviewed-by: Michael Stahl 

diff --git a/download.lst b/download.lst
index 99538799357f..8558f095b4a9 100644
--- a/download.lst
+++ b/download.lst
@@ -193,8 +193,8 @@ export MYTHES_SHA256SUM := 
1e81f395d8c851c3e4e75b568e20fa2fa549354e75ab397f9de4b
 export MYTHES_TARBALL := a8c2c5b8f09e7ede322d5c602ff6a4b6-mythes-1.2.4.tar.gz
 export NEON_SHA256SUM := 
c9dfcee723050df37ce18ba449d7707b78e7ab8230f3a4c59d9112e17dc2718d
 export NEON_TARBALL := neon-0.31.1.tar.gz
-export NSS_SHA256SUM := 
861a4510b7c21516f49a4cfa5b871aa796e4e1ef2dfe949091970e56f9d60cdf
-export NSS_TARBALL := nss-3.53-with-nspr-4.25.tar.gz
+export NSS_SHA256SUM := 
ec6032d78663c6ef90b4b83eb552dedf721d2bce208cec3bf527b8f637db7e45
+export NSS_TARBALL := nss-3.55-with-nspr-4.27.tar.gz
 export ODFGEN_SHA256SUM := 
2c7b21892f84a4c67546f84611eccdad6259875c971e98ddb027da66ea0ac9c2
 export ODFGEN_VERSION_MICRO := 6
 export ODFGEN_TARBALL := libodfgen-0.1.$(ODFGEN_VERSION_MICRO).tar.bz2
diff --git a/external/nss/UnpackedTarball_nss.mk 
b/external/nss/UnpackedTarball_nss.mk
index 5904e267b668..beb9afe11890 100644
--- a/external/nss/UnpackedTarball_nss.mk
+++ b/external/nss/UnpackedTarball_nss.mk
@@ -25,7 +25,6 @@ $(eval $(call gb_UnpackedTarball_add_patches,nss,\
 external/nss/nss.vs2015.pdb.patch \
 external/nss/nss.bzmozilla1238154.patch \
 external/nss/macos-dlopen.patch.0 \
-external/nss/nss.nspr-parallel-win-debug_build.patch \
external/nss/nss.getopt.patch.0 \
 $(if $(filter iOS,$(OS)), \
 external/nss/nss-ios.patch) \
diff --git a/external/nss/macos-dlopen.patch.0 
b/external/nss/macos-dlopen.patch.0
index 8c484e4c6841..1889b8df7cd3 100644
--- a/external/nss/macos-dlopen.patch.0
+++ b/external/nss/macos-dlopen.patch.0
@@ -1,14 +1,14 @@
 --- nspr/pr/src/linking/prlink.c
 +++ nspr/pr/src/linking/prlink.c
-@@ -793,7 +793,7 @@
- /* ensure the file exists if it contains a slash character i.e. path 
*/
- /* DARWIN's dlopen ignores the provided path and checks for the */
- /* plain filename in DYLD_LIBRARY_PATH */
--if (strchr(name, PR_DIRECTORY_SEPARATOR) == NULL ||
-+if (strchr(name, PR_DIRECTORY_SEPARATOR) == NULL || strncmp(name, 
"@loader_path/", 13) == 0 ||
- PR_Access(name, PR_ACCESS_EXISTS) == PR_SUCCESS) {
- h = dlopen(name, dl_flags);
- }
+@@ -799,7 +799,7 @@
+  * The reason is that DARWIN's dlopen ignores the provided path
+  * and checks for the plain filename in DYLD_LIBRARY_PATH,
+  * which could load an unexpected version of a library. */
+-if (strchr(name, PR_DIRECTORY_SEPARATOR) == NULL) {
++if (strchr(name, PR_DIRECTORY_SEPARATOR) == NULL || strncmp(name, 
"@loader_path/", 13) == 0) {
+   /* no slash, allow to load from any location */
+   okToLoad = PR_TRUE;
+ } else {
 --- nss/lib/libpkix/pkix_pl_nss/module/pkix_pl_httpcertstore.c
 +++ nss/lib/libpkix/pkix_pl_nss/module/pkix_pl_httpcertstore.c
 @@ -224,7 +224,11 @@
diff --git a/external/nss/nss.nspr-parallel-win-debug_build.patch 
b/external/nss/nss.nspr-parallel-win-debug_build.patch
deleted file mode 100644
index 86b55e1ccf7f..
--- a/external/nss/nss.nspr-parallel-win-debug_build.patch
+++ /dev/null
@@ -1,40 +0,0 @@
-Änderung:4866:23940b78e965
-Nutzer:  Jan-Marek Glogowski 
-Datum:   Fri May 01 22:50:55 2020 +
-Dateien: pr/tests/Makefile.in
-Beschreibung:
-Bug 290526 Write separate PDBs for test OBJs r=glandium
-
-Quite often when running a parallel NSS build, I get the following
-compiler error message, resulting in a build failure, despite
-compiling with the -FS flag:
-
-.../nss/nspr/pr/tests/zerolen.c: fatal error C1041:
-Programmdatenbank "...\nss\nspr\out\pr\tests\vc140.pdb" kann nicht
-ge<94>ffnet werden; verwenden Sie /FS, wenn mehrere CL.EXE in
-dieselbe .PDB-Datei schreiben.
-
-The failing source file is always one of the last test object
-files. But the actual problem is not the compiler accessing the
-PDB file, but

[Libreoffice-commits] core.git: Branch 'libreoffice-7-0' - sw/source

2020-08-10 Thread Caolán McNamara (via logerrit)
 sw/source/uibase/sidebar/ThemePanel.cxx |   23 ++-
 1 file changed, 14 insertions(+), 9 deletions(-)

New commits:
commit bfcb7d1ef8185739de3fae4940bf46b3fe0e754b
Author: Caolán McNamara 
AuthorDate: Thu Aug 6 11:30:49 2020 +0100
Commit: Xisco Fauli 
CommitDate: Mon Aug 10 11:38:03 2020 +0200

tdf#135488 ensure something is selected in experimental theme colorset

so double click without something selected can't normally happen,
and bail out and do nothing if it does happen

Change-Id: Ic9ed756d4b2b51b87f09786ab7b8506b2d720d84
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100217
Tested-by: Jenkins
Reviewed-by: Xisco Fauli 

diff --git a/sw/source/uibase/sidebar/ThemePanel.cxx 
b/sw/source/uibase/sidebar/ThemePanel.cxx
index 18abd157b71a..290ca4106fb3 100644
--- a/sw/source/uibase/sidebar/ThemePanel.cxx
+++ b/sw/source/uibase/sidebar/ThemePanel.cxx
@@ -444,6 +444,9 @@ ThemePanel::ThemePanel(vcl::Window* pParent,
 }
 
 mxValueSetColors->SetOptimalSize();
+
+if (!aColorSets.empty())
+mxValueSetColors->SelectItem(1); // ItemId 1, position 0
 }
 
 ThemePanel::~ThemePanel()
@@ -480,17 +483,19 @@ IMPL_LINK_NOARG(ThemePanel, DoubleClickHdl, 
weld::TreeView&, bool)
 void ThemePanel::DoubleClickHdl()
 {
 SwDocShell* pDocSh = static_cast(SfxObjectShell::Current());
-if (pDocSh)
-{
-OUString sEntryFonts = mxListBoxFonts->get_selected_text();
-sal_uInt32 nItemId = mxValueSetColors->GetSelectedItemId();
-sal_uInt32 nIndex = nItemId - 1;
-OUString sEntryColors = maColorSets.getColorSet(nIndex).getName();
+if (!pDocSh)
+return;
 
-StyleSet aStyleSet = setupThemes();
+sal_uInt32 nItemId = mxValueSetColors->GetSelectedItemId();
+if (!nItemId)
+return;
+OUString sEntryFonts = mxListBoxFonts->get_selected_text();
+sal_uInt32 nIndex = nItemId - 1;
+OUString sEntryColors = maColorSets.getColorSet(nIndex).getName();
 
-applyTheme(pDocSh->GetStyleSheetPool(), sEntryFonts, sEntryColors, 
aStyleSet, maColorSets);
-}
+StyleSet aStyleSet = setupThemes();
+
+applyTheme(pDocSh->GetStyleSheetPool(), sEntryFonts, sEntryColors, 
aStyleSet, maColorSets);
 }
 
 void ThemePanel::NotifyItemUpdate(const sal_uInt16 /*nSId*/,
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-7-0' - sd/source

2020-08-10 Thread Jim Raykowski (via logerrit)
 sd/source/ui/sidebar/DocumentHelper.cxx |5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

New commits:
commit 41e180f500a42a3afd53b673f538becd75a8f120
Author: Jim Raykowski 
AuthorDate: Thu Jul 16 13:25:47 2020 -0800
Commit: Xisco Fauli 
CommitDate: Mon Aug 10 11:31:25 2020 +0200

tdf#134847 fix nullptr use crash

Grabbing focus back to the document when OLE object last had focus sets
focus to the frame containing the OLE object. sd::ViewShell is no good
in this case and crashes when undo manager tries to pass ViewShellId to
EnterListAction. Other areas in the code set ViewShellId(-1) and then
check if ViewShell is good before getting ViewShellId. This patch
follows these practices.

Change-Id: I89093686c4d98148ac032832711f80479366741d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99759
Tested-by: Jenkins
Reviewed-by: Jim Raykowski 
(cherry picked from commit f52f86a3905fc099832187ad20fd29f0ab73b43f)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100226
Reviewed-by: Xisco Fauli 

diff --git a/sd/source/ui/sidebar/DocumentHelper.cxx 
b/sd/source/ui/sidebar/DocumentHelper.cxx
index 151cc510622e..08fab9f247d3 100644
--- a/sd/source/ui/sidebar/DocumentHelper.cxx
+++ b/sd/source/ui/sidebar/DocumentHelper.cxx
@@ -312,9 +312,12 @@ void DocumentHelper::AssignMasterPageToPageList (
 if (aCleanedList.empty() )
 return;
 
+ViewShellId  nViewShellId(-1);
+if (sd::ViewShell* pViewShell = rTargetDocument.GetDocSh()->GetViewShell())
+nViewShellId = pViewShell->GetViewShellBase().GetViewShellId();
 SfxUndoManager* pUndoMgr = rTargetDocument.GetDocSh()->GetUndoManager();
 if( pUndoMgr )
-pUndoMgr->EnterListAction(SdResId(STR_UNDO_SET_PRESLAYOUT), 
OUString(), 0, 
rTargetDocument.GetDocSh()->GetViewShell()->GetViewShellBase().GetViewShellId());
+pUndoMgr->EnterListAction(SdResId(STR_UNDO_SET_PRESLAYOUT), 
OUString(), 0, nViewShellId);
 
 SdPage* pMasterPageInDocument = 
ProvideMasterPage(rTargetDocument,pMasterPage,rpPageList);
 if (pMasterPageInDocument == nullptr)
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-7-0' - basic/qa basic/source

2020-08-10 Thread Mike Kaganski (via logerrit)
 basic/qa/basic_coverage/test_for_each.vb |   43 +++
 basic/source/runtime/runtime.cxx |5 ++-
 2 files changed, 46 insertions(+), 2 deletions(-)

New commits:
commit 82ab191a074141aafa7567fc3c3cad9bc7ea0b24
Author: Mike Kaganski 
AuthorDate: Thu Aug 6 18:50:08 2020 +0300
Commit: Xisco Fauli 
CommitDate: Mon Aug 10 11:28:13 2020 +0200

tdf#135470: Fix checks

The check should prevent error in case when the variable has
wrong type. IsObject returns false e.g. for arrays, which are
valid iterables. So proper check is using GetFullType, which
will give SbxOBJECT for any variable for which GetObject does
not set error.

"Next" should also generate an error for uninitialized loops.

Regression after 5760c94b8847164f9a7a181f031c7c86643944af

Change-Id: Ib58e1cc76ef63aa1ff8a86c9d5dc956e4780fb49
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100258
Tested-by: Jenkins
Reviewed-by: Mike Kaganski 
(cherry picked from commit e67ffd5a62c8c27697da594a0fea2362317f221f)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100225
Reviewed-by: Xisco Fauli 

diff --git a/basic/qa/basic_coverage/test_for_each.vb 
b/basic/qa/basic_coverage/test_for_each.vb
new file mode 100644
index ..654513e88332
--- /dev/null
+++ b/basic/qa/basic_coverage/test_for_each.vb
@@ -0,0 +1,43 @@
+'
+' This file is part of the LibreOffice project.
+'
+' This Source Code Form is subject to the terms of the Mozilla Public
+' License, v. 2.0. If a copy of the MPL was not distributed with this
+' file, You can obtain one at http://mozilla.org/MPL/2.0/.
+'
+
+Function doUnitTest as Integer
+Dim n As Integer, i
+Dim a(3)
+n = 0
+For Each i In a
+n = n + 1
+Next i
+If n <> 4 Then
+doUnitTest = "For Each over array failed"
+Exit Function
+End If
+
+If TestInvalidForEachWithErrorHandler <> "13 91 14 " Then
+doUnitTest = "For Each doesn't generate proper errors on bad arguments"
+Exit Function
+End If
+
+doUnitTest = 1
+End Function
+
+Function TestInvalidForEachWithErrorHandler
+Dim s As String
+On Error Goto ErrHandler
+' This For Each is given a bad iterable; it must generate first error ("Data 
type mismatch") for b;
+For Each a In b
+' Then proceed here (Resume Next from ErrHandler), and generate "Object 
variable not set" for c;
+c.d
+' Then proceed here (Resume Next from ErrHandler), and generate "Invalid 
parameter" at Next.
+Next
+TestInvalidForEachWithErrorHandler = s
+Exit Function
+ErrHandler:
+s = s & Err & " "
+Resume Next
+End Function
diff --git a/basic/source/runtime/runtime.cxx b/basic/source/runtime/runtime.cxx
index 774d58cf084a..7eb0d022e8ee 100644
--- a/basic/source/runtime/runtime.cxx
+++ b/basic/source/runtime/runtime.cxx
@@ -1144,7 +1144,7 @@ void SbiRuntime::PushForEach()
 pForStk = p;
 
 SbxVariableRef xObjVar = PopVar();
-SbxBase* pObj = xObjVar.is() && xObjVar->IsObject() ? xObjVar->GetObject() 
: nullptr;
+SbxBase* pObj = xObjVar && xObjVar->GetFullType() == SbxOBJECT ? 
xObjVar->GetObject() : nullptr;
 
 if (SbxDimArray* pArray = dynamic_cast(pObj))
 {
@@ -3118,8 +3118,9 @@ void SbiRuntime::StepTESTFOR( sal_uInt32 nOp1 )
 }
 case ForType::Error:
 {
-// We are in Resume Next mode, and we already had one iteration
+// We are in Resume Next mode after failed loop initialization
 bEndLoop = true;
+Error(ERRCODE_BASIC_BAD_PARAMETER);
 break;
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-7-0' - oox/qa oox/source

2020-08-10 Thread Miklos Vajna (via logerrit)
 oox/qa/unit/data/gradient-multistep-transparency.pptx |binary
 oox/qa/unit/drawingml.cxx |   27 +
 oox/source/drawingml/fillproperties.cxx   |   36 --
 3 files changed, 60 insertions(+), 3 deletions(-)

New commits:
commit 6f1cbda681545fa3667b107830ebea804b08c98c
Author: Miklos Vajna 
AuthorDate: Thu Aug 6 12:31:35 2020 +0200
Commit: Xisco Fauli 
CommitDate: Mon Aug 10 10:43:36 2020 +0200

tdf#134183 PPTX: improve import of transparency in multi-step gradients

Impress core only support a single step, improve the conversion from
multi-step to single step to take transparency into account explicitly.

Once we select the widest segment, look backwards and forward if there
are other next segments which have the same RGB color, just different
transparency and include them. This helps in general, because in case a
0% or 100% transparency is mishandled, that's very visible.

(cherry picked from commit 73993fdb5d4b507694cd0edf80887d19f7e2bf9a)

Change-Id: I11d593c01a6a4b16149ce74c1408c2a39895e941
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100283
Tested-by: Jenkins
Reviewed-by: Xisco Fauli 

diff --git a/oox/qa/unit/data/gradient-multistep-transparency.pptx 
b/oox/qa/unit/data/gradient-multistep-transparency.pptx
new file mode 100644
index ..83946c71b403
Binary files /dev/null and 
b/oox/qa/unit/data/gradient-multistep-transparency.pptx differ
diff --git a/oox/qa/unit/drawingml.cxx b/oox/qa/unit/drawingml.cxx
index ee5ae764af70..d88f91797304 100644
--- a/oox/qa/unit/drawingml.cxx
+++ b/oox/qa/unit/drawingml.cxx
@@ -10,7 +10,9 @@
 #include 
 #include 
 
+#include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -199,6 +201,31 @@ CPPUNIT_TEST_FIXTURE(OoxDrawingmlTest, 
testChartDataLabelCharColor)
 CPPUNIT_ASSERT_EQUAL(static_cast(0xff), nCharColor);
 }
 
+CPPUNIT_TEST_FIXTURE(OoxDrawingmlTest, testGradientMultiStepTransparency)
+{
+// Load a document with a multi-step gradient.
+OUString aURL
+= m_directories.getURLFromSrc(DATA_DIRECTORY) + 
"gradient-multistep-transparency.pptx";
+load(aURL);
+
+// Check the end transparency of the gradient.
+uno::Reference 
xDrawPagesSupplier(getComponent(), uno::UNO_QUERY);
+uno::Reference 
xDrawPage(xDrawPagesSupplier->getDrawPages()->getByIndex(0),
+ uno::UNO_QUERY);
+uno::Reference xShape(xDrawPage->getByIndex(1), 
uno::UNO_QUERY);
+CPPUNIT_ASSERT_EQUAL(OUString("Rectangle 4"), xShape->getName());
+uno::Reference xShapeProps(xShape, uno::UNO_QUERY);
+awt::Gradient aTransparence;
+xShapeProps->getPropertyValue("FillTransparenceGradient") >>= 
aTransparence;
+
+// Without the accompanying fix in place, this test would have failed with:
+// - Expected: 16777215 (0xff)
+// - Actual  : 3487029 (0x353535)
+// i.e. the end transparency was not 100%, but was 21%, leading to an 
unexpected visible line on
+// the right of this shape.
+CPPUNIT_ASSERT_EQUAL(static_cast(0xff), 
aTransparence.EndColor);
+}
+
 CPPUNIT_PLUGIN_IMPLEMENT();
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/oox/source/drawingml/fillproperties.cxx 
b/oox/source/drawingml/fillproperties.cxx
index f203e9e2823e..7082cbee0186 100644
--- a/oox/source/drawingml/fillproperties.cxx
+++ b/oox/source/drawingml/fillproperties.cxx
@@ -512,21 +512,51 @@ void FillProperties::pushToPropMap( ShapePropertyMap& 
rPropMap,
 // convert DrawingML angle (in 1/6 degrees) to API 
angle (in 1/10 degrees)
 aGradient.Angle = static_cast< sal_Int16 >( (8100 - 
(nDmlAngle / (PER_DEGREE / 10))) % 3600 );
 Color aStartColor, aEndColor;
+
+// Try to grow the widest segment backwards: if a previous 
segment has the same
+// color, just different transparency, include it.
+while (aWidestSegmentStart != aGradientStops.begin())
+{
+auto it = std::prev(aWidestSegmentStart);
+if (it->second.getColor(rGraphicHelper, nPhClr)
+!= 
aWidestSegmentStart->second.getColor(rGraphicHelper, nPhClr))
+{
+break;
+}
+
+aWidestSegmentStart = it;
+}
+
+auto aWidestSegmentEnd = std::next(aWidestSegmentStart);
+// Try to grow the widest segment forward: if a neext 
segment has the same
+// color, just different transparency, include it.
+while (aWidestSegmentEnd != 
std::prev(aGradientStops.end()))
+{
+auto it = std::next(aWidestSegmentEnd);
+if (it->second.getCo

[Libreoffice-commits] core.git: Branch 'libreoffice-7-0' - sw/qa writerfilter/source

2020-08-10 Thread Serge Krot (via logerrit)
 sw/qa/extras/ooxmlexport/ooxmlexport13.cxx |2 +-
 sw/qa/extras/ooxmlexport/ooxmlexport4.cxx  |5 +++--
 sw/qa/extras/ooxmlexport/ooxmlexport8.cxx  |3 ++-
 writerfilter/source/dmapper/SdtHelper.cxx  |9 +
 4 files changed, 15 insertions(+), 4 deletions(-)

New commits:
commit 73b2bacf2f3f531896fc20aeecb51d433ecd5a9c
Author: Serge Krot 
AuthorDate: Mon Jul 20 15:15:47 2020 +0200
Commit: Xisco Fauli 
CommitDate: Mon Aug 10 10:39:39 2020 +0200

tdf#134572 DOCX: Incorrect default value in dropdown text fields

Change-Id: I3169e817c2f033d1525adc3b02ac3680ad220d70
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99074
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens 
(cherry picked from commit 89b6b2dbb728abea2186ff1ae158c0adb67d05be)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100216
Reviewed-by: Xisco Fauli 

diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport13.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport13.cxx
index 430749768862..1b9621b473b0 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport13.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport13.cxx
@@ -1052,7 +1052,7 @@ DECLARE_OOXMLEXPORT_TEST(tdf119809, "tdf119809.docx")
 
CPPUNIT_ASSERT(xServiceInfo->supportsService("com.sun.star.text.textfield.DropDown"));
 
 uno::Sequence aItems = getProperty< uno::Sequence 
>(aField, "Items");
-CPPUNIT_ASSERT_EQUAL(sal_Int32(0), aItems.getLength());
+CPPUNIT_ASSERT_EQUAL(sal_Int32(1), aItems.getLength());
 }
 }
 
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport4.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport4.cxx
index 22d069a90864..c6cb3c351843 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport4.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport4.cxx
@@ -741,9 +741,10 @@ void Test::verifyComboBoxExport(bool aComboBoxAsDropDown)
 
CPPUNIT_ASSERT(xServiceInfo->supportsService("com.sun.star.text.textfield.DropDown"));
 
 uno::Sequence aItems = getProperty< uno::Sequence 
>(aField, "Items");
-CPPUNIT_ASSERT_EQUAL(sal_Int32(2), aItems.getLength());
+CPPUNIT_ASSERT_EQUAL(sal_Int32(3), aItems.getLength());
 CPPUNIT_ASSERT_EQUAL(OUString("manolo"), aItems[0]);
 CPPUNIT_ASSERT_EQUAL(OUString("pepito"), aItems[1]);
+CPPUNIT_ASSERT_EQUAL(OUString("Manolo"), aItems[2]);
 }
 else
 {
@@ -767,7 +768,7 @@ DECLARE_OOXMLEXPORT_EXPORTONLY_TEST(testComboBoxControl, 
"combobox-control.docx"
 xmlDocUniquePtr pXmlDoc = parseExport("word/document.xml");
 assertXPath(pXmlDoc, 
"/w:document/w:body/w:p/w:sdt/w:sdtPr/w:dropDownList/w:listItem[1]", "value", 
"manolo");
 assertXPath(pXmlDoc, 
"/w:document/w:body/w:p/w:sdt/w:sdtPr/w:dropDownList/w:listItem[2]", "value", 
"pepito");
-assertXPathContent(pXmlDoc, 
"/w:document/w:body/w:p/w:sdt/w:sdtContent/w:r/w:t", "manolo");
+assertXPathContent(pXmlDoc, 
"/w:document/w:body/w:p/w:sdt/w:sdtContent/w:r/w:t", "Manolo");
 
 // check imported control
 verifyComboBoxExport(getShapes() == 0);
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport8.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport8.cxx
index 5f401b266bfc..5b09147a0ad2 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport8.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport8.cxx
@@ -948,9 +948,10 @@ DECLARE_OOXMLEXPORT_TEST(testN779630, "n779630.docx")
 
CPPUNIT_ASSERT(xServiceInfo->supportsService("com.sun.star.text.textfield.DropDown"));
 
 uno::Sequence aItems = getProperty< uno::Sequence 
>(aField, "Items");
-CPPUNIT_ASSERT_EQUAL(sal_Int32(2), aItems.getLength());
+CPPUNIT_ASSERT_EQUAL(sal_Int32(3), aItems.getLength());
 CPPUNIT_ASSERT_EQUAL(OUString("Yes"), aItems[0]);
 CPPUNIT_ASSERT_EQUAL(OUString("No"), aItems[1]);
+CPPUNIT_ASSERT_EQUAL(OUString("dropdown default text"), aItems[2]);
 }
 }
 
diff --git a/writerfilter/source/dmapper/SdtHelper.cxx 
b/writerfilter/source/dmapper/SdtHelper.cxx
index 4e437ae0e562..60153d4c6126 100644
--- a/writerfilter/source/dmapper/SdtHelper.cxx
+++ b/writerfilter/source/dmapper/SdtHelper.cxx
@@ -90,6 +90,15 @@ void SdtHelper::createDropDownControl()
 
m_rDM_Impl.GetTextFactory()->createInstance("com.sun.star.text.TextField.DropDown"),
 uno::UNO_QUERY);
 
+const auto it = std::find_if(
+m_aDropDownItems.begin(), m_aDropDownItems.end(),
+[aDefaultText](const OUString& item) -> bool { return 
!item.compareTo(aDefaultText); });
+
+if (m_aDropDownItems.end() == it)
+{
+m_aDropDownItems.push_back(aDefaultText);
+}
+
 // set properties
 uno::Reference xPropertySet(xControlModel, 
uno::UNO_QUERY);
 xPropertySet->setPropertyValue("SelectedItem", 
uno::makeAny(aDefaultText));
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: dbaccess/source

2020-08-10 Thread Caolán McNamara (via logerrit)
 dbaccess/source/ui/app/AppDetailView.cxx |   24 +---
 dbaccess/source/ui/app/AppDetailView.hxx |4 +++-
 2 files changed, 20 insertions(+), 8 deletions(-)

New commits:
commit 6ad2f463784a24c566477cdd60ae729651bb8564
Author: Caolán McNamara 
AuthorDate: Sat Aug 8 14:06:49 2020 +0100
Commit: Caolán McNamara 
CommitDate: Mon Aug 10 10:24:30 2020 +0200

restore removing the entries which are not enabled currently

Change-Id: If24474e362aa80a3a0240a6ab899aa9e71066aef
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100389
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 

diff --git a/dbaccess/source/ui/app/AppDetailView.cxx 
b/dbaccess/source/ui/app/AppDetailView.cxx
index 2f77244699d3..e6e318c661d4 100644
--- a/dbaccess/source/ui/app/AppDetailView.cxx
+++ b/dbaccess/source/ui/app/AppDetailView.cxx
@@ -51,10 +51,11 @@ using namespace ::com::sun::star::beans;
 using ::com::sun::star::util::URL;
 using ::com::sun::star::sdb::application::NamedDatabaseObject;
 
-TaskEntry::TaskEntry( const char* _pAsciiUNOCommand, const char* _pHelpID, 
const char* pTitleResourceID )
+TaskEntry::TaskEntry( const char* _pAsciiUNOCommand, const char* _pHelpID, 
const char* pTitleResourceID, bool _bHideWhenDisabled )
 :sUNOCommand( OUString::createFromAscii( _pAsciiUNOCommand ) )
 ,pHelpID( _pHelpID )
 ,sTitle( DBA_RES(pTitleResourceID) )
+,bHideWhenDisabled( _bHideWhenDisabled )
 {
 }
 
@@ -341,9 +342,7 @@ void OApplicationDetailView::impl_createPage( ElementType 
_eType, const Referenc
 Resize();
 }
 
-namespace {
-
-void impl_fillTaskPaneData(ElementType _eType, TaskPaneData& _rData)
+void OApplicationDetailView::impl_fillTaskPaneData(ElementType _eType, 
TaskPaneData& _rData) const
 {
 TaskEntryList& rList( _rData.aTasks );
 rList.clear(); rList.reserve( 4 );
@@ -353,7 +352,7 @@ void impl_fillTaskPaneData(ElementType _eType, 
TaskPaneData& _rData)
 case E_TABLE:
 rList.emplace_back( ".uno:DBNewTable", 
RID_STR_TABLES_HELP_TEXT_DESIGN, RID_STR_NEW_TABLE );
 rList.emplace_back( ".uno:DBNewTableAutoPilot", 
RID_STR_TABLES_HELP_TEXT_WIZARD, RID_STR_NEW_TABLE_AUTO );
-rList.emplace_back( ".uno:DBNewView", RID_STR_VIEWS_HELP_TEXT_DESIGN, 
RID_STR_NEW_VIEW );
+rList.emplace_back( ".uno:DBNewView", RID_STR_VIEWS_HELP_TEXT_DESIGN, 
RID_STR_NEW_VIEW, true );
 _rData.pTitleId = RID_STR_TABLES_CONTAINER;
 break;
 
@@ -364,7 +363,7 @@ void impl_fillTaskPaneData(ElementType _eType, 
TaskPaneData& _rData)
 break;
 
 case E_REPORT:
-rList.emplace_back( ".uno:DBNewReport", RID_STR_REPORT_HELP_TEXT, 
RID_STR_NEW_REPORT );
+rList.emplace_back( ".uno:DBNewReport", RID_STR_REPORT_HELP_TEXT, 
RID_STR_NEW_REPORT, true );
 rList.emplace_back( ".uno:DBNewReportAutoPilot", 
RID_STR_REPORTS_HELP_TEXT_WIZARD, RID_STR_NEW_REPORT_AUTO );
 _rData.pTitleId = RID_STR_REPORTS_CONTAINER;
 break;
@@ -379,8 +378,19 @@ void impl_fillTaskPaneData(ElementType _eType, 
TaskPaneData& _rData)
 default:
 OSL_FAIL( "OApplicationDetailView::impl_fillTaskPaneData: illegal 
element type!" );
 }
-}
 
+// remove the entries which are not enabled currently
+for (TaskEntryList::iterator pTask = rList.begin(); pTask != rList.end();)
+{
+if  (   pTask->bHideWhenDisabled
+&&  
!getBorderWin().getView()->getCommandController().isCommandEnabled( 
pTask->sUNOCommand )
+)
+pTask = rList.erase( pTask );
+else
+{
+++pTask;
+}
+}
 }
 
 const TaskPaneData& OApplicationDetailView::impl_getTaskPaneData( ElementType 
_eType )
diff --git a/dbaccess/source/ui/app/AppDetailView.hxx 
b/dbaccess/source/ui/app/AppDetailView.hxx
index bca1145f2010..a00e7f4ba642 100644
--- a/dbaccess/source/ui/app/AppDetailView.hxx
+++ b/dbaccess/source/ui/app/AppDetailView.hxx
@@ -47,11 +47,12 @@ namespace dbaui
 OUStringsUNOCommand;
 const char* pHelpID;
 OUStringsTitle;
+boolbHideWhenDisabled;
 // TODO: we should be consistent in the task pane and the 
menus/toolbars:
 // If an entry is disabled in the latter, it should also be 
disabled in the former.
 // If an entry is *hidden* in the former, it should also be hidden 
in the latter.
 
-TaskEntry( const char* _pAsciiUNOCommand, const char* pHelpID, const 
char* pTitleResourceID );
+TaskEntry( const char* _pAsciiUNOCommand, const char* pHelpID, const 
char* pTitleResourceID, bool _bHideWhenDisabled = false );
 };
 typedef std::vector< TaskEntry >  TaskEntryList;
 
@@ -313,6 +314,7 @@ namespace dbaui
 );
 
 const TaskPaneData& impl_getTaskPaneData( ElementType _eType );
+voidimpl_fillTaskPaneData( ElementType _eType, 
TaskPaneData& _rData ) const;
 };
 }
 #endif // INCLUDED_DBACCESS_

[Libreoffice-commits] core.git: include/svtools svtools/source svtools/uiconfig

2020-08-10 Thread Caolán McNamara (via logerrit)
 include/svtools/editbrowsebox.hxx |1 
 svtools/source/brwbox/ebbcontrols.cxx |4 ++
 svtools/uiconfig/ui/datewindow.ui |   67 --
 3 files changed, 45 insertions(+), 27 deletions(-)

New commits:
commit dd0ee0ccf79918526cd1753e66a55f289a7b6ea3
Author: Caolán McNamara 
AuthorDate: Sun Aug 9 16:01:37 2020 +0100
Commit: Caolán McNamara 
CommitDate: Mon Aug 10 10:24:04 2020 +0200

tdf#135529 today/none only used by DateControl

Change-Id: Id8987e65ef3fae9c009af59d3a1f9d0e83726648
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100404
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 

diff --git a/include/svtools/editbrowsebox.hxx 
b/include/svtools/editbrowsebox.hxx
index 130dae510486..725cc081d044 100644
--- a/include/svtools/editbrowsebox.hxx
+++ b/include/svtools/editbrowsebox.hxx
@@ -687,6 +687,7 @@ namespace svt
 std::unique_ptr m_xCalendarBuilder;
 std::unique_ptr m_xTopLevel;
 std::unique_ptr m_xCalendar;
+std::unique_ptr m_xExtras;
 std::unique_ptr m_xTodayBtn;
 std::unique_ptr m_xNoneBtn;
 
diff --git a/svtools/source/brwbox/ebbcontrols.cxx 
b/svtools/source/brwbox/ebbcontrols.cxx
index 6958a1f63fab..b15927af9e58 100644
--- a/svtools/source/brwbox/ebbcontrols.cxx
+++ b/svtools/source/brwbox/ebbcontrols.cxx
@@ -429,6 +429,7 @@ namespace svt
 , m_xCalendarBuilder(Application::CreateBuilder(m_xMenuButton.get(), 
"svt/ui/datewindow.ui"))
 , m_xTopLevel(m_xCalendarBuilder->weld_widget("date_popup_window"))
 , m_xCalendar(m_xCalendarBuilder->weld_calendar("date"))
+, m_xExtras(m_xCalendarBuilder->weld_widget("extras"))
 , m_xTodayBtn(m_xCalendarBuilder->weld_button("today"))
 , m_xNoneBtn(m_xCalendarBuilder->weld_button("none"))
 {
@@ -439,6 +440,8 @@ namespace svt
 m_xMenuButton->set_visible(bDropDown);
 m_xMenuButton->connect_toggled(LINK(this, DateControl, ToggleHdl));
 
+m_xExtras->show();
+
 m_xTodayBtn->connect_clicked(LINK(this, DateControl, ImplClickHdl));
 m_xNoneBtn->connect_clicked(LINK(this, DateControl, ImplClickHdl));
 
@@ -484,6 +487,7 @@ namespace svt
 {
 m_xTodayBtn.reset();
 m_xNoneBtn.reset();
+m_xExtras.reset();
 m_xCalendar.reset();
 m_xTopLevel.reset();
 m_xCalendarBuilder.reset();
diff --git a/svtools/uiconfig/ui/datewindow.ui 
b/svtools/uiconfig/ui/datewindow.ui
index 0e7729afe74e..b3c1b2d73085 100644
--- a/svtools/uiconfig/ui/datewindow.ui
+++ b/svtools/uiconfig/ui/datewindow.ui
@@ -27,46 +27,59 @@
   
 
 
-  
-True
-False
-  
-  
-False
-True
-1
-  
-
-
-  
-True
+  
 False
+True
+vertical
 6
-spread
 
-  
-Today
+  
 True
-True
-True
-True
-True
+False
   
   
-True
+False
 True
 0
   
 
 
-  
-None
+  
 True
-True
-True
+False
+6
+spread
+
+  
+Today
+True
+True
+True
+True
+True
+  
+  
+True
+True
+0
+  
+
+
+  
+None
+True
+True
+True
+  
+  
+True
+True
+1
+  
+
   
   
-True
+False
 True
 1
   
@@ -75,7 +88,7 @@
   
 False
 True
-2
+1
   
 
   
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-6.4' - include/tools tools/source

2020-08-10 Thread Tomaž Vajngerl (via logerrit)
 include/tools/json_writer.hxx |1 +
 tools/source/misc/json_writer.cxx |8 
 2 files changed, 9 insertions(+)

New commits:
commit cf6388b6a5f3e3baa54ed8b02020e9a25487b32b
Author: Tomaž Vajngerl 
AuthorDate: Sat Aug 1 08:40:10 2020 +0200
Commit: Tomaž Vajngerl 
CommitDate: Mon Aug 10 09:35:19 2020 +0200

add extractAsOString to JsonWriter

Change-Id: I761fcf885a4965f88107f84b839108960805a1b8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99909
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Tomaž Vajngerl 

diff --git a/include/tools/json_writer.hxx b/include/tools/json_writer.hxx
index 34930584730d..588577d41303 100644
--- a/include/tools/json_writer.hxx
+++ b/include/tools/json_writer.hxx
@@ -47,6 +47,7 @@ public:
 /** Hands ownership of the the underlying storage buffer to the caller,
  * after this no more document modifications may be written. */
 char* extractData();
+OString extractAsOString();
 
 private:
 void endNode();
diff --git a/tools/source/misc/json_writer.cxx 
b/tools/source/misc/json_writer.cxx
index 3040fda8a080..ab109dae669c 100644
--- a/tools/source/misc/json_writer.cxx
+++ b/tools/source/misc/json_writer.cxx
@@ -255,5 +255,13 @@ char* JsonWriter::extractData()
 return pRet;
 }
 
+OString JsonWriter::extractAsOString()
+{
+char* pChar = extractData();
+OString ret(pChar);
+free(pChar);
+return ret;
+}
+
 } // namespace tools
 /* vim:set shiftwidth=4 softtabstop=4 expandtab cinoptions=b1,g0,N-s 
cinkeys+=0=break: */
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits