[Libreoffice-bugs] [Bug 61558] High CPU load when scrolling a document with many comments

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=61558

--- Comment #54 from Dominic  ---
If I scroll continuously (by placing the cursor in the scroll bar and scroll
down the page at a time) the CPU usage falls to about half of what it was. The
CPU usage rises as soon as the scrolling stops. That is strange!
 We have a guide for the setup, which you can find on the following
link:https://www.techsupportdubai.com/led-tv-repair/
If I take a screen dump with Alt/PrtScr (or it might be AltGr/PrtScr) the CPU
usage jumps while the screenshot is taken, and then falls to virtually zero.
This last bit is a bit intermittent - mostly it does. It might depend on
whether the Writer Window is active when the screen dump is taken. Why does the
Writer CPU fall after the screenshot?

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: cui/Library_cui.mk cui/source cui/uiconfig include/editeng include/svl include/svx offapi/com offapi/UnoApi_offapi.mk RepositoryExternal.mk svx/source

2019-07-24 Thread Shubham Goyal (via logerrit)
 RepositoryExternal.mk |2 
 cui/Library_cui.mk|2 
 cui/source/dialogs/QrCodeGenDialog.cxx|  288 ++
 cui/source/inc/QrCodeGenDialog.hxx|   58 +++
 cui/uiconfig/ui/qrcodegen.ui  |  273 +
 include/editeng/unoprnms.hxx  |1 
 include/svl/solar.hrc |2 
 include/svx/svdograf.hxx  |   15 
 include/svx/unoshprp.hxx  |7 
 offapi/UnoApi_offapi.mk   |2 
 offapi/com/sun/star/drawing/GraphicObjectShape.idl|5 
 offapi/com/sun/star/drawing/QRCode.idl|   42 ++
 offapi/com/sun/star/drawing/QRCodeErrorCorrection.idl |   50 +++
 svx/source/svdraw/svdograf.cxx|   10 
 svx/source/unodraw/unoshap2.cxx   |   22 +
 15 files changed, 775 insertions(+), 4 deletions(-)

New commits:
commit 2de42b53b7c23223c38e64a75eae248d8a0cd4ec
Author: Shubham Goyal <22shub...@gmail.com>
AuthorDate: Wed Jul 3 12:15:08 2019 +0530
Commit: Samuel Mehrbrodt 
CommitDate: Thu Jul 25 07:15:10 2019 +0200

QRCode Dialog Box feature

The patch handles the created QR code as a Customized Shape (Graphic
Object)

Change-Id: I1cee6f0e7fac585de880a9ac34e3bc441a4b7390
Reviewed-on: https://gerrit.libreoffice.org/74167
Reviewed-by: Stephan Bergmann 
Tested-by: Jenkins
Reviewed-by: Mike Kaganski 
Reviewed-by: Samuel Mehrbrodt 

diff --git a/RepositoryExternal.mk b/RepositoryExternal.mk
index a66077d3bcbb..021d9a7b650e 100644
--- a/RepositoryExternal.mk
+++ b/RepositoryExternal.mk
@@ -4172,7 +4172,7 @@ gb_ExternalProject__use_qrcodegen :=
 else # !SYSTEM_QRCODEGEN
 
 define gb_LinkTarget__use_qrcodegen
-l$(call gb_LinkTarget_use_unpacked,$(1),qrcodegen)
+$(call gb_LinkTarget_use_unpacked,$(1),qrcodegen)
 $(call gb_LinkTarget_set_include,$(1),\
-I$(call gb_UnpackedTarball_get_dir,qrcodegen/cpp/)\
$$(INCLUDE) \
diff --git a/cui/Library_cui.mk b/cui/Library_cui.mk
index e3cd0c36ba4f..620d3de8fa69 100644
--- a/cui/Library_cui.mk
+++ b/cui/Library_cui.mk
@@ -73,6 +73,7 @@ $(eval $(call gb_Library_use_externals,cui,\
 libxml2 \
 orcus-parser \
 orcus \
+qrcodegen \
 ))
 ifeq ($(DISABLE_GUI),)
 $(eval $(call gb_Library_use_externals,cui,\
@@ -129,6 +130,7 @@ $(eval $(call gb_Library_add_exception_objects,cui,\
 cui/source/dialogs/screenshotannotationdlg \
 cui/source/dialogs/pastedlg \
 cui/source/dialogs/postdlg \
+cui/source/dialogs/QrCodeGenDialog \
 cui/source/dialogs/scriptdlg \
 cui/source/dialogs/SignatureLineDialogBase \
 cui/source/dialogs/SignatureLineDialog \
diff --git a/cui/source/dialogs/QrCodeGenDialog.cxx 
b/cui/source/dialogs/QrCodeGenDialog.cxx
new file mode 100644
index ..628ada1df38b
--- /dev/null
+++ b/cui/source/dialogs/QrCodeGenDialog.cxx
@@ -0,0 +1,288 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4; 
fill-column: 100 -*- */
+/*
+ * 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/.
+ */
+
+#include 
+
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+
+#include 
+
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+
+using namespace css;
+using namespace css::uno;
+using namespace css::beans;
+using namespace css::container;
+using namespace css::frame;
+using namespace css::io;
+using namespace css::lang;
+using namespace css::frame;
+using namespace css::sheet;
+using namespace css::text;
+using namespace css::drawing;
+using namespace css::graphic;
+using namespace qrcodegen;
+
+QrCodeGenDialog::QrCodeGenDialog(weld::Widget* pParent, Reference 
xModel,
+ bool bEditExisting)
+: GenericDialogController(pParent, "cui/ui/qrcodegen.ui", 
"QrCodeGenDialog")
+, m_xModel(std::move(xModel))
+, m_xEdittext(m_xBuilder->weld_entry("edit_text"))
+, m_xRadioLow(m_xBuilder->weld_radio_button("button_low"))
+, m_xRadioMedium(m_xBuilder->weld_radio_button("button_medium"))
+, m_xRadioQuartile(m_xBuilder->weld_radio_button("button_quartile"))
+, m_xRadioHigh(m_xBuilder->weld_radio_button("button_high"))
+, m_xSpinBorder(m_xBuilder->weld_spin_button("edit_border"))
+{
+maBox.AddButton(m_xRadioLow.get());
+maBox.AddButton(m_xRadioMedium.get());
+maBox.AddButton(m_xRadioQuartile.get());
+maBox.AddButton(m_xRadioHigh.get());
+
+// Set ECC to Low by Default.
+if 

[Libreoffice-commits] core.git: extensions/source include/unotools lotuswordpro/inc offapi/com reportbuilder/java sc/source sd/source sw/inc sw/source unotools/source xmloff/source xmlsecurity/inc

2019-07-24 Thread Andrea Gelmini (via logerrit)
 extensions/source/ole/unoconversionutilities.hxx   
 |2 +-
 include/unotools/streamwrap.hxx
 |6 +++---
 lotuswordpro/inc/xfilter/xfparastyle.hxx   
 |2 +-
 offapi/com/sun/star/document/TypeDetection.idl 
 |2 +-
 offapi/com/sun/star/form/FormComponentType.idl 
 |2 +-
 offapi/com/sun/star/ui/XImageManager.idl   
 |2 +-
 
reportbuilder/java/org/libreoffice/report/pentaho/model/OfficeStylesCollection.java
 |2 +-
 reportbuilder/java/org/libreoffice/report/pentaho/output/StyleUtilities.java   
 |6 +++---
 sc/source/core/tool/interpr2.cxx   
 |2 +-
 sc/source/core/tool/interpr3.cxx   
 |2 +-
 sc/source/core/tool/interpr7.cxx   
 |2 +-
 sd/source/ui/inc/tools/AsynchronousTask.hxx
 |2 +-
 sd/source/ui/unoidl/unocpres.cxx   
 |2 +-
 sw/inc/tox.hxx 
 |2 +-
 sw/source/core/doc/CntntIdxStore.cxx   
 |2 +-
 sw/source/core/view/viewsh.cxx 
 |2 +-
 unotools/source/config/configitem.cxx  
 |4 ++--
 xmloff/source/draw/shapeimport.cxx 
 |2 +-
 xmlsecurity/inc/framework/saxeventkeeperimpl.hxx   
 |2 +-
 19 files changed, 24 insertions(+), 24 deletions(-)

New commits:
commit 5ccc8124a03cffca3a1848f754524a06a063cb51
Author: Andrea Gelmini 
AuthorDate: Thu Jul 25 00:42:27 2019 +0200
Commit: Andrea Gelmini 
CommitDate: Thu Jul 25 07:07:25 2019 +0200

Fix typos

Change-Id: I59a0fd175fa5185c15d093d2d9bed9f95bb4cfd5
Reviewed-on: https://gerrit.libreoffice.org/76280
Tested-by: Jenkins
Reviewed-by: Andrea Gelmini 

diff --git a/extensions/source/ole/unoconversionutilities.hxx 
b/extensions/source/ole/unoconversionutilities.hxx
index c75428c87caa..8948de229205 100644
--- a/extensions/source/ole/unoconversionutilities.hxx
+++ b/extensions/source/ole/unoconversionutilities.hxx
@@ -1821,7 +1821,7 @@ Reference 
UnoConversionUtilities::createAdapter(const Sequence xIntAdapterFac;
 xIntAdapterFac= m_smgr->createInstance(INTERFACE_ADAPTER_FACTORY);
 // We create an adapter object that does not only implement the required 
type but also
-// all types that the COM object pretends to implement. An COM object must 
therefore
+// all types that the COM object pretends to implement. A COM object must 
therefore
 // support the property "_implementedInterfaces".
 Reference xIntAdapted;
 Reference xInv(receiver, UNO_QUERY);
diff --git a/include/unotools/streamwrap.hxx b/include/unotools/streamwrap.hxx
index 299b6dfd081f..bb34dd7af7e6 100644
--- a/include/unotools/streamwrap.hxx
+++ b/include/unotools/streamwrap.hxx
@@ -39,7 +39,7 @@ namespace utl
 // workaround for incremental linking bugs in MSVC2015
 class SAL_DLLPUBLIC_TEMPLATE OInputStreamWrapper_Base : public 
cppu::WeakImplHelper< css::io::XInputStream > {};
 
-/// helper class for wrapping an SvStream into an com.sun.star.io::XInputStream
+/// helper class for wrapping an SvStream into a com.sun.star.io::XInputStream
 class UNOTOOLS_DLLPUBLIC OInputStreamWrapper : public OInputStreamWrapper_Base
 {
 protected:
@@ -73,7 +73,7 @@ protected:
 
 //= OSeekableInputStreamWrapper
 
-/** helper class for wrapping an SvStream into an com.sun.star.io::XInputStream
+/** helper class for wrapping an SvStream into a com.sun.star.io::XInputStream
 which is seekable (i.e. supports the com.sun.star.io::XSeekable interface).
 */
 class UNOTOOLS_DLLPUBLIC OSeekableInputStreamWrapper
@@ -119,7 +119,7 @@ protected:
 
 typedef ::cppu::ImplHelper1 <   css::io::XSeekable
 >   OSeekableOutputStreamWrapper_Base;
-/** helper class for wrapping an SvStream into an 
com.sun.star.io::XOutputStream
+/** helper class for wrapping an SvStream into a com.sun.star.io::XOutputStream
 which is seekable (i.e. supports the com.sun.star.io::XSeekable interface).
 */
 class UNOTOOLS_DLLPUBLIC OSeekableOutputStreamWrapper
diff --git a/lotuswordpro/inc/xfilter/xfparastyle.hxx 
b/lotuswordpro/inc/xfilter/xfparastyle.hxx
index 7c066d61c513..8b947a9ea8be 100644
--- a/lotuswordpro/inc/xfilter/xfparastyle.hxx
+++ b/lotuswordpro/inc/xfilter/xfparastyle.hxx
@@ -104,7 +104,7 @@ public:
 public:
 /**
  * @descr   Set layout for the paragraph. When such property was set, this 
paragraph will
- *  start at an 

[Libreoffice-bugs] [Bug 103512] [META] AutoFilter-related bugs and enhancements

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=103512
Bug 103512 depends on bug 122772, which changed state.

Bug 122772 Summary: Table splits after AutoFilter popup menu use
https://bugs.documentfoundation.org/show_bug.cgi?id=122772

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 122772] Table splits after AutoFilter popup menu use

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=122772

Jim Raykowski  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 103512] [META] AutoFilter-related bugs and enhancements

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=103512
Bug 103512 depends on bug 122774, which changed state.

Bug 122774 Summary: All items in AutoFilter popup menu are not keyboard 
accessible
https://bugs.documentfoundation.org/show_bug.cgi?id=122774

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126519] UI - Calculation incorrect with VLOOKUP function, will not correct on recalculate

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126519

--- Comment #2 from a...@patternjugglers.co.nz ---
Created attachment 152972
  --> https://bugs.documentfoundation.org/attachment.cgi?id=152972=edit
Copy of sample problem spreadsheet, with unnecessary extra data removed

• the yellow highlighted cells in column C contain the same value
• the green hightlighted cell in column J has calculated correctly from the
VLOOKUP
• the red highlighted cells in column J are displaying 0.00, which is
incorrect

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126527] find (and replace) by format does not find anything (in some cases)

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126527

--- Comment #4 from stragu  ---
Very interesting! Thanks for testing, m.a.riosv

I should add that using the "Find & Replace > Attributes... > Strikethrough"
works as expected in both documents.

I also just tried to restart in safe mode with user profile as factory
settings, which didn't fix it.

I also noticed that, when selecting single strikethrough as the format searched
for, we can see underneath the Find box:
"Single strikethrough, Indent left 0.0 inch, From top 0.0 inch, From bottom 0.0
inch"
Is that expected? It is confusing, as the search didn't specify any values for
indentation.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126527] find (and replace) by format does not find anything (in some cases)

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126527

m.a.riosv  changed:

   What|Removed |Added

 CC||miguelangelrv@libreoffice.o
   ||rg

--- Comment #3 from m.a.riosv  ---
Works for me
Version: 6.2.5.2 (x64)
Build ID: 1ec314fa52f458adc18c4f025c545a4e8b22c159
CPU threads: 4; OS: Windows 10.0; UI render: GL; VCL: win; 
Locale: es-ES (es_ES); UI-Language: en-US Calc: threaded

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 118384] Sidebar: Style is not added back to hierarchy after deleting and redoing it

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=118384

Jim Raykowski  changed:

   What|Removed |Added

   Assignee|libreoffice-b...@lists.free |rayk...@gmail.com
   |desktop.org |

--- Comment #5 from Jim Raykowski  ---
Here is a patch for this: 
https://gerrit.libreoffice.org/#/c/76279/

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 47807] EDITING: Undo button not active for style change

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=47807

--- Comment #16 from Commit Notification 
 ---
Jim Raykowski committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/+/ed882d693f37779e3a09641e7cd43b7a925d2312%5E%21

tdf#47807 Invalidate bindings to Undo Redo

It will be available in 6.4.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 47807] EDITING: Undo button not active for style change

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=47807

Commit Notification  changed:

   What|Removed |Added

 Whiteboard||target:6.4.0

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

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

2019-07-24 Thread Jim Raykowski (via logerrit)
 include/svx/sdrundomanager.hxx   |2 ++
 svx/source/svdraw/sdrundomanager.cxx |   20 
 2 files changed, 22 insertions(+)

New commits:
commit ed882d693f37779e3a09641e7cd43b7a925d2312
Author: Jim Raykowski 
AuthorDate: Mon Jul 22 21:28:54 2019 -0800
Commit: Jim Raykowski 
CommitDate: Thu Jul 25 00:50:05 2019 +0200

tdf#47807 Invalidate bindings to Undo Redo

Change-Id: I71a16e60395003402372202a755b976d37690a27
Reviewed-on: https://gerrit.libreoffice.org/76146
Tested-by: Jenkins
Reviewed-by: Jim Raykowski 

diff --git a/include/svx/sdrundomanager.hxx b/include/svx/sdrundomanager.hxx
index 60a06bd546c5..854c4be54c51 100644
--- a/include/svx/sdrundomanager.hxx
+++ b/include/svx/sdrundomanager.hxx
@@ -63,6 +63,8 @@ public:
 // by a last undo during text edit
 bool isEndTextEditTriggeredFromUndo() { return 
mbEndTextEditTriggeredFromUndo; }
 void SetDocShell(SfxObjectShell* pDocShell);
+
+virtual void AddUndoAction( std::unique_ptr pAction, bool 
bTryMerg=false ) override;
 };
 
 #endif // INCLUDED_SVX_SDRUNDOMANAGER_HXX
diff --git a/svx/source/svdraw/sdrundomanager.cxx 
b/svx/source/svdraw/sdrundomanager.cxx
index 78e72e76196a..5d13d8e5e8a8 100644
--- a/svx/source/svdraw/sdrundomanager.cxx
+++ b/svx/source/svdraw/sdrundomanager.cxx
@@ -18,7 +18,10 @@
  */
 
 #include 
+#include 
+#include 
 #include 
+#include 
 #include 
 
 SdrUndoManager::SdrUndoManager()
@@ -146,4 +149,21 @@ void SdrUndoManager::EmptyActionsChanged()
 }
 }
 
+void SdrUndoManager::AddUndoAction( std::unique_ptr pAction, 
bool bTryMerg )
+{
+EditUndoManager::AddUndoAction( std::move(pAction), bTryMerg);
+
+if (m_pDocSh)
+{
+SfxViewFrame* pViewFrame = SfxViewFrame::GetFirst(  m_pDocSh );
+while ( pViewFrame )
+{
+pViewFrame->GetBindings().Invalidate( SID_UNDO );
+pViewFrame->GetBindings().Invalidate( SID_REDO );
+pViewFrame = SfxViewFrame::GetNext( *pViewFrame, m_pDocSh );
+}
+}
+}
+
+
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

SHA1/MD5 in sal/rtl/source/digest.c patch license able for OO

2019-07-24 Thread Peter Kovacs
Hello all,


I would like to ask if I or any other from OpenOffice Development team
can port the patch from LibreOffice to OpenOffice to fix this Issue
under the APL2 License, so the Issue can be fixed in OpenOffice without
more work then necessary.

LO Issue: 114939

OO Issue: 127661

Thanks for your considerations.


All the Best

Peter


___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice

[Libreoffice-bugs] [Bug 126532] New: Export PDF Overwrite Original on Save

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126532

Bug ID: 126532
   Summary: Export PDF Overwrite Original on Save
   Product: LibreOffice
   Version: 6.2.4.2 release
  Hardware: x86-64 (AMD64)
OS: Windows (All)
Status: UNCONFIRMED
  Severity: enhancement
  Priority: medium
 Component: Draw
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: grieb.k...@gmail.com

Description:
I open a file (PDF) for editing and want to save (export) that file overwriting
the original file.

Open > Export as PDF > Save (replace currently opened file) > General/Input
Output error (Unable to save export).

I am able to save other file types with them open, but would also like this
feature for the export functionality.


Steps to Reproduce:
1. Open PDF
2. Export as PDF
3. Select the currently opened file

Actual Results:
General Input/Output Error

Expected Results:
File Save


Reproducible: Always


User Profile Reset: Yes



Additional Info:

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 87064] [META] Color picker dialog bugs and enhancements

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=87064

V Stuart Foote  changed:

   What|Removed |Added

 Depends on||126525


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=126525
[Bug 126525] Suite-wide color picker
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126525] Suite-wide color picker

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126525

V Stuart Foote  changed:

   What|Removed |Added

 CC||vstuart.fo...@utsa.edu
 Blocks||87064


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=87064
[Bug 87064] [META] Color picker dialog bugs and enhancements
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 107237] [META] Notebookbar Tabbed

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=107237

V Stuart Foote  changed:

   What|Removed |Added

 Depends on||126530


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=126530
[Bug 126530] Tabbed Notebook Bar Usability Issues on Windows 10
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126530] Tabbed Notebook Bar Usability Issues on Windows 10

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126530

V Stuart Foote  changed:

   What|Removed |Added

 CC||vstuart.fo...@utsa.edu
 Blocks||107237


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=107237
[Bug 107237] [META] Notebookbar Tabbed
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: include/rtl offapi/com sal/osl sc/inc svtools/source xmloff/source

2019-07-24 Thread Andrea Gelmini (via logerrit)
 include/rtl/uri.h   |2 +-
 offapi/com/sun/star/rdf/URI.idl |8 
 offapi/com/sun/star/rdf/XDocumentMetadataAccess.idl |4 ++--
 offapi/com/sun/star/rdf/XLiteral.idl|2 +-
 offapi/com/sun/star/rdf/XURI.idl|6 +++---
 sal/osl/unx/file_url.cxx|2 +-
 sc/inc/externalrefmgr.hxx   |4 ++--
 svtools/source/svhtml/htmlkywd.cxx  |2 +-
 xmloff/source/transform/AttrTransformerAction.hxx   |4 ++--
 9 files changed, 17 insertions(+), 17 deletions(-)

New commits:
commit 6131288f2a85d43ba21210e677648c9245793e52
Author: Andrea Gelmini 
AuthorDate: Tue Jul 23 22:25:57 2019 +0200
Commit: Andrea Gelmini 
CommitDate: Wed Jul 24 23:36:21 2019 +0200

Fix typo

"an URI", to complete:
https://gerrit.libreoffice.org/#/c/75985/

Change-Id: I57489b05117fd12ae6aa22544437ab5bc6b5154f
Reviewed-on: https://gerrit.libreoffice.org/76037
Tested-by: Jenkins
Reviewed-by: Andrea Gelmini 

diff --git a/include/rtl/uri.h b/include/rtl/uri.h
index af472dae4d52..3b134fbb80af 100644
--- a/include/rtl/uri.h
+++ b/include/rtl/uri.h
@@ -309,7 +309,7 @@ SAL_DLLPUBLIC void SAL_CALL rtl_uriDecode(
 signaled.
 
 @param pRelUriRef
-An URI reference that may be either absolute or relative.  If it is
+A URI reference that may be either absolute or relative.  If it is
 absolute, it will be returned unmodified.
 
 @param pResult
diff --git a/offapi/com/sun/star/rdf/URI.idl b/offapi/com/sun/star/rdf/URI.idl
index b057e42660d6..dbdbc2248227 100644
--- a/offapi/com/sun/star/rdf/URI.idl
+++ b/offapi/com/sun/star/rdf/URI.idl
@@ -27,7 +27,7 @@
 
 module com {   module sun {   module star {   module rdf {
 
-/** represents an URI node that may occur in a RDF graph.
+/** represents a URI node that may occur in a RDF graph.
 
 @since OOo 3.0
 
@@ -36,7 +36,7 @@ module com {   module sun {   module star {   module rdf {
 service URI : XURI
 {
 
-/** creates an URI RDF node.
+/** creates a URI RDF node.
 
 @param Value
 the URI, represented as `string`.
@@ -47,7 +47,7 @@ service URI : XURI
 create( [in] string Value )
 raises( com::sun::star::lang::IllegalArgumentException );
 
-/** creates an URI RDF node from namespace prefix and local name.
+/** creates a URI RDF node from namespace prefix and local name.
 
 @param Namespace
 the namespace prefix of the URI, represented as
@@ -62,7 +62,7 @@ service URI : XURI
 createNS( [in] string Namespace, [in] string LocalName )
 raises( com::sun::star::lang::IllegalArgumentException );
 
-/** creates an URI RDF node for a well-known URI.
+/** creates a URI RDF node for a well-known URI.
 
 @param Id
 the URI, represented as a constant from URIs.
diff --git a/offapi/com/sun/star/rdf/XDocumentMetadataAccess.idl 
b/offapi/com/sun/star/rdf/XDocumentMetadataAccess.idl
index 8760322ddbe1..b7d143555feb 100644
--- a/offapi/com/sun/star/rdf/XDocumentMetadataAccess.idl
+++ b/offapi/com/sun/star/rdf/XDocumentMetadataAccess.idl
@@ -77,10 +77,10 @@ interface XDocumentMetadataAccess
 XMetadatable getElementByMetadataReference(
 [in] com::sun::star::beans::StringPair MetadataReference);
 
-/** get the ODF element that corresponds to an URI.
+/** get the ODF element that corresponds to a URI.
 
 @param URI
-an URI that may identify an ODF element
+a URI that may identify an ODF element
 
 @returns
 the ODF element that corresponds to the given URI, or `NULL`
diff --git a/offapi/com/sun/star/rdf/XLiteral.idl 
b/offapi/com/sun/star/rdf/XLiteral.idl
index de67596b35d1..7366d021fe7f 100644
--- a/offapi/com/sun/star/rdf/XLiteral.idl
+++ b/offapi/com/sun/star/rdf/XLiteral.idl
@@ -33,7 +33,7 @@ module com {   module sun {   module star {   module rdf {
 
 just a string Value
 Value and Language
-typed literal: Value and Datatype (represented by an URI)
+typed literal: Value and Datatype (represented by a URI)
 
 Note that there is no literal with both Language and Datatype.
 
diff --git a/offapi/com/sun/star/rdf/XURI.idl b/offapi/com/sun/star/rdf/XURI.idl
index 1d08b9cfeb45..5386186d9ede 100644
--- a/offapi/com/sun/star/rdf/XURI.idl
+++ b/offapi/com/sun/star/rdf/XURI.idl
@@ -26,7 +26,7 @@
 
 module com {   module sun {   module star {   module rdf {
 
-/** represents an URI node that may occur in a RDF graph.
+/** represents a URI node that may occur in a RDF graph.
 
 
 Note that this is actually an IRI, but the RDF literature speaks of URIs
@@ -41,8 +41,8 @@ module com {   module sun {   module star {   module rdf {
 after the last occurrence of the path separator: "/"
 after the last occurrence of the scheme separator: ":"
 
-An URI without a ":" is 

[Libreoffice-commits] core.git: basic/source qadevOOo/tests scaddins/source sdext/source svx/source sw/source vcl/unx xmerge/source xmloff/inc xmloff/source xmlsecurity/inc xmlsecurity/source

2019-07-24 Thread Andrea Gelmini (via logerrit)
 basic/source/classes/sbunoobj.cxx  
|2 -
 qadevOOo/tests/java/ifc/ucb/_XCachedDynamicResultSetFactory.java   
|2 -
 qadevOOo/tests/java/ifc/ucb/_XCachedDynamicResultSetStubFactory.java   
|2 -
 scaddins/source/datefunc/datefunc.cxx  
|2 -
 sdext/source/pdfimport/tree/writertreevisiting.cxx 
|2 -
 svx/source/table/tablelayouter.cxx 
|2 -
 svx/source/table/tablelayouter.hxx 
|2 -
 sw/source/filter/html/htmlatr.cxx  
|2 -
 sw/source/uibase/shells/textsh1.cxx
|2 -
 vcl/unx/generic/print/common_gfx.cxx   
|2 -
 
xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/DocumentMergerImpl.java
  |2 -
 
xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/xslt/DocumentMergerImpl.java
 |2 -
 xmloff/inc/xmlmultiimagehelper.hxx 
|2 -
 xmloff/source/chart/SchXMLExport.cxx   
|2 -
 xmloff/source/script/XMLEventImportHelper.cxx  
|2 -
 xmlsecurity/inc/framework/saxeventkeeperimpl.hxx   
|6 ++--
 xmlsecurity/inc/framework/securityengine.hxx   
|2 -
 xmlsecurity/source/framework/buffernode.cxx
|6 ++--
 xmlsecurity/source/framework/buffernode.hxx
|   14 +-
 xmlsecurity/source/framework/elementcollector.cxx  
|4 +-
 xmlsecurity/source/framework/elementmark.cxx   
|4 +-
 xmlsecurity/source/framework/saxeventkeeperimpl.cxx
|2 -
 xmlsecurity/source/framework/signatureengine.cxx   
|2 -
 xmlsecurity/source/helper/xsecctl.cxx  
|2 -
 24 files changed, 36 insertions(+), 36 deletions(-)

New commits:
commit afad1a0cbc893291ea32baa7234c4ed7a5ffcf80
Author: Andrea Gelmini 
AuthorDate: Wed Jul 24 19:48:51 2019 +0200
Commit: Andrea Gelmini 
CommitDate: Wed Jul 24 23:34:49 2019 +0200

Fix typos

Change-Id: Ib778a6598f3797fe871a25b56b7b4b7761889296
Reviewed-on: https://gerrit.libreoffice.org/76269
Tested-by: Jenkins
Reviewed-by: Andrea Gelmini 

diff --git a/basic/source/classes/sbunoobj.cxx 
b/basic/source/classes/sbunoobj.cxx
index c4bec7580d48..321ac7faedeb 100644
--- a/basic/source/classes/sbunoobj.cxx
+++ b/basic/source/classes/sbunoobj.cxx
@@ -1011,7 +1011,7 @@ static Any sbxToUnoValueImpl( const SbxValue* pVar, bool 
bBlockConversionToSmall
 
 if( !bBlockConversionToSmallestType )
 {
-// #79615 Choose "smallest" represention for int values
+// #79615 Choose "smallest" representation for int values
 // because up cast is allowed, downcast not
 switch( eType )
 {
diff --git a/qadevOOo/tests/java/ifc/ucb/_XCachedDynamicResultSetFactory.java 
b/qadevOOo/tests/java/ifc/ucb/_XCachedDynamicResultSetFactory.java
index 595033744c2e..814f4a9436f3 100644
--- a/qadevOOo/tests/java/ifc/ucb/_XCachedDynamicResultSetFactory.java
+++ b/qadevOOo/tests/java/ifc/ucb/_XCachedDynamicResultSetFactory.java
@@ -36,7 +36,7 @@ import com.sun.star.ucb.XDynamicResultSet;
 * 
 *   'CachedDynamicResultSetStub' (of type
 *   com.sun.star.sdbc.XDynamicResultSet):
-*   this must be an imlementation of 
+*   this must be an implementation of 
 *   com.sun.star.ucb.CachedDynamicResultSetStub service.
 *  
 * Test is  NOT  multithread compliant. 
diff --git 
a/qadevOOo/tests/java/ifc/ucb/_XCachedDynamicResultSetStubFactory.java 
b/qadevOOo/tests/java/ifc/ucb/_XCachedDynamicResultSetStubFactory.java
index b0c91ffdc74e..ac505faf4435 100644
--- a/qadevOOo/tests/java/ifc/ucb/_XCachedDynamicResultSetStubFactory.java
+++ b/qadevOOo/tests/java/ifc/ucb/_XCachedDynamicResultSetStubFactory.java
@@ -39,7 +39,7 @@ import com.sun.star.uno.UnoRuntime;
 * 
 *   'DynamicResultSet' (of type
 *   com.sun.star.sdbc.XDynamicResultSet):
-*   this must be an imlementation of 
+*   this must be an implementation of 
 *   com.sun.star.ucb.DynamicResultSet service.
 *  
 * Test is  NOT  multithread compliant. 
diff --git a/scaddins/source/datefunc/datefunc.cxx 
b/scaddins/source/datefunc/datefunc.cxx
index 3011ccb32480..e09f871c8ae1 100644
--- a/scaddins/source/datefunc/datefunc.cxx
+++ 

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

2019-07-24 Thread Caolán McNamara (via logerrit)
 connectivity/source/commontools/FValue.cxx |   12 ++--
 1 file changed, 10 insertions(+), 2 deletions(-)

New commits:
commit 30320abbef8fabfeadda2b31b72f595c4a1c1ed9
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 11:22:58 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 22:21:32 2019 +0200

cid#1448360 Assignment of overlapping memory

Change-Id: Ifefa007ff7534e65cc35cdbcb4983012b36ed6e0
Reviewed-on: https://gerrit.libreoffice.org/76248
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/connectivity/source/commontools/FValue.cxx 
b/connectivity/source/commontools/FValue.cxx
index 61f344bb92b5..51adab4f3e98 100644
--- a/connectivity/source/commontools/FValue.cxx
+++ b/connectivity/source/commontools/FValue.cxx
@@ -2086,11 +2086,19 @@ void ORowSetValue::setSigned(bool _bMod)
 }
 break;
 case DataType::BIGINT:
+{
 if ( m_bSigned )
-m_aValue.m_nInt64 = 
static_cast(m_aValue.m_uInt64);
+{
+auto nTmp = static_cast(m_aValue.m_uInt64);
+m_aValue.m_nInt64 = nTmp;
+}
 else
-m_aValue.m_uInt64 = 
static_cast(m_aValue.m_nInt64);
+{
+auto nTmp = static_cast(m_aValue.m_nInt64);
+m_aValue.m_uInt64 = nTmp;
+}
 break;
+}
 }
 m_eTypeKind = nType;
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 126531] New: Connector Behavior Broken since 6.2.x

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126531

Bug ID: 126531
   Summary: Connector Behavior Broken since 6.2.x
   Product: LibreOffice
   Version: 6.2.5.2 release
  Hardware: x86-64 (AMD64)
OS: Windows (All)
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Draw
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: ralf.kr...@bluewin.ch

Created attachment 152971
  --> https://bugs.documentfoundation.org/attachment.cgi?id=152971=edit
Connector_Bug_example

Hi

The Behavior of the connector is broken since LO 6.2.x. At least since LO
6.2.4. It worked without problems in LO 6.1.x:
To reproduce the Bug
1. Open the attached File
2. Take the Arrow from a gluepoint of the circle to another -> it should be
updated in realtime. But it is only updated when the curser released.

Hope this is enough...
Thanks
Ralf

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: Branch 'libreoffice-6-3' - sc/source

2019-07-24 Thread Miklos Vajna (via logerrit)
 sc/source/ui/drawfunc/futext3.cxx |5 +
 1 file changed, 5 insertions(+)

New commits:
commit f5f894e1608a75dffc07ddb3644b566fee1b6848
Author: Miklos Vajna 
AuthorDate: Mon Jul 22 13:58:57 2019 +0200
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 21:35:47 2019 +0200

tdf#106000 sc ui: fix missing invalidation when cancelling note creation

Regression from commit dca01def7885ad69cf66edd75cf8207a5adb64f9
(refactor ListBox/ComboBox to use RenderContext, 2015-05-07),
invalidating the area of note anchors worked like this in the past:

- user ended text edit of a new, empty comment
- the Calc shell was updated from shape text edit to the tab view, which
  called ImplListBoxWindow::SelectEntry()
- this called ImplPaint(), which invoked maUserDrawHdl (if it was set)
- and at the end this invoked ScViewFunc::UpdateSelectionArea(), all
  triggered by the font name and font size listboxes

So the update of the cell of the former note depended on the presence of
font name/size listboxes, by accident.

This broke, because ImplListBoxWindow::SelectEntry() now calls
Invalidate(), and Paint() calls ImplDoPaint(), which doesn't invoke
maUserDrawHdl anymore. Fix the problem by invalidating the cell
explicitly, so in case the user customizes the toolbar to not have these
listboxes, the behavior is still correct.

Do this in FuText::StopEditMode(), because there we know the position of
the deleted note and that's where we delete the matching SdrCaptionObj
(via the RemoveLastUndoAction()).

(cherry picked from commit d001492faddb0b95356c7744aa4d193aacb38d7d)

Change-Id: Iaa4b609477ac6c7f23efc575fddac6e39aa6a7b5
Reviewed-on: https://gerrit.libreoffice.org/76128
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/sc/source/ui/drawfunc/futext3.cxx 
b/sc/source/ui/drawfunc/futext3.cxx
index 5c1d3a6ed00b..cbdcefb7d893 100644
--- a/sc/source/ui/drawfunc/futext3.cxx
+++ b/sc/source/ui/drawfunc/futext3.cxx
@@ -157,6 +157,11 @@ void FuText::StopEditMode()
 if( bNewNote && bDeleteNote )
 {
 pUndoMgr->RemoveLastUndoAction();
+
+// Make sure the former area of the note anchor is invalidated.
+ScRangeList aRangeList(aNotePos);
+ScMarkData aMarkData(aRangeList);
+rViewShell.UpdateSelectionArea(aMarkData);
 }
 else if( bNewNote || bDeleteNote )
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-commits] core.git: qadevOOo/tests

2019-07-24 Thread Caolán McNamara (via logerrit)
 qadevOOo/tests/java/ifc/text/_XSimpleText.java |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit ceac24b4a140a471a36f423bf48dce32860e8911
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 10:53:42 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 21:34:01 2019 +0200

cid#1448461 RV: Bad use of return value

Change-Id: I11dce56c00931b0e04c5fa52f2598a8f80625d5b
Reviewed-on: https://gerrit.libreoffice.org/76243
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/qadevOOo/tests/java/ifc/text/_XSimpleText.java 
b/qadevOOo/tests/java/ifc/text/_XSimpleText.java
index 8f1c32d40911..f4ab856f3b5d 100644
--- a/qadevOOo/tests/java/ifc/text/_XSimpleText.java
+++ b/qadevOOo/tests/java/ifc/text/_XSimpleText.java
@@ -98,7 +98,7 @@ public class _XSimpleText extends MultiMethodTest {
 }
 catch(com.sun.star.lang.IllegalArgumentException e ) {
 // Some exception.FAILED
-Status.failed( e.toString() );
+log.println(e.toString());
 bOK = false;
 }
 String gStr = oObj.getString() ;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-commits] core.git: 2 commits - qadevOOo/tests sdext/source

2019-07-24 Thread Caolán McNamara (via logerrit)
 qadevOOo/tests/java/ifc/text/_XText.java   |2 +-
 sdext/source/pdfimport/tree/writertreevisiting.cxx |8 
 2 files changed, 5 insertions(+), 5 deletions(-)

New commits:
commit bb6f5ae90332fee2eb650398d1ae0b554ac89ce2
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 10:57:27 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 21:33:38 2019 +0200

cid#1448446 Unchecked dynamic_cast

Change-Id: I790f55e5edfce37b28dfc8b915970e123425f434
Reviewed-on: https://gerrit.libreoffice.org/76244
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/sdext/source/pdfimport/tree/writertreevisiting.cxx 
b/sdext/source/pdfimport/tree/writertreevisiting.cxx
index b84e5f2fcb07..96b9e4bcfab9 100644
--- a/sdext/source/pdfimport/tree/writertreevisiting.cxx
+++ b/sdext/source/pdfimport/tree/writertreevisiting.cxx
@@ -671,8 +671,8 @@ void WriterXmlOptimizer::checkHeaderAndFooter( PageElement& 
rElem )
 auto it = std::find_if(rElem.Children.begin(), rElem.Children.end(), 
isParagraphElement);
 if (it != rElem.Children.end())
 {
-ParagraphElement* pPara = dynamic_cast(it->get());
-if( pPara->y+pPara->h < rElem.h*0.15 && pPara->isSingleLined( 
m_rProcessor ) )
+ParagraphElement& rPara = dynamic_cast(*it->get());
+if( rPara.y+rPara.h < rElem.h*0.15 && rPara.isSingleLined( 
m_rProcessor ) )
 {
 auto next_it = it;
 ParagraphElement* pNextPara = nullptr;
@@ -680,10 +680,10 @@ void WriterXmlOptimizer::checkHeaderAndFooter( 
PageElement& rElem )
 {
 pNextPara = dynamic_cast(next_it->get());
 }
-if( pNextPara && pNextPara->y > pPara->y+pPara->h*2 )
+if( pNextPara && pNextPara->y > rPara.y+rPara.h*2 )
 {
 rElem.HeaderElement = std::move(*it);
-pPara->Parent = nullptr;
+rPara.Parent = nullptr;
 rElem.Children.erase( it );
 }
 }
commit 1779f7fcbaf11428e9064653a61045dc267778be
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 11:43:15 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 21:33:14 2019 +0200

cid#1448285 RV: Bad use of return value

Change-Id: I3509620b37884b75a00c89ae38b68adc324da00c
Reviewed-on: https://gerrit.libreoffice.org/76252
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/qadevOOo/tests/java/ifc/text/_XText.java 
b/qadevOOo/tests/java/ifc/text/_XText.java
index 234dd5f1882f..0e2d8a3d6ea9 100644
--- a/qadevOOo/tests/java/ifc/text/_XText.java
+++ b/qadevOOo/tests/java/ifc/text/_XText.java
@@ -145,7 +145,7 @@ public class _XText extends MultiMethodTest {
 }
 catch( com.sun.star.container.NoSuchElementException nseE ){
 // Some exception.FAILED
-Status.failed( nseE.toString() );
+log.println(nseE.toString());
 return;
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-commits] core.git: 2 commits - reportbuilder/java sd/source

2019-07-24 Thread Caolán McNamara (via logerrit)
 
reportbuilder/java/org/libreoffice/report/pentaho/output/OfficeDocumentReportTarget.java
 |1 -
 sd/source/ui/dlg/BulletAndPositionDlg.cxx  
  |1 +
 2 files changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 852d5b7b3bc289c1018da5373dea50b7aa68ef0b
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 11:45:00 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 21:32:49 2019 +0200

cid#1448272 RV: Bad use of return value

Change-Id: I716c4e617ae2493385391f96567e2fb4571cd440
Reviewed-on: https://gerrit.libreoffice.org/76253
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git 
a/reportbuilder/java/org/libreoffice/report/pentaho/output/OfficeDocumentReportTarget.java
 
b/reportbuilder/java/org/libreoffice/report/pentaho/output/OfficeDocumentReportTarget.java
index 0a3f23c6986e..330a5e091aa0 100644
--- 
a/reportbuilder/java/org/libreoffice/report/pentaho/output/OfficeDocumentReportTarget.java
+++ 
b/reportbuilder/java/org/libreoffice/report/pentaho/output/OfficeDocumentReportTarget.java
@@ -1026,7 +1026,6 @@ public abstract class OfficeDocumentReportTarget extends 
AbstractReportTarget
 DocumentBuilder dBuilder = dbFactory.newDocumentBuilder();
 Document document = dBuilder.parse(new 
InputSource(inputStream));
 
-document.getElementsByTagName("document-meta/meta/generator");
 Node node = 
document.getFirstChild().getFirstChild().getFirstChild().getFirstChild();
 String creator = node.getNodeValue();
 node.setNodeValue(creator + "/report_builder");
commit 554e7d8ee6bee44d87392476a498fb7f3f6068c6
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 11:51:10 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 21:32:22 2019 +0200

cid#1448222 Out-of-bounds read

possibly coverity gold

Change-Id: I6443225482eab5be31e1e12d282e420e767f23b2
Reviewed-on: https://gerrit.libreoffice.org/76254
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/sd/source/ui/dlg/BulletAndPositionDlg.cxx 
b/sd/source/ui/dlg/BulletAndPositionDlg.cxx
index 26e81878661f..17e7b20136f2 100644
--- a/sd/source/ui/dlg/BulletAndPositionDlg.cxx
+++ b/sd/source/ui/dlg/BulletAndPositionDlg.cxx
@@ -521,6 +521,7 @@ void SvxBulletAndPositionDlg::InitControls()
 {
 nNumberingType = SVX_NUM_NUMBER_NONE;
 bAllLevel = false;
+bSameDistBorderNum = false;
 bSameBulRelSize = false;
 bSameBulColor = false;
 bSameStart = false;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

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

2019-07-24 Thread Caolán McNamara (via logerrit)
 sw/source/core/frmedt/fefly1.cxx   |2 +-
 sw/source/core/frmedt/feshview.cxx |2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit fd40f94e6bfcd8df200a30c933e4d09d1b99e897
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 11:54:59 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 21:31:11 2019 +0200

cid#1401334 Uncaught exception

Change-Id: I371610e76e513a40da800eb11de0883775fdbd2d
Reviewed-on: https://gerrit.libreoffice.org/76255
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/sw/source/core/frmedt/fefly1.cxx b/sw/source/core/frmedt/fefly1.cxx
index f81fb14a647c..bd6b66a5dabe 100644
--- a/sw/source/core/frmedt/fefly1.cxx
+++ b/sw/source/core/frmedt/fefly1.cxx
@@ -592,7 +592,7 @@ Point SwFEShell::FindAnchorPos( const Point& rAbsPos, bool 
bMoveIt )
 // re-created. Thus, delete all fly frames except the 
 before the
 // anchor attribute is change and re-create them 
afterwards.
 {
-std::unique_ptr 
pHandleAnchorNodeChg;
+std::unique_ptr> pHandleAnchorNodeChg;
 SwFlyFrameFormat* pFlyFrameFormat( 
dynamic_cast() );
 if ( pFlyFrameFormat )
 {
diff --git a/sw/source/core/frmedt/feshview.cxx 
b/sw/source/core/frmedt/feshview.cxx
index 136a5fbfe385..736b10235109 100644
--- a/sw/source/core/frmedt/feshview.cxx
+++ b/sw/source/core/frmedt/feshview.cxx
@@ -576,7 +576,7 @@ bool SwFEShell::MoveAnchor( SwMove nDir )
 // re-created. Thus, delete all fly frames except the  
before the
 // anchor attribute is change and re-create them afterwards.
 {
-std::unique_ptr pHandleAnchorNodeChg;
+std::unique_ptr> pHandleAnchorNodeChg;
 SwFlyFrameFormat* pFlyFrameFormat( 
dynamic_cast() );
 if ( pFlyFrameFormat )
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-commits] core.git: sw/inc

2019-07-24 Thread Caolán McNamara (via logerrit)
 sw/inc/swmodule.hxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 40b5af334464b7e30bc29e479ec6a626a4ad0d48
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 11:56:23 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 21:30:46 2019 +0200

cid#1401328 Uncaught exception

Change-Id: I15b1157220631e984f30f6e3c0a0362167236c9c
Reviewed-on: https://gerrit.libreoffice.org/76256
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/sw/inc/swmodule.hxx b/sw/inc/swmodule.hxx
index be21755a08cc..5d96a31da341 100644
--- a/sw/inc/swmodule.hxx
+++ b/sw/inc/swmodule.hxx
@@ -77,7 +77,7 @@ class SW_DLLPUBLIC SwModule final : public SfxModule, public 
SfxListener, public
 OUStringm_sActAuthor;
 
 // ConfigItems
-std::unique_ptr m_pModuleConfig;
+std::unique_ptr> 
m_pModuleConfig;
 std::unique_ptr> 
m_pUsrPref;
 std::unique_ptr> 
m_pWebUsrPref;
 std::unique_ptr  m_pPrintOptions;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

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

2019-07-24 Thread Caolán McNamara (via logerrit)
 vcl/source/font/fontcharmap.cxx |   22 +++---
 1 file changed, 11 insertions(+), 11 deletions(-)

New commits:
commit 63922db083dc70c9b248c9eb34b24382048adf08
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 12:07:57 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 21:29:49 2019 +0200

rename search+replaced Getsal_uInt16 result back to GetUShort

Change-Id: Ia6e35d0ca15b0ac2310ad847c6eda6db548b25f6
Reviewed-on: https://gerrit.libreoffice.org/76258
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/vcl/source/font/fontcharmap.cxx b/vcl/source/font/fontcharmap.cxx
index 8d8dd5815dc1..9c8b54682041 100644
--- a/vcl/source/font/fontcharmap.cxx
+++ b/vcl/source/font/fontcharmap.cxx
@@ -87,7 +87,7 @@ bool ImplFontCharMap::isDefaultMap() const
 }
 
 static unsigned GetUInt( const unsigned char* p ) { 
return((p[0]<<24)+(p[1]<<16)+(p[2]<<8)+p[3]);}
-static unsigned Getsal_uInt16( const unsigned char* p ){ return((p[0]<<8) | 
p[1]);}
+static unsigned GetUShort( const unsigned char* p ){ return((p[0]<<8) | p[1]);}
 static int GetSShort( const unsigned char* p ){ return 
static_cast((p[0]<<8)|p[1]);}
 
 // TODO: move CMAP parsing directly into the ImplFontCharMap class
@@ -104,10 +104,10 @@ bool ParseCMAP( const unsigned char* pCmap, int nLength, 
CmapResult& rResult )
 if( !pCmap || (nLength < 24) )
 return false;
 
-if( Getsal_uInt16( pCmap ) != 0x ) // simple check for CMAP corruption
+if( GetUShort( pCmap ) != 0x ) // simple check for CMAP corruption
 return false;
 
-int nSubTables = Getsal_uInt16( pCmap + 2 );
+int nSubTables = GetUShort( pCmap + 2 );
 if( (nSubTables <= 0) || (nLength < (24 + 8*nSubTables)) )
 return false;
 
@@ -120,8 +120,8 @@ bool ParseCMAP( const unsigned char* pCmap, int nLength, 
CmapResult& rResult )
 int nBestVal = 0;
 for( const unsigned char* p = pCmap + 4; --nSubTables >= 0; p += 8 )
 {
-int nPlatform = Getsal_uInt16( p );
-int nEncoding = Getsal_uInt16( p+2 );
+int nPlatform = GetUShort( p );
+int nEncoding = GetUShort( p+2 );
 int nPlatformEncoding = (nPlatform << 8) + nEncoding;
 
 int nValue;
@@ -150,7 +150,7 @@ bool ParseCMAP( const unsigned char* pCmap, int nLength, 
CmapResult& rResult )
 continue;
 
 int nTmpOffset = GetUInt( p+4 );
-int nTmpFormat = Getsal_uInt16( pCmap + nTmpOffset );
+int nTmpFormat = GetUShort( pCmap + nTmpOffset );
 if( nTmpFormat == 12 )  // 32bit code -> glyph map 
format
 nValue += 3;
 else if( nTmpFormat != 4 )  // 16bit code -> glyph map 
format
@@ -177,7 +177,7 @@ bool ParseCMAP( const unsigned char* pCmap, int nLength, 
CmapResult& rResult )
 // format 4, the most common 16bit char mapping table
 if( (nFormat == 4) && ((nOffset+16) < nLength) )
 {
-int nSegCountX2 = Getsal_uInt16( pCmap + nOffset + 6 );
+int nSegCountX2 = GetUShort( pCmap + nOffset + 6 );
 nRangeCount = nSegCountX2/2 - 1;
 pCodePairs = new sal_UCS4[ nRangeCount * 2 ];
 pStartGlyphs = new int[ nRangeCount ];
@@ -188,10 +188,10 @@ bool ParseCMAP( const unsigned char* pCmap, int nLength, 
CmapResult& rResult )
 sal_UCS4* pCP = pCodePairs;
 for( int i = 0; i < nRangeCount; ++i )
 {
-const sal_UCS4 cMinChar = Getsal_uInt16( pBeginBase + 2*i );
-const sal_UCS4 cMaxChar = Getsal_uInt16( pLimitBase + 2*i );
+const sal_UCS4 cMinChar = GetUShort( pBeginBase + 2*i );
+const sal_UCS4 cMaxChar = GetUShort( pLimitBase + 2*i );
 const int nGlyphDelta  = GetSShort( pDeltaBase + 2*i );
-const int nRangeOffset = Getsal_uInt16( pOffsetBase + 2*i );
+const int nRangeOffset = GetUShort( pOffsetBase + 2*i );
 if( cMinChar > cMaxChar ) {  // no sane font should trigger this
 SAL_WARN("vcl.gdi", "Min char should never be more than the 
max char!");
 break;
@@ -219,7 +219,7 @@ bool ParseCMAP( const unsigned char* pCmap, int nLength, 
CmapResult& rResult )
 break;
 }
 for( sal_UCS4 c = cMinChar; c <= cMaxChar; ++c, pGlyphIdPtr+=2 
) {
-const int nGlyphIndex = Getsal_uInt16( pGlyphIdPtr ) + 
nGlyphDelta;
+const int nGlyphIndex = GetUShort( pGlyphIdPtr ) + 
nGlyphDelta;
 aGlyphIdArray.push_back( 
static_cast(nGlyphIndex) );
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

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

2019-07-24 Thread Caolán McNamara (via logerrit)
 vcl/source/font/fontcharmap.cxx |5 +
 1 file changed, 5 insertions(+)

New commits:
commit 52b92a591943c1988fd3d660bd6fc5ac53ce0f33
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 12:18:10 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 21:28:26 2019 +0200

cid#1209863 Untrusted loop bound

Change-Id: Ic8d20e92b4021dfebe01e1265c3afb2bcd509827
Reviewed-on: https://gerrit.libreoffice.org/76259
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/vcl/source/font/fontcharmap.cxx b/vcl/source/font/fontcharmap.cxx
index b057a2056645..8d8dd5815dc1 100644
--- a/vcl/source/font/fontcharmap.cxx
+++ b/vcl/source/font/fontcharmap.cxx
@@ -232,6 +232,11 @@ bool ParseCMAP( const unsigned char* pCmap, int nLength, 
CmapResult& rResult )
 else if( (nFormat == 12) && ((nOffset+16) < nLength) )
 {
 nRangeCount = GetUInt( pCmap + nOffset + 12 );
+if (nRangeCount < 0)
+{
+SAL_WARN("vcl.gdi", "negative RangeCount");
+nRangeCount = 0;
+}
 
 const int nGroupOffset = nOffset + 16;
 const int nRemainingLen = nLength - nGroupOffset;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-commits] core.git: sd/inc sd/source

2019-07-24 Thread Ashod Nakashian (via logerrit)
 sd/inc/CustomAnimationPreset.hxx|6 +++-
 sd/source/core/CustomAnimationPreset.cxx|   34 ++--
 sd/source/ui/animations/CustomAnimationPane.cxx |   20 --
 sd/source/ui/animations/CustomAnimationPane.hxx |   12 +---
 4 files changed, 34 insertions(+), 38 deletions(-)

New commits:
commit 3b55741586883d4fbb7f3e4c5368b5382c3ce5f8
Author: Ashod Nakashian 
AuthorDate: Mon Jan 28 10:03:39 2019 -0500
Commit: Noel Grandin 
CommitDate: Wed Jul 24 21:01:51 2019 +0200

sd: LOK: support per-user localization of CustomAnimationPreset

This only fixes the caching in CustomAnimationPreset and
CustomAnimationPane. Other cached labels will be done separately.

Reviewed-on: https://gerrit.libreoffice.org/68263
Reviewed-by: Andras Timar 
Tested-by: Andras Timar 
(cherry picked from commit b44679e47211a27f4fe22d05762d9d48c8f88935)

Change-Id: Iaf511168d26b3a5567ca53556e242d3c071d2623
Reviewed-on: https://gerrit.libreoffice.org/76241
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/sd/inc/CustomAnimationPreset.hxx b/sd/inc/CustomAnimationPreset.hxx
index ac55ad77..57669734014b 100644
--- a/sd/inc/CustomAnimationPreset.hxx
+++ b/sd/inc/CustomAnimationPreset.hxx
@@ -88,6 +88,9 @@ public:
 SAL_DLLPRIVATE CustomAnimationPresets();
 SAL_DLLPRIVATE ~CustomAnimationPresets();
 
+/** This method gets presets instance, which is localized
+ * for the current user's locale.
+*/
 static const CustomAnimationPresets& getCustomAnimationPresets();
 
 SAL_DLLPRIVATE css::uno::Reference< css::animations::XAnimationNode > 
getRandomPreset( sal_Int16 nPresetClass ) const;
@@ -125,7 +128,8 @@ private:
 PresetCategoryList maMotionPathsPresets;
 PresetCategoryList maMiscPresets;
 
-SAL_DLLPRIVATE static CustomAnimationPresets*  mpCustomAnimationPresets;
+//! Maps per-language the animation presets.
+SAL_DLLPRIVATE static std::map  
mpCustomAnimationPresetsMap;
 };
 
 
diff --git a/sd/source/core/CustomAnimationPreset.cxx 
b/sd/source/core/CustomAnimationPreset.cxx
index 1f2066e6af9c..4c02acc25d6c 100644
--- a/sd/source/core/CustomAnimationPreset.cxx
+++ b/sd/source/core/CustomAnimationPreset.cxx
@@ -35,6 +35,9 @@
 #include 
 #include 
 #include 
+#include 
+#include 
+#include 
 #include 
 
 #include 
@@ -521,22 +524,27 @@ void CustomAnimationPresets::changePresetSubType( const 
CustomAnimationEffectPtr
 }
 }
 
-CustomAnimationPresets* CustomAnimationPresets::mpCustomAnimationPresets = 
nullptr;
+std::map  
CustomAnimationPresets::mpCustomAnimationPresetsMap;
 
 const CustomAnimationPresets& 
CustomAnimationPresets::getCustomAnimationPresets()
 {
-if( !mpCustomAnimationPresets )
-{
-SolarMutexGuard aGuard;
-
-if( !mpCustomAnimationPresets )
-{
-mpCustomAnimationPresets = new sd::CustomAnimationPresets();
-mpCustomAnimationPresets->importResources();
-}
-}
-
-return *mpCustomAnimationPresets;
+// Support localization per-view. Currently not useful for Desktop
+// but very much critical for LOK. The cache now is per-language.
+const OUString aLang = comphelper::LibreOfficeKit::isActive()
+   ? 
comphelper::LibreOfficeKit::getLanguageTag().getLanguage()
+   : 
SvtSysLocaleOptions().GetLanguageTag().getLanguage();
+
+SolarMutexGuard aGuard;
+const auto it = mpCustomAnimationPresetsMap.find(aLang);
+if (it != mpCustomAnimationPresetsMap.end())
+return *it->second;
+
+// Note: we are invoked recursively(!), so we must set the instance pointer
+// in the cache map before we importResources, lest we get in infinite 
loop.
+sd::CustomAnimationPresets* pCustomAnimationPresets = new 
sd::CustomAnimationPresets();
+mpCustomAnimationPresetsMap[aLang] = pCustomAnimationPresets;
+pCustomAnimationPresets->importResources();
+return *pCustomAnimationPresets;
 }
 
 Reference< XAnimationNode > CustomAnimationPresets::getRandomPreset( sal_Int16 
nPresetClass ) const
diff --git a/sd/source/ui/animations/CustomAnimationPane.cxx 
b/sd/source/ui/animations/CustomAnimationPane.cxx
index 6971328837e8..ee1807187fbd 100644
--- a/sd/source/ui/animations/CustomAnimationPane.cxx
+++ b/sd/source/ui/animations/CustomAnimationPane.cxx
@@ -124,7 +124,6 @@ CustomAnimationPane::CustomAnimationPane( Window* pParent, 
ViewShellBase& rBase,
   const 
css::uno::Reference& rxFrame )
 :   PanelLayout( pParent, "CustomAnimationsPanel", 
"modules/simpress/ui/customanimationspanel.ui", rxFrame ),
 mrBase( rBase ),
-mpCustomAnimationPresets(nullptr),
 mnPropertyType( nPropertyTypeNone ),
 mnCurvePathPos( LISTBOX_ENTRY_NOTFOUND ),
 mnPolygonPathPos( LISTBOX_ENTRY_NOTFOUND ),
@@ -532,7 +531,7 @@ void CustomAnimationPane::updateControls()
 {
 

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

2019-07-24 Thread Ashod Nakashian (via logerrit)
 sd/source/ui/framework/configuration/ConfigurationTracer.cxx |7 ---
 sd/source/ui/framework/configuration/ConfigurationTracer.hxx |3 ++-
 2 files changed, 6 insertions(+), 4 deletions(-)

New commits:
commit 0c4a2b126c4aaa6f87baa1d86973b25fe7691009
Author: Ashod Nakashian 
AuthorDate: Sun Nov 18 14:58:45 2018 -0500
Commit: Noel Grandin 
CommitDate: Wed Jul 24 21:01:33 2019 +0200

sd: use OSL_DEBUG_LEVEL instead of DEBUG

Change-Id: I0703a1564a23657e956e43fe4cbf1d9549acb94f
Reviewed-on: https://gerrit.libreoffice.org/68258
Reviewed-by: Andras Timar 
Tested-by: Andras Timar 
Reviewed-on: https://gerrit.libreoffice.org/76240
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/sd/source/ui/framework/configuration/ConfigurationTracer.cxx 
b/sd/source/ui/framework/configuration/ConfigurationTracer.cxx
index aef2366824c6..7ddaa23ae906 100644
--- a/sd/source/ui/framework/configuration/ConfigurationTracer.cxx
+++ b/sd/source/ui/framework/configuration/ConfigurationTracer.cxx
@@ -19,6 +19,7 @@
 
 #include "ConfigurationTracer.hxx"
 
+#include 
 #include 
 
 using namespace ::com::sun::star::uno;
@@ -30,11 +31,11 @@ void ConfigurationTracer::TraceConfiguration (
 const Reference& rxConfiguration,
 const char* pMessage)
 {
-#ifdef DEBUG
+#if OSL_DEBUG_LEVEL >=1
 SAL_INFO("sd.ui","" << pMessage << " at " << rxConfiguration.get() << " 
{");
 if (rxConfiguration.is())
 {
-TraceBoundResources(rxConfiguration, NULL, 0);
+TraceBoundResources(rxConfiguration, nullptr, 0);
 }
 else
 {
@@ -47,7 +48,7 @@ void ConfigurationTracer::TraceConfiguration (
 #endif
 }
 
-#ifdef DEBUG
+#if OSL_DEBUG_LEVEL >=1
 void ConfigurationTracer::TraceBoundResources (
 const Reference& rxConfiguration,
 const Reference& rxResourceId,
diff --git a/sd/source/ui/framework/configuration/ConfigurationTracer.hxx 
b/sd/source/ui/framework/configuration/ConfigurationTracer.hxx
index c2fc122b499a..facedd3f4600 100644
--- a/sd/source/ui/framework/configuration/ConfigurationTracer.hxx
+++ b/sd/source/ui/framework/configuration/ConfigurationTracer.hxx
@@ -23,6 +23,7 @@
 #include 
 
 namespace com { namespace sun { namespace star { namespace drawing { namespace 
framework { class XConfiguration; } } } } }
+namespace com { namespace sun { namespace star { namespace drawing { namespace 
framework { class XResourceId; } } } } }
 namespace com { namespace sun { namespace star { namespace uno { template 
 class Reference; } } } }
 
 namespace sd { namespace framework {
@@ -36,7 +37,7 @@ public:
 static void TraceConfiguration (
 const css::uno::Reference& 
rxConfiguration,
 const char* pMessage);
-#ifdef DEBUG
+#if OSL_DEBUG_LEVEL >=1
 static void TraceBoundResources (
 const css::uno::Reference& 
rxConfiguration,
 const css::uno::Reference& 
rxResourceId,
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 126530] Tabbed Notebook Bar Usability Issues on Windows 10

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126530

--- Comment #1 from Damian Hofmann  ---
Created attachment 152970
  --> https://bugs.documentfoundation.org/attachment.cgi?id=152970=edit
Tabbed Notebar on Windows 10 HiDPI

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126530] New: Tabbed Notebook Bar Usability Issues on Windows 10

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126530

Bug ID: 126530
   Summary: Tabbed Notebook Bar Usability Issues on Windows 10
   Product: LibreOffice
   Version: 6.2.5.2 release
  Hardware: All
OS: Windows (All)
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: UI
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: damian.hofm...@gmx.ch

Description:
The tabbed Notebook Bar has a host of display/usability issues on Windows 10:

1. Tabs are too small in height
2. Underlined Mnemonics add visual noise
3. Padding between text and border of tabs is too small
4. Not enough contrast between colors for currently active and inactive tabs
5. Buttons are poorly aligned, have no space between and are missing borders

Overall, the Notebook Bar looks cramped and noisy. Lack of contrast makes it
difficult to see, which tab is currently active. These layout problems impact
usability.

Tested on Windows 10 1809 with LO 6.2.5
Screen resolution: 4K (3840 x 2160) with 250% text scaling

Steps to Reproduce:
1.View > User Interface > Tabbed

Actual Results:
cramped and noisy UI

Expected Results:
clean UI


Reproducible: Always


User Profile Reset: No



Additional Info:

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 50916] Allow more than 1024 columns in calc

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=50916

--- Comment #90 from Gerry  ---
Test of the attachment to comment 86 with Gnumeric 1.12.43. Gnumeric loads the
spreadsheet nicely up to column 3201. No problem.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 125179] Writer crash footer Insert Field More Fields

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=125179

Ian Eales  changed:

   What|Removed |Added

 Resolution|--- |NOTOURBUG
 Status|NEEDINFO|RESOLVED

--- Comment #6 from Ian Eales  ---
Uninstalled 6.2.3
Huge hassle as programs folder locked by system
Installed 6.2.5 Version: 6.2.5.2 (x64)
Still crashes.
uninstalled 6.2.5
Deleted %AppData%\Roaming\LibreOffice
Installed 6.2.5
No more crashes on inserting Additional Fields

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 50916] Allow more than 1024 columns in calc

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=50916

--- Comment #89 from Gerry  ---
Another test with the attachement to comment 86: 
I opened the file in the document viewer of UBPorts (Ubuntu Touch). The
document viewer is based on LibreOffice 5.x Kit (is it version 5.4?).

The document loads fine. There is no error message. However, the application
crashes when I scroll to column 1024. Actually, it crashes when I am in column
1019, because then 1024 would be visible, which seems to cause the crash.

System:
UBports with document viewer (LibreOfficeKit 5.x)

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 100853] HiDPI: Check marks in Windows menus are drawn pixellated

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=100853

--- Comment #13 from Damian Hofmann  ---
Created attachment 152969
  --> https://bugs.documentfoundation.org/attachment.cgi?id=152969=edit
LO 6.2.5 on Windows 10 1803

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 100853] HiDPI: Check marks in Windows menus are drawn pixellated

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=100853

--- Comment #12 from Damian Hofmann  ---
Still looks bad.

Retested with LO 6.2.5 on Windows 10 1803. See attached screenshot.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126499] Chart in a form: setmodified(true) won't be executed

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126499

Oliver Brinzing  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|UNCONFIRMED |NEW

--- Comment #4 from Oliver Brinzing  ---
i can confirm with LO 6.1.3/LO 5.4.7.2:

- macro ChangeData() is called onLoad "Temperaturverlauf".

- macro is called after changinag a value,
- and chart model data is updated with new values
- but chart does not refresh

but it works with LO 4.4.7.2

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126529] Crash in: SwCursorShell::GetCurrFrame

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126529

--- Comment #1 from opensuse.lietuviu.ka...@gmail.com ---
See also
http://crashreport.libreoffice.org/stats/crash_details/1023a717-541c-4422-9ffd-095919bb4f09

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126529] New: Crash in: SwCursorShell::GetCurrFrame

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126529

Bug ID: 126529
   Summary: Crash in: SwCursorShell::GetCurrFrame
   Product: LibreOffice
   Version: 6.2.4.2 release
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Writer
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: opensuse.lietuviu.ka...@gmail.com

This bug was filed from the crash reporting server and is
br-2c9d5aea-982b-413d-85c7-79e80685e4fe.
=

I copied big part of text from one document. This text contained a lot of
tracked changes. I pasted text into another document as DDE link. LO Writer
crashed. I successed to reproduce bug with same steps and same text. But I can
not reproduce with simple text or liitle tracked changes.

Versija: 6.2.4.2
Darinio identifikatorius: 2412653d852ce75f65fbfa83fb7e7b669a126d64
Procesoriaus gijos: 4; OS:Linux 4.4; Sąsajos pateikimas: numatytasis; VCL:
kde4; 
Lokalė: lt-LT (lt_LT.UTF-8); UI-Language: lt-LT
Calc: threaded

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: Branch 'feature/lok-clipboard' - 3 commits - desktop/source

2019-07-24 Thread Ashod Nakashian (via logerrit)
Rebased ref, commits from common ancestor:
commit ad28cf191d24dfbd0b0c29ab1d72a5b20f4b66ef
Author: Ashod Nakashian 
AuthorDate: Mon Jul 22 19:00:23 2019 -0400
Commit: Michael Meeks 
CommitDate: Tue Jul 23 09:51:36 2019 +0100

clipboard: log before erasing

Change-Id: I6d59cdd31c62bbc7db81ccb8985745f68d19cd28

diff --git a/desktop/source/lib/lokclipboard.cxx 
b/desktop/source/lib/lokclipboard.cxx
index 7eefd09370d1..f44f4e9fed81 100644
--- a/desktop/source/lib/lokclipboard.cxx
+++ b/desktop/source/lib/lokclipboard.cxx
@@ -51,8 +51,8 @@ void LOKClipboardFactory::releaseClipboardForView(int nViewId)
 auto it = gClipboards.get()->find(nViewId);
 if (it != gClipboards.get()->end())
 {
+SAL_INFO("lok", "Releasing clip: " << it->second.get() << " for 
destroyed " << nViewId);
 gClipboards.get()->erase(it);
-SAL_INFO("lok", "Released clip: " << it->second.get() << " for 
destroyed " << nViewId);
 }
 }
 }
commit 26255a756c308ec296a670da4f758980b8fa105c
Author: Michael Meeks 
AuthorDate: Tue Jul 23 09:49:11 2019 +0100
Commit: Michael Meeks 
CommitDate: Tue Jul 23 09:49:11 2019 +0100

clipboard: check for null.

Change-Id: I07060035a9744642c98d4a0946695408dc0f6b42

diff --git a/desktop/source/lib/lokclipboard.cxx 
b/desktop/source/lib/lokclipboard.cxx
index 3285965c846f..7eefd09370d1 100644
--- a/desktop/source/lib/lokclipboard.cxx
+++ b/desktop/source/lib/lokclipboard.cxx
@@ -37,7 +37,6 @@ rtl::Reference 
LOKClipboardFactory::getClipboardForCurView()
 return xClip;
 }
 
-/// FIXME: should really copy and stash its content for a bit.
 void LOKClipboardFactory::releaseClipboardForView(int nViewId)
 {
 osl::MutexGuard aGuard(gMutex);
@@ -47,7 +46,7 @@ void LOKClipboardFactory::releaseClipboardForView(int nViewId)
 gClipboards.get()->clear();
 SAL_INFO("lok", "Released all clipboards on doc destroy\n");
 }
-else
+else if (gClipboards.get())
 {
 auto it = gClipboards.get()->find(nViewId);
 if (it != gClipboards.get()->end())
commit c6ad8b963e64217ccdac1914720b9e20c81fd28a
Author: Michael Meeks 
AuthorDate: Mon Jul 22 09:48:46 2019 +0100
Commit: Michael Meeks 
CommitDate: Mon Jul 22 11:39:22 2019 +0100

lok: share more of the clipboard mime-type handling code.

Change-Id: I98fc9fa0fd1b5afa00c33b642d2d504eab0b62ba

diff --git a/desktop/source/lib/lokclipboard.cxx 
b/desktop/source/lib/lokclipboard.cxx
index 17a9c5c54e4f..3285965c846f 100644
--- a/desktop/source/lib/lokclipboard.cxx
+++ b/desktop/source/lib/lokclipboard.cxx
@@ -189,20 +189,7 @@ LOKTransferable::LOKTransferable(const size_t nInCount, 
const char** pInMimeType
 m_aFlavors = css::uno::Sequence(nInCount);
 for (size_t i = 0; i < nInCount; ++i)
 {
-OUString aMimeType = OUString::fromUtf8(pInMimeTypes[i]);
-
-// cf. sot/source/base/exchange.cxx for these two exceptional types.
-if (aMimeType.startsWith("text/plain"))
-{
-aMimeType = "text/plain;charset=utf-16";
-m_aFlavors[i].DataType = cppu::UnoType::get();
-}
-else if (aMimeType == "application/x-libreoffice-tsvc")
-m_aFlavors[i].DataType = cppu::UnoType::get();
-else
-m_aFlavors[i].DataType = 
cppu::UnoType>::get();
-m_aFlavors[i].MimeType = aMimeType;
-m_aFlavors[i].HumanPresentableName = aMimeType;
+initFlavourFromMime(m_aFlavors[i], 
OUString::fromUtf8(pInMimeTypes[i]));
 
 uno::Any aContent;
 if (m_aFlavors[i].DataType == cppu::UnoType::get())
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-commits] core.git: qadevOOo/tests

2019-07-24 Thread Caolán McNamara (via logerrit)
 qadevOOo/tests/java/ifc/style/_ParagraphProperties.java |   10 +-
 1 file changed, 5 insertions(+), 5 deletions(-)

New commits:
commit 426eff7c6652033518c22d124f22c423e1ae054b
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 11:31:58 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 18:14:12 2019 +0200

cid#1448303 RV: Bad use of return value

Change-Id: I779ba87054cbfc10d2d1b26159c5c4873364ebfa
Reviewed-on: https://gerrit.libreoffice.org/76250
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/qadevOOo/tests/java/ifc/style/_ParagraphProperties.java 
b/qadevOOo/tests/java/ifc/style/_ParagraphProperties.java
index d1f134b9f258..88b1cae74dce 100644
--- a/qadevOOo/tests/java/ifc/style/_ParagraphProperties.java
+++ b/qadevOOo/tests/java/ifc/style/_ParagraphProperties.java
@@ -329,23 +329,23 @@ public class _ParagraphProperties extends 
MultiPropertyTest {
 NumberingRules = (XIndexReplace) AnyConverter.toObject(
new Type(XIndexReplace.class), 
oObj.getPropertyValue("NumberingRules"));
 } catch (com.sun.star.lang.IllegalArgumentException ex) {
-Status.failed( "could not get NumberingRuels: "+ ex.toString() );
+log.println("could not get NumberingRules: "+ ex.toString() );
 return;
 } catch (UnknownPropertyException ex) {
-Status.failed( "could not get NumberingRuels: "+ ex.toString() );
+log.println( "could not get NumberingRules: "+ ex.toString() );
 return;
 } catch (WrappedTargetException ex) {
-Status.failed( "could not get NumberingRuels: "+ ex.toString() );
+log.println( "could not get NumberingRules: "+ ex.toString() );
 return;
 }
 try {
 propertyValues = (PropertyValue[]) NumberingRules.getByIndex(0);
 
 } catch (com.sun.star.lang.IndexOutOfBoundsException ex) {
-Status.failed( "could not get NumberlingLevel-Array from 
NumberingRuels: "+ ex.toString() );
+log.println( "could not get NumberlingLevel-Array from 
NumberingRules: "+ ex.toString() );
 return;
 } catch (WrappedTargetException ex) {
-Status.failed( "could not get NumberlingLevel-Array from 
NumberingRuels: "+ ex.toString() );
+log.println( "could not get NumberlingLevel-Array from 
NumberingRules: "+ ex.toString() );
 return;
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

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

2019-07-24 Thread Caolán McNamara (via logerrit)
 idlc/source/astexpression.cxx |  390 +-
 1 file changed, 312 insertions(+), 78 deletions(-)

New commits:
commit d4dae168e4d82864636952b7cd4b4ef7e9befd6e
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 10:01:29 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 17:44:29 2019 +0200

cid#1448213 Assignment of overlapping memory

Change-Id: I6611f09000a0901b0d016cda7d8a418fc399fc63
Reviewed-on: https://gerrit.libreoffice.org/76229
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/idlc/source/astexpression.cxx b/idlc/source/astexpression.cxx
index 48963d43a0ac..aab83ba58948 100644
--- a/idlc/source/astexpression.cxx
+++ b/idlc/source/astexpression.cxx
@@ -106,55 +106,82 @@ coerce_value(AstExprValue *ev, ExprType t)
 case ET_short:
 return true;
 case ET_ushort:
+{
 if (ev->u.usval > SAL_MAX_INT16)
 return false;
-ev->u.sval = static_cast(ev->u.usval);
+auto tmp = static_cast(ev->u.usval);
+ev->u.sval = tmp;
 ev->et = ET_short;
 return true;
+}
 case ET_long:
+{
 if (ev->u.lval < SAL_MIN_INT16 || ev->u.lval > 
SAL_MAX_INT16)
 return false;
-ev->u.sval = static_cast(ev->u.lval);
+auto tmp = static_cast(ev->u.lval);
+ev->u.sval = tmp;
 ev->et = ET_short;
 return true;
+}
 case ET_ulong:
+{
 if (ev->u.ulval > SAL_MAX_INT16)
 return false;
-ev->u.sval = static_cast(ev->u.ulval);
+auto tmp = static_cast(ev->u.ulval);
+ev->u.sval = tmp;
 ev->et = ET_short;
 return true;
+}
 case ET_hyper:
+{
 if (ev->u.hval < SAL_MIN_INT16 || ev->u.hval > 
SAL_MAX_INT16)
 return false;
-ev->u.sval = static_cast(ev->u.hval);
+auto tmp = static_cast(ev->u.hval);
+ev->u.sval = tmp;
 ev->et = ET_short;
 return true;
+}
 case ET_uhyper:
+{
 if (ev->u.uhval > SAL_MAX_INT16)
 return false;
-ev->u.sval = static_cast(ev->u.uhval);
+auto tmp = static_cast(ev->u.uhval);
+ev->u.sval = tmp;
 ev->et = ET_short;
 return true;
+}
 case ET_boolean:
-ev->u.sval = static_cast(ev->u.bval);
+{
+auto tmp = static_cast(ev->u.bval);
+ev->u.sval = tmp;
 ev->et = ET_short;
 return true;
+}
 case ET_float:
+{
 if (ev->u.fval < SAL_MIN_INT16 || ev->u.fval > 
SAL_MAX_INT16)
 return false;
-ev->u.sval = static_cast(ev->u.fval);
+auto tmp = static_cast(ev->u.fval);
+ev->u.sval = tmp;
 ev->et = ET_short;
 return true;
+}
 case ET_double:
+{
 if (ev->u.dval < SAL_MIN_INT16 || ev->u.dval > 
SAL_MAX_INT16)
 return false;
-ev->u.sval = static_cast(ev->u.dval);
+auto tmp = static_cast(ev->u.dval);
+ev->u.sval = tmp;
 ev->et = ET_short;
 return true;
+}
 case ET_byte:
-ev->u.sval = static_cast(ev->u.byval);
+{
+auto tmp = static_cast(ev->u.byval);
+ev->u.sval = tmp;
 ev->et = ET_short;
 return true;
+}
 default:
 OSL_ASSERT(false);
 return false;
@@ -163,57 +190,84 @@ coerce_value(AstExprValue *ev, ExprType t)
 switch (ev->et)
 {
 case ET_short:
+{
 if (ev->u.sval < 0)
 return false;
-ev->u.usval = static_cast(ev->u.sval);
+auto tmp = static_cast(ev->u.sval);
+ev->u.usval = tmp;
 ev->et = ET_ushort;
 return true;
+}
 

[Libreoffice-bugs] [Bug 126519] UI - Calculation incorrect with VLOOKUP function, will not correct on recalculate

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126519

m.a.riosv  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 CC||miguelangelrv@libreoffice.o
   ||rg
 Status|UNCONFIRMED |NEEDINFO

--- Comment #1 from m.a.riosv  ---
Please attach a sample file showing the issues.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126528] [LocalHelp] Add Python info inside "Organize macros" help page

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126528

Olivier Hallot  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|UNCONFIRMED |NEW

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

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

2019-07-24 Thread Caolán McNamara (via logerrit)
 sax/source/fastparser/fastparser.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit da346f7afbe5e42b07c205c1c61c48d0f7036931
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 11:16:14 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 17:37:22 2019 +0200

cid#1448394 can't change xml_parse_config setting

Change-Id: I8c45c9d5e0a6bc7439aa4d0827e4752ec364d12c
Reviewed-on: https://gerrit.libreoffice.org/76247
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/sax/source/fastparser/fastparser.cxx 
b/sax/source/fastparser/fastparser.cxx
index 1ac7b209d311..6114cbc469d1 100644
--- a/sax/source/fastparser/fastparser.cxx
+++ b/sax/source/fastparser/fastparser.cxx
@@ -1055,6 +1055,7 @@ void FastSaxParserImpl::parse()
 throw SAXException("Couldn't create parser", Reference< 
XInterface >(), Any() );
 
 // Tell libxml2 parser to decode entities in attribute values.
+// coverity[unsafe-xml-parse-config] - entity support is required
 xmlCtxtUseOptions(rEntity.mpParser, XML_PARSE_NOENT);
 }
 else
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 126528] [LocalHelp] Add Python info inside "Organize macros" help page

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126528

LibreOfficiant  changed:

   What|Removed |Added

Version|6.3.0.0.alpha0+ |6.3.0.2 rc

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126528] [LocalHelp] Add Python info inside "Organize macros" help page

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126528

LibreOfficiant  changed:

   What|Removed |Added

URL||https://help.libreoffice.or
   ||g/6.4/en-US/text/shared/01/
   ||06130200.html?DbPAR=SHARED#
   ||bm_id3237403
   Keywords||easyHack,
   ||skillDocumentation,
   ||skillPython

--- Comment #1 from LibreOfficiant  ---
Python main page is
https://help.libreoffice.org/6.3/en-US/text/sbasic/python/main.html?DbPAR=BASIC#bm_id491543348147708

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 99296] Calc: Disable headers/footers by default

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=99296

--- Comment #23 from Heiko Tietze  ---
(In reply to Roman Kuznetsov from comment #22)
> Heiko, I suggest merge back your patch disabling header/footer by default.
> 
> For solution of the problem with "Insert->Headers and Footers" menu item I
> suggest follow variants for discuss:
> 
> 1. Just delete menu item Insert->Headers and Footers at all and update
> guides and Help.
> 2. Insert->Headers and Footers should be split to two menu items
> Insert->Header and Insert->Footer that will be open Format->Page dialog on
> Header and Footer tabs and will activate options "Header on" / "Footer on"
> respectively.
> 
> I think disabling of Headers/Footers by default is more important and useful
> thing than disabled "Insert->Headers and Footers" menu item that btw. has
> workaround.

Samuel, would that reassess your concerns?

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126528] New: [LocalHelp] Add Python info inside "Organize macros" help page

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126528

Bug ID: 126528
   Summary: [LocalHelp] Add Python info inside "Organize macros"
help page
   Product: LibreOffice
   Version: 6.3.0.0.alpha0+
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: minor
  Priority: medium
 Component: Documentation
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: libreoffici...@sfr.fr
CC: olivier.hal...@libreoffice.org

Description:
https://help.libreoffice.org/6.4/en-US/text/shared/01/06130200.html?DbPAR=SHARED#bm_id3237403
help page details Basic/BeanShell/JavaScript while it omits Python.
A new help Python page must be created, modelled as the above 3, that solely
document "Run" button and proposes to consult created Python help pages. 
An short presentation of APSO extension is a plus!

Steps to Reproduce:
1. Open
https://help.libreoffice.org/6.4/en-US/text/shared/01/06130200.html?DbPAR=SHARED#bm_id3237403
2. Observe Python information is missing
3. I'm done with steps

Actual Results:
3 links are available

Expected Results:
a 4th link would be welcome


Reproducible: Always


User Profile Reset: No



Additional Info:

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: soltools/cpp

2019-07-24 Thread Caolán McNamara (via logerrit)
 soltools/cpp/_cpp.c   |1 +
 soltools/cpp/_lex.c   |1 +
 soltools/cpp/_macro.c |7 +++
 soltools/cpp/_unix.c  |1 +
 4 files changed, 10 insertions(+)

New commits:
commit 421c7d874d62f3d0f696df540744845753902583
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 10:25:09 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 17:20:04 2019 +0200

cid#1448523 silence bogus Out-of-bounds read

Change-Id: I0911133287ace66454e741f7ea994f191d06931c
Reviewed-on: https://gerrit.libreoffice.org/76232
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/soltools/cpp/_cpp.c b/soltools/cpp/_cpp.c
index 0a7ce78d1749..003611c3f53b 100644
--- a/soltools/cpp/_cpp.c
+++ b/soltools/cpp/_cpp.c
@@ -74,6 +74,7 @@ void
 {
 trp->tp = trp->lp = trp->bp;
 outptr = outbuf;
+// coverity[overrun-buffer-arg: FALSE] - a multiple of trp->max is 
allocated, not trp->max itself
 anymacros |= gettokens(trp, 1);
 trp->tp = trp->bp;
 }
diff --git a/soltools/cpp/_lex.c b/soltools/cpp/_lex.c
index 0efd303d242e..dfd8605a3950 100644
--- a/soltools/cpp/_lex.c
+++ b/soltools/cpp/_lex.c
@@ -364,6 +364,7 @@ continue2:
 {
 trp->lp = tp;
 tp = growtokenrow(trp);
+// coverity[overrun-local : FALSE] - a multiple of trp->max is 
allocated, not trp->max itself
 maxp = >bp[trp->max];
 }
 tp->type = UNCLASS;
diff --git a/soltools/cpp/_macro.c b/soltools/cpp/_macro.c
index 8a89e9379d36..eaafb86034ef 100644
--- a/soltools/cpp/_macro.c
+++ b/soltools/cpp/_macro.c
@@ -287,7 +287,10 @@ void
 if (np->flag & ISMAC)
 builtin(trp, np->val);
 else
+{
+// coverity[overrun-buffer-arg: FALSE] - a multiple of trp->max is 
allocated, not trp->max itself
 expand(trp, np, );
+}
 tp = trp->tp;
 }   // end for
 if (flag)
@@ -430,7 +433,10 @@ int
 while (parens > 0)
 {
 if (trp->tp >= trp->lp)
+{
+// coverity[overrun-buffer-arg: FALSE] - a multiple of trp->max is 
allocated, not trp->max itself
 gettokens(trp, 0);
+}
 if (needspace)
 {
 needspace = 0;
@@ -602,6 +608,7 @@ void
 tt[len] = '\0';
 setsource("<##>", -1, -1, tt, 0);
 maketokenrow(3, );
+// coverity[overrun-buffer-arg: FALSE] - a multiple of 
trp->max is allocated, not trp->max itself
 gettokens(, 1);
 unsetsource();
 if (ntr.bp->type == UNCLASS)
diff --git a/soltools/cpp/_unix.c b/soltools/cpp/_unix.c
index 2ee9215524a2..ac4b7624a00a 100644
--- a/soltools/cpp/_unix.c
+++ b/soltools/cpp/_unix.c
@@ -87,6 +87,7 @@ void
 case 'A':
 setsource("", -1, -1, optarg, 0);
 maketokenrow(3, );
+// coverity[overrun-buffer-arg: FALSE] - a multiple of 
trp->max is allocated, not trp->max itself
 gettokens(, 1);
 doadefine(, c);
 dofree(tr.bp);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

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

2019-07-24 Thread Caolán McNamara (via logerrit)
 sc/source/core/data/validat.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 672da0902b406c88042dfa3b48dd565700514343
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 11:28:35 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 17:16:12 2019 +0200

cid#1448321 Dereference after null check

Change-Id: I724e4c19df9aa446f496b21ea5f3918e2def28da
Reviewed-on: https://gerrit.libreoffice.org/76249
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/sc/source/core/data/validat.cxx b/sc/source/core/data/validat.cxx
index 0444dbf2ef5b..4e0e62ddcada 100644
--- a/sc/source/core/data/validat.cxx
+++ b/sc/source/core/data/validat.cxx
@@ -846,7 +846,7 @@ bool ScValidationData::GetSelectionFromFormula(
 
 if( pEntry )
 {
-assert(pEntry);
+assert(pStrings);
 pStrings->push_back(*pEntry);
 n++;
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

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

2019-07-24 Thread Caolán McNamara (via logerrit)
 vcl/source/fontsubset/sft.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 29a2296af8c0d0e6a630bce1c064e1211047d014
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 11:37:16 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 17:05:43 2019 +0200

cid#1448301 silence Untrusted loop bound

Change-Id: Ib5b592fdcf234a78747058eb3cdc7efc5d9cddc6
Reviewed-on: https://gerrit.libreoffice.org/76251
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/vcl/source/fontsubset/sft.cxx b/vcl/source/fontsubset/sft.cxx
index 2e974a10d3c6..e9aae0ba9356 100644
--- a/vcl/source/fontsubset/sft.cxx
+++ b/vcl/source/fontsubset/sft.cxx
@@ -436,6 +436,7 @@ static int GetSimpleTTOutline(TrueTypeFont const *ttf, 
sal_uInt32 glyphID, Contr
 pa[i++].flags = static_cast(flag);
 if (flag & 8) { /*- repeat flag */
 n = *p++;
+// coverity[tainted_data] - i > lastPoint extra checks the n loop 
bound
 for (j=0; j lastPoint) {/*- if the font is 
really broken */
 free(pa);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

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

2019-07-24 Thread Caolán McNamara (via logerrit)
 unoidl/source/sourceprovider-parser.y |   21 +++--
 1 file changed, 15 insertions(+), 6 deletions(-)

New commits:
commit d397372f5cf9ff1fa67720c76768c3cc29f7de59
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 09:09:06 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 17:04:24 2019 +0200

cid#1448548 Assignment of overlapping memory

Change-Id: I146abd4635db993a5a0c5e612bb3f6dc23cdfe24
Reviewed-on: https://gerrit.libreoffice.org/76227
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/unoidl/source/sourceprovider-parser.y 
b/unoidl/source/sourceprovider-parser.y
index 94573b1ea147..965a3b9b8c37 100644
--- a/unoidl/source/sourceprovider-parser.y
+++ b/unoidl/source/sourceprovider-parser.y
@@ -228,8 +228,11 @@ bool coerce(
 ok = true;
 break;
 case unoidl::detail::SourceProviderExpr::TYPE_FLOAT:
-lhs->fval = lhs->uval;
-ok = true;
+{
+auto nTmp = lhs->uval;
+lhs->fval = nTmp;
+ok = true;
+}
 break;
 }
 break;
@@ -239,12 +242,18 @@ bool coerce(
 ok = false;
 break;
 case unoidl::detail::SourceProviderExpr::TYPE_INT:
-rhs->fval = rhs->ival;
-ok = true;
+{
+auto tmp = rhs->ival;
+rhs->fval = tmp;
+ok = true;
+}
 break;
 case unoidl::detail::SourceProviderExpr::TYPE_UINT:
-rhs->fval = rhs->uval;
-ok = true;
+{
+auto tmp = rhs->uval;
+rhs->fval = tmp;
+ok = true;
+}
 break;
 case unoidl::detail::SourceProviderExpr::TYPE_FLOAT:
 ok = true;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

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

2019-07-24 Thread Caolán McNamara (via logerrit)
 filter/source/msfilter/dffpropset.cxx |   17 +
 1 file changed, 13 insertions(+), 4 deletions(-)

New commits:
commit a657260c419a7138971aeb0151c81dcb23df3748
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 10:34:01 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 17:04:04 2019 +0200

cid#1448519 Untrusted loop bound

Change-Id: I92952fdef3e1066082d7ba8c3befebfcb7f9adc1
Reviewed-on: https://gerrit.libreoffice.org/76242
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/filter/source/msfilter/dffpropset.cxx 
b/filter/source/msfilter/dffpropset.cxx
index 73c0e45d64db..5b0b1194ae06 100644
--- a/filter/source/msfilter/dffpropset.cxx
+++ b/filter/source/msfilter/dffpropset.cxx
@@ -20,6 +20,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -1109,14 +1110,22 @@ void DffPropSet::ReadPropSet( SvStream& rIn, bool 
bSetUninitializedOnly )
 
 sal_uInt32 nComplexDataFilePos = rIn.Tell() + ( nPropCount * 6 );
 
-for( sal_uInt32 nPropNum = 0; nPropNum < nPropCount; nPropNum++ )
+const size_t nMaxPossibleRecords = rIn.remainingSize() / 
(sizeof(sal_uInt16) + sizeof(sal_uInt32));
+if (nPropCount > nMaxPossibleRecords)
 {
-sal_uInt16 nTmp;
-sal_uInt32 nRecType, nContent;
+SAL_WARN("filter.ms", "Parsing error: " << nMaxPossibleRecords <<
+ " max possible entries, but " << nPropCount << " claimed, 
truncating");
+nPropCount = nMaxPossibleRecords;
+}
+
+for (sal_uInt32 nPropNum = 0; nPropNum < nPropCount; ++nPropNum)
+{
+sal_uInt16 nTmp(0);
+sal_uInt32 nContent(0);
 rIn.ReadUInt16( nTmp )
.ReadUInt32( nContent );
 
-nRecType = nTmp & 0x3fff;
+sal_uInt32 nRecType = nTmp & 0x3fff;
 
 if ( nRecType > 0x3ff )
 break;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 115709] [META] DOCX (OOXML) Tracking changes-related issues

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=115709
Bug 115709 depends on bug 126516, which changed state.

Bug 126516 Summary: FILEOPEN DOCX Rejecting multi-paragraph deletion destroys 
change tracking of formatting
https://bugs.documentfoundation.org/show_bug.cgi?id=126516

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126516] FILEOPEN DOCX Rejecting multi-paragraph deletion destroys change tracking of formatting

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126516

László Németh  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #3 from László Németh  ---
Fixed in https://gerrit.libreoffice.org/#/c/76154/

https://gerrit.libreoffice.org/plugins/gitiles/core/+/5ab9cbe7f8d0c0bfd6a2fe1de2e14dfe201d008b%5E%21

tdf#126516 DOCX change tracking: fix paragraph style rejection

by keeping paragraph-level ParagraphFormat redlines.

(regression from commit a5abe0fc4d435d3a7a7de8bf55ec74087fdd299a
"tdf#125546 DOCX import: fix overgrowth of change tracking entries")

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126527] find (and replace) by format does not find anything (in some cases)

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126527

--- Comment #2 from stragu  ---
Created attachment 152968
  --> https://bugs.documentfoundation.org/attachment.cgi?id=152968=edit
preexisting doc converted from RTF to ODT

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126527] find (and replace) by format does not find anything (in some cases)

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126527

--- Comment #1 from stragu  ---
Created attachment 152967
  --> https://bugs.documentfoundation.org/attachment.cgi?id=152967=edit
new doc created with Writer

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126527] New: find (and replace) by format does not find anything (in some cases)

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126527

Bug ID: 126527
   Summary: find (and replace) by format does not find anything
(in some cases)
   Product: LibreOffice
   Version: 6.2.5.2 release
  Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Writer
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: stephane.guil...@member.fsf.org

Description:
Using "Find and Replace > Format..." does not work in a new document, when
looking for text formatted in a certain way. It does however work in other
documents.

Steps to Reproduce:
1. Open attachment new_doc.odt (brand new document created with LibreOffice)
2. Find and Replace > Format... > Font Effects > Strikethrough > Single
3. Click "OK"
4. Click "Find all"

Actual Results:
The words that are strikethrough are not selected

Expected Results:
Nothing was found


Reproducible: Always


User Profile Reset: No



Additional Info:
The same steps done in preexisting_doc.odt work as expected for some reason.
This file used to be an RTF document, that had (possibly) been edited with
Microsoft Office at some stage.

Not sure how to consistently create a document in which the format search will
work.

Tested with:

Version: 6.2.5.2
Build ID: 1:6.2.5-0ubuntu0.18.04.1~lo1
CPU threads: 8; OS: Linux 4.15; UI render: default; VCL: gtk3; 
Locale: en-AU (en_AU.UTF-8); UI-Language: en-GB
Calc: threaded

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 56959] libvisio: Draw does not import properly MS Visio connectors

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=56959

--- Comment #20 from Bartosz  ---
Detection of connectors are implemented here:
https://gerrit.libreoffice.org/#/c/75649/

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: dtrans/test

2019-07-24 Thread Andrea Gelmini (via logerrit)
 dtrans/test/win32/dnd/atlwindow.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit e622abbab580434038c117b013757885d3f0fb58
Author: Andrea Gelmini 
AuthorDate: Wed Jul 24 08:42:54 2019 +0200
Commit: Andrea Gelmini 
CommitDate: Wed Jul 24 16:40:36 2019 +0200

Fix typo

Change-Id: I6632893980c8f71addda60c9bf447ceca962804d
Reviewed-on: https://gerrit.libreoffice.org/72691
Tested-by: Jenkins
Reviewed-by: Andrea Gelmini 

diff --git a/dtrans/test/win32/dnd/atlwindow.cxx 
b/dtrans/test/win32/dnd/atlwindow.cxx
index 9e1e2e1fc693..95aca10e75fc 100644
--- a/dtrans/test/win32/dnd/atlwindow.cxx
+++ b/dtrans/test/win32/dnd/atlwindow.cxx
@@ -115,7 +115,7 @@ LRESULT AWindow::OnCreate(UINT uMsg, WPARAM wParam, LPARAM 
lParam, BOOL& bHandle
 
 m_xDropTarget->addDropTargetListener( static_cast
 ( new DropTargetListener( m_hwndEdit)) );
-//  // make this window tho a drop target
+//  // make this window a drop target
 m_xDropTarget->setActive(sal_True);
 
 return 0;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 126526] New: Overlays of cell's borders using Rows to repeat

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126526

Bug ID: 126526
   Summary: Overlays of cell's borders using Rows to repeat
   Product: LibreOffice
   Version: 6.1.6.3 release
  Hardware: All
OS: Windows (All)
Status: UNCONFIRMED
  Severity: minor
  Priority: medium
 Component: Calc
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: micael.camo...@gmail.com

Description:
the top border of the free cells overlays the bottom border of the Rows to
repeat staring from the 2nd page

Steps to Reproduce:
1.Set full borders in black color in A1
2.Set "Rows to repeat" to row 1
3.Set full borders in red color from A2 to A80
4.Click on "print preview"

Actual Results:
If you look at the preview, the borders of the A1 cell are black in 1st page,
but when you go to next page, the bottom border becomes red. The border of the
free cells overlays the fixex cells.

Expected Results:
If I want to repeat a row, I expect to see its full features, including the
bottom borders.


Reproducible: Always


User Profile Reset: No



Additional Info:

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 93331] LibreOffice.app bundle integrity is destroyed by language pack installer

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=93331

--- Comment #14 from Tor Lillqvist  ---
Allow me a bit of marketing: The LibreOffice Vanilla app in the Mac App Store
does not suffer from this problem. It includes a bunch of languages already.
(The localisations included are those that have "good enough" coverage to be
worth it, and yes, that is not something that can be decided objectively, but a
guesstimate.)

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126521] Incorrect export to Excel of Calc spreadsheet containing Multiple Operations

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126521

Roman Kuznetsov <79045_79...@mail.ru> changed:

   What|Removed |Added

   Keywords||filter:xlsx
 CC||er...@redhat.com

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126525] New: Suite-wide color picker

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126525

Bug ID: 126525
   Summary: Suite-wide color picker
   Product: LibreOffice
   Version: unspecified
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: enhancement
  Priority: medium
 Component: LibreOffice
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: gtomor...@libero.it

I was reading https://bugs.documentfoundation.org/show_bug.cgi?id=93813;>Bug 93813
and got to thinking about how the entire suite, not just Draw, needs this
feature as swatch creation in LO is non-trivial. 

I'm pretty sure some developer on your team had already considered this...but
just in case...

I was thinking LO could hook into the existing system-wide color picker
available in Win/Mac/Linux...ah, Linux (regardless of DE) doesn't have a
system-wide color picker.

So, instead of "reinventing the wheel" and coding from scratch, couldn't an
existing GPL/MIT solution be integrated, thus eliminating the need for (at
least) three different solutions.

Pardon me for being presumptious. I admit I'm not in anyway a programmer, just
an end-user, but I feel in this day and age a suite-wide color picker is
commonplace, and judging by questions and requests posted here, on
ask.libreoffice.org and elsewhere, I'm not alone. Thank you.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 93331] LibreOffice.app bundle integrity is destroyed by language pack installer

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=93331

--- Comment #13 from Dale  ---
I believe this bug is still present.  Information from "About LibreOffice":

Version: 6.2.5.2
Build ID: 1ec314fa52f458adc18c4f025c545a4e8b22c159
CPU threads: 16; OS: Mac OS X 10.14.6; UI render: default; VCL: osx; 
Locale: en-US (en_US.UTF-8); UI-Language: en-US
Calc: threaded

To reproduce: install LibreOffice.app into ~/Applications (or anywhere your
user can write to it), start LibreOffice, open a blank Writer document. 
Afterwards:

$ codesign -vv ~/Applications/LibreOffice.app/
/Users/dale/Applications/LibreOffice.app/: a sealed resource is missing or
invalid
file added:
/Users/dale/Applications/LibreOffice.app/Contents/Resources/__pycache__/uno.cpython-35.pyc
file added:
/Users/dale/Applications/LibreOffice.app/Contents/Resources/__pycache__/pythonloader.cpython-35.pyc
file added:
/Users/dale/Applications/LibreOffice.app/Contents/Resources/__pycache__/unohelper.cpython-35.pyc
file added:
/Users/dale/Applications/LibreOffice.app/Contents/Resources/extensions/dict-en/pythonpath/__pycache__/lightproof_opts_en.cpython-35.pyc
file added:
/Users/dale/Applications/LibreOffice.app/Contents/Resources/extensions/dict-en/pythonpath/__pycache__/lightproof_impl_en.cpython-35.pyc
file added:
/Users/dale/Applications/LibreOffice.app/Contents/Resources/extensions/dict-en/pythonpath/__pycache__/lightproof_en.cpython-35.pyc
file added:
/Users/dale/Applications/LibreOffice.app/Contents/Resources/extensions/dict-en/pythonpath/__pycache__/lightproof_handler_en.cpython-35.pyc

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 112470] EDITING: Exiting edit mode via clicking on header skewes column selection

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=112470

Miklos Vajna  changed:

   What|Removed |Added

 Blocks|118739  |

--- Comment #2 from Miklos Vajna  ---
I just tried this again with older versions, I can even see rendering errors
with OpenOffice.org on Linux. So at least it does not belong to the
"List-Combobox-RenderContext-regressions" tracker.


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=118739
[Bug 118739] [META] Refactor ListBox/ComboBox to use RenderContext regressions
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 118739] [META] Refactor ListBox/ComboBox to use RenderContext regressions

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=118739

Miklos Vajna  changed:

   What|Removed |Added

 Depends on|112470  |


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=112470
[Bug 112470] EDITING: Exiting edit mode via clicking on header skewes column
selection
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 99296] Calc: Disable headers/footers by default

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=99296

--- Comment #22 from Roman Kuznetsov <79045_79...@mail.ru> ---
Heiko, I suggest merge back your patch disabling header/footer by default.

For solution of the problem with "Insert->Headers and Footers" menu item I
suggest follow variants for discuss:

1. Just delete menu item Insert->Headers and Footers at all and update guides
and Help.
2. Insert->Headers and Footers should be split to two menu items Insert->Header
and Insert->Footer that will be open Format->Page dialog on Header and Footer
tabs and will activate options "Header on" / "Footer on" respectively.

I think disabling of Headers/Footers by default is more important and useful 
thing than disabled "Insert->Headers and Footers" menu item that btw. has
workaround.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126446] Pixelated and blurry icons on a fresh Windows 10 install with dpi higher than 125%

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126446

V Stuart Foote  changed:

   What|Removed |Added

 CC||vstuart.fo...@utsa.edu
 Ever confirmed|0   |1
 Status|UNCONFIRMED |NEEDINFO

--- Comment #3 from V Stuart Foote  ---
Default 'Automatic' icon theme selection on Windows are the Colibre icons in
PNG. The PNG will be scaled in bulk in response to setting os UI magnification.

That scaling of the PNG bitmaped icons can appear pixeleated at higher
valuse--I find 175% to be ugly, %150 is marginal.

Fortunately to support scaling to support HiDPI displays, LibreOffice now
provides icon themes as SVG.  The SVG are scaled and then saved in bulk to new
PNG corresponding to the UI magnification.

Give the SVGs a test from Tools -> Options -> View: 'Icon Style' droplist then
close and reopen LibreOffice--a new SVG derived icon set will be built for the
UI magnification set.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

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

2019-07-24 Thread Caolán McNamara (via logerrit)
 sd/source/ui/unoidl/UnoDocumentSettings.cxx |4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

New commits:
commit 1733b527ac07fcb0067905e7277227f086b53afb
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 08:43:28 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 16:04:39 2019 +0200

cid#1451642 silence Division or modulo by zero

Change-Id: I728528ecbb69c23b63807944e4ae0beb9cb86cbb
Reviewed-on: https://gerrit.libreoffice.org/76222
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/sd/source/ui/unoidl/UnoDocumentSettings.cxx 
b/sd/source/ui/unoidl/UnoDocumentSettings.cxx
index 3add9902b8c7..2b426a632380 100644
--- a/sd/source/ui/unoidl/UnoDocumentSettings.cxx
+++ b/sd/source/ui/unoidl/UnoDocumentSettings.cxx
@@ -701,7 +701,9 @@ DocumentSettings::_setPropertyValues(const 
PropertyMapEntry** ppEntries,
 sal_Int32 nValue = 0;
 if( *pValues >>= nValue )
 {
-Fraction aFract( pDoc->GetUIScale().GetNumerator(), 
nValue );
+auto nNumerator = pDoc->GetUIScale().GetNumerator();
+assert(nNumerator != 0);
+Fraction aFract(nNumerator, nValue);
 pDoc->SetUIScale( aFract );
 bOk = true;
 bChanged = true;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

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

2019-07-24 Thread Ashod Nakashian (via logerrit)
 sd/source/ui/view/drviews1.cxx |   26 +-
 1 file changed, 5 insertions(+), 21 deletions(-)

New commits:
commit 8213923c183c95c58b8c82ab6ad5e3f1611a4202
Author: Ashod Nakashian 
AuthorDate: Wed Nov 28 22:06:05 2018 -0500
Commit: Noel Grandin 
CommitDate: Wed Jul 24 16:02:18 2019 +0200

LOK: sd: use native SdPage to flag selection

The slide-sorter is actually still necessary to apply/change
the master slide of a given slide, so we must enable it.

Change-Id: I3f59f58be76ab1c63453b4f6288044572800a556

Change-Id: I7554ba4afd28d7ea4f3ed6ba375d9765a89ef21c
Reviewed-on: https://gerrit.libreoffice.org/69618
Reviewed-by: Ashod Nakashian 
Tested-by: Ashod Nakashian 
Reviewed-on: https://gerrit.libreoffice.org/73497
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/sd/source/ui/view/drviews1.cxx b/sd/source/ui/view/drviews1.cxx
index 768bf33ef473..67a651122497 100644
--- a/sd/source/ui/view/drviews1.cxx
+++ b/sd/source/ui/view/drviews1.cxx
@@ -774,31 +774,15 @@ bool DrawViewShell::ActivateObject(SdrOle2Obj* pObj, long 
nVerb)
  */
 bool DrawViewShell::SelectPage(sal_uInt16 nPage, sal_uInt16 nSelect)
 {
-slidesorter::SlideSorterViewShell* pSlideSorterViewShell
-= 
slidesorter::SlideSorterViewShell::GetSlideSorter(GetViewShellBase());
-if (pSlideSorterViewShell != nullptr)
+SdPage* pPage = GetDoc()->GetSdPage(nPage, PageKind::Standard);
+if (pPage)
 {
-slidesorter::controller::PageSelector& aPageSelector
-= 
pSlideSorterViewShell->GetSlideSorter().GetController().GetPageSelector();
 if (nSelect == 0)
-{
-// Deselect.
-aPageSelector.DeselectPage(nPage);
-
-}
+pPage->SetSelected(false); // Deselect.
 else if (nSelect == 1)
-{
-// Select.
-aPageSelector.SelectPage(nPage);
-}
+pPage->SetSelected(true); // Select.
 else
-{
-// Toggle.
-if (aPageSelector.IsPageSelected(nPage))
-aPageSelector.DeselectPage(nPage);
-else
-aPageSelector.SelectPage(nPage);
-}
+pPage->SetSelected(!pPage->IsSelected()); // Toggle.
 
 return true;
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 126400] LibreOffice math does not get focus back after using tool window

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126400

V Stuart Foote  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Blocks||103461
 Status|UNCONFIRMED |NEEDINFO
 CC||rb.hensc...@t-online.de,
   ||vstuart.fo...@utsa.edu

--- Comment #1 from V Stuart Foote  ---
Can not confirm on Winndows 10 Home 64-bit en-US (1903) with
Version: 6.2.5.2 (x64)
Build ID: 1ec314fa52f458adc18c4f025c545a4e8b22c159
CPU threads: 4; OS: Windows 10.0; UI render: GL; VCL: win; 
Locale: en-US (en_US); UI-Language: en-US
Calc: threaded

Focus after selection from the elements pane shifts back to the input window. I
didn't check with (experimental) visual mode--but that is not likely relevant
here.

What os/DE distro are you using? And, have you tried using a different VCL
plugin?


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=103461
[Bug 103461] [META] Elements pane/dock/window/sidebar in Math
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 103461] [META] Elements pane/dock/window/sidebar in Math

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=103461

V Stuart Foote  changed:

   What|Removed |Added

 Depends on||126400


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=126400
[Bug 126400] LibreOffice math does not get focus back after using tool window
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126524] Default unnamed data series name longer than in Excel

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126524

Gabor Kelemen  changed:

   What|Removed |Added

   Assignee|libreoffice-b...@lists.free |kelem...@ubuntu.com
   |desktop.org |
 Ever confirmed|0   |1
 Status|UNCONFIRMED |ASSIGNED

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126524] Default unnamed data series name longer than in Excel

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126524

NISZ LibreOffice Team  changed:

   What|Removed |Added

 Blocks||75057
 CC||kelem...@ubuntu.com,
   ||nem...@numbertext.org


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=75057
[Bug 75057] [META] OOXML Chart object issues
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 75057] [META] OOXML Chart object issues

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=75057

NISZ LibreOffice Team  changed:

   What|Removed |Added

 Depends on||126524


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=126524
[Bug 126524] Default unnamed data series name longer than in Excel
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126524] Default unnamed data series name longer than in Excel

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126524

--- Comment #2 from NISZ LibreOffice Team  ---
Created attachment 152966
  --> https://bugs.documentfoundation.org/attachment.cgi?id=152966=edit
Screenshot of the original document side by side in Word and Writer

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126524] Default unnamed data series name longer than in Excel

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126524

--- Comment #1 from NISZ LibreOffice Team  ---
Created attachment 152965
  --> https://bugs.documentfoundation.org/attachment.cgi?id=152965=edit
Example file from Word

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126524] New: Default unnamed data series name longer than in Excel

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126524

Bug ID: 126524
   Summary: Default unnamed data series name longer than in Excel
   Product: LibreOffice
   Version: 6.4.0.0.alpha0+ Master
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Calc
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: libreoff...@nisz.hu

Description:
If there is an unnamed data series in a chart data source then the default
placeholder name is much longer than in MSO. This can affect the layout of the
chart. 

Steps to Reproduce:
Steps to reproduce:
1.  Open attached file in Word and recent Writer


Actual Results:
The default data series name is “Unnamed Series N” instead of “SeriesN”. As
consequence the chart draw area is a bit narrower.

Expected Results:
Same default name.


Reproducible: Always


User Profile Reset: No



Additional Info:
LibreOffice details:
Version: 6.4.0.0.alpha0+ (x86)
Build ID: 382aceb80d3c0f420d51762e1f88eee2334ff11b
CPU threads: 4; OS: Windows 6.3; UI render: default; VCL: win; 
TinderBox: Win-x86@42, Branch:master, Time: 2019-07-21_23:45:18
Locale: hu-HU (hu_HU); UI-Language: en-US
Calc: CL

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: compilerplugins/clang framework/inc framework/source

2019-07-24 Thread Noel Grandin (via logerrit)
 compilerplugins/clang/unusedfields.only-used-in-constructor.results |   30 
+++---
 compilerplugins/clang/unusedfields.readonly.results |   26 
++---
 compilerplugins/clang/unusedfields.untouched.results|   14 +-
 compilerplugins/clang/unusedfields.writeonly.results|   50 
+-
 framework/inc/xml/imagesdocumenthandler.hxx |5 -
 framework/source/xml/imagesdocumenthandler.cxx  |   10 --
 6 files changed, 63 insertions(+), 72 deletions(-)

New commits:
commit 5ae7908ecec19d2d7f5464bf89fcd26a4aef3ace
Author: Noel Grandin 
AuthorDate: Wed Jul 24 11:26:48 2019 +0200
Commit: Noel Grandin 
CommitDate: Wed Jul 24 15:36:24 2019 +0200

loplugin:unusedfields

Change-Id: I80ed513cf690f94bf4320b040b36d7f904c63482
Reviewed-on: https://gerrit.libreoffice.org/76233
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git 
a/compilerplugins/clang/unusedfields.only-used-in-constructor.results 
b/compilerplugins/clang/unusedfields.only-used-in-constructor.results
index 8d2d4d9bb5d5..283d724286d2 100644
--- a/compilerplugins/clang/unusedfields.only-used-in-constructor.results
+++ b/compilerplugins/clang/unusedfields.only-used-in-constructor.results
@@ -76,9 +76,9 @@ connectivity/source/inc/flat/EResultSet.hxx:42
 connectivity::flat::OFlatResultSet m_bBookmarkable _Bool
 connectivity/source/inc/java/lang/Object.hxx:45
 connectivity::SDBThreadAttach m_aGuard jvmaccess::class 
VirtualMachine::AttachGuard
-cppcanvas/source/mtfrenderer/textaction.cxx:809
+cppcanvas/source/mtfrenderer/textaction.cxx:811
 cppcanvas::internal::(anonymous namespace)::EffectTextAction 
maTextLineInfo const tools::TextLineInfo
-cppcanvas/source/mtfrenderer/textaction.cxx:1638
+cppcanvas/source/mtfrenderer/textaction.cxx:1644
 cppcanvas::internal::(anonymous namespace)::OutlineAction maTextLineInfo 
const tools::TextLineInfo
 cppu/source/threadpool/threadpool.cxx:354
 _uno_ThreadPool dummy sal_Int32
@@ -186,6 +186,16 @@ filter/source/graphicfilter/icgm/chart.hxx:46
 DataNode nBoxX2 sal_Int16
 filter/source/graphicfilter/icgm/chart.hxx:47
 DataNode nBoxY2 sal_Int16
+framework/inc/xml/imagesdocumenthandler.hxx:99
+framework::OReadImagesDocumentHandler m_bExternalImagesStartFound _Bool
+framework/inc/xml/imagesdocumenthandler.hxx:100
+framework::OReadImagesDocumentHandler m_bExternalImageStartFound _Bool
+framework/inc/xml/imagesdocumenthandler.hxx:101
+framework::OReadImagesDocumentHandler m_nHashMaskModeBitmap sal_Int32
+framework/inc/xml/imagesdocumenthandler.hxx:102
+framework::OReadImagesDocumentHandler m_nHashMaskModeColor sal_Int32
+framework/inc/xml/imagesdocumenthandler.hxx:133
+framework::OWriteImagesDocumentHandler m_aXMLXlinkNS class rtl::OUString
 helpcompiler/inc/HelpCompiler.hxx:197
 HelpCompiler lang const std::string
 include/basic/basmgr.hxx:56
@@ -736,9 +746,9 @@ sw/inc/unosett.hxx:145
 SwXNumberingRules m_pImpl ::sw::UnoImplPtr
 sw/qa/core/test_ToxTextGenerator.cxx:139
 ToxTextGeneratorWithMockedChapterField mChapterFieldType class 
SwChapterFieldType
-sw/qa/extras/layout/layout.cxx:2608
-  class SvtSysLocaleOptions &
-sw/qa/extras/uiwriter/uiwriter.cxx:4068
+sw/qa/extras/uiwriter/uiwriter2.cxx:65
+  class SwUiWriterTest2 *
+sw/qa/extras/uiwriter/uiwriter.cxx:4065
 IdleTask maIdle class Idle
 sw/source/core/crsr/crbm.cxx:66
 (anonymous namespace)::CursorStateHelper m_aSaveState const class 
SwCursorSaveState
@@ -750,10 +760,6 @@ sw/source/core/text/inftxt.hxx:679
 SwTextSlot aText class rtl::OUString
 sw/source/core/text/porfld.cxx:143
 SwFieldSlot aText class rtl::OUString
-sw/source/filter/inc/fltshell.hxx:214
-SwFltRedline aStampPrev const class DateTime
-sw/source/filter/inc/fltshell.hxx:216
-SwFltRedline eTypePrev const enum RedlineType
 sw/source/uibase/docvw/romenu.hxx:35
 SwReadOnlyPopup m_aBuilder class VclBuilder
 sw/source/uibase/inc/condedit.hxx:43
@@ -832,9 +838,9 @@ vcl/source/app/salvtables.cxx:2568
 SalInstanceEntry m_aTextFilter class (anonymous namespace)::WeldTextFilter
 vcl/source/app/salvtables.cxx:5012
 SalInstanceComboBoxWithEdit m_aTextFilter class (anonymous 
namespace)::WeldTextFilter
-vcl/source/gdi/jobset.cxx:36
+vcl/source/gdi/jobset.cxx:35
 ImplOldJobSetupData cDeviceName char [32]
-vcl/source/gdi/jobset.cxx:37
+vcl/source/gdi/jobset.cxx:36
 ImplOldJobSetupData cPortName char [32]
 vcl/source/gdi/pdfbuildin_fonts.hxx:34
 vcl::pdf::BuildinFont m_nAscent const int
@@ -842,7 +848,7 @@ vcl/source/gdi/pdfbuildin_fonts.hxx:35
 vcl::pdf::BuildinFont m_nDescent const int
 vcl/source/gdi/pdfbuildin_fonts.hxx:42
 vcl::pdf::BuildinFont m_aWidths const int [256]
-vcl/unx/gtk3/gtk3gtkinst.cxx:3593
+vcl/unx/gtk3/gtk3gtkinst.cxx:3597
 CrippledViewport viewport GtkViewport
 vcl/unx/gtk/a11y/atkhypertext.cxx:29
 HyperLink atk_hyper_link const AtkHyperlink
diff 

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

2019-07-24 Thread Tor Lillqvist (via logerrit)
 extensions/source/ole/unoobjw.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 4bd1b38633d6cb288eb559afc0ac6b961538ae60
Author: Tor Lillqvist 
AuthorDate: Thu Mar 7 14:55:57 2019 +0200
Commit: Tor Lillqvist 
CommitDate: Wed Jul 24 15:19:10 2019 +0200

Add FIXME

Change-Id: Id8a553ef880c79a1d71b6df9586f75a83f53746a
Reviewed-on: https://gerrit.libreoffice.org/76266
Reviewed-by: Tor Lillqvist 
Tested-by: Tor Lillqvist 

diff --git a/extensions/source/ole/unoobjw.cxx 
b/extensions/source/ole/unoobjw.cxx
index 3c8313e4c83e..1a476117e739 100644
--- a/extensions/source/ole/unoobjw.cxx
+++ b/extensions/source/ole/unoobjw.cxx
@@ -951,6 +951,7 @@ HRESULT STDMETHODCALLTYPE 
CXTypeInfo::GetDocumentation(MEMBERID memid,
 }
 else
 {
+// FIXME: Shouldn't we be able to know the names of the members of 
UNO interfaces?
 *pBstrName = 
SysAllocString(o3tl::toW(OUString("UnknownNameOfMember#" + 
OUString::number(memid)).getStr()));
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 125681] Clicking on Insert Symbol hangs session

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=125681

Commit Notification  changed:

   What|Removed |Added

 Whiteboard||target:6.4.0

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 125681] Clicking on Insert Symbol hangs session

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=125681

--- Comment #4 from Commit Notification 
 ---
Tor Lillqvist committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/online/+/f7a70ba9d3411953ca4147ac8cc484dd8745b5b7%5E%21

tdf#125681: Get rid of ChildSession::getDocumentMutex() and associated code

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] online.git: 2 commits - common/Session.cpp common/Session.hpp kit/ChildSession.cpp kit/ChildSession.hpp kit/Kit.cpp test/data test/Makefile.am test/UnitWOPITemplate.cpp test/Whit

2019-07-24 Thread Henry Castro (via logerrit)
 common/Session.cpp|7 +
 common/Session.hpp|2 
 kit/ChildSession.cpp  |  184 +++---
 kit/ChildSession.hpp  |7 -
 kit/Kit.cpp   |   30 +--
 test/Makefile.am  |9 +-
 test/UnitWOPITemplate.cpp |  172 +++
 test/WhiteBoxTests.cpp|8 --
 test/data/test.ott|binary
 wsd/ClientSession.cpp |   16 +++-
 wsd/DocumentBroker.cpp|   10 ++
 wsd/LOOLWSD.cpp   |3 
 wsd/Storage.cpp   |   24 --
 wsd/Storage.hpp   |   14 ++-
 14 files changed, 311 insertions(+), 175 deletions(-)

New commits:
commit 760864870fa8992fda58be03f3f55232b247506f
Author: Henry Castro 
AuthorDate: Tue May 21 23:33:26 2019 -0400
Commit: Jan Holesovsky 
CommitDate: Wed Jul 24 12:10:15 2019 +0200

wsd: introduce "TemplateSource" WOPI property

Change-Id: I9df1d5d0d4be7fe10ee15c40c36195c86ccf857e
Reviewed-on: https://gerrit.libreoffice.org/76190
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Jan Holesovsky 

diff --git a/common/Session.cpp b/common/Session.cpp
index 1c37e5ba2..5c8d11568 100644
--- a/common/Session.cpp
+++ b/common/Session.cpp
@@ -75,7 +75,7 @@ bool Session::sendBinaryFrame(const char *buffer, int length)
 return sendMessage(buffer, length, WSOpCode::Binary) >= length;
 }
 
-void Session::parseDocOptions(const std::vector& tokens, int& 
part, std::string& timestamp)
+void Session::parseDocOptions(const std::vector& tokens, int& 
part, std::string& timestamp, std::string& doctemplate)
 {
 // First token is the "load" command itself.
 size_t offset = 1;
@@ -161,6 +161,11 @@ void Session::parseDocOptions(const 
std::vector& tokens, int& part,
 timestamp = value;
 ++offset;
 }
+else if (name == "template")
+{
+doctemplate = value;
+++offset;
+}
 }
 
 Util::mapAnonymized(_userId, _userIdAnonym);
diff --git a/common/Session.hpp b/common/Session.hpp
index f09af7574..ddee50d55 100644
--- a/common/Session.hpp
+++ b/common/Session.hpp
@@ -124,7 +124,7 @@ protected:
 
 /// Parses the options of the "load" command,
 /// shared between MasterProcessSession::loadDocument() and 
ChildProcessSession::loadDocument().
-void parseDocOptions(const std::vector& tokens, int& part, 
std::string& timestamp);
+void parseDocOptions(const std::vector& tokens, int& part, 
std::string& timestamp, std::string& doctemplate);
 
 void updateLastActivityTime()
 {
diff --git a/kit/ChildSession.cpp b/kit/ChildSession.cpp
index 60cf4c087..a229ae5f0 100644
--- a/kit/ChildSession.cpp
+++ b/kit/ChildSession.cpp
@@ -534,8 +534,8 @@ bool ChildSession::loadDocument(const char * /*buffer*/, 
int /*length*/, const s
 return false;
 }
 
-std::string timestamp;
-parseDocOptions(tokens, part, timestamp);
+std::string timestamp, doctemplate;
+parseDocOptions(tokens, part, timestamp, doctemplate);
 
 std::string renderOpts;
 if (!getDocOptions().empty())
@@ -556,7 +556,7 @@ bool ChildSession::loadDocument(const char * /*buffer*/, 
int /*length*/, const s
 const bool loaded = _docManager.onLoad(getId(), getJailedFilePath(), 
getJailedFilePathAnonym(),
getUserName(), getUserNameAnonym(),
getDocPassword(), renderOpts, 
getHaveDocPassword(),
-   getLang(), getWatermarkText());
+   getLang(), getWatermarkText(), 
doctemplate);
 if (!loaded || _viewId < 0)
 {
 LOG_ERR("Failed to get LoKitDocument instance for [" << 
getJailedFilePathAnonym() << "].");
@@ -566,6 +566,17 @@ bool ChildSession::loadDocument(const char * /*buffer*/, 
int /*length*/, const s
 LOG_INF("Created new view with viewid: [" << _viewId << "] for username: 
[" <<
 getUserNameAnonym() << "] in session: [" << getId() << "].");
 
+if (!doctemplate.empty())
+{
+std::string url = getJailedFilePath();
+bool success = getLOKitDocument()->saveAs(url.c_str(), nullptr, 
nullptr);
+if (!success)
+{
+LOG_ERR("Failed to save template [" << getJailedFilePath() << 
"].");
+return false;
+}
+}
+
 getLOKitDocument()->setView(_viewId);
 
 _docType = LOKitHelper::getDocumentTypeAsString(getLOKitDocument()->get());
diff --git a/kit/ChildSession.hpp b/kit/ChildSession.hpp
index 215d1ad4b..bb6a3d720 100644
--- a/kit/ChildSession.hpp
+++ b/kit/ChildSession.hpp
@@ -46,7 +46,8 @@ public:
 const std::string& renderOpts,
 const bool haveDocPassword,
 const std::string& lang,
-const std::string& watermarkText) = 0;
+const std::string& watermarkText,
+ 

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

2019-07-24 Thread Noel Grandin (via logerrit)
 compilerplugins/clang/unusedenumconstants.readonly.results  |  482 
 compilerplugins/clang/unusedenumconstants.untouched.results |  124 
 compilerplugins/clang/unusedenumconstants.writeonly.results | 6638 +---
 cui/source/inc/cuihyperdlg.hxx  |   10 
 cui/source/inc/iconcdlg.hxx |1 
 sc/source/ui/inc/PivotLayoutTreeListBase.hxx|3 
 6 files changed, 3596 insertions(+), 3662 deletions(-)

New commits:
commit 8d73da6696e42f57abbd8317d227054a4b5475ed
Author: Noel Grandin 
AuthorDate: Wed Jul 24 12:17:29 2019 +0200
Commit: Noel Grandin 
CommitDate: Wed Jul 24 15:10:17 2019 +0200

loplugin:unusedenumconstants

Change-Id: I484b39d1060e6a563dabdde03867989b8ba067f9
Reviewed-on: https://gerrit.libreoffice.org/76237
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/compilerplugins/clang/unusedenumconstants.readonly.results 
b/compilerplugins/clang/unusedenumconstants.readonly.results
index bf5ce11700ea..57147a88192c 100644
--- a/compilerplugins/clang/unusedenumconstants.readonly.results
+++ b/compilerplugins/clang/unusedenumconstants.readonly.results
@@ -5,7 +5,7 @@ bridges/source/cpp_uno/gcc3_linux_x86-64/abi.cxx:78
 bridges/source/cpp_uno/gcc3_linux_x86-64/abi.cxx:79
 enum x86_64_reg_class X86_64_X87UP_CLASS
 chart2/source/inc/CharacterProperties.hxx:121
-enum chart::CharacterProperties::(anonymous at 
/home/noel/libo2/chart2/source/inc/CharacterProperties.hxx:42:5) 
FAST_PROPERTY_ID_END_CHAR_PROP
+enum chart::CharacterProperties::(anonymous at 
/media/noel/disk3/libo6/chart2/source/inc/CharacterProperties.hxx:42:5) 
FAST_PROPERTY_ID_END_CHAR_PROP
 chart2/source/inc/TitleHelper.hxx:48
 enum chart::TitleHelper::eTitleType NORMAL_TITLE_END
 chart2/source/view/inc/ShapeFactory.hxx:50
@@ -38,45 +38,19 @@ chart2/source/view/inc/ShapeFactory.hxx:63
 enum chart::SymbolEnum Symbol_HorizontalBar
 chart2/source/view/inc/ShapeFactory.hxx:64
 enum chart::SymbolEnum Symbol_VerticalBar
-configmgr/source/access.hxx:455
-enum configmgr::Access::(anonymous at 
/home/noel/libo2/configmgr/source/access.hxx:453:5) IS_GROUP_MEMBER
-configmgr/source/access.hxx:455
-enum configmgr::Access::(anonymous at 
/home/noel/libo2/configmgr/source/access.hxx:453:5) IS_SET_MEMBER
-configmgr/source/parsemanager.hxx:47
-enum configmgr::ParseManager::(anonymous at 
/home/noel/libo2/configmgr/source/parsemanager.hxx:47:5) NAMESPACE_XS
-configmgr/source/parsemanager.hxx:47
-enum configmgr::ParseManager::(anonymous at 
/home/noel/libo2/configmgr/source/parsemanager.hxx:47:5) NAMESPACE_OOR
-configmgr/source/parsemanager.hxx:47
-enum configmgr::ParseManager::(anonymous at 
/home/noel/libo2/configmgr/source/parsemanager.hxx:47:5) NAMESPACE_XSI
+configmgr/source/access.hxx:445
+enum configmgr::Access::(anonymous at 
/media/noel/disk3/libo6/configmgr/source/access.hxx:443:5) IS_GROUP_MEMBER
+configmgr/source/access.hxx:445
+enum configmgr::Access::(anonymous at 
/media/noel/disk3/libo6/configmgr/source/access.hxx:443:5) IS_SET_MEMBER
+configmgr/source/parsemanager.hxx:44
+enum configmgr::ParseManager::(anonymous at 
/media/noel/disk3/libo6/configmgr/source/parsemanager.hxx:44:5) NAMESPACE_XSI
+configmgr/source/parsemanager.hxx:44
+enum configmgr::ParseManager::(anonymous at 
/media/noel/disk3/libo6/configmgr/source/parsemanager.hxx:44:5) NAMESPACE_OOR
+configmgr/source/parsemanager.hxx:44
+enum configmgr::ParseManager::(anonymous at 
/media/noel/disk3/libo6/configmgr/source/parsemanager.hxx:44:5) NAMESPACE_XS
 connectivity/source/drivers/evoab2/NConnection.hxx:41
 connectivity::evoab::SDBCAddress::sdbc_address_type Unknown
-cppcanvas/source/mtfrenderer/emfppen.cxx:53
-enum cppcanvas::internal::EmfPlusPenData PenDataTransform
-cppcanvas/source/mtfrenderer/emfppen.cxx:54
-enum cppcanvas::internal::EmfPlusPenData PenDataStartCap
-cppcanvas/source/mtfrenderer/emfppen.cxx:55
-enum cppcanvas::internal::EmfPlusPenData PenDataEndCap
-cppcanvas/source/mtfrenderer/emfppen.cxx:56
-enum cppcanvas::internal::EmfPlusPenData PenDataJoin
-cppcanvas/source/mtfrenderer/emfppen.cxx:57
-enum cppcanvas::internal::EmfPlusPenData PenDataMiterLimit
-cppcanvas/source/mtfrenderer/emfppen.cxx:58
-enum cppcanvas::internal::EmfPlusPenData PenDataLineStyle
-cppcanvas/source/mtfrenderer/emfppen.cxx:59
-enum cppcanvas::internal::EmfPlusPenData PenDataDashedLineCap
-cppcanvas/source/mtfrenderer/emfppen.cxx:60
-enum cppcanvas::internal::EmfPlusPenData PenDataDashedLineOffset
-cppcanvas/source/mtfrenderer/emfppen.cxx:61
-enum cppcanvas::internal::EmfPlusPenData PenDataDashedLine
-cppcanvas/source/mtfrenderer/emfppen.cxx:62
-enum cppcanvas::internal::EmfPlusPenData PenDataNonCenter
-cppcanvas/source/mtfrenderer/emfppen.cxx:63
-enum cppcanvas::internal::EmfPlusPenData PenDataCompoundLine
-cppcanvas/source/mtfrenderer/emfppen.cxx:64
-enum 

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

2019-07-24 Thread Noel Grandin (via logerrit)
 include/vcl/svlbitm.hxx |3 +--
 include/vcl/treelistbox.hxx |3 +--
 vcl/source/treelist/svlbitm.cxx |9 -
 3 files changed, 6 insertions(+), 9 deletions(-)

New commits:
commit be07f2b19ddce09eb4f18b0a7cc4f1583b3f86e0
Author: Noel Grandin 
AuthorDate: Wed Jul 24 11:55:34 2019 +0200
Commit: Noel Grandin 
CommitDate: Wed Jul 24 15:05:22 2019 +0200

remove unused SvLBoxButtonKind::StaticImage

and consequently unused SvBmp::STATICIMAGE

Change-Id: Ie48deb0c757e6eb9e22e7b29cde72b0145c16943
Reviewed-on: https://gerrit.libreoffice.org/76235
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/include/vcl/svlbitm.hxx b/include/vcl/svlbitm.hxx
index 8d267e0754af..c28bed22f460 100644
--- a/include/vcl/svlbitm.hxx
+++ b/include/vcl/svlbitm.hxx
@@ -39,8 +39,7 @@ enum class SvBmp
 TRISTATE = 2,
 HIUNCHECKED  = 3,
 HICHECKED= 4,
-HITRISTATE   = 5,
-STATICIMAGE  = 6
+HITRISTATE   = 5
 };
 
 enum class SvItemStateFlags
diff --git a/include/vcl/treelistbox.hxx b/include/vcl/treelistbox.hxx
index c6a13397ba80..ef515aebf5a8 100644
--- a/include/vcl/treelistbox.hxx
+++ b/include/vcl/treelistbox.hxx
@@ -57,8 +57,7 @@ namespace utl {
 enum class SvLBoxButtonKind
 {
 EnabledCheckbox,
-DisabledCheckbox,
-StaticImage
+DisabledCheckbox
 };
 
 enum class SvButtonState { Unchecked, Checked, Tristate };
diff --git a/vcl/source/treelist/svlbitm.cxx b/vcl/source/treelist/svlbitm.cxx
index 56b4f016d9e9..58433644ee93 100644
--- a/vcl/source/treelist/svlbitm.cxx
+++ b/vcl/source/treelist/svlbitm.cxx
@@ -40,7 +40,7 @@ void SvLBoxButtonData::InitData( bool _bRadioBtn, const 
Control* pCtrl )
 {
 nWidth = nHeight = 0;
 
-aBmps.resize(int(SvBmp::STATICIMAGE)+1);
+aBmps.resize(int(SvBmp::HITRISTATE)+1);
 
 bDataOk = false;
 pImpl->bDefaultImages = true;
@@ -307,14 +307,13 @@ void SvLBoxButton::Paint(
 const Point& rPos, SvTreeListBox& rDev, vcl::RenderContext& rRenderContext,
 const SvViewDataEntry* /*pView*/, const SvTreeListEntry& /*rEntry*/)
 {
-SvBmp nIndex = eKind == SvLBoxButtonKind::StaticImage ? SvBmp::STATICIMAGE 
: SvLBoxButtonData::GetIndex(nItemFlags);
+SvBmp nIndex = SvLBoxButtonData::GetIndex(nItemFlags);
 DrawImageFlags nStyle = eKind != SvLBoxButtonKind::DisabledCheckbox && 
rDev.IsEnabled() ? DrawImageFlags::NONE : DrawImageFlags::Disable;
 
 //Native drawing
 bool bNativeOK = false;
 ControlType eCtrlType = (pData->IsRadio())? ControlType::Radiobutton : 
ControlType::Checkbox;
-if ( nIndex != SvBmp::STATICIMAGE && 
rRenderContext.IsNativeControlSupported( eCtrlType, ControlPart::Entire) )
-
+if ( rRenderContext.IsNativeControlSupported( eCtrlType, 
ControlPart::Entire) )
 {
 Size aSize(pData->Width(), pData->Height());
 ImplAdjustBoxSize(aSize, eCtrlType, rRenderContext);
@@ -386,7 +385,7 @@ void SvLBoxButton::InitViewData(SvTreeListBox* 
pView,SvTreeListEntry* pEntry, Sv
 Size aSize( pData->Width(), pData->Height() );
 
 ControlType eCtrlType = (pData->IsRadio())? ControlType::Radiobutton : 
ControlType::Checkbox;
-if ( eKind != SvLBoxButtonKind::StaticImage && pView )
+if ( pView )
 ImplAdjustBoxSize(aSize, eCtrlType, *pView);
 pViewData->mnWidth = aSize.Width();
 pViewData->mnHeight = aSize.Height();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

[Libreoffice-bugs] [Bug 126523] New: Crash in: com::sun::star::uno::operator<<= >(com::sun::star::uno::Any &, com::sun::star::uno::R

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126523

Bug ID: 126523
   Summary: Crash in:
com::sun::star::uno::operator<<=
>(com::sun::star::uno::Any
&,com::sun::star::uno::Reference const &)
   Product: LibreOffice
   Version: 6.1.5.2 release
  Hardware: x86-64 (AMD64)
OS: Windows (All)
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: framework
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: j...@hp-homepages.de

This bug was filed from the crash reporting server and is
br-a1ec90c7-7e6f-4ca9-a628-2660bbbe2535.
=
The bug occurs in a database form inside a LibreOffice Writer document and
leads to crashes without any error message.
The crashes seem to occur randomly/nondeterministic when clicking different
parts of the form. Often it continues crashing on the same data set after
reopening the document until it suddenly works without any changes.
Sometimes the form can be used for hours without any problems before it starts
crashing.

Related crash reports:
http://crashreport.libreoffice.org/stats/crash_details/b20278ac-5c25-498f-9468-cbf4257f275b
http://crashreport.libreoffice.org/stats/crash_details/15c39ed8-d785-44a9-87ad-b11a888f8b2a

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 118739] [META] Refactor ListBox/ComboBox to use RenderContext regressions

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=118739

Miklos Vajna  changed:

   What|Removed |Added

 Depends on|122830  |


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=122830
[Bug 122830] UI: Hint is a empty gray box for line-setting in the cell
appearance dialog of the sidebar with OpenGL enabled (very flickery without)
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 122830] UI: Hint is a empty gray box for line-setting in the cell appearance dialog of the sidebar with OpenGL enabled (very flickery without)

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=122830

Miklos Vajna  changed:

   What|Removed |Added

 Blocks|118739  |

--- Comment #4 from Miklos Vajna  ---
Adjusting the tracker bug list, this is not a regression from
dca01def7885ad69cf66edd75cf8207a5adb64f9.


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=118739
[Bug 118739] [META] Refactor ListBox/ComboBox to use RenderContext regressions
-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-commits] core.git: sc/inc sc/qa sc/source

2019-07-24 Thread Noel Grandin (via logerrit)
 sc/inc/types.hxx   |   11 +--
 sc/qa/unit/ucalc.cxx   |8 
 sc/qa/unit/ucalc_sharedformula.cxx |6 +++---
 sc/source/core/data/bcaslot.cxx|   13 -
 sc/source/core/data/table3.cxx |6 +++---
 5 files changed, 19 insertions(+), 25 deletions(-)

New commits:
commit 8c1d25e0dbbcc9b6d4ed58f4bc9e87e10e5e
Author: Noel Grandin 
AuthorDate: Wed Jul 24 11:51:57 2019 +0200
Commit: Noel Grandin 
CommitDate: Wed Jul 24 14:43:54 2019 +0200

convert AreaOverlapType to scoped enum

and remove unused AreaPartialOverlap

Change-Id: I21e94d8d15c59485c1287e99dcc6558e0570a8eb
Reviewed-on: https://gerrit.libreoffice.org/76234
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/sc/inc/types.hxx b/sc/inc/types.hxx
index c4e62ae63452..5c68c606d0fe 100644
--- a/sc/inc/types.hxx
+++ b/sc/inc/types.hxx
@@ -107,13 +107,12 @@ struct MultiDataCellState
 MultiDataCellState( StateType eState );
 };
 
-enum AreaOverlapType
+enum class AreaOverlapType
 {
-AreaInside,
-AreaPartialOverlap,
-AreaInsideOrOverlap,
-OneRowInsideArea,
-OneColumnInsideArea
+Inside,
+InsideOrOverlap,
+OneRowInside,
+OneColumnInside
 };
 
 enum class ListenerGroupType
diff --git a/sc/qa/unit/ucalc.cxx b/sc/qa/unit/ucalc.cxx
index 4db44ce1b6c1..c4a6f84cd3e6 100644
--- a/sc/qa/unit/ucalc.cxx
+++ b/sc/qa/unit/ucalc.cxx
@@ -4225,7 +4225,7 @@ void Test::testCopyPasteRepeatOneFormula()
 ScRange aWholeSheet(0,0,0,MAXCOL,MAXROW,0);
 ScBroadcastAreaSlotMachine* pBASM = m_pDoc->GetBASM();
 CPPUNIT_ASSERT(pBASM);
-std::vector aListeners = 
pBASM->GetAllListeners(aWholeSheet, sc::AreaInside);
+std::vector aListeners = 
pBASM->GetAllListeners(aWholeSheet, sc::AreaOverlapType::Inside);
 CPPUNIT_ASSERT_EQUAL(static_cast(1), aListeners.size());
 const sc::AreaListener* pListener = aListeners.data();
 CPPUNIT_ASSERT_EQUAL(ScRange(0,0,0,1,0,0), pListener->maArea);
@@ -4258,7 +4258,7 @@ void Test::testCopyPasteRepeatOneFormula()
 #if !defined(USE_FORMULA_GROUP_LISTENER) || USE_FORMULA_GROUP_LISTENER
 // At this point, there should only be one area listener and it should be
 // a group listener listening on A1:B10.
-aListeners = pBASM->GetAllListeners(aWholeSheet, sc::AreaInside);
+aListeners = pBASM->GetAllListeners(aWholeSheet, 
sc::AreaOverlapType::Inside);
 CPPUNIT_ASSERT_EQUAL(static_cast(1), aListeners.size());
 pListener = aListeners.data();
 CPPUNIT_ASSERT_EQUAL(ScRange(0,0,0,1,9,0), pListener->maArea);
@@ -4276,7 +4276,7 @@ void Test::testCopyPasteRepeatOneFormula()
 // This check makes only sense if group listeners are activated.
 #if !defined(USE_FORMULA_GROUP_LISTENER) || USE_FORMULA_GROUP_LISTENER
 // Make there we only have one group area listener listening on A2:B11.
-aListeners = pBASM->GetAllListeners(aWholeSheet, sc::AreaInside);
+aListeners = pBASM->GetAllListeners(aWholeSheet, 
sc::AreaOverlapType::Inside);
 CPPUNIT_ASSERT_EQUAL(static_cast(1), aListeners.size());
 pListener = aListeners.data();
 CPPUNIT_ASSERT_EQUAL(ScRange(0,1,0,1,10,0), pListener->maArea);
@@ -4303,7 +4303,7 @@ void Test::testCopyPasteRepeatOneFormula()
 // This check makes only sense if group listeners are activated.
 #if !defined(USE_FORMULA_GROUP_LISTENER) || USE_FORMULA_GROUP_LISTENER
 // Check the group area listener again to make sure it's listening on 
A1:B10 once again.
-aListeners = pBASM->GetAllListeners(aWholeSheet, sc::AreaInside);
+aListeners = pBASM->GetAllListeners(aWholeSheet, 
sc::AreaOverlapType::Inside);
 CPPUNIT_ASSERT_EQUAL(static_cast(1), aListeners.size());
 pListener = aListeners.data();
 CPPUNIT_ASSERT_EQUAL(ScRange(0,0,0,1,9,0), pListener->maArea);
diff --git a/sc/qa/unit/ucalc_sharedformula.cxx 
b/sc/qa/unit/ucalc_sharedformula.cxx
index ea22e3651d4d..d07d7edaf391 100644
--- a/sc/qa/unit/ucalc_sharedformula.cxx
+++ b/sc/qa/unit/ucalc_sharedformula.cxx
@@ -648,7 +648,7 @@ void Test::testSharedFormulasRefUpdateRangeDeleteRow()
 // Check the area listener status.
 ScBroadcastAreaSlotMachine* pBASM = m_pDoc->GetBASM();
 CPPUNIT_ASSERT(pBASM);
-std::vector aListeners = 
pBASM->GetAllListeners(aWholeArea, sc::AreaInside);
+std::vector aListeners = 
pBASM->GetAllListeners(aWholeArea, sc::AreaOverlapType::Inside);
 std::sort(aListeners.begin(), aListeners.end(), SortByArea());
 
 // This check makes only sense if group listeners are activated.
@@ -688,7 +688,7 @@ void Test::testSharedFormulasRefUpdateRangeDeleteRow()
 // This check makes only sense if group listeners are activated.
 #if !defined(USE_FORMULA_GROUP_LISTENER) || USE_FORMULA_GROUP_LISTENER
 // We should only have one listener group-listening on A1:B4.
-aListeners = pBASM->GetAllListeners(aWholeArea, sc::AreaInside);
+aListeners = pBASM->GetAllListeners(aWholeArea, 
sc::AreaOverlapType::Inside);
 

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

2019-07-24 Thread Noel Grandin (via logerrit)
 include/svx/DescriptionGenerator.hxx  |   13 ---
 svx/source/accessibility/DescriptionGenerator.cxx |   76 --
 2 files changed, 1 insertion(+), 88 deletions(-)

New commits:
commit ba031cc41321c7817c5b23e0b5378e532f7e1dea
Author: Noel Grandin 
AuthorDate: Wed Jul 24 11:57:52 2019 +0200
Commit: Noel Grandin 
CommitDate: Wed Jul 24 14:38:59 2019 +0200

remove unused accessibility::DescriptionGenerator::PropertyType::FillStyle

Change-Id: I20005ad23088d41262ff988eac2132918db56ab8
Reviewed-on: https://gerrit.libreoffice.org/76236
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/include/svx/DescriptionGenerator.hxx 
b/include/svx/DescriptionGenerator.hxx
index 8d3010ea5b94..7769a25dbe6a 100644
--- a/include/svx/DescriptionGenerator.hxx
+++ b/include/svx/DescriptionGenerator.hxx
@@ -44,8 +44,7 @@ public:
 enum class PropertyType {
 Color,
 Integer,
-String,
-FillStyle
+String
 };
 
 /** Creates a new description generator with an empty description
@@ -165,16 +164,6 @@ private:
 */
 SVX_DLLPRIVATE void AddString (const OUString& sPropertyName,
 const OUString& sLocalizedName, sal_uInt16 nWhichId);
-
-/** Add a property value formatted as fill style to the description
-string.  If the fill style is HATCH,
-GRADIENT, or BITMAP, then the of the
-hatch, gradient, or bitmap is appended as well.
-@param sPropertyName
-Name of the property.  Usually this will be "FillStyle".
-*/
-SVX_DLLPRIVATE void AddFillStyle (const OUString& sPropertyName,
-const OUString& sLocalizedName);
 };
 
 
diff --git a/svx/source/accessibility/DescriptionGenerator.cxx 
b/svx/source/accessibility/DescriptionGenerator.cxx
index cefe8e104fcc..ff5bb21f20eb 100644
--- a/svx/source/accessibility/DescriptionGenerator.cxx
+++ b/svx/source/accessibility/DescriptionGenerator.cxx
@@ -169,9 +169,6 @@ void DescriptionGenerator::AddProperty (const OUString& 
sPropertyName,
 case PropertyType::String:
 AddString (sPropertyName, sLocalizedName, nWhichId);
 break;
-case PropertyType::FillStyle:
-AddFillStyle (sPropertyName, sLocalizedName);
-break;
 }
 }
 }
@@ -266,79 +263,6 @@ void DescriptionGenerator::AddString (const OUString& 
sPropertyName,
 }
 
 
-void DescriptionGenerator::AddFillStyle (const OUString& sPropertyName,
-const OUString& sLocalizedName)
-{
-msDescription.append(sLocalizedName);
-msDescription.append('=');
-
-try
-{
-if (mxSet.is())
-{
-uno::Any aValue = mxSet->getPropertyValue (sPropertyName);
-drawing::FillStyle aFillStyle;
-aValue >>= aFillStyle;
-
-// Get the fill style name from the resource.
-OUString sFillStyleName;
-{
-SolarMutexGuard aGuard;
-switch (aFillStyle)
-{
-case drawing::FillStyle_NONE:
-sFillStyleName = 
SvxResId(RID_SVXSTR_A11Y_FILLSTYLE_NONE);
-break;
-case drawing::FillStyle_SOLID:
-sFillStyleName = 
SvxResId(RID_SVXSTR_A11Y_FILLSTYLE_SOLID);
-break;
-case drawing::FillStyle_GRADIENT:
-sFillStyleName = 
SvxResId(RID_SVXSTR_A11Y_FILLSTYLE_GRADIENT);
-break;
-case drawing::FillStyle_HATCH:
-sFillStyleName = 
SvxResId(RID_SVXSTR_A11Y_FILLSTYLE_HATCH);
-break;
-case drawing::FillStyle_BITMAP:
-sFillStyleName = 
SvxResId(RID_SVXSTR_A11Y_FILLSTYLE_BITMAP);
-break;
-default:
-break;
-}
-}
-msDescription.append (sFillStyleName);
-
-// Append the appropriate properties.
-switch (aFillStyle)
-{
-case drawing::FillStyle_NONE:
-break;
-case drawing::FillStyle_SOLID:
-AddProperty ("FillColor", PropertyType::Color, 
SIP_XA_FILLCOLOR);
-break;
-case drawing::FillStyle_GRADIENT:
-AddProperty ("FillGradientName", PropertyType::String, 
SIP_XA_FILLGRADIENT,
-XATTR_FILLGRADIENT);
-break;
-case drawing::FillStyle_HATCH:
-AddProperty ("FillColor", PropertyType::Color, 
SIP_XA_FILLCOLOR);
-AddProperty ("FillHatchName", PropertyType::String, 
SIP_XA_FILLHATCH,
-XATTR_FILLHATCH);
-break;
-case drawing::FillStyle_BITMAP:

[Libreoffice-bugs] [Bug 126493] EDITING: Objects don't snap to grid properly

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126493

--- Comment #2 from Harald Berger  ---
See also

https://ask.libreoffice.org/en/question/201719/objects-dont-snap-exactly-to-gridlines-in-writer/

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

[Libreoffice-bugs] [Bug 126493] EDITING: Objects don't snap to grid properly

2019-07-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=126493

Harald Berger  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever confirmed|0   |1

--- Comment #1 from Harald Berger  ---
I confirm the bug below

Windows 10 Home; Version 1903; 64-Bit

Version: 6.2.5.2 (x64)
Build ID: 1ec314fa52f458adc18c4f025c545a4e8b22c159
CPU threads: 8; OS: Windows 10.0; UI render: default; VCL: win; 
Locale: en-GB (en_GB); UI-Language: en-GB
Calc: threaded

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

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

2019-07-24 Thread Caolán McNamara (via logerrit)
 cui/source/dialogs/SpellDialog.cxx |1 +
 sw/source/filter/ww8/ww8scan.cxx   |3 ++-
 2 files changed, 3 insertions(+), 1 deletion(-)

New commits:
commit 00af4b1e5157d1a7f71da168dc7b48707e9113cd
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 09:02:41 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 14:25:29 2019 +0200

cid#1451635 silence Dereference after null check

Change-Id: Ie5978be908647fe1215e1517a2bf638c7936dae2
Reviewed-on: https://gerrit.libreoffice.org/76225
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/cui/source/dialogs/SpellDialog.cxx 
b/cui/source/dialogs/SpellDialog.cxx
index 9c6e90b74c25..dc2a9b4ef83e 100644
--- a/cui/source/dialogs/SpellDialog.cxx
+++ b/cui/source/dialogs/SpellDialog.cxx
@@ -1483,6 +1483,7 @@ bool SentenceEditWindow_Impl::KeyInput(const KeyEvent& 
rKeyEvt)
 if (pErrorAttrib->nStart != m_nErrorStart || 
pErrorAttrib->nEnd != m_nErrorEnd)
 {
 std::unique_ptr 
xNewError(pErrorAttrib->pAttr->Clone());
+assert(pErrorAttr);
 m_xEditEngine->RemoveAttribs(ESelection(0, 
pErrorAttr->nStart, 0, pErrorAttr->nEnd), false, EE_CHAR_GRABBAG);
 SetAttrib(*xNewError, m_nErrorStart, m_nErrorEnd);
 }
commit b055dda3c82086f5c633403bd952209d6ed4288f
Author: Caolán McNamara 
AuthorDate: Wed Jul 24 10:05:59 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 14:25:08 2019 +0200

cid#1448539 try silence Double free

Change-Id: I05f30f33acc06325eba59965a84fa43749d6f394
Reviewed-on: https://gerrit.libreoffice.org/76230
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/sw/source/filter/ww8/ww8scan.cxx b/sw/source/filter/ww8/ww8scan.cxx
index 19e0d81e7850..196c0eef3936 100644
--- a/sw/source/filter/ww8/ww8scan.cxx
+++ b/sw/source/filter/ww8/ww8scan.cxx
@@ -2859,13 +2859,14 @@ WW8PLCFx_Fc_FKP::WW8Fkp::Entry&
 mnIStd = rEntry.mnIStd;
 mbMustDelete = rEntry.mbMustDelete;
 
-if (mbMustDelete)
+if (rEntry.mbMustDelete)
 {
 mpData = new sal_uInt8[mnLen];
 memcpy(mpData, rEntry.mpData, mnLen);
 }
 else
 mpData = rEntry.mpData;
+
 return *this;
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

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

2019-07-24 Thread Caolán McNamara (via logerrit)
 vcl/source/app/salvtables.cxx |2 +-
 vcl/source/window/aboutdialog.cxx |2 +-
 vcl/source/window/builder.cxx |2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 2d63403fa3bf7fe6019b58ce4af5c49968e03a5b
Author: Caolán McNamara 
AuthorDate: Tue Jul 23 21:31:26 2019 +0100
Commit: Caolán McNamara 
CommitDate: Wed Jul 24 14:24:55 2019 +0200

move aboutdialog header inside vcl

Change-Id: I1906992759afec0176b723917662fa0644a03c1c
Reviewed-on: https://gerrit.libreoffice.org/76221
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/include/vcl/aboutdialog.hxx b/vcl/inc/aboutdialog.hxx
similarity index 100%
rename from include/vcl/aboutdialog.hxx
rename to vcl/inc/aboutdialog.hxx
diff --git a/vcl/source/app/salvtables.cxx b/vcl/source/app/salvtables.cxx
index 5aab75be773d..6f8d11cb7886 100644
--- a/vcl/source/app/salvtables.cxx
+++ b/vcl/source/app/salvtables.cxx
@@ -36,7 +36,6 @@
 #include 
 #include 
 #include 
-#include 
 #include 
 #include 
 #include 
@@ -65,6 +64,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 
 SalFrame::SalFrame()
diff --git a/vcl/source/window/aboutdialog.cxx 
b/vcl/source/window/aboutdialog.cxx
index b8a8aa595982..ba09f423ab05 100644
--- a/vcl/source/window/aboutdialog.cxx
+++ b/vcl/source/window/aboutdialog.cxx
@@ -12,11 +12,11 @@
 #include 
 #include 
 #include 
-#include 
 #include 
 #include 
 #include 
 #include 
+#include 
 
 #include 
 #include 
diff --git a/vcl/source/window/builder.cxx b/vcl/source/window/builder.cxx
index 604d9950dece..0a4c66e0cbf1 100644
--- a/vcl/source/window/builder.cxx
+++ b/vcl/source/window/builder.cxx
@@ -20,7 +20,6 @@
 #include 
 #include 
 #include 
-#include 
 #include 
 #include 
 #include 
@@ -61,6 +60,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits

  1   2   >