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

2021-12-24 Thread Arnaud VERSINI (via logerrit)
 idl/source/objects/types.cxx |   10 --
 1 file changed, 4 insertions(+), 6 deletions(-)

New commits:
commit 7a6fb939053f0c4f5c3e653fe4b21bc2d904d7dc
Author: Arnaud VERSINI 
AuthorDate: Fri Dec 24 20:23:28 2021 +0100
Commit: Mike Kaganski 
CommitDate: Sat Dec 25 08:32:26 2021 +0100

idl : remove useless define and copy the code directly !

Change-Id: I92d53beb110a44624b72f7ec76ab8d1089f9caf0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127438
Reviewed-by: Noel Grandin 
Reviewed-by: Mike Kaganski 
Tested-by: Mike Kaganski 

diff --git a/idl/source/objects/types.cxx b/idl/source/objects/types.cxx
index 46262e7b98d9..a3dde07d96a7 100644
--- a/idl/source/objects/types.cxx
+++ b/idl/source/objects/types.cxx
@@ -119,17 +119,15 @@ void SvMetaAttribute::Insert(SvSlotElementList&)
 {
 }
 
-#define CTOR\
-: nType( MetaTypeType::Base )   \
-, bIsItem( false )  \
-
 SvMetaType::SvMetaType()
-CTOR
+: nType( MetaTypeType::Base )
+, bIsItem( false )
 {
 }
 
 SvMetaType::SvMetaType( const OString& rName )
-CTOR
+: nType( MetaTypeType::Base )
+, bIsItem( false )
 {
 SetName( rName );
 }


[Libreoffice-bugs] [Bug 146410] Colibre: The Clone Formatting icons does not look like a paint brush

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146410

Rizal Muttaqin  changed:

   What|Removed |Added

 Blocks||117497


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=117497
[Bug 117497] [META] Colibre icons
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 117497] [META] Colibre icons

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=117497

Rizal Muttaqin  changed:

   What|Removed |Added

 Depends on||146410


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=146410
[Bug 146410] Colibre: The Clone Formatting icons does not look like a paint
brush
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146410] Colibre: The Clone Formatting icons does not look like a paint brush

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146410

--- Comment #1 from Rizal Muttaqin  ---
The patch is here
https://gerrit.libreoffice.org/c/core/+/127448

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-commits] core.git: icon-themes/colibre icon-themes/colibre_svg

2021-12-24 Thread Rizal Muttaqin (via logerrit)
 icon-themes/colibre/cmd/32/changepicture.png  |binary
 icon-themes/colibre/cmd/32/compressgraphic.png|binary
 icon-themes/colibre/cmd/32/crop.png   |binary
 icon-themes/colibre/cmd/32/gallery.png|binary
 icon-themes/colibre/cmd/32/graphicdialog.png  |binary
 icon-themes/colibre/cmd/32/graphicfiltersepia.png |binary
 icon-themes/colibre/cmd/32/imagebutton.png|binary
 icon-themes/colibre/cmd/32/imagecontrol.png   |binary
 icon-themes/colibre/cmd/32/imagemapdialog.png |binary
 icon-themes/colibre/cmd/32/insertgraphic.png  |binary
 icon-themes/colibre/cmd/32/photoalbumdialog.png   |binary
 icon-themes/colibre/cmd/lc_changepicture.png  |binary
 icon-themes/colibre/cmd/lc_compressgraphic.png|binary
 icon-themes/colibre/cmd/lc_crop.png   |binary
 icon-themes/colibre/cmd/lc_gallery.png|binary
 icon-themes/colibre/cmd/lc_graphicdialog.png  |binary
 icon-themes/colibre/cmd/lc_graphicfiltersepia.png |binary
 icon-themes/colibre/cmd/lc_imagebutton.png|binary
 icon-themes/colibre/cmd/lc_imagecontrol.png   |binary
 icon-themes/colibre/cmd/lc_imagemapdialog.png |binary
 icon-themes/colibre/cmd/lc_insertgraphic.png  |binary
 icon-themes/colibre/cmd/lc_photoalbumdialog.png   |binary
 icon-themes/colibre/cmd/sc_changepicture.png  |binary
 icon-themes/colibre/cmd/sc_compressgraphic.png|binary
 icon-themes/colibre/cmd/sc_gallery.png|binary
 icon-themes/colibre/cmd/sc_graphicdialog.png  |binary
 icon-themes/colibre/cmd/sc_graphicfiltersepia.png |binary
 icon-themes/colibre/cmd/sc_imagebutton.png|binary
 icon-themes/colibre/cmd/sc_imagecontrol.png   |binary
 icon-themes/colibre/cmd/sc_imagemapdialog.png |binary
 icon-themes/colibre/cmd/sc_insertgraphic.png  |binary
 icon-themes/colibre/cmd/sc_photoalbumdialog.png   |binary
 icon-themes/colibre/res/grafikde.png  |binary
 icon-themes/colibre/res/grafikei.png  |binary
 icon-themes/colibre/res/lx03125.png   |binary
 icon-themes/colibre/res/sx03125.png   |binary
 icon-themes/colibre/sd/res/image.png  |binary
 icon-themes/colibre/sd/res/placeholder_image_large.png|binary
 icon-themes/colibre/sd/res/placeholder_image_large_hover.png  |binary
 icon-themes/colibre/sd/res/placeholder_image_small.png|binary
 icon-themes/colibre/sd/res/placeholder_image_small_hover.png  |binary
 icon-themes/colibre/svx/res/cd02.png  |binary
 icon-themes/colibre/svx/res/galdefl.png   |binary
 icon-themes/colibre/svx/res/galdefs.png   |binary
 icon-themes/colibre/svx/res/galnorl.png   |binary
 icon-themes/colibre/svx/res/galnors.png   |binary
 icon-themes/colibre/svx/res/galrdol.png   |binary
 icon-themes/colibre/svx/res/galrdos.png   |binary
 icon-themes/colibre/svx/res/symphony/ColorModeNormal_16x16.png|binary
 icon-themes/colibre/svx/res/symphony/ColorModeWaterMark_16x16.png |binary
 icon-themes/colibre_svg/cmd/32/changepicture.svg  |2 +-
 icon-themes/colibre_svg/cmd/32/compressgraphic.svg|2 +-
 icon-themes/colibre_svg/cmd/32/crop.svg   |2 +-
 icon-themes/colibre_svg/cmd/32/gallery.svg|2 +-
 icon-themes/colibre_svg/cmd/32/graphicdialog.svg  |2 +-
 icon-themes/colibre_svg/cmd/32/graphicfiltersepia.svg |2 +-
 icon-themes/colibre_svg/cmd/32/imagebutton.svg|2 +-
 icon-themes/colibre_svg/cmd/32/imagecontrol.svg   |2 +-
 icon-themes/colibre_svg/cmd/32/imagemapdialog.svg |2 +-
 icon-themes/colibre_svg/cmd/32/insertgraphic.svg  |2 +-
 icon-themes/colibre_svg/cmd/32/photoalbumdialog.svg   |2 +-
 icon-themes/colibre_svg/cmd/lc_changepicture.svg  |2 +-
 icon-themes/colibre_svg/cmd/lc_compressgraphic.svg|

[Libreoffice-commits] core.git: Branch 'libreoffice-7-3' - icon-themes/colibre icon-themes/colibre_svg

2021-12-24 Thread Rizal Muttaqin (via logerrit)
 icon-themes/colibre/cmd/32/changepicture.png  |binary
 icon-themes/colibre/cmd/32/compressgraphic.png|binary
 icon-themes/colibre/cmd/32/crop.png   |binary
 icon-themes/colibre/cmd/32/gallery.png|binary
 icon-themes/colibre/cmd/32/graphicdialog.png  |binary
 icon-themes/colibre/cmd/32/graphicfiltersepia.png |binary
 icon-themes/colibre/cmd/32/imagebutton.png|binary
 icon-themes/colibre/cmd/32/imagecontrol.png   |binary
 icon-themes/colibre/cmd/32/imagemapdialog.png |binary
 icon-themes/colibre/cmd/32/insertgraphic.png  |binary
 icon-themes/colibre/cmd/32/photoalbumdialog.png   |binary
 icon-themes/colibre/cmd/lc_changepicture.png  |binary
 icon-themes/colibre/cmd/lc_compressgraphic.png|binary
 icon-themes/colibre/cmd/lc_crop.png   |binary
 icon-themes/colibre/cmd/lc_gallery.png|binary
 icon-themes/colibre/cmd/lc_graphicdialog.png  |binary
 icon-themes/colibre/cmd/lc_graphicfiltersepia.png |binary
 icon-themes/colibre/cmd/lc_imagebutton.png|binary
 icon-themes/colibre/cmd/lc_imagecontrol.png   |binary
 icon-themes/colibre/cmd/lc_imagemapdialog.png |binary
 icon-themes/colibre/cmd/lc_insertgraphic.png  |binary
 icon-themes/colibre/cmd/lc_photoalbumdialog.png   |binary
 icon-themes/colibre/cmd/sc_changepicture.png  |binary
 icon-themes/colibre/cmd/sc_compressgraphic.png|binary
 icon-themes/colibre/cmd/sc_gallery.png|binary
 icon-themes/colibre/cmd/sc_graphicdialog.png  |binary
 icon-themes/colibre/cmd/sc_graphicfiltersepia.png |binary
 icon-themes/colibre/cmd/sc_imagebutton.png|binary
 icon-themes/colibre/cmd/sc_imagecontrol.png   |binary
 icon-themes/colibre/cmd/sc_imagemapdialog.png |binary
 icon-themes/colibre/cmd/sc_insertgraphic.png  |binary
 icon-themes/colibre/cmd/sc_photoalbumdialog.png   |binary
 icon-themes/colibre/res/grafikde.png  |binary
 icon-themes/colibre/res/grafikei.png  |binary
 icon-themes/colibre/res/lx03125.png   |binary
 icon-themes/colibre/res/sx03125.png   |binary
 icon-themes/colibre/sd/res/image.png  |binary
 icon-themes/colibre/sd/res/placeholder_image_large.png|binary
 icon-themes/colibre/sd/res/placeholder_image_large_hover.png  |binary
 icon-themes/colibre/sd/res/placeholder_image_small.png|binary
 icon-themes/colibre/sd/res/placeholder_image_small_hover.png  |binary
 icon-themes/colibre/svx/res/cd02.png  |binary
 icon-themes/colibre/svx/res/galdefl.png   |binary
 icon-themes/colibre/svx/res/galdefs.png   |binary
 icon-themes/colibre/svx/res/galnorl.png   |binary
 icon-themes/colibre/svx/res/galnors.png   |binary
 icon-themes/colibre/svx/res/galrdol.png   |binary
 icon-themes/colibre/svx/res/galrdos.png   |binary
 icon-themes/colibre/svx/res/symphony/ColorModeNormal_16x16.png|binary
 icon-themes/colibre/svx/res/symphony/ColorModeWaterMark_16x16.png |binary
 icon-themes/colibre_svg/cmd/32/changepicture.svg  |2 +-
 icon-themes/colibre_svg/cmd/32/compressgraphic.svg|2 +-
 icon-themes/colibre_svg/cmd/32/crop.svg   |2 +-
 icon-themes/colibre_svg/cmd/32/gallery.svg|2 +-
 icon-themes/colibre_svg/cmd/32/graphicdialog.svg  |2 +-
 icon-themes/colibre_svg/cmd/32/graphicfiltersepia.svg |2 +-
 icon-themes/colibre_svg/cmd/32/imagebutton.svg|2 +-
 icon-themes/colibre_svg/cmd/32/imagecontrol.svg   |2 +-
 icon-themes/colibre_svg/cmd/32/imagemapdialog.svg |2 +-
 icon-themes/colibre_svg/cmd/32/insertgraphic.svg  |2 +-
 icon-themes/colibre_svg/cmd/32/photoalbumdialog.svg   |2 +-
 icon-themes/colibre_svg/cmd/lc_changepicture.svg  |2 +-
 icon-themes/colibre_svg/cmd/lc_compressgraphic.svg|

[Libreoffice-bugs] [Bug 146410] New: Colibre: The Clone Formatting icons does not look like a paint brush

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146410

Bug ID: 146410
   Summary: Colibre: The Clone Formatting icons does not look like
a paint brush
   Product: LibreOffice
   Version: unspecified
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: UI
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: riz...@libreoffice.org

Created attachment 177133
  --> https://bugs.documentfoundation.org/attachment.cgi?id=177133=edit
Clone Formatting looks a flimsy brush

While it's not quite necessary, the icon for "Clone Formatting" in current
Colibre theme does not look like a paint brush at all.

The suggestion is to update it.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-commits] core.git: 2 commits - unotools/source unoxml/inc unoxml/source

2021-12-24 Thread Noel Grandin (via logerrit)
 unotools/source/ucbhelper/XTempFile.hxx |4 +-
 unotools/source/ucbhelper/xtempfile.cxx |   61 
 unoxml/inc/event.hxx|3 +
 unoxml/source/events/event.cxx  |   18 -
 unoxml/source/events/mouseevent.cxx |   21 +--
 unoxml/source/events/mutationevent.cxx  |   15 ---
 unoxml/source/events/uievent.cxx|7 +--
 7 files changed, 66 insertions(+), 63 deletions(-)

New commits:
commit 870a3b5bc34d5d4082b46940822d871106cac2d9
Author: Noel Grandin 
AuthorDate: Thu Dec 23 21:38:41 2021 +0200
Commit: Noel Grandin 
CommitDate: Sat Dec 25 07:15:33 2021 +0100

osl::Mutex->std::mutex in CEvent

Change-Id: I644ebf86803448e824818f571d6612741408d02c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127402
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/unoxml/inc/event.hxx b/unoxml/inc/event.hxx
index c33d8f9d3b3c..216bf2775ed4 100644
--- a/unoxml/inc/event.hxx
+++ b/unoxml/inc/event.hxx
@@ -27,6 +27,7 @@
 #include 
 
 #include 
+#include 
 
 namespace DOM::events
 {
@@ -35,7 +36,7 @@ class CEvent : public cppu::WeakImplHelper< 
css::xml::dom::events::XEvent >
 friend class CEventDispatcher;
 
 protected:
-::osl::Mutex m_Mutex;
+std::mutex m_Mutex;
 bool m_canceled;
 OUString m_eventType;
 css::uno::Reference< css::xml::dom::events::XEventTarget > m_target;
diff --git a/unoxml/source/events/event.cxx b/unoxml/source/events/event.cxx
index bc12bb5f83c1..e3b092ff3831 100644
--- a/unoxml/source/events/event.cxx
+++ b/unoxml/source/events/event.cxx
@@ -39,52 +39,52 @@ namespace DOM::events
 
 OUString SAL_CALL CEvent::getType()
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 return m_eventType;
 }
 
 Reference< XEventTarget > SAL_CALL
 CEvent::getTarget()
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 return m_target;
 }
 
 Reference< XEventTarget > SAL_CALL
 CEvent::getCurrentTarget()
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 return m_currentTarget;
 }
 
 PhaseType SAL_CALL CEvent::getEventPhase()
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 return m_phase;
 }
 
 sal_Bool SAL_CALL CEvent::getBubbles()
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 return m_bubbles;
 }
 
 sal_Bool SAL_CALL CEvent::getCancelable()
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 return m_cancelable;
 }
 
 css::util::Time SAL_CALL
 CEvent::getTimeStamp()
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 return m_time;
 }
 
 void SAL_CALL CEvent::stopPropagation()
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 if (m_cancelable) { m_canceled = true; }
 }
 
@@ -96,7 +96,7 @@ namespace DOM::events
 CEvent::initEvent(OUString const& eventTypeArg, sal_Bool canBubbleArg,
 sal_Bool cancelableArg)
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 
 m_eventType = eventTypeArg;
 m_bubbles = canBubbleArg;
diff --git a/unoxml/source/events/mouseevent.cxx 
b/unoxml/source/events/mouseevent.cxx
index ee22b8905c70..4ae8a1b55741 100644
--- a/unoxml/source/events/mouseevent.cxx
+++ b/unoxml/source/events/mouseevent.cxx
@@ -40,47 +40,47 @@ namespace DOM::events
 
 sal_Int32 SAL_CALL CMouseEvent::getScreenX()
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 return m_screenX;
 }
 sal_Int32 SAL_CALL CMouseEvent::getScreenY()
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 return m_screenY;
 }
 sal_Int32 SAL_CALL CMouseEvent::getClientX()
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 return m_clientX;
 }
 sal_Int32 SAL_CALL CMouseEvent::getClientY()
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 return m_clientY;
 }
 sal_Bool SAL_CALL CMouseEvent::getCtrlKey()
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 return m_ctrlKey;
 }
 sal_Bool SAL_CALL CMouseEvent::getShiftKey()
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 return m_shiftKey;
 }
 sal_Bool SAL_CALL CMouseEvent::getAltKey()
 {
-::osl::MutexGuard const g(m_Mutex);
+std::unique_lock const g(m_Mutex);
 return m_altKey;
 }
 

[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-6.4' - filter/qa filter/source sfx2/source

2021-12-24 Thread Miklos Vajna (via logerrit)
 filter/qa/unit/textfilterdetect.cxx |   20 
 filter/source/textfilterdetect/filterdetect.cxx |6 +-
 sfx2/source/doc/objstor.cxx |   10 +-
 3 files changed, 34 insertions(+), 2 deletions(-)

New commits:
commit da88d0e570493cc6b0f56744bfbc00d829b26ee7
Author: Miklos Vajna 
AuthorDate: Tue Dec 21 14:31:34 2021 +0100
Commit: Mike Kaganski 
CommitDate: Sat Dec 25 06:48:11 2021 +0100

filter: try to detect 0-byte DOC files based on extension

Commit ae1f51b4888a3aa14837ac6e4083f33b2176ca45 (tdf#123476 filter: try
to detect 0-byte files based on extension, 2020-10-28), already
implemented this UNO-based import filters, do the same for built-in
filters as well.

Another problem in filter/ was to pick the WW6 filter for .doc --
require export+preferred support in the filter to get WW8 instead.

An additional filter that may kick in is MS Word 2003 XML: this is
avoided by requiring "preferred".

(cherry picked from commit e519d7465f38367205b37bc80f0244029e352306)

Conflicts:
filter/qa/unit/textfilterdetect.cxx

Change-Id: I46e280beb5341213b0fe7a09a549b52c0c1ea3f6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127296
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Mike Kaganski 
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127432

diff --git a/filter/qa/unit/data/empty.doc b/filter/qa/unit/data/empty.doc
new file mode 100644
index ..e69de29bb2d1
diff --git a/filter/qa/unit/textfilterdetect.cxx 
b/filter/qa/unit/textfilterdetect.cxx
index 74d967490ffe..4083610b0736 100644
--- a/filter/qa/unit/textfilterdetect.cxx
+++ b/filter/qa/unit/textfilterdetect.cxx
@@ -116,6 +116,26 @@ CPPUNIT_TEST_FIXTURE(TextFilterDetectTest, testEmptyFile)
 // Without the accompanying fix in place, this test would have failed, as 
it was opened in
 // Writer instead.
 
CPPUNIT_ASSERT(xServiceInfo->supportsService("com.sun.star.presentation.PresentationDocument"));
+getComponent()->dispose();
+
+// ... and DOC
+aURL = m_directories.getURLFromSrc(DATA_DIRECTORY) + "empty.doc";
+// Without the accompanying fix in place, this test would have failed, the 
import filter aborted
+// loading.
+getComponent() = loadFromDesktop(aURL);
+xServiceInfo.set(getComponent(), uno::UNO_QUERY);
+CPPUNIT_ASSERT(xServiceInfo.is());
+
CPPUNIT_ASSERT(xServiceInfo->supportsService("com.sun.star.text.TextDocument"));
+uno::Reference xModel(getComponent(), uno::UNO_QUERY);
+uno::Sequence aArgs = xModel->getArgs();
+comphelper::SequenceAsHashMap aMap(aArgs);
+OUString aFilterName;
+aMap["FilterName"] >>= aFilterName;
+// Without the accompanying fix in place, this test would have failed with:
+// - Expected: MS Word 97
+// - Actual  : MS WinWord 6.0
+// i.e. opening worked, but saving back failed instead of producing a WW8 
binary file.
+CPPUNIT_ASSERT_EQUAL(OUString("MS Word 97"), aFilterName);
 }
 }
 
diff --git a/filter/source/textfilterdetect/filterdetect.cxx 
b/filter/source/textfilterdetect/filterdetect.cxx
index 1f5080ad0951..4a6314c9224a 100644
--- a/filter/source/textfilterdetect/filterdetect.cxx
+++ b/filter/source/textfilterdetect/filterdetect.cxx
@@ -149,7 +149,11 @@ bool HandleEmptyFileUrlByExtension(MediaDescriptor& 
rMediaDesc, const OUString&
 return false;
 }
 
-std::shared_ptr 
pFilter(SfxFilterMatcher().GetFilter4Extension(rExt));
+// Requiring the export+preferred flags helps to find the relevant filter, 
e.g. .doc -> WW8 (and
+// not WW6 or Mac_Word).
+SfxFilterFlags nMust
+= SfxFilterFlags::IMPORT | SfxFilterFlags::EXPORT | 
SfxFilterFlags::PREFERED;
+std::shared_ptr 
pFilter(SfxFilterMatcher().GetFilter4Extension(rExt, nMust));
 if (!pFilter)
 {
 return false;
diff --git a/sfx2/source/doc/objstor.cxx b/sfx2/source/doc/objstor.cxx
index 49d90e54c599..abdb402e507e 100644
--- a/sfx2/source/doc/objstor.cxx
+++ b/sfx2/source/doc/objstor.cxx
@@ -774,7 +774,15 @@ bool SfxObjectShell::DoLoad( SfxMedium *pMed )
 }
 else
 {
-bOk = ConvertFrom(*pMedium);
+if (tools::isEmptyFileUrl(pMedium->GetName()))
+{
+// The import filter would fail with empty input.
+bOk = true;
+}
+else
+{
+bOk = ConvertFrom(*pMedium);
+}
 InitOwnModel_Impl();
 }
 }


[Libreoffice-bugs] [Bug 146409] New: customize keyboard simpler

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146409

Bug ID: 146409
   Summary: customize keyboard simpler
   Product: LibreOffice
   Version: 7.4.0.0 alpha0+ Master
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: enhancement
  Priority: medium
 Component: LibreOffice
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: solot...@gmail.com

as a user it would be cool that customize keyboard would be simpler in the
following aspects: search for a function, and display what keys do.

two suggestions which would make the access quicker:
first: have 3 instead of 2 columnfs for shortcut keys, instead of key +
function have key + writer, key + libreoffice. the radio butteon "libreoffcie"
"writer" could then go away. instead have 2 modify buttons, modify writer,
modify libreoffice.

second: make the search for all commands work. currently typing "inden" does
not retrieve style Text body indent, one needs to click on styles to find it.
in version 7.2.4.1 this is, linux.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146172] Window "Table of Contents, Index or Bibliography." out screen

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146172

QA Administrators  changed:

   What|Removed |Added

 Whiteboard|| QA:needsComment

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146171] Tracked changes with no date (0000-00-00) in .doc(x) documents are no longer tracked when converted to ODF format.

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146171

QA Administrators  changed:

   What|Removed |Added

 Whiteboard|| QA:needsComment

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146169] LibreOffice Writer prints one copy when I requested 20

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146169

QA Administrators  changed:

   What|Removed |Added

 Whiteboard|| QA:needsComment

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146168] Font attributes of certain list numbers have changed after 7.2, and I can't change them back

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146168

QA Administrators  changed:

   What|Removed |Added

 Whiteboard|| QA:needsComment

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146162] JUMBO SHEETS: cannot hide all columns

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146162

QA Administrators  changed:

   What|Removed |Added

 Whiteboard|| QA:needsComment

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146161] Consider renaming button "open" in templates dialog

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146161

QA Administrators  changed:

   What|Removed |Added

 Whiteboard|| QA:needsComment

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146157] Please make the inverted question marks after trailing equality signs optional

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146157

QA Administrators  changed:

   What|Removed |Added

 Whiteboard|| QA:needsComment

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146154] Wrong amount of selected text is highlighted in mixed LTR/RTL text

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146154

QA Administrators  changed:

   What|Removed |Added

 Whiteboard|| QA:needsComment

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146391] hlookup: when the search criteria contains a number, the search result will be NV

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146391

QA Administrators  changed:

   What|Removed |Added

 Status|NEEDINFO|UNCONFIRMED
 Ever confirmed|1   |0

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146391] hlookup: when the search criteria contains a number, the search result will be NV

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146391

--- Comment #3 from QA Administrators  ---
[Automated Action] NeedInfo-To-Unconfirmed

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 90380] VIEWING: Navigator causes constant cpu usage when it does not have focus in documents with lots of text frames and images

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=90380

--- Comment #9 from QA Administrators  ---
Dear Gordo,

To make sure we're focusing on the bugs that affect our users today,
LibreOffice QA is asking bug reporters and confirmers to retest open, confirmed
bugs which have not been touched for over a year.

There have been thousands of bug fixes and commits since anyone checked on this
bug report. During that time, it's possible that the bug has been fixed, or the
details of the problem have changed. We'd really appreciate your help in
getting confirmation that the bug is still present.

If you have time, please do the following:

Test to see if the bug is still present with the latest version of LibreOffice
from https://www.libreoffice.org/download/

If the bug is present, please leave a comment that includes the information
from Help - About LibreOffice.

If the bug is NOT present, please set the bug's Status field to
RESOLVED-WORKSFORME and leave a comment that includes the information from Help
- About LibreOffice.

Please DO NOT

Update the version field
Reply via email (please reply directly on the bug tracker)
Set the bug's Status field to RESOLVED - FIXED (this status has a particular
meaning that is not 
appropriate in this case)


If you want to do more to help you can test to see if your issue is a
REGRESSION. To do so:
1. Download and install oldest version of LibreOffice (usually 3.3 unless your
bug pertains to a feature added after 3.3) from
https://downloadarchive.documentfoundation.org/libreoffice/old/

2. Test your bug
3. Leave a comment with your results.
4a. If the bug was present with 3.3 - set version to 'inherited from OOo';
4b. If the bug was not present in 3.3 - add 'regression' to keyword


Feel free to come ask questions or to say hello in our QA chat:
https://web.libera.chat/?settings=#libreoffice-qa

Thank you for helping us make LibreOffice even better for everyone!

Warm Regards,
QA Team

MassPing-UntouchedBug

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 142493] Footnote Anchor and Footnote characters don't save the styles when I save as doc or docx end reopen the saved document

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=142493

--- Comment #4 from QA Administrators  ---
Dear Antonio,

This bug has been in NEEDINFO status with no change for at least
6 months. Please provide the requested information as soon as
possible and mark the bug as UNCONFIRMED. Due to regular bug
tracker maintenance, if the bug is still in NEEDINFO status with
no change in 30 days the QA team will close the bug as INSUFFICIENTDATA
due to lack of needed information.

For more information about our NEEDINFO policy please read the
wiki located here:
https://wiki.documentfoundation.org/QA/Bugzilla/Fields/Status/NEEDINFO

If you have already provided the requested information, please
mark the bug as UNCONFIRMED so that the QA team knows that the
bug is ready to be confirmed.

Thank you for helping us make LibreOffice even better for everyone!

Warm Regards,
QA Team

MassPing-NeedInfo-Ping

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 131063] Navigate document content when selection is made by single click in the Navigator

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=131063

--- Comment #19 from Jim Raykowski  ---
(In reply to golemus from comment #18)
> ok. Would it be difficult to implement it so that in future you could change
> context menu of navigator through Tools -- Customize -- Context menus? Or
> perhaps even the navigator toolbar (although I don't use the toolbar but I
> guess somebody does as it is there).
Best to open an enhancement request for adding a way to customizing the
navigator context menu directly in Writer.

You can already customize the navigator context menu by manually editing the
'navigatorcontextmenu.ui' file.

usr/lib/libreoffice/share/config/soffice.cfg/modules/swriter/ui/navigatorcontextmenu.ui

ALWAYS GOOD TO MAKE A BACKUP COPY BEFORE MODIFYING :-)

To make the promote/demote menu items not appear in the context menu either
remove the following lines or set  to False as I have
done.


  
False
False
Promote Chapter
True

  


  
False
False
Demote Chapter
True

  


  
False
False
Promote Level
True

  


  
False
False
Demote Level
True

  


-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 145700] FiLEOPEN DOC lighting direction has inverted z-direction for extruded custom shapes

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=145700

--- Comment #8 from Regina Henschel  ---
Created attachment 177132
  --> https://bugs.documentfoundation.org/attachment.cgi?id=177132=edit
Test file in doc format, so that you can open it directly in LO

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 145700] FiLEOPEN DOC lighting direction has inverted z-direction for extruded custom shapes

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=145700

--- Comment #7 from Regina Henschel  ---
Created attachment 177131
  --> https://bugs.documentfoundation.org/attachment.cgi?id=177131=edit
Test file with non-default settings as mentioned in previous attachment

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 145700] FiLEOPEN DOC lighting direction has inverted z-direction for extruded custom shapes

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=145700

--- Comment #6 from Regina Henschel  ---
Created attachment 177130
  --> https://bugs.documentfoundation.org/attachment.cgi?id=177130=edit
Proof, that z-axis points towards observer

I'm sure now, that in MS Office the z-axis points towards observer. For
screenshots and calculations see the attached files.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 106613] UI: Sidebar navigator in Impress/Draw not updated immediately when page/slide is renamed

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=106613

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

https://git.libreoffice.org/core/commit/9bfc42015acd6ae3475ab7927ccc006507cc38a2

tdf#146003 tdf#106613 Draw: fix slide rename not updated in Navigator

It will be available in 7.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] [Bug 106613] UI: Sidebar navigator in Impress/Draw not updated immediately when page/slide is renamed

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=106613

Commit Notification  changed:

   What|Removed |Added

 Whiteboard|target:7.1.0 target:7.0.4   |target:7.1.0 target:7.0.4
   ||target:7.4.0

-- 
You are receiving this mail because:
You are the assignee for the bug.

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

2021-12-24 Thread Jim Raykowski (via logerrit)
 sd/source/ui/slidesorter/controller/SlsSlotManager.cxx |9 +++--
 sd/source/ui/view/drviewsb.cxx |8 +---
 sd/source/ui/view/drviewsd.cxx |3 ---
 3 files changed, 8 insertions(+), 12 deletions(-)

New commits:
commit 9bfc42015acd6ae3475ab7927ccc006507cc38a2
Author: Jim Raykowski 
AuthorDate: Thu Dec 23 20:46:59 2021 -0900
Commit: Jim Raykowski 
CommitDate: Fri Dec 24 23:21:24 2021 +0100

tdf#146003 tdf#106613 Draw: fix slide rename not updated in Navigator

Change-Id: Iee2d34e3192e3f7875ab10f3e79e9dc1a2fb20b8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127397
Tested-by: Jenkins
Reviewed-by: Jim Raykowski 

diff --git a/sd/source/ui/slidesorter/controller/SlsSlotManager.cxx 
b/sd/source/ui/slidesorter/controller/SlsSlotManager.cxx
index 89d10e310725..8ade9b296462 100644
--- a/sd/source/ui/slidesorter/controller/SlsSlotManager.cxx
+++ b/sd/source/ui/slidesorter/controller/SlsSlotManager.cxx
@@ -1029,12 +1029,9 @@ bool SlotManager::RenameSlideFromDrawViewShell( 
sal_uInt16 nPageId, const OUStri
 pDocument->SetChanged();
 
 // inform navigator about change
-SfxBoolItem aItem( SID_NAVIGATOR_INIT, true );
-if (mrSlideSorter.GetViewShell() != nullptr)
-mrSlideSorter.GetViewShell()->GetDispatcher()->ExecuteList(
-SID_NAVIGATOR_INIT,
-SfxCallMode::ASYNCHRON | SfxCallMode::RECORD,
-{  });
+if (mrSlideSorter.GetViewShell() && 
mrSlideSorter.GetViewShell()->GetViewFrame())
+
mrSlideSorter.GetViewShell()->GetViewFrame()->GetBindings().Invalidate(
+SID_NAVIGATOR_STATE, true);
 }
 
 return bSuccess;
diff --git a/sd/source/ui/view/drviewsb.cxx b/sd/source/ui/view/drviewsb.cxx
index 5bc741c91269..6f6bba8555d5 100644
--- a/sd/source/ui/view/drviewsb.cxx
+++ b/sd/source/ui/view/drviewsb.cxx
@@ -97,9 +97,11 @@ bool DrawViewShell::RenameSlide( sal_uInt16 nPageId, const 
OUString & rName  )
 GetDoc()->SetChanged();
 
 // inform navigator about change
-SfxBoolItem aItem( SID_NAVIGATOR_INIT, true );
-GetViewFrame()->GetDispatcher()->ExecuteList(SID_NAVIGATOR_INIT,
-SfxCallMode::ASYNCHRON | SfxCallMode::RECORD, {  });
+if (GetViewFrame())
+{
+SfxBindings& rBindings = GetViewFrame()->GetBindings();
+rBindings.Invalidate(SID_NAVIGATOR_STATE, true);
+}
 
 // Tell the slide sorter about the name change (necessary for
 // accessibility.)
diff --git a/sd/source/ui/view/drviewsd.cxx b/sd/source/ui/view/drviewsd.cxx
index 763ae13eac91..0926e02990e5 100644
--- a/sd/source/ui/view/drviewsd.cxx
+++ b/sd/source/ui/view/drviewsd.cxx
@@ -61,9 +61,6 @@ void DrawViewShell::ExecNavigatorWin( SfxRequest& rReq )
 if( pNavWin )
 pNavWin->InitTreeLB( GetDoc() );
 }
-SfxBindings& rBindings = GetViewFrame()->GetBindings();
-rBindings.Invalidate(SID_NAVIGATOR_STATE, true);
-rBindings.Invalidate(SID_NAVIGATOR_PAGENAME, true);
 }
 break;
 


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

2021-12-24 Thread Mike Kaganski (via logerrit)
 tools/source/fsys/urlobj.cxx |   91 +--
 1 file changed, 37 insertions(+), 54 deletions(-)

New commits:
commit 132d652ad1af4ec067fe08e388e8ddba963ddc2b
Author: Mike Kaganski 
AuthorDate: Fri Dec 24 21:37:17 2021 +0100
Commit: Mike Kaganski 
CommitDate: Fri Dec 24 23:09:14 2021 +0100

Simplify INetURLObject::parseHostOrNetBiosName

Change-Id: I18818e89b2e6f0ef203caa51ba554c281753d3f5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127383
Tested-by: Jenkins
Reviewed-by: Mike Kaganski 

diff --git a/tools/source/fsys/urlobj.cxx b/tools/source/fsys/urlobj.cxx
index 7b86fe0f5261..67cb1f1cd024 100644
--- a/tools/source/fsys/urlobj.cxx
+++ b/tools/source/fsys/urlobj.cxx
@@ -2831,67 +2831,50 @@ bool INetURLObject::parseHostOrNetBiosName(
 EncodeMechanism eMechanism, rtl_TextEncoding eCharset, bool bNetBiosName,
 OUStringBuffer* pCanonic)
 {
+if (pBegin >= pEnd)
+return true;
 sal_Int32 nOriginalCanonicLength = pCanonic ? pCanonic->getLength() : 0;
-if (pBegin < pEnd)
+if (sal_Unicode const* p = pBegin; parseHost(p, pEnd, pCanonic) && p == 
pEnd)
+return true;
+if (pCanonic)
+pCanonic->setLength(nOriginalCanonicLength); // discard parseHost 
results
+if (!bNetBiosName)
+return false;
+while (pBegin < pEnd)
 {
-sal_Unicode const * p = pBegin;
-if (!parseHost(p, pEnd, pCanonic) || p != pEnd)
+EscapeType eEscapeType;
+sal_uInt32 nUTF32 = getUTF32(pBegin, pEnd, eMechanism, eCharset, 
eEscapeType);
+switch (nUTF32)
 {
-if (bNetBiosName)
-{
-OUStringBuffer buf;
-while (pBegin < pEnd)
-{
-EscapeType eEscapeType;
-sal_uInt32 nUTF32 = getUTF32(pBegin, pEnd,
- eMechanism, eCharset,
- eEscapeType);
-if (!INetMIME::isVisible(nUTF32))
-{
-if (pCanonic)
-pCanonic->setLength(nOriginalCanonicLength);
-return false;
-}
-if (!rtl::isAsciiAlphanumeric(nUTF32))
-switch (nUTF32)
-{
-case '"':
-case '*':
-case '+':
-case ',':
-case '/':
-case ':':
-case ';':
-case '<':
-case '=':
-case '>':
-case '?':
-case '[':
-case '\\':
-case ']':
-case '`':
-case '|':
-return false;
-}
-if (pCanonic != nullptr) {
-appendUCS4(
-buf, nUTF32, eEscapeType, PART_URIC,
-eCharset, true);
-}
-}
-if (pCanonic)
-{
-pCanonic->setLength(nOriginalCanonicLength);
-pCanonic->append(buf);
-}
-}
-else
-{
+case '"':
+case '*':
+case '+':
+case ',':
+case '/':
+case ':':
+case ';':
+case '<':
+case '=':
+case '>':
+case '?':
+case '[':
+case '\\':
+case ']':
+case '`':
+case '|':
 if (pCanonic)
 pCanonic->setLength(nOriginalCanonicLength);
 return false;
-}
+default:
+if (!INetMIME::isVisible(nUTF32))
+{
+if (pCanonic)
+pCanonic->setLength(nOriginalCanonicLength);
+return false;
+}
 }
+if (pCanonic)
+appendUCS4(*pCanonic, nUTF32, eEscapeType, PART_URIC, eCharset, 
true);
 }
 return true;
 }


[Libreoffice-bugs] [Bug 146408] Неверное получение сохраненной записи при её открытии

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146408

aleksandrakorobov...@gmail.com changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever confirmed|0   |1

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146408] New: Неверное получение сохраненной записи при её открытии

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146408

Bug ID: 146408
   Summary: Неверное получение сохраненной записи при её открытии
   Product: cppunit
   Version: unspecified
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: General
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: aleksandrakorobov...@gmail.com
CC: markus.mohrh...@googlemail.com

Created attachment 177129
  --> https://bugs.documentfoundation.org/attachment.cgi?id=177129=edit
Неверное получение сохраненной записи "Выполнение лабораторной работы"

После открытия созданной записи, вместо требуемой информации выводится
сообщение «System error».

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146406] Неверное получение сохраненной записи

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146406

aleksandrakorobov...@gmail.com changed:

   What|Removed |Added

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

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146406] Неверное получение сохраненной записи

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146406

aleksandrakorobov...@gmail.com changed:

   What|Removed |Added

 Status|NEEDINFO|NEW

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-commits] core.git: include/tools sal/rtl sal/textenc sax/source stoc/source tools/source vcl/source vcl/win

2021-12-24 Thread Mike Kaganski (via logerrit)
 include/tools/inetmime.hxx|   14 +--
 sal/rtl/uri.cxx   |   31 
---
 sal/rtl/ustrbuf.cxx   |   12 ---
 sal/textenc/convertbig5hkscs.cxx  |   14 +--
 sal/textenc/converteuctw.cxx  |   10 +-
 sal/textenc/convertgb18030.cxx|   13 +--
 sal/textenc/convertisciidevangari.cxx |8 +-
 sal/textenc/convertiso2022cn.cxx  |   10 +-
 sal/textenc/convertiso2022jp.cxx  |8 +-
 sal/textenc/convertiso2022kr.cxx  |8 +-
 sal/textenc/convertsinglebytetobmpunicode.cxx |8 +-
 sal/textenc/handleundefinedunicodetotextchar.cxx  |4 -
 sal/textenc/tcvtutf8.cxx  |   13 +--
 sal/textenc/unichars.hxx  |   24 --
 sax/source/expatwrap/saxwriter.cxx|   40 
+-
 stoc/source/uriproc/UriSchemeParser_vndDOTsunDOTstarDOTscript.cxx |9 --
 tools/source/inet/inetmime.cxx|   28 
---
 vcl/source/gdi/scrptrun.cxx   |   10 +-
 vcl/win/window/salframe.cxx   |5 -
 19 files changed, 87 insertions(+), 182 deletions(-)

New commits:
commit 8b333c76945960fc62a01829666ba234f59a6d94
Author: Mike Kaganski 
AuthorDate: Fri Dec 24 15:25:52 2021 +0300
Commit: Mike Kaganski 
CommitDate: Fri Dec 24 21:42:08 2021 +0100

Use rtl functions instead of own surrogate checking/combining

Change-Id: I3eb05d8f5b0761bc3b672d4c855eb469f8cc1a29
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127375
Tested-by: Jenkins
Reviewed-by: Mike Kaganski 

diff --git a/include/tools/inetmime.hxx b/include/tools/inetmime.hxx
index ce0469fa131a..a3c7de49690c 100644
--- a/include/tools/inetmime.hxx
+++ b/include/tools/inetmime.hxx
@@ -227,16 +227,12 @@ inline int INetMIME::getHexWeight(sal_uInt32 nChar)
 inline sal_uInt32 INetMIME::getUTF32Character(const sal_Unicode *& rBegin,
   const sal_Unicode * pEnd)
 {
-DBG_ASSERT(rBegin && rBegin < pEnd,
+assert(rBegin && rBegin < pEnd &&
"INetMIME::getUTF32Character(): Bad sequence");
-if (rBegin + 1 < pEnd && rBegin[0] >= 0xD800 && rBegin[0] <= 0xDBFF
-&& rBegin[1] >= 0xDC00 && rBegin[1] <= 0xDFFF)
-{
-sal_uInt32 nUTF32 = sal_uInt32(*rBegin++ & 0x3FF) << 10;
-return (nUTF32 | (*rBegin++ & 0x3FF)) + 0x1;
-}
-else
-return *rBegin++;
+sal_uInt32 nUTF32 = *rBegin++;
+if (rBegin < pEnd && rtl::isHighSurrogate(nUTF32) && 
rtl::isLowSurrogate(rBegin[0]))
+nUTF32 = rtl::combineSurrogates(nUTF32, *rBegin++);
+return nUTF32;
 }
 
 
diff --git a/sal/rtl/uri.cxx b/sal/rtl/uri.cxx
index 441a6c69f1e8..0c5479563405 100644
--- a/sal/rtl/uri.cxx
+++ b/sal/rtl/uri.cxx
@@ -219,21 +219,7 @@ namespace {
 
 void writeUcs4(rtl_uString ** pBuffer, sal_Int32 * pCapacity, sal_uInt32 
nUtf32)
 {
-assert(rtl::isUnicodeCodePoint(nUtf32));
-if (nUtf32 <= 0x)
-{
-writeUnicode(pBuffer, pCapacity, static_cast< sal_Unicode >(nUtf32));
-}
-else
-{
-nUtf32 -= 0x1;
-writeUnicode(
-pBuffer, pCapacity,
-static_cast< sal_Unicode >(nUtf32 >> 10 | 0xD800));
-writeUnicode(
-pBuffer, pCapacity,
-static_cast< sal_Unicode >((nUtf32 & 0x3FF) | 0xDC00));
-}
+rtl_uStringbuffer_insertUtf32(pBuffer, pCapacity, (*pBuffer)->length, 
nUtf32);
 }
 
 void writeEscapeOctet(rtl_uString ** pBuffer, sal_Int32 * pCapacity,
@@ -284,20 +270,7 @@ bool writeEscapeChar(rtl_uString ** pBuffer, sal_Int32 * 
pCapacity,
 rtl_UnicodeToTextConverter aConverter
 = rtl_createUnicodeToTextConverter(eCharset);
 sal_Unicode aSrc[2];
-sal_Size nSrcSize;
-if (nUtf32 <= 0x)
-{
-aSrc[0] = static_cast< sal_Unicode >(nUtf32);
-nSrcSize = 1;
-}
-else
-{
-aSrc[0] = static_cast< sal_Unicode >(
-((nUtf32 - 0x1) >> 10) | 0xD800);
-aSrc[1] = static_cast< sal_Unicode >(
-((nUtf32 - 0x1) & 0x3FF) | 0xDC00);
-nSrcSize = 2;
-}
+sal_Size nSrcSize = rtl::splitSurrogates(nUtf32, aSrc);
 
 char aDst[32]; // FIXME  random value
 sal_uInt32 nInfo;
diff --git a/sal/rtl/ustrbuf.cxx b/sal/rtl/ustrbuf.cxx
index 53e178e5881a..7c5b187edbcb 100644
--- a/sal/rtl/ustrbuf.cxx
+++ b/sal/rtl/ustrbuf.cxx
@@ -174,17 +174,7 @@ void rtl_uStringbuffer_insertUtf32(
 SAL_THROW_EXTERN_C()
 {
 sal_Unicode 

[Libreoffice-bugs] [Bug 146406] Неверное получение сохраненной записи

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146406

Michael Warner  changed:

   What|Removed |Added

 Status|NEW |NEEDINFO

--- Comment #1 from Michael Warner  ---
Machine translation
-
Failed Retrieval of Stored Lab Progress Record

After opening the created record, instead of the required information, the
message “System error” is displayed.
—-

How is this related to LibreOffice?

-- 
You are receiving this mail because:
You are the assignee for the bug.

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

2021-12-24 Thread Noel Grandin (via logerrit)
 unotools/source/config/moduleoptions.cxx |   25 +
 1 file changed, 13 insertions(+), 12 deletions(-)

New commits:
commit fd4acfaca9fc012313f03f46e927add6feb6a553
Author: Noel Grandin 
AuthorDate: Thu Dec 23 19:37:42 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Dec 24 20:19:22 2021 +0100

osl::Mutex->std::mutex in SvtModuleOptions

Change-Id: I6882b30c823d06aa04c04c8a17bfe33cc45d7307
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127415
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/unotools/source/config/moduleoptions.cxx 
b/unotools/source/config/moduleoptions.cxx
index 324736359b6e..967191c06e1f 100644
--- a/unotools/source/config/moduleoptions.cxx
+++ b/unotools/source/config/moduleoptions.cxx
@@ -762,9 +762,9 @@ namespace {
 //global
 std::weak_ptr g_pModuleOptions;
 
-osl::Mutex& impl_GetOwnStaticMutex()
+std::mutex& impl_GetOwnStaticMutex()
 {
-static osl::Mutex s_Mutex;
+static std::mutex s_Mutex;
 return s_Mutex;
 }
 }
@@ -785,13 +785,14 @@ SvtModuleOptions::SvtModuleOptions()
 return;
 
 // take the mutex, so we don't accidentally create more than one
-::osl::MutexGuard aGuard( impl_GetOwnStaticMutex() );
+std::unique_lock aGuard( impl_GetOwnStaticMutex() );
 
 m_pImpl = g_pModuleOptions.lock();
 if( !m_pImpl )
 {
 m_pImpl = std::make_shared();
 g_pModuleOptions = m_pImpl;
+aGuard.unlock(); // because holdConfigItem will call this constructor
 ItemHolder1::holdConfigItem(EItem::ModuleOptions);
 }
 }
@@ -823,32 +824,32 @@ const OUString & SvtModuleOptions::GetFactoryName( 
EFactory eFactory ) const
 
 OUString SvtModuleOptions::GetFactoryStandardTemplate( EFactory eFactory ) 
const
 {
-::osl::MutexGuard aGuard( impl_GetOwnStaticMutex() );
+std::unique_lock aGuard( impl_GetOwnStaticMutex() );
 return m_pImpl->GetFactoryStandardTemplate( eFactory );
 }
 
 OUString SvtModuleOptions::GetFactoryEmptyDocumentURL( EFactory eFactory ) 
const
 {
-::osl::MutexGuard aGuard( impl_GetOwnStaticMutex() );
+std::unique_lock aGuard( impl_GetOwnStaticMutex() );
 return SvtModuleOptions_Impl::GetFactoryEmptyDocumentURL( eFactory );
 }
 
 OUString SvtModuleOptions::GetFactoryDefaultFilter( EFactory eFactory ) const
 {
-::osl::MutexGuard aGuard( impl_GetOwnStaticMutex() );
+std::unique_lock aGuard( impl_GetOwnStaticMutex() );
 return m_pImpl->GetFactoryDefaultFilter( eFactory );
 }
 
 bool SvtModuleOptions::IsDefaultFilterReadonly( EFactory eFactory   ) const
 {
-::osl::MutexGuard aGuard( impl_GetOwnStaticMutex() );
+std::unique_lock aGuard( impl_GetOwnStaticMutex() );
 m_pImpl->MakeReadonlyStatesAvailable();
 return m_pImpl->IsDefaultFilterReadonly( eFactory );
 }
 
 sal_Int32 SvtModuleOptions::GetFactoryIcon( EFactory eFactory ) const
 {
-::osl::MutexGuard aGuard( impl_GetOwnStaticMutex() );
+std::unique_lock aGuard( impl_GetOwnStaticMutex() );
 return m_pImpl->GetFactoryIcon( eFactory );
 }
 
@@ -862,14 +863,14 @@ bool SvtModuleOptions::ClassifyFactoryByName( 
std::u16string_view sName,
 void SvtModuleOptions::SetFactoryStandardTemplate(   EFactory 
eFactory   ,
const OUString& sTemplate  )
 {
-::osl::MutexGuard aGuard( impl_GetOwnStaticMutex() );
+std::unique_lock aGuard( impl_GetOwnStaticMutex() );
 m_pImpl->SetFactoryStandardTemplate( eFactory, sTemplate );
 }
 
 void SvtModuleOptions::SetFactoryDefaultFilter(   EFactory 
eFactory,
 const OUString& sFilter )
 {
-::osl::MutexGuard aGuard( impl_GetOwnStaticMutex() );
+std::unique_lock aGuard( impl_GetOwnStaticMutex() );
 m_pImpl->SetFactoryDefaultFilter( eFactory, sFilter );
 }
 
@@ -899,7 +900,7 @@ bool SvtModuleOptions::IsDraw() const
 
 bool SvtModuleOptions::IsWriter() const
 {
-::osl::MutexGuard aGuard( impl_GetOwnStaticMutex() );
+std::unique_lock aGuard( impl_GetOwnStaticMutex() );
 return m_pImpl->IsModuleInstalled( EModule::WRITER );
 }
 
@@ -1086,7 +1087,7 @@ SvtModuleOptions::EFactory 
SvtModuleOptions::ClassifyFactoryByModel(const css::u
 
 css::uno::Sequence < OUString > SvtModuleOptions::GetAllServiceNames()
 {
-::osl::MutexGuard aGuard( impl_GetOwnStaticMutex() );
+std::unique_lock aGuard( impl_GetOwnStaticMutex() );
 return m_pImpl->GetAllServiceNames();
 }
 


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

2021-12-24 Thread Noel Grandin (via logerrit)
 svx/source/xml/xmlgrhlp.cxx |   11 ---
 1 file changed, 4 insertions(+), 7 deletions(-)

New commits:
commit addd00fefe246e3045380e97e9a66a39a23ecb4a
Author: Noel Grandin 
AuthorDate: Fri Dec 24 14:20:44 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Dec 24 20:17:15 2021 +0100

use comphelper::WeakComponentImplHelper in SvXMLGraphicImportExportHelper

Change-Id: Ieb76e57ac8c9e2f456e5d7d2c127668d5f2b8490
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127418
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/svx/source/xml/xmlgrhlp.cxx b/svx/source/xml/xmlgrhlp.cxx
index 96be71b78769..a20db0d073d2 100644
--- a/svx/source/xml/xmlgrhlp.cxx
+++ b/svx/source/xml/xmlgrhlp.cxx
@@ -31,8 +31,7 @@
 #include 
 #include 
 #include 
-#include 
-#include 
+#include 
 #include 
 #include 
 
@@ -930,7 +929,7 @@ namespace {
 
 namespace impl
 {
-typedef cppu::WeakComponentImplHelper xComp( m_xGraphicObjectResolver, UNO_QUERY );
 OSL_ASSERT( xComp.is());


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

2021-12-24 Thread Noel Grandin (via logerrit)
 canvas/inc/parametricpolypolygon.hxx  |   10 --
 canvas/source/tools/parametricpolypolygon.cxx |   10 +++---
 2 files changed, 7 insertions(+), 13 deletions(-)

New commits:
commit 439cb9d60d853b9521def7931a68226b95de48a8
Author: Noel Grandin 
AuthorDate: Fri Dec 24 09:58:44 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Dec 24 20:16:38 2021 +0100

use comphelper::WeakComponentImplHelper in ParametricPolyPolygon

Change-Id: I923970389870ced5e8e310c0f73859117b1cedb9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127405
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/canvas/inc/parametricpolypolygon.hxx 
b/canvas/inc/parametricpolypolygon.hxx
index 089c42e76f57..0d39700ff488 100644
--- a/canvas/inc/parametricpolypolygon.hxx
+++ b/canvas/inc/parametricpolypolygon.hxx
@@ -21,8 +21,7 @@
 
 #include 
 #include 
-#include 
-#include 
+#include 
 #include 
 
 #include 
@@ -35,11 +34,10 @@ namespace com::sun::star::rendering { class XGraphicDevice; 
}
 
 namespace canvas
 {
-typedef cppu::WeakComponentImplHelper< 
css::rendering::XParametricPolyPolygon2D,
+typedef comphelper::WeakComponentImplHelper< 
css::rendering::XParametricPolyPolygon2D,
css::lang::XServiceInfo > 
ParametricPolyPolygon_Base;
 
-class CANVASTOOLS_DLLPUBLIC ParametricPolyPolygon final : public 
::cppu::BaseMutex,
-  public ParametricPolyPolygon_Base
+class CANVASTOOLS_DLLPUBLIC ParametricPolyPolygon final : public 
ParametricPolyPolygon_Base
 {
 public:
 enum class GradientType
@@ -92,7 +90,7 @@ namespace canvas
 const css::uno::Sequence< css::uno::Any >& rArgs );
 
 /// Dispose all internal references
-virtual void SAL_CALL disposing() override;
+virtual void disposing() override;
 
 // XParametricPolyPolygon2D
 virtual css::uno::Reference< css::rendering::XPolyPolygon2D > SAL_CALL 
getOutline( double t ) override;
diff --git a/canvas/source/tools/parametricpolypolygon.cxx 
b/canvas/source/tools/parametricpolypolygon.cxx
index 331b4e47535b..870f25f2ddbf 100644
--- a/canvas/source/tools/parametricpolypolygon.cxx
+++ b/canvas/source/tools/parametricpolypolygon.cxx
@@ -147,10 +147,8 @@ namespace canvas
 colors, stops, fAspectRatio );
 }
 
-void SAL_CALL ParametricPolyPolygon::disposing()
+void ParametricPolyPolygon::disposing()
 {
-::osl::MutexGuard aGuard( m_aMutex );
-
 mxDevice.clear();
 }
 
@@ -174,7 +172,7 @@ namespace canvas
 
 uno::Reference< rendering::XColorSpace > SAL_CALL 
ParametricPolyPolygon::getColorSpace()
 {
-::osl::MutexGuard aGuard( m_aMutex );
+std::unique_lock aGuard( m_aMutex );
 
 return mxDevice.is() ? mxDevice->getDeviceColorSpace() : 
uno::Reference< rendering::XColorSpace >();
 }
@@ -205,7 +203,6 @@ namespace canvas
   const uno::Sequence< 
uno::Sequence< double > >&   rColors,
   const uno::Sequence< double 
>&rStops,
   double   
 nAspectRatio ) :
-ParametricPolyPolygon_Base( m_aMutex ),
 mxDevice( rDevice ),
 maValues( rGradientPoly,
   rColors,
@@ -219,7 +216,6 @@ namespace canvas
   GradientType 
 eType,
   const uno::Sequence< 
uno::Sequence< double > >&   rColors,
   const uno::Sequence< double 
>&rStops ) :
-ParametricPolyPolygon_Base( m_aMutex ),
 mxDevice( rDevice ),
 maValues( ::basegfx::B2DPolygon(),
   rColors,
@@ -231,7 +227,7 @@ namespace canvas
 
 ParametricPolyPolygon::Values ParametricPolyPolygon::getValues() const
 {
-::osl::MutexGuard aGuard( m_aMutex );
+std::unique_lock aGuard( m_aMutex );
 
 return maValues;
 }


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

2021-12-24 Thread Noel Grandin (via logerrit)
 include/unotools/cmdoptions.hxx   |   11 ---
 unotools/source/config/cmdoptions.cxx |   27 +--
 2 files changed, 13 insertions(+), 25 deletions(-)

New commits:
commit 1576ef8532cf2cea2da361db9c5707f1994e65e2
Author: Noel Grandin 
AuthorDate: Thu Dec 23 18:40:44 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Dec 24 19:50:02 2021 +0100

osl::Mutex->std::mutex in SvtCommandOptions

Change-Id: I80eaba645351e2f3a3c67586fdced3325608633e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127413
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/include/unotools/cmdoptions.hxx b/include/unotools/cmdoptions.hxx
index d0e6792ba8b9..bb3414f8ff41 100644
--- a/include/unotools/cmdoptions.hxx
+++ b/include/unotools/cmdoptions.hxx
@@ -109,17 +109,6 @@ public:
 
 void EstablishFrameCallback(const css::uno::Reference& 
xFrame);
 
-private:
-
/*-
-@short  return a reference to a static mutex
-@descr  These class is partially threadsafe (for 
de-/initialization only).
-All access methods aren't safe!
-We create a static mutex only for one ime and use at 
different times.
-@return A reference to a static mutex member.
-
**-***/
-
-UNOTOOLS_DLLPRIVATE static ::osl::Mutex& GetOwnStaticMutex();
-
 private:
 std::shared_ptr m_pImpl;
 
diff --git a/unotools/source/config/cmdoptions.cxx 
b/unotools/source/config/cmdoptions.cxx
index d9e061224a4a..1408f22c2480 100644
--- a/unotools/source/config/cmdoptions.cxx
+++ b/unotools/source/config/cmdoptions.cxx
@@ -84,6 +84,12 @@ class SvtCmdOptions
 CommandHashMap m_aCommandHashMap;
 };
 
+std::mutex& GetOwnStaticMutex()
+{
+static std::mutex theCommandOptionsMutex;
+return theCommandOptionsMutex;
+}
+
 }
 
 typedef ::std::vector< css::uno::WeakReference< css::frame::XFrame > > 
SvtFrameVector;
@@ -185,7 +191,7 @@ SvtCommandOptions_Impl::~SvtCommandOptions_Impl()
 
 void SvtCommandOptions_Impl::Notify( const Sequence< OUString >& )
 {
-MutexGuard aGuard( SvtCommandOptions::GetOwnStaticMutex() );
+std::unique_lock aGuard( GetOwnStaticMutex() );
 
 Sequence< OUString >lNames   = impl_GetPropertyNames ();
 Sequence< Any > lValues  = GetProperties ( lNames 
);
@@ -296,13 +302,14 @@ std::weak_ptr g_pCommandOptions;
 SvtCommandOptions::SvtCommandOptions()
 {
 // Global access, must be guarded (multithreading!).
-MutexGuard aGuard( GetOwnStaticMutex() );
+std::unique_lock aGuard( GetOwnStaticMutex() );
 
 m_pImpl = g_pCommandOptions.lock();
 if( !m_pImpl )
 {
 m_pImpl = std::make_shared();
 g_pCommandOptions = m_pImpl;
+aGuard.unlock(); // because holdConfigItem will call this constructor
 ItemHolder1::holdConfigItem(EItem::CmdOptions);
 }
 }
@@ -310,7 +317,7 @@ SvtCommandOptions::SvtCommandOptions()
 SvtCommandOptions::~SvtCommandOptions()
 {
 // Global access, must be guarded (multithreading!)
-MutexGuard aGuard( GetOwnStaticMutex() );
+std::unique_lock aGuard( GetOwnStaticMutex() );
 
 m_pImpl.reset();
 }
@@ -319,7 +326,7 @@ SvtCommandOptions::~SvtCommandOptions()
 
 bool SvtCommandOptions::HasEntries( CmdOption eOption ) const
 {
-MutexGuard aGuard( GetOwnStaticMutex() );
+std::unique_lock aGuard( GetOwnStaticMutex() );
 return m_pImpl->HasEntries( eOption );
 }
 
@@ -327,7 +334,7 @@ bool SvtCommandOptions::HasEntries( CmdOption eOption ) 
const
 
 bool SvtCommandOptions::Lookup( CmdOption eCmdOption, const OUString& 
aCommandURL ) const
 {
-MutexGuard aGuard( GetOwnStaticMutex() );
+std::unique_lock aGuard( GetOwnStaticMutex() );
 return m_pImpl->Lookup( eCmdOption, aCommandURL );
 }
 
@@ -335,16 +342,8 @@ bool SvtCommandOptions::Lookup( CmdOption eCmdOption, 
const OUString& aCommandUR
 
 void SvtCommandOptions::EstablishFrameCallback(const css::uno::Reference< 
css::frame::XFrame >& xFrame)
 {
-MutexGuard aGuard( GetOwnStaticMutex() );
+std::unique_lock aGuard( GetOwnStaticMutex() );
 m_pImpl->EstablishFrameCallback(xFrame);
 }
 
-//  private method
-
-Mutex& SvtCommandOptions::GetOwnStaticMutex()
-{
-static osl::Mutex theCommandOptionsMutex;
-return theCommandOptionsMutex;
-}
-
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */


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

2021-12-24 Thread Noel Grandin (via logerrit)
 unotools/source/config/pathoptions.cxx |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit d94db4056a49fc4fee73173e1ef1c943f8ecc21f
Author: Noel Grandin 
AuthorDate: Thu Dec 23 19:39:05 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Dec 24 19:45:55 2021 +0100

osl::Mutex->std::mutex in SvtPathOptions_Impl

Change-Id: I04d0bf2a6b00ab0da24735158ad267632356128f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127416
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/unotools/source/config/pathoptions.cxx 
b/unotools/source/config/pathoptions.cxx
index 60335e00dfb9..e7793d07321c 100644
--- a/unotools/source/config/pathoptions.cxx
+++ b/unotools/source/config/pathoptions.cxx
@@ -82,7 +82,7 @@ class SvtPathOptions_Impl
 VarNameSet  m_aSystemPathVarNames;
 
 OUStringm_aEmptyString;
-mutable ::osl::Mutexm_aMutex;
+mutable std::mutex  m_aMutex;
 
 public:
 SvtPathOptions_Impl();
@@ -210,7 +210,7 @@ const VarNameAttribute aVarNameAttribute[] =
 
 const OUString& SvtPathOptions_Impl::GetPath( SvtPathOptions::Paths ePath )
 {
-::osl::MutexGuard aGuard( m_aMutex );
+std::unique_lock aGuard( m_aMutex );
 
 try
 {
@@ -263,7 +263,7 @@ const OUString& SvtPathOptions_Impl::GetPath( 
SvtPathOptions::Paths ePath )
 
 void SvtPathOptions_Impl::SetPath( SvtPathOptions::Paths ePath, const 
OUString& rNewPath )
 {
-::osl::MutexGuard aGuard( m_aMutex );
+std::unique_lock aGuard( m_aMutex );
 
 OUStringaResult;
 OUStringaNewValue;


[Libreoffice-bugs] [Bug 97663] SVG line spacing not correct

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=97663

--- Comment #10 from Commit Notification 
 ---
Hossein committed a patch related to this issue.
It has been pushed to "libreoffice-7-3":

https://git.libreoffice.org/core/commit/8e1aea537ffe35311cc6d43d4b0cef3f4ec82367

tdf#97663 SVGIO: Fix line spacing for 

It will be available in 7.3.0.2.

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] [Bug 97663] SVG line spacing not correct

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=97663

Commit Notification  changed:

   What|Removed |Added

 Whiteboard|target:7.4.0|target:7.4.0 target:7.3.0.2

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-commits] core.git: Branch 'libreoffice-7-3' - svgio/inc svgio/qa svgio/source

2021-12-24 Thread Hossein (via logerrit)
 svgio/inc/svgnode.hxx  |2 +-
 svgio/inc/svgtextnode.hxx  |2 --
 svgio/inc/svgtspannode.hxx |2 +-
 svgio/qa/cppunit/SvgImportTest.cxx |   20 
 svgio/qa/cppunit/data/em_units.svg |   14 ++
 svgio/source/svgreader/svgtextnode.cxx |4 
 6 files changed, 36 insertions(+), 8 deletions(-)

New commits:
commit 8e1aea537ffe35311cc6d43d4b0cef3f4ec82367
Author: Hossein 
AuthorDate: Mon Dec 20 05:44:23 2021 +0100
Commit: Xisco Fauli 
CommitDate: Fri Dec 24 19:35:15 2021 +0100

tdf#97663 SVGIO: Fix line spacing for 

tdf#97663 is a regression caused by the commit
701324a1e1f7e0c181ff1a50956ced686785ea53. The previous patch caused
LO to forget the size of the font which was needed to calculate line
height based on em units.

em, px, pt, cm, in...
https://www.w3.org/Style/Examples/007/units.en.html

Accompanied with this fix is a unit test provided to avoid this
issue in the future.

The fix can be tested with:

make CPPUNIT_TEST_NAME="testTdf97663" -sr \
CppunitTest_svgio

The em_units.svg is opened with Firefox, Chrome and Inkscape and the
rendering in LibreOffice is compatible with the rendering in these
applications.

Change-Id: Idaecd9fb18101f7925fe2a917f7fc3fe7257ebc5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127130
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens 
(cherry picked from commit 4e2e57b530544736804ab663f832173ba1d78559)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127376
Reviewed-by: Xisco Fauli 

diff --git a/svgio/inc/svgnode.hxx b/svgio/inc/svgnode.hxx
index 63c6b2318406..58b1682506f2 100644
--- a/svgio/inc/svgnode.hxx
+++ b/svgio/inc/svgnode.hxx
@@ -159,7 +159,7 @@ namespace svgio::svgreader
 virtual double getCurrentFontSizeInherited() const override;
 virtual double getCurrentXHeightInherited() const override;
 
-virtual double getCurrentFontSize() const;
+double getCurrentFontSize() const;
 double getCurrentXHeight() const;
 
 /// Id access
diff --git a/svgio/inc/svgtextnode.hxx b/svgio/inc/svgtextnode.hxx
index b72d1043b8c0..37983ad31531 100644
--- a/svgio/inc/svgtextnode.hxx
+++ b/svgio/inc/svgtextnode.hxx
@@ -58,8 +58,6 @@ namespace svgio::svgreader
 virtual void parseAttribute(const OUString& rTokenName, SVGToken 
aSVGToken, const OUString& aContent) override;
 virtual void 
decomposeSvgNode(drawinglayer::primitive2d::Primitive2DContainer& rTarget, bool 
bReferenced) const override;
 
-virtual double getCurrentFontSize() const override;
-
 /// transform content, set if found in current context
 const std::optional& getTransform() const { 
return mpaTransform; }
 void setTransform(const std::optional& 
pMatrix) { mpaTransform = pMatrix; }
diff --git a/svgio/inc/svgtspannode.hxx b/svgio/inc/svgtspannode.hxx
index af4ae8268b5c..10a7b7ee16a9 100644
--- a/svgio/inc/svgtspannode.hxx
+++ b/svgio/inc/svgtspannode.hxx
@@ -42,7 +42,7 @@ namespace svgio::svgreader
 virtual const SvgStyleAttributes* getSvgStyleAttributes() const 
override;
 virtual void parseAttribute(const OUString& rTokenName, SVGToken 
aSVGToken, const OUString& aContent) override;
 
-virtual double getCurrentFontSize() const override;
+double getCurrentFontSize() const;
 
 /// access to SvgTextPositions
 const SvgTextPositions& getSvgTextPositions() const { return 
maSvgTextPositions; }
diff --git a/svgio/qa/cppunit/SvgImportTest.cxx 
b/svgio/qa/cppunit/SvgImportTest.cxx
index 25baf76fdad8..3034c997015e 100644
--- a/svgio/qa/cppunit/SvgImportTest.cxx
+++ b/svgio/qa/cppunit/SvgImportTest.cxx
@@ -69,6 +69,7 @@ class Test : public test::BootstrapFixture, public 
XmlTestTools
 void testTdf101237();
 void testTdf94765();
 void testBehaviourWhenWidthAndHeightIsOrIsNotSet();
+void testTdf97663();
 
 Primitive2DSequence parseSvg(std::u16string_view aSource);
 
@@ -104,6 +105,7 @@ public:
 CPPUNIT_TEST(testTdf101237);
 CPPUNIT_TEST(testTdf94765);
 CPPUNIT_TEST(testBehaviourWhenWidthAndHeightIsOrIsNotSet);
+CPPUNIT_TEST(testTdf97663);
 CPPUNIT_TEST_SUITE_END();
 };
 
@@ -804,6 +806,24 @@ void Test::testBehaviourWhenWidthAndHeightIsOrIsNotSet()
 }
 }
 
+void Test::testTdf97663()
+{
+Primitive2DSequence aSequence = 
parseSvg(u"/svgio/qa/cppunit/data/em_units.svg");
+CPPUNIT_ASSERT_EQUAL(1, static_cast(aSequence.getLength()));
+
+drawinglayer::Primitive2dXmlDump dumper;
+// This can be dumped to a file using dumper.dump(container, file_url)
+Primitive2DContainer container = 
comphelper::sequenceToContainer(aSequence);
+xmlDocUniquePtr pDocument = dumper.dumpAndParse(container);
+
+CPPUNIT_ASSERT (pDocument);
+
+ 

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

2021-12-24 Thread Jan-Marek Glogowski (via logerrit)
 idl/source/objects/types.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 03faef951a0dd4352a371ead89ca79040359234b
Author: Jan-Marek Glogowski 
AuthorDate: Fri Dec 24 14:42:00 2021 +0100
Commit: Jan-Marek Glogowski 
CommitDate: Fri Dec 24 19:21:29 2021 +0100

mergelibs: fix duplicate aSfxInt16Item_Impl

The callgrind / mergelibs build fails with:

/svx/source/form/typemap.o: in function `aSvxColorItem_Impl´: \
   multiple definition of `aSfxInt16Item_Impl´; \
/sfx2/source/appl/appbas.o: first defined here

Regression from commit 60d7796624f1c56eb036d3e73d7f8a8216cce2a4
("sd theme: allow setting color effects in the sidebar").

Change-Id: Ibe382e2e12d3359caa3e853e264de0ede6a5e601
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127429
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski 

diff --git a/idl/source/objects/types.cxx b/idl/source/objects/types.cxx
index 57d62c49c24c..46262e7b98d9 100644
--- a/idl/source/objects/types.cxx
+++ b/idl/source/objects/types.cxx
@@ -231,6 +231,7 @@ void SvMetaType::WriteSfxItem(
 bool bExport = false, bReturn = false;
 // these are exported from sfx library
 if (rItemName == "SfxBoolItem" ||
+rItemName == "SfxInt16Item" ||
 rItemName == "SfxStringItem" ||
 rItemName == "SfxUInt16Item" ||
 rItemName == "SfxUInt32Item" ||


[Libreoffice-bugs] [Bug 136845] clicking on libreoffice help or typing F1 opens a white page NewHelp0.html in write/web

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=136845

himajin100...@gmail.com changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=14
   ||6403

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146403] Help or F1 does not open my default browser

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146403

himajin100...@gmail.com changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=13
   ||6845

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146405] SPAM (deletion request)

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146405

V Stuart Foote  changed:

   What|Removed |Added

 Resolution|FIXED   |INVALID
Summary|Ok  |SPAM (deletion request)

-- 
You are receiving this mail because:
You are the assignee for the bug.

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

2021-12-24 Thread Noel Grandin (via logerrit)
 include/unotools/useroptions.hxx   |2 -
 unotools/source/config/useroptions.cxx |   34 +
 2 files changed, 18 insertions(+), 18 deletions(-)

New commits:
commit ac511d90cdf9d28eb8809c30be9fa08b42ea0bd3
Author: Noel Grandin 
AuthorDate: Thu Dec 23 19:42:19 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Dec 24 19:10:25 2021 +0100

osl::Mutex->std::mutex in SvtUserOptions

Change-Id: Ib16cc05a8d9c3e7ef828223e8b1067eeb7faf809
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127399
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/include/unotools/useroptions.hxx b/include/unotools/useroptions.hxx
index adc5f5055d5b..d21482dcca2d 100644
--- a/include/unotools/useroptions.hxx
+++ b/include/unotools/useroptions.hxx
@@ -60,8 +60,6 @@ public:
 SvtUserOptions ();
 virtual ~SvtUserOptions () override;
 
-static osl::Mutex& GetInitMutex ();
-
 // get the address token
 OUString GetCompany() const;
 OUString GetFirstName  () const;
diff --git a/unotools/source/config/useroptions.cxx 
b/unotools/source/config/useroptions.cxx
index ffe01cca497c..448850fb84fc 100644
--- a/unotools/source/config/useroptions.cxx
+++ b/unotools/source/config/useroptions.cxx
@@ -259,34 +259,36 @@ bool SvtUserOptions::Impl::IsTokenReadonly (UserOptToken 
nToken) const
 beans::PropertyAttribute::READONLY);
 }
 
+static std::mutex& GetInitMutex()
+{
+static std::mutex gMutex;
+return gMutex;
+}
+
+
 SvtUserOptions::SvtUserOptions ()
 {
 // Global access, must be guarded (multithreading)
-osl::MutexGuard aGuard(GetInitMutex());
+std::unique_lock aGuard(GetInitMutex());
 
-if (xSharedImpl.expired())
+xImpl = xSharedImpl.lock();
+if (!xImpl)
 {
 xImpl = std::make_shared();
 xSharedImpl = xImpl;
+aGuard.unlock(); // because holdConfigItem will call this constructor
 ItemHolder1::holdConfigItem(EItem::UserOptions);
 }
-xImpl = xSharedImpl.lock();
 xImpl->AddListener(this);
 }
 
 SvtUserOptions::~SvtUserOptions()
 {
 // Global access, must be guarded (multithreading)
-osl::MutexGuard aGuard( GetInitMutex() );
+std::unique_lock aGuard( GetInitMutex() );
 xImpl->RemoveListener(this);
 }
 
-osl::Mutex& SvtUserOptions::GetInitMutex()
-{
-static osl::Mutex gMutex;
-return gMutex;
-}
-
 OUString SvtUserOptions::GetCompany() const { return 
GetToken(UserOptToken::Company); }
 OUString SvtUserOptions::GetFirstName  () const { return 
GetToken(UserOptToken::FirstName); }
 OUString SvtUserOptions::GetLastName   () const { return 
GetToken(UserOptToken::LastName); }
@@ -307,37 +309,37 @@ OUString SvtUserOptions::GetEncryptionKey  () const { 
return GetToken(UserOptTok
 
 bool SvtUserOptions::IsTokenReadonly (UserOptToken nToken) const
 {
-osl::MutexGuard aGuard(GetInitMutex());
+std::unique_lock aGuard(GetInitMutex());
 return xImpl->IsTokenReadonly(nToken);
 }
 
 OUString SvtUserOptions::GetToken (UserOptToken nToken) const
 {
-osl::MutexGuard aGuard(GetInitMutex());
+std::unique_lock aGuard(GetInitMutex());
 return xImpl->GetToken(nToken);
 }
 
 void SvtUserOptions::SetToken (UserOptToken nToken, OUString const& rNewToken)
 {
-osl::MutexGuard aGuard(GetInitMutex());
+std::unique_lock aGuard(GetInitMutex());
 xImpl->SetToken(nToken, rNewToken);
 }
 
 void SvtUserOptions::SetBoolValue (UserOptToken nToken, bool bNewValue)
 {
-osl::MutexGuard aGuard(GetInitMutex());
+std::unique_lock aGuard(GetInitMutex());
 xImpl->SetBoolValue(nToken, bNewValue);
 }
 
 bool SvtUserOptions::GetEncryptToSelf() const
 {
-osl::MutexGuard aGuard(GetInitMutex());
+std::unique_lock aGuard(GetInitMutex());
 return xImpl->GetBoolValue(UserOptToken::EncryptToSelf);
 }
 
 OUString SvtUserOptions::GetFullName () const
 {
-osl::MutexGuard aGuard(GetInitMutex());
+std::unique_lock aGuard(GetInitMutex());
 return xImpl->GetFullName();
 }
 


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

2021-12-24 Thread Noel Grandin (via logerrit)
 include/svx/xmlgrhlp.hxx|8 ++--
 svx/source/xml/xmlgrhlp.cxx |   16 +++-
 2 files changed, 5 insertions(+), 19 deletions(-)

New commits:
commit 4621e719c283cb24ec6b884b55a6719a321336f3
Author: Noel Grandin 
AuthorDate: Fri Dec 24 10:06:17 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Dec 24 18:42:25 2021 +0100

use comphelper::WeakComponentImplHelper in SvXMLGraphicHelper

and remove unused disposing() override

Change-Id: I6423d9c3c418bb07f84e1622c8df6eac8ad762c4
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127406
Tested-by: Noel Grandin 
Reviewed-by: Noel Grandin 

diff --git a/include/svx/xmlgrhlp.hxx b/include/svx/xmlgrhlp.hxx
index 8a72664a5b55..0e54e1cb102b 100644
--- a/include/svx/xmlgrhlp.hxx
+++ b/include/svx/xmlgrhlp.hxx
@@ -20,8 +20,7 @@
 #ifndef INCLUDED_SVX_XMLGRHLP_HXX
 #define INCLUDED_SVX_XMLGRHLP_HXX
 
-#include 
-#include 
+#include 
 #include 
 #include 
 
@@ -47,8 +46,7 @@ struct SvxGraphicHelperStream_Impl
 };
 
 class SVXCORE_DLLPUBLIC SvXMLGraphicHelper final :
-public cppu::BaseMutex,
-public 
cppu::WeakComponentImplHelper
 {
@@ -82,8 +80,6 @@ private:
   SvXMLGraphicHelperMode eCreateMode,
   const OUString& rGraphicMimeType = 
OUString() );
 
-virtual void SAL_CALL   disposing() override;
-
 SVX_DLLPRIVATE OUString 
implSaveGraphic(css::uno::Reference const & rxGraphic,
 OUString & rOutMimeType,
 std::u16string_view rRequestName);
diff --git a/svx/source/xml/xmlgrhlp.cxx b/svx/source/xml/xmlgrhlp.cxx
index 35d3a247c56a..96be71b78769 100644
--- a/svx/source/xml/xmlgrhlp.cxx
+++ b/svx/source/xml/xmlgrhlp.cxx
@@ -367,18 +367,12 @@ const GraphicObject& 
SvXMLGraphicOutputStream::GetGraphicObject()
 }
 
 SvXMLGraphicHelper::SvXMLGraphicHelper(SvXMLGraphicHelperMode eCreateMode)
-: cppu::WeakComponentImplHelper(m_aMutex)
 {
 Init( nullptr, eCreateMode );
 }
 
 SvXMLGraphicHelper::SvXMLGraphicHelper()
-: cppu::WeakComponentImplHelper(m_aMutex)
-, meCreateMode(SvXMLGraphicHelperMode::Read)
+: meCreateMode(SvXMLGraphicHelperMode::Read)
 {
 }
 
@@ -386,10 +380,6 @@ SvXMLGraphicHelper::~SvXMLGraphicHelper()
 {
 }
 
-void SAL_CALL SvXMLGraphicHelper::disposing()
-{
-}
-
 bool SvXMLGraphicHelper::ImplGetStreamNames( const OUString& rURLStr,
  OUString& rPictureStorageName,
  OUString& rPictureStreamName )
@@ -580,7 +570,7 @@ OUString SAL_CALL 
SvXMLGraphicHelper::resolveGraphicObjectURL( const OUString& /
 // XGraphicStorageHandler
 uno::Reference SAL_CALL 
SvXMLGraphicHelper::loadGraphic(OUString const & rURL)
 {
-osl::MutexGuard aGuard(m_aMutex);
+std::unique_lock aGuard(m_aMutex);
 
 uno::Reference xGraphic;
 
@@ -612,7 +602,7 @@ uno::Reference SAL_CALL 
SvXMLGraphicHelper::loadGraphic(OUStr
 
 uno::Reference SAL_CALL 
SvXMLGraphicHelper::loadGraphicFromOutputStream(uno::Reference
 const & rxOutputStream)
 {
-osl::MutexGuard aGuard(m_aMutex);
+std::unique_lock aGuard(m_aMutex);
 
 uno::Reference xGraphic;
 


[Libreoffice-bugs] [Bug 146382] Can’t open file from Explorer-attached remote location

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146382

--- Comment #5 from Commit Notification 
 ---
Noel Grandin committed a patch related to this issue.
It has been pushed to "libreoffice-7-3":

https://git.libreoffice.org/core/commit/efa20d38fa51675e738ce50d6894b5df75debea0

tdf#146382 cant open file from Explorer-attached remote location

It will be available in 7.3.0.2.

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] [Bug 146382] Can’t open file from Explorer-attached remote location

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146382

Commit Notification  changed:

   What|Removed |Added

 Whiteboard|target:7.4.0|target:7.4.0 target:7.3.0.2

-- 
You are receiving this mail because:
You are the assignee for the bug.

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

2021-12-24 Thread Noel Grandin (via logerrit)
 tools/qa/cppunit/test_urlobj.cxx |9 +
 tools/source/fsys/urlobj.cxx |3 +++
 2 files changed, 12 insertions(+)

New commits:
commit efa20d38fa51675e738ce50d6894b5df75debea0
Author: Noel Grandin 
AuthorDate: Fri Dec 24 09:35:44 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Dec 24 18:21:34 2021 +0100

tdf#146382 cant open file from Explorer-attached remote location

problem spotted by mikekaganski

regression from
commit 0b46361ef84a61100a0274a007062317607d097a
Author: Noel Grandin 
Date:   Sat Nov 20 08:03:49 2021 +0200
tdf#133835 speedup calc autofilter (4)

where I changed the behaviour of parseHostOrNetBiosName from overwriting
the destination buffer to appending to it.

Change-Id: I1bd3c7eade035a867c026364f5e114c1e652ca32
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127352
Tested-by: Gabor Kelemen 
Tested-by: Jenkins
Reviewed-by: Noel Grandin 
(cherry picked from commit 15bc49228997a5da556961b923bb607642b6)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127381

diff --git a/tools/qa/cppunit/test_urlobj.cxx b/tools/qa/cppunit/test_urlobj.cxx
index a335701c5cab..ec64b5d66777 100644
--- a/tools/qa/cppunit/test_urlobj.cxx
+++ b/tools/qa/cppunit/test_urlobj.cxx
@@ -311,6 +311,14 @@ namespace tools_urlobj
 obj.GetMainURL(INetURLObject::DecodeMechanism::NONE));
 }
 
+void testTd146382() {
+INetURLObject 
obj("file://share.allotropia.de@SSL/DavWWWRoot/remote.php");
+CPPUNIT_ASSERT(!obj.HasError());
+CPPUNIT_ASSERT_EQUAL(
+
OUString("file://share.allotropia.de@SSL/DavWWWRoot/remote.php"),
+obj.GetMainURL(INetURLObject::DecodeMechanism::NONE));
+}
+
 // Change the following lines only, if you add, remove or rename
 // member functions of the current class,
 // because these macros are need by auto register mechanism.
@@ -326,6 +334,7 @@ namespace tools_urlobj
 CPPUNIT_TEST( testSetName );
 CPPUNIT_TEST( testSetExtension );
 CPPUNIT_TEST( testChangeScheme );
+CPPUNIT_TEST( testTd146382 );
 CPPUNIT_TEST_SUITE_END(  );
 };  // class createPool
 
diff --git a/tools/source/fsys/urlobj.cxx b/tools/source/fsys/urlobj.cxx
index 8c3d2845eab2..7b86fe0f5261 100644
--- a/tools/source/fsys/urlobj.cxx
+++ b/tools/source/fsys/urlobj.cxx
@@ -2880,7 +2880,10 @@ bool INetURLObject::parseHostOrNetBiosName(
 }
 }
 if (pCanonic)
+{
+pCanonic->setLength(nOriginalCanonicLength);
 pCanonic->append(buf);
+}
 }
 else
 {


[Libreoffice-commits] core.git: Makefile.in

2021-12-24 Thread Jan-Marek Glogowski (via logerrit)
 Makefile.in |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 69b6c87eca7d7fa1e83653750e79e04ae1d5d384
Author: Jan-Marek Glogowski 
AuthorDate: Fri Dec 24 13:47:46 2021 +0100
Commit: Jan-Marek Glogowski 
CommitDate: Fri Dec 24 18:20:46 2021 +0100

autogen.sh is actually in $(SRCDIR)

Change-Id: I9c58294fb607683929d370a5f2707901e2a88548
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127421
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski 

diff --git a/Makefile.in b/Makefile.in
index 92ff372429b2..3ac9ce6d3c22 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -39,6 +39,7 @@ Makefile $(BUILDDIR)/Makefile: $(BUILDDIR)/config_host.mk 
$(BUILDDIR)/config_hos
 
 # run configure in an environment not polluted by config_host.mk
 $(BUILDDIR)/config_host.mk : $(wildcard \
+   $(SRCDIR)/autogen.sh \
$(SRCDIR)/config_host.mk.in \
$(SRCDIR)/config_host_lang.mk.in \
$(SRCDIR)/Makefile.in \
@@ -49,7 +50,6 @@ $(BUILDDIR)/config_host.mk : $(wildcard \
$(SRCDIR)/m4/*.m4 \
$(BUILDDIR)/autogen.input \
$(BUILDDIR)/autogen.lastrun \
-   $(BUILDDIR)/autogen.sh \
) \
$(shell . $(SRCDIR)/bin/get_config_variables JAVA_HOME && \
if test -n "$${JAVA_HOME}" -a ! -d "$${JAVA_HOME}/bin"; 
then echo force-restart; fi)


[Libreoffice-commits] core.git: Makefile.in

2021-12-24 Thread Jan-Marek Glogowski (via logerrit)
 Makefile.in |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 6bed8aa4462ac553921392533be646ef26f7b9e7
Author: Jan-Marek Glogowski 
AuthorDate: Fri Dec 24 13:45:47 2021 +0100
Commit: Jan-Marek Glogowski 
CommitDate: Fri Dec 24 18:20:01 2021 +0100

Re-run autogen.sh on m4 macro change

Change-Id: I8cd1203bcdf1b98bb34c0fd1ec38ad3f6af0eaed
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127422
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski 

diff --git a/Makefile.in b/Makefile.in
index bf756f205915..92ff372429b2 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -46,6 +46,7 @@ $(BUILDDIR)/config_host.mk : $(wildcard \
$(SRCDIR)/configure.ac \
$(SRCDIR)/config_host/*.h.in \
$(SRCDIR)/download.lst \
+   $(SRCDIR)/m4/*.m4 \
$(BUILDDIR)/autogen.input \
$(BUILDDIR)/autogen.lastrun \
$(BUILDDIR)/autogen.sh \


[Libreoffice-commits] help.git: AllLangHelp_sdatabase.mk AllLangHelp_shared.mk source/text

2021-12-24 Thread Olivier Hallot (via logerrit)
 AllLangHelp_sdatabase.mk   |9 +
 AllLangHelp_shared.mk  |9 -
 source/text/sdatabase/querywizard00.xhp|   28 ++--
 source/text/sdatabase/querywizard01.xhp|   16 
 source/text/sdatabase/querywizard02.xhp|   17 +++--
 source/text/sdatabase/querywizard03.xhp|   19 ---
 source/text/sdatabase/querywizard04.xhp|   21 +
 source/text/sdatabase/querywizard05.xhp|   14 +++---
 source/text/sdatabase/querywizard06.xhp|   19 ---
 source/text/sdatabase/querywizard07.xhp|   11 ---
 source/text/sdatabase/querywizard08.xhp|   17 +++--
 source/text/shared/guide/data_queries.xhp  |2 +-
 source/text/shared/guide/database_main.xhp |2 +-
 source/text/shared/guide/main.xhp  |2 +-
 14 files changed, 84 insertions(+), 102 deletions(-)

New commits:
commit 215707177c272f9fcf73a79b9150fe677b991716
Author: Olivier Hallot 
AuthorDate: Fri Dec 24 11:32:32 2021 -0300
Commit: Olivier Hallot 
CommitDate: Fri Dec 24 17:57:56 2021 +0100

Refactor database files

Change-Id: If6acf26ea094234d06522c26c8bb8274c0abdaa5
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/127430
Tested-by: Jenkins
Reviewed-by: Olivier Hallot 

diff --git a/AllLangHelp_sdatabase.mk b/AllLangHelp_sdatabase.mk
index 736b1abb4..6d607cb60 100644
--- a/AllLangHelp_sdatabase.mk
+++ b/AllLangHelp_sdatabase.mk
@@ -75,6 +75,15 @@ $(eval $(call gb_AllLangHelp_add_helpfiles,sdatabase,\
 helpcontent2/source/text/sdatabase/menuview \
 helpcontent2/source/text/sdatabase/migrate_macros \
 helpcontent2/source/text/sdatabase/password \
+helpcontent2/source/text/sdatabase/querywizard00 \
+helpcontent2/source/text/sdatabase/querywizard01 \
+helpcontent2/source/text/sdatabase/querywizard02 \
+helpcontent2/source/text/sdatabase/querywizard03 \
+helpcontent2/source/text/sdatabase/querywizard04 \
+helpcontent2/source/text/sdatabase/querywizard05 \
+helpcontent2/source/text/sdatabase/querywizard06 \
+helpcontent2/source/text/sdatabase/querywizard07 \
+helpcontent2/source/text/sdatabase/querywizard08 \
 helpcontent2/source/text/sdatabase/tablewizard00 \
 helpcontent2/source/text/sdatabase/tablewizard01 \
 helpcontent2/source/text/sdatabase/tablewizard02 \
diff --git a/AllLangHelp_shared.mk b/AllLangHelp_shared.mk
index 6e19148d1..03f5e3d5b 100644
--- a/AllLangHelp_shared.mk
+++ b/AllLangHelp_shared.mk
@@ -870,15 +870,6 @@ $(eval $(call gb_AllLangHelp_add_helpfiles,shared,\
 ))
 
 $(eval $(call gb_AllLangHelp_add_helpfiles,shared,\
-helpcontent2/source/text/shared/explorer/database/querywizard00 \
-helpcontent2/source/text/shared/explorer/database/querywizard01 \
-helpcontent2/source/text/shared/explorer/database/querywizard02 \
-helpcontent2/source/text/shared/explorer/database/querywizard03 \
-helpcontent2/source/text/shared/explorer/database/querywizard04 \
-helpcontent2/source/text/shared/explorer/database/querywizard05 \
-helpcontent2/source/text/shared/explorer/database/querywizard06 \
-helpcontent2/source/text/shared/explorer/database/querywizard07 \
-helpcontent2/source/text/shared/explorer/database/querywizard08 \
 helpcontent2/source/text/shared/explorer/database/rep_datetime \
 helpcontent2/source/text/shared/explorer/database/rep_insertfield \
 helpcontent2/source/text/shared/explorer/database/rep_main \
diff --git a/source/text/shared/explorer/database/querywizard00.xhp 
b/source/text/sdatabase/querywizard00.xhp
similarity index 64%
rename from source/text/shared/explorer/database/querywizard00.xhp
rename to source/text/sdatabase/querywizard00.xhp
index ffe747400..314f14c93 100644
--- a/source/text/shared/explorer/database/querywizard00.xhp
+++ b/source/text/sdatabase/querywizard00.xhp
@@ -1,6 +1,6 @@
 
 
-   
+
 
- 
-   
+
+
 
 
 Query Wizard
-/text/shared/explorer/database/querywizard00.xhp
+/text/sdatabase/querywizard00.xhp
 
 
 
 mw deleted 2 index entries
 
-Query 
Wizard
+Query 
Wizard
 The Query Wizard helps you to design a database query. The 
saved query can be called later, either from the graphical user interface, or 
using the automatically created SQL language command.
 
-
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
 
 
 
 
-Query Wizard - Field selection
+Query Wizard - Field selection
 
 
diff --git a/source/text/shared/explorer/database/querywizard01.xhp 
b/source/text/sdatabase/querywizard01.xhp
similarity index 79%
rename from source/text/shared/explorer/database/querywizard01.xhp
rename to source/text/sdatabase/querywizard01.xhp
index 16eb42baa..7570c3daa 100644
--- a/source/text/shared/explorer/database/querywizard01.xhp
+++ b/source/text/sdatabase/querywizard01.xhp
@@ -1,6 +1,6 @@
 
 
-   
+
 
- 
-   
+
+
 
 
 Query Wizard - Field Selection

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

2021-12-24 Thread Olivier Hallot (via logerrit)
 helpcontent2 |2 +-
 svx/source/accessibility/ChildrenManagerImpl.cxx |   10 +++---
 svx/source/accessibility/ChildrenManagerImpl.hxx |7 +++
 3 files changed, 7 insertions(+), 12 deletions(-)

New commits:
commit cf6976280f5371c01251c0ffce206535433207a3
Author: Olivier Hallot 
AuthorDate: Fri Dec 24 13:57:59 2021 -0300
Commit: Gerrit Code Review 
CommitDate: Fri Dec 24 17:57:59 2021 +0100

Update git submodules

* Update helpcontent2 from branch 'master'
  to 215707177c272f9fcf73a79b9150fe677b991716
  - Refactor database files

Change-Id: If6acf26ea094234d06522c26c8bb8274c0abdaa5
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/127430
Tested-by: Jenkins
Reviewed-by: Olivier Hallot 

diff --git a/helpcontent2 b/helpcontent2
index a35f8b648fd4..215707177c27 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit a35f8b648fd4bac70bbde0554a2d4051c7633cde
+Subproject commit 215707177c272f9fcf73a79b9150fe677b991716
commit a3af03ec84ff5e8a89ccfa6fb2350eec6f09883c
Author: Noel Grandin 
AuthorDate: Fri Dec 24 12:00:57 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Dec 24 17:57:41 2021 +0100

use comphelper::WeakComponentImplHelper in ChildrenManagerImpl

Change-Id: I24b1f497a77b7ff896060a8f7dacc5d3a5e27ea2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127411
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/svx/source/accessibility/ChildrenManagerImpl.cxx 
b/svx/source/accessibility/ChildrenManagerImpl.cxx
index 2b8f5674b0d9..080d2a204b9d 100644
--- a/svx/source/accessibility/ChildrenManagerImpl.cxx
+++ b/svx/source/accessibility/ChildrenManagerImpl.cxx
@@ -69,10 +69,7 @@ ChildrenManagerImpl::ChildrenManagerImpl (
 const uno::Reference& rxShapeList,
 const AccessibleShapeTreeInfo& rShapeTreeInfo,
 AccessibleContextBase& rContext)
-: ::cppu::WeakComponentImplHelper<
-  css::document::XEventListener,
-  css::view::XSelectionChangeListener>(m_aMutex),
-  mxShapeList (rxShapeList),
+: mxShapeList (rxShapeList),
   mxParent (rxParent),
   maShapeTreeInfo (rShapeTreeInfo),
   mrContext (rContext),
@@ -83,8 +80,7 @@ ChildrenManagerImpl::ChildrenManagerImpl (
 
 ChildrenManagerImpl::~ChildrenManagerImpl()
 {
-DBG_ASSERT (rBHelper.bDisposed || rBHelper.bInDispose,
-"~AccessibleDrawDocumentView: object has not been disposed");
+DBG_ASSERT (m_bDisposed, "~AccessibleDrawDocumentView: object has not been 
disposed");
 }
 
 
@@ -694,7 +690,7 @@ void ChildrenManagerImpl::impl_dispose()
 }
 
 
-void SAL_CALL ChildrenManagerImpl::disposing()
+void ChildrenManagerImpl::disposing()
 {
 impl_dispose();
 }
diff --git a/svx/source/accessibility/ChildrenManagerImpl.hxx 
b/svx/source/accessibility/ChildrenManagerImpl.hxx
index 1956a7650fc4..6cd9a76305b5 100644
--- a/svx/source/accessibility/ChildrenManagerImpl.hxx
+++ b/svx/source/accessibility/ChildrenManagerImpl.hxx
@@ -24,7 +24,7 @@
 #include 
 #include 
 #include 
-#include 
+#include 
 #include 
 #include 
 #include 
@@ -72,8 +72,7 @@ typedef ::std::vector 
ChildDescriptorListType;
 @see ChildrenManager
 */
 class ChildrenManagerImpl final
-:   public cppu::BaseMutex,
-public cppu::WeakComponentImplHelper<
+:   public comphelper::WeakComponentImplHelper<
 css::document::XEventListener,
 css::view::XSelectionChangeListener>,
 public IAccessibleViewForwarderListener,
@@ -316,7 +315,7 @@ private:
 /** This method is called from the component helper base class while
 disposing.
 */
-virtual void SAL_CALL disposing() override;
+virtual void disposing() override;
 
 void impl_dispose();
 


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

2021-12-24 Thread Noel Grandin (via logerrit)
 svx/source/sidebar/PanelFactory.cxx |9 +++--
 1 file changed, 3 insertions(+), 6 deletions(-)

New commits:
commit c55d1affc6299feed36145a01279b641bb60ac57
Author: Noel Grandin 
AuthorDate: Fri Dec 24 13:04:32 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Dec 24 17:57:10 2021 +0100

use comphelper::WeakComponentImplHelper in PanelFactory

Change-Id: I936fd58fdb626d6707cf672f188cd8570269bc44
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127412
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/svx/source/sidebar/PanelFactory.cxx 
b/svx/source/sidebar/PanelFactory.cxx
index b839100fbcf6..f55575d1ab97 100644
--- a/svx/source/sidebar/PanelFactory.cxx
+++ b/svx/source/sidebar/PanelFactory.cxx
@@ -41,8 +41,7 @@
 #include 
 #include 
 #include 
-#include 
-#include 
+#include 
 #include 
 #include 
 #include 
@@ -61,12 +60,11 @@ constexpr OUStringLiteral IMPLEMENTATION_NAME = 
u"org.apache.openoffice.comp.svx
 constexpr OUStringLiteral SERVICE_NAME = u"com.sun.star.ui.UIElementFactory";
 */
 
-typedef ::cppu::WeakComponentImplHelper< css::ui::XUIElementFactory, 
css::lang::XServiceInfo >
+typedef comphelper::WeakComponentImplHelper< css::ui::XUIElementFactory, 
css::lang::XServiceInfo >
 PanelFactoryInterfaceBase;
 
 class PanelFactory
-: private ::cppu::BaseMutex,
-  public PanelFactoryInterfaceBase
+: public PanelFactoryInterfaceBase
 {
 public:
 PanelFactory();
@@ -89,7 +87,6 @@ public:
 };
 
 PanelFactory::PanelFactory()
-: PanelFactoryInterfaceBase(m_aMutex)
 {
 }
 


[Libreoffice-bugs] [Bug 146407] New: Writer - Dysfunction of Diagonal & Criss-Cross Border.

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146407

Bug ID: 146407
   Summary: Writer - Dysfunction of Diagonal & Criss-Cross Border.
   Product: LibreOffice
   Version: 7.3.0.0 beta1+
  Hardware: x86-64 (AMD64)
OS: Windows (All)
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Writer
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: jo3...@jarl.com

Description:
Toolbar icons about Diagonal & Criss-Cross Border seem not to work.

Steps to Reproduce:
1. Make table in Writer.
2. Select some cells.
3. Select "Borders" icon in "Table" toolbar.
4. Select icon of "Diagonal Down Border" or "Diagonal Up Border" or
"Criss-Cross Border".

Actual Results:
Nothing happens.

Expected Results:
Diagonal lines expected to be drawn.


Reproducible: Always


User Profile Reset: Yes



Additional Info:
Version: 7.3.0.0.beta1 (x64) / LibreOffice Community
Build ID: 436f14c25ec1847646b953cf13d0db4f7ca3be57
CPU threads: 8; OS: Windows 10.0 Build 19042; UI render: Skia/Vulkan; VCL: win
Locale: ja-JP (ja_JP); UI: ja-JP
Calc: CL

Version: 7.3.0.0.beta1+ (x64) / LibreOffice Community
Build ID: c25067e46d7d849a584295de365e32c6c7af11bf
CPU threads: 8; OS: Windows 10.0 Build 19042; UI render: Skia/Vulkan; VCL: win
Locale: ja-JP (ja_JP); UI: ja-JP
Calc: CL

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146397] Writer: the text cursor is barely visible

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146397

--- Comment #5 from Paolo Benvenuto  ---
Version: 7.2.2.2 / LibreOffice Community
Build ID: 20(Build:2)
CPU threads: 8; OS: Linux 5.11; UI render: default; VCL: gtk3
Locale: es-DO (it_IT.UTF-8); UI: it-IT
Ubuntu package version: 1:7.2.2~rc2-0ubuntu0.20.04.1~lo1
Calc: threaded

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 105581] [META] Help viewer dialog bugs and enhancements

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=105581

V Stuart Foote  changed:

   What|Removed |Added

 Depends on||146403


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=146403
[Bug 146403] Help or F1 does not open my default browser
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146403] Help or F1 does not open my default browser

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146403

V Stuart Foote  changed:

   What|Removed |Added

  Component|LibreOffice |Documentation
 CC||olivier.hallot@libreoffice.
   ||org

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 120248] Add an option to choose the application, which is used to show the local help

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=120248

V Stuart Foote  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=14
   ||6403

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146403] Help or F1 does not open my default browser

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146403

V Stuart Foote  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=12
   ||0248
 Blocks||105581
 CC||vstuart.fo...@utsa.edu

--- Comment #1 from V Stuart Foote  ---
There is no Java in the on-line or locally installed Help system.

It is javascript and the JRE is not in anyway related.

Since you remain on Windows 8.1, assistance may be limited but you need to
configure your default programs correctly. LibreOffice help system does not set
its browser and follows the os/DE settings.  

There is enhancement bug 120248 to provide that capability, but the issue
probably lays with your system.


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=105581
[Bug 105581] [META] Help viewer dialog bugs and enhancements
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146405] Ok

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146405

aleksandrakorobov...@gmail.com changed:

   What|Removed |Added

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

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146406] Неверное получение сохраненной записи

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146406

aleksandrakorobov...@gmail.com 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] [Bug 146406] New: Неверное получение сохраненной записи

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146406

Bug ID: 146406
   Summary: Неверное получение сохраненной записи
   Product: cppunit
   Version: unspecified
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: General
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: aleksandrakorobov...@gmail.com
CC: markus.mohrh...@googlemail.com

Created attachment 177128
  --> https://bugs.documentfoundation.org/attachment.cgi?id=177128=edit
Неверное получение сохраненной записи "Выполнение лабораторной работы"

После открытия созданной записи, вместо требуемой информации выводится
сообщение «System error».

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146405] New: Ok

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146405

Bug ID: 146405
   Summary: Ok
   Product: LibreOffice Online
   Version: unspecified
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: LibreOffice
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: aleksandrakorobov...@gmail.com

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

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146402] LibreOffice aborts when starting with an empty profile (Windows)

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146402

V Stuart Foote  changed:

   What|Removed |Added

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

--- Comment #3 from V Stuart Foote  ---
Can not reproduce on Windows 10 (21H2) with Intel Iris GPU

Vendor: 0x8086
Device: 0x8a52
API: 1.2.195
Driver: 0.404.1069
DeviceType: integrated
DeviceName: Intel(R) Iris(R) Plus Graphics
Denylisted: no

@Aaron, maybe figure out what AMD GPU details you have (the Skia.log from
release build) and manually add an appropriate stanza to the
"skia_denylist_vulkan.xml" in the <$ORIGIN>\share\skia folder?  Having the deny
list may bypass the SkiaHelper util?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146392] LO build broken due to x509.h

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146392

--- Comment #5 from Caolán McNamara  ---
At the top of each file in xmlsecurity/inc/pch/precompiled_x*hxx there is a
line to update the pch e.g. ./bin/update_pch ... If you run each of the
update_pch suggestions and retry with --enable-pch=full does it make a
difference. If it does you can commit and submit the change as a fix for this.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146233] Wiggling letter when typing certain letter on certain spot. Independent of sidebar. Zoom matters

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146233

--- Comment #7 from Caolán McNamara  ---
its very new, yesterday or the day before:
https://gerrit.libreoffice.org/c/core/+/127089

-- 
You are receiving this mail because:
You are the assignee for the bug.

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

2021-12-24 Thread Jim Raykowski (via logerrit)
 sw/source/uibase/docvw/edtwin.cxx |   13 +
 1 file changed, 9 insertions(+), 4 deletions(-)

New commits:
commit df77dcbbb2ecd3a272a3ea55877a1b1c9fd8ef03
Author: Jim Raykowski 
AuthorDate: Wed Dec 22 23:58:09 2021 -0900
Commit: Xisco Fauli 
CommitDate: Fri Dec 24 15:56:12 2021 +0100

Outline folding: check pointer before use

Change-Id: Ifa79ef5f286da77fdac9464b4e271021107bf34e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127348
Tested-by: Jenkins
Reviewed-by: Jim Raykowski 
(cherry picked from commit 1609bb6ab5e34fc8fff67240e3c49dde93f2d2d7)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127379
Reviewed-by: Xisco Fauli 

diff --git a/sw/source/uibase/docvw/edtwin.cxx 
b/sw/source/uibase/docvw/edtwin.cxx
index 5fea06590626..0fc6e91b594d 100644
--- a/sw/source/uibase/docvw/edtwin.cxx
+++ b/sw/source/uibase/docvw/edtwin.cxx
@@ -3908,10 +3908,15 @@ void SwEditWin::MouseMove(const MouseEvent& _rMEvt)
 SwContentFrame* pContentFrame = 
aSwContentAtPos.aFnd.pNode->GetTextNode()->getLayoutFrame(nullptr);
 if (pContentFrame != m_pSavedOutlineFrame)
 {
-if (m_pSavedOutlineFrame && 
!m_pSavedOutlineFrame->IsInDtor() &&
-
rNds.GetOutLineNds().Seek_Entry(static_cast(m_pSavedOutlineFrame)->GetTextNodeFirst(),
 ) &&
-rSh.GetAttrOutlineContentVisible(nPos))
-
GetFrameControlsManager().RemoveControlsByType(FrameControlType::Outline, 
m_pSavedOutlineFrame);
+if (m_pSavedOutlineFrame && 
!m_pSavedOutlineFrame->IsInDtor())
+{
+SwTextNode* pTextNode =
+
static_cast(m_pSavedOutlineFrame)->GetTextNodeFirst();
+if (pTextNode && 
rNds.GetOutLineNds().Seek_Entry(pTextNode, ) &&
+rSh.GetAttrOutlineContentVisible(nPos))
+GetFrameControlsManager().RemoveControlsByType(
+FrameControlType::Outline, 
m_pSavedOutlineFrame);
+}
 m_pSavedOutlineFrame = pContentFrame;
 }
 // show button


[Libreoffice-bugs] [Bug 106613] UI: Sidebar navigator in Impress/Draw not updated immediately when page/slide is renamed

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=106613

Aron Budea  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=14
   ||6003

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 85905] Allow undocking of Sidebar decks

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=85905

V Stuart Foote  changed:

   What|Removed |Added

   See Also|https://bugs.documentfounda |
   |tion.org/show_bug.cgi?id=14 |
   |6003|

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146116] sticky points move inwards when shape is roated

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146116

Regina Henschel  changed:

   What|Removed |Added

 Ever confirmed|1   |0
 Status|NEEDINFO|UNCONFIRMED

--- Comment #6 from Regina Henschel  ---
(In reply to birnb...@posteo.de from comment #5)
> I just noticed that the error is only observable if rotation is done via the
> properties palette "position and size". 
> 
> No problem if the shape is rotated directly i.e. by selecting it on the
> drawing canvas.

I can still not reproduce the error, neither with the rotation section in the
side bar nor with the rotation tab in the 'Position and Size' dialog.

A problem only on Mac?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146402] LibreOffice aborts when starting with an empty profile (Windows)

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146402

Aron Budea  changed:

   What|Removed |Added

 Attachment #177126|text/x-log  |text/plain
  mime type||

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146402] LibreOffice aborts when starting with an empty profile (Windows)

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146402

Aron Budea  changed:

   What|Removed |Added

   Priority|medium  |high
Summary|LibreOffice aborts when |LibreOffice aborts when
   |starting with an empty  |starting with an empty
   |profile (debug build,   |profile (Windows)
   |Windows)|
   Severity|normal  |major

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146402] LibreOffice aborts when starting with an empty profile (debug build, Windows)

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146402

--- Comment #2 from Aron Budea  ---
Created attachment 177126
  --> https://bugs.documentfoundation.org/attachment.cgi?id=177126=edit
Skia log

Daily (release) build from 12-22 doesn't start at all. So this isn't
debug-only.

-- 
You are receiving this mail because:
You are the assignee for the bug.

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

2021-12-24 Thread Arnaud VERSINI (via logerrit)
 bridges/source/cpp_uno/gcc3_linux_x86-64/rtti.cxx |1 -
 1 file changed, 1 deletion(-)

New commits:
commit 82807f810b3300111820e9b2fa9a82e3e35a313a
Author: Arnaud VERSINI 
AuthorDate: Sun Dec 19 12:34:40 2021 +0100
Commit: Noel Grandin 
CommitDate: Fri Dec 24 14:53:13 2021 +0100

bridges : remove redundant public

Change-Id: I836412674448acb2a047d3d8b4711fa8d0b67257
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127410
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/bridges/source/cpp_uno/gcc3_linux_x86-64/rtti.cxx 
b/bridges/source/cpp_uno/gcc3_linux_x86-64/rtti.cxx
index 242e2a476e23..206446a81d0c 100644
--- a/bridges/source/cpp_uno/gcc3_linux_x86-64/rtti.cxx
+++ b/bridges/source/cpp_uno/gcc3_linux_x86-64/rtti.cxx
@@ -57,7 +57,6 @@ private:
 };
 
 class GeneratedPad: public Generated {
-public:
 public:
 GeneratedPad(std::unique_ptr && pad): pad_(std::move(pad)) {};
 


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

2021-12-24 Thread Noel Grandin (via logerrit)
 xmlsecurity/source/xmlsec/nss/ciphercontext.cxx |6 ++
 xmlsecurity/source/xmlsec/nss/ciphercontext.hxx |4 ++--
 2 files changed, 4 insertions(+), 6 deletions(-)

New commits:
commit b0163975efc6924c154793590a759784ffa77d08
Author: Noel Grandin 
AuthorDate: Thu Dec 23 22:24:27 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Dec 24 14:45:11 2021 +0100

osl::Mutex->std::mutex in OCipherContext

Change-Id: I0a457dc8ddccc0fce42032956aff6d661d1ae80a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127403
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/xmlsecurity/source/xmlsec/nss/ciphercontext.cxx 
b/xmlsecurity/source/xmlsec/nss/ciphercontext.cxx
index c4d1c6113bf6..ff3441249435 100644
--- a/xmlsecurity/source/xmlsec/nss/ciphercontext.cxx
+++ b/xmlsecurity/source/xmlsec/nss/ciphercontext.cxx
@@ -63,8 +63,6 @@ uno::Reference< xml::crypto::XCipherContext > 
OCipherContext::Create( CK_MECHANI
 
 void OCipherContext::Dispose()
 {
-::osl::MutexGuard aGuard( m_aMutex );
-
 if ( m_pContext )
 {
 PK11_DestroyContext( m_pContext, PR_TRUE );
@@ -94,7 +92,7 @@ void OCipherContext::Dispose()
 
 uno::Sequence< ::sal_Int8 > SAL_CALL OCipherContext::convertWithCipherContext( 
const uno::Sequence< ::sal_Int8 >& aData )
 {
-::osl::MutexGuard aGuard( m_aMutex );
+std::unique_lock aGuard( m_aMutex );
 
 if ( m_bBroken )
 throw uno::RuntimeException();
@@ -174,7 +172,7 @@ uno::Sequence< ::sal_Int8 > SAL_CALL 
OCipherContext::convertWithCipherContext( c
 
 uno::Sequence< ::sal_Int8 > SAL_CALL 
OCipherContext::finalizeCipherContextAndDispose()
 {
-::osl::MutexGuard aGuard( m_aMutex );
+std::unique_lock aGuard( m_aMutex );
 
 if ( m_bBroken )
 throw uno::RuntimeException();
diff --git a/xmlsecurity/source/xmlsec/nss/ciphercontext.hxx 
b/xmlsecurity/source/xmlsec/nss/ciphercontext.hxx
index 3ac5df6c7fd0..40b610ef512e 100644
--- a/xmlsecurity/source/xmlsec/nss/ciphercontext.hxx
+++ b/xmlsecurity/source/xmlsec/nss/ciphercontext.hxx
@@ -22,14 +22,14 @@
 #include 
 
 #include 
-#include 
+#include 
 #include 
 #include 
 
 class OCipherContext : public cppu::WeakImplHelper< 
css::xml::crypto::XCipherContext >
 {
 private:
-::osl::Mutex m_aMutex;
+std::mutex m_aMutex;
 
 PK11SlotInfo* m_pSlot;
 PK11SymKey* m_pSymKey;


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

2021-12-24 Thread Noel Grandin (via logerrit)
 accessibility/inc/extended/accessibleeditbrowseboxcell.hxx|9 -
 accessibility/source/extended/accessibleeditbrowseboxcell.cxx |5 ++---
 2 files changed, 6 insertions(+), 8 deletions(-)

New commits:
commit c5a0bcf94403a29490d707ae1878fba3e0423d13
Author: Noel Grandin 
AuthorDate: Fri Dec 24 09:44:18 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Dec 24 14:44:52 2021 +0100

use comphelper::WeakComponentImplHelper in EditBrowseBoxTableCellAccess

Change-Id: I36a1dd4fa488517a1e69d3a1c4b03c850d1a933e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127404
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/accessibility/inc/extended/accessibleeditbrowseboxcell.hxx 
b/accessibility/inc/extended/accessibleeditbrowseboxcell.hxx
index 6d827ac13a05..ab45db70af77 100644
--- a/accessibility/inc/extended/accessibleeditbrowseboxcell.hxx
+++ b/accessibility/inc/extended/accessibleeditbrowseboxcell.hxx
@@ -20,7 +20,7 @@
 
 #include 
 #include 
-#include 
+#include 
 #include 
 
 namespace accessibility
@@ -89,8 +89,7 @@ namespace accessibility
 
 // XAccessible providing an EditBrowseBoxTableCell
 class EditBrowseBoxTableCellAccess final :
-public cppu::BaseMutex,
-public cppu::WeakComponentImplHelper
+public 
comphelper::WeakComponentImplHelper
 {
 css::uno::WeakReference< css::accessibility::XAccessibleContext >
 m_aContext;
@@ -120,8 +119,8 @@ namespace accessibility
 // XAccessible
 virtual css::uno::Reference< css::accessibility::XAccessibleContext > 
SAL_CALL getAccessibleContext(  ) override;
 
-// XComponent/OComponentHelper
-virtual void SAL_CALL disposing() override;
+// XComponent/WeakComponentImplHelper
+virtual void disposing() override;
 
 EditBrowseBoxTableCellAccess( const EditBrowseBoxTableCellAccess& ) = 
delete;
 EditBrowseBoxTableCellAccess& operator=( const 
EditBrowseBoxTableCellAccess& ) = delete;
diff --git a/accessibility/source/extended/accessibleeditbrowseboxcell.cxx 
b/accessibility/source/extended/accessibleeditbrowseboxcell.cxx
index 06165071b621..f9985e48a268 100644
--- a/accessibility/source/extended/accessibleeditbrowseboxcell.cxx
+++ b/accessibility/source/extended/accessibleeditbrowseboxcell.cxx
@@ -183,8 +183,7 @@ namespace accessibility
 const css::uno::Reference< css::accessibility::XAccessible >& 
_rxParent, const css::uno::Reference< css::accessibility::XAccessible >& 
_rxControlAccessible,
 const css::uno::Reference< css::awt::XWindow >& _rxFocusWindow,
 ::vcl::IAccessibleTableProvider& _rBrowseBox, sal_Int32 _nRowPos, 
sal_uInt16 _nColPos )
-:WeakComponentImplHelper( m_aMutex )
-,m_xParent( _rxParent )
+:m_xParent( _rxParent )
 ,m_xControlAccessible( _rxControlAccessible )
 ,m_xFocusWindow( _rxFocusWindow )
 ,m_pBrowseBox( &_rBrowseBox )
@@ -213,7 +212,7 @@ namespace accessibility
 return xMyContext;
 }
 
-void SAL_CALL EditBrowseBoxTableCellAccess::disposing()
+void EditBrowseBoxTableCellAccess::disposing()
 {
 // dispose our context, if it still alive
 css::uno::Reference< XComponent > xMyContext( m_aContext.get(), 
UNO_QUERY );


[Libreoffice-bugs] [Bug 146376] Calc: Wrong font when I open a new csv file

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146376

--- Comment #7 from Pat War  ---
I tried to use the Menu/Help but the Restart in Safe Mode option was not
enabled so eventually I had to use Mac OS Terminal with this code:
/Applications/LibreOffice.app/Contents/MacOS/soffice --safe-mode and reverted
to Normal that way.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146404] New: Number Format Code: Not all color codes are saved

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146404

Bug ID: 146404
   Summary: Number Format Code: Not all color codes are saved
   Product: LibreOffice
   Version: 7.1.6.2 release
  Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: UNCONFIRMED
  Severity: minor
  Priority: medium
 Component: Calc
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: mmuel...@datastreamsolutions.com

1  Description

When using special color codes in number format codes, e.g.
  COLOR23¹
that aren't 1 of the 8 documented² color codes, i.e.
  CYAN, GREEN, BLACK, BLUE, MAGENTA, RED, WHITE, YELLOW
the application will correctly display the color, but after saving and
reloading the document, the color code will no longer be part of the number
format, and no color will be displayed.


2  Steps to reproduce

2.1  Enter "-1234" into a cell.
2.2  Open cell formatting for that cell, and change the number format to
   [COLOR23]0.0
2.3  The number in the cell will now appear green-ish.
2.4  Save the document. Close the document. Re-open the document.
2.5  The number in the cell will appear with the default font color. (NOT
green-ish) If you open the cell formatting, you will see the color code
vanished from the number format code.


3  Is this a bug?

While it is documented that there are only 8 number format code colors², the
application still reacts to input such as COLOR23. This indicates to the user
that this is an undocumented feature. The loss of these settings after
reloading the document is highly unintuitive. So either:
  a) (Documentation okay, Application bug)
 The bug is that the application allows undocumented color codes.
  b) (Documentation bug, Application bug)
 The documentation forgot to mention the COLOR{NN} codes and the
application doesn't properly save them.



4  System parameters

Operating system: Gentoo

Package manager: Portage

Package: app-office/libreoffice 7.1.7.2
- Note: Compiled from source, as opposed to the app-office/libreoffice-bin
package.

Portage flags:
- USE: bluetooth branding cups dbus gtk mariadb -accessibility -base -clang
-coinmp -custom-cflags -debug -eds -firebird -googledrive -gstreamer -java -kde
-ldap -odk -pdfimport -postgres -test -vulkan
- ELIBC: -FreeBSD
- LIBREOFFICE_EXTENSIONS: -nlpsolver -scripting-beanshell -scripting-javascript
-wiki-publisher
- PYTHON_SINGLE_TARGET: "python3_9 -python3_8 -python3_10


5  Footnotes

¹  COLOR23 is a valid Excel number format code color.
² 
https://help.libreoffice.org/7.2/en-US/text/shared/01/05020301.html#hd_id3155994

-- 
You are receiving this mail because:
You are the assignee for the bug.

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

2021-12-24 Thread Noel Grandin (via logerrit)
 svx/inc/sdr/primitive2d/primitivefactory2d.hxx |   13 +
 1 file changed, 5 insertions(+), 8 deletions(-)

New commits:
commit d1e270557a62bfe81f63a72b6802a11eff9e047d
Author: Noel Grandin 
AuthorDate: Fri Dec 24 11:54:21 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Dec 24 13:52:13 2021 +0100

use comphelper::WeakComponentImplHelper in PrimitiveFactory2D

Change-Id: Iead37062710d64705fca2d00404922b3ad3b6f17
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127409
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/svx/inc/sdr/primitive2d/primitivefactory2d.hxx 
b/svx/inc/sdr/primitive2d/primitivefactory2d.hxx
index fd6ded05ab18..1fa65d52e01a 100644
--- a/svx/inc/sdr/primitive2d/primitivefactory2d.hxx
+++ b/svx/inc/sdr/primitive2d/primitivefactory2d.hxx
@@ -21,8 +21,7 @@
 #include 
 #include 
 #include 
-#include 
-#include 
+#include 
 #include 
 #include 
 #include 
@@ -30,17 +29,15 @@
 #include 
 #include 
 
-typedef cppu::WeakComponentImplHelper
+typedef comphelper::WeakComponentImplHelper
 PrimitiveFactory2DImplBase;
 
 // base class for C++ implementation of css::graphic::XPrimitiveFactory2D
-class PrimitiveFactory2D : protected cppu::BaseMutex, public 
PrimitiveFactory2DImplBase
+class PrimitiveFactory2D final : public PrimitiveFactory2DImplBase
 {
 public:
-PrimitiveFactory2D()
-: PrimitiveFactory2DImplBase(m_aMutex)
-{
-}
+PrimitiveFactory2D() {}
 
 // Methods from XPrimitiveFactory2D
 virtual css::uno::Sequence>


[Libreoffice-bugs] [Bug 146116] sticky points move inwards when shape is roated

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146116

--- Comment #5 from birnb...@posteo.de  ---
I just noticed that the error is only observable if rotation is done via the
properties palette "position and size". 

No problem if the shape is rotated directly i.e. by selecting it on the drawing
canvas.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146116] sticky points move inwards when shape is roated

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146116

--- Comment #4 from birnb...@posteo.de  ---
Created attachment 177125
  --> https://bugs.documentfoundation.org/attachment.cgi?id=177125=edit
shape after 10 rotations and 10 rotations back

The right shape was rotated 10 times clockwise and then 10 times counter
clockwise. It can be seen that the positioning error of the sticky point adds
up regardless of the sense of rotation.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146116] sticky points move inwards when shape is roated

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146116

--- Comment #3 from birnb...@posteo.de  ---
Created attachment 177124
  --> https://bugs.documentfoundation.org/attachment.cgi?id=177124=edit
shape after 10 rotations

I rotated the right shape clockwise 10 times, here is the result.

My test machine is a Retina Mac but the same behaviour was observed on a
non-Retina (MacBook Air 2011, OSX 10.13.4)

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 136187] TOC counts extra page in some cases

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=136187

--- Comment #4 from Alexander Polkhovskiy  ---
(In reply to BogdanB from comment #3)
> I think is NOT about TOC.
> I have simplified the document. And this simple document it is showing in
> the bottom Page 1 of 3 (Page 1 of 2 to print). Somewhere there is a page 3.
> I think this is the problem and not the TOC.
> 
> Or maybe the problem is that the Page break is inserted with Page Position
> "Before".

After you have wrote it, I noticed, that "somewhere there is a page" 2, not 3.
And it is the page after 1 and before 3 - even page 2, that isn't visible. If
you change Page break's start Page number to any even number, everything works
fine.
And there is a bug about even virtual page automatically added and counted by
"Show page number" variable if you break an odd page. But this time "Page"
field and TOC count pages differently - and this happens only in particular
case.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 72043] FORMATTING: Extra page breaks in docx from iOS Pages

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=72043

--- Comment #37 from jasoneck  ---
When creating your PPT poster, you will need to pay close attention to its
format. By failing to arrange the format in an appropriate way, all the efforts
you have put into writing your paper were just a waste of time
https://gold-essays.com/pay-someone-to-make-my-ppt-poster.html

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146391] hlookup: when the search criteria contains a number, the search result will be NV

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146391

--- Comment #2 from rupert.graspeunt...@gmail.com ---
Created attachment 177123
  --> https://bugs.documentfoundation.org/attachment.cgi?id=177123=edit
the error can be reproduced on tables "bezeichnung" and "Budget"

to vary the occurrence of the error, in Table "Bezeichnung" modify Cells A23,
A24, A25
you can see the result in Table "Budget",Cells e.g. e23, e24, e25

If you add in Table "Bezeichnung", Cell A21 a space as the first character,
Table "Budget",Cells e.g. e23, e24, e25 show correct results

-- 
You are receiving this mail because:
You are the assignee for the bug.

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

2021-12-24 Thread Noel Grandin (via logerrit)
 unotools/source/ucbhelper/ucblockbytes.cxx|   56 +-
 unotools/source/ucbhelper/ucblockbytes.hxx|   16 +++
 unotools/source/ucbhelper/ucbstreamhelper.cxx |4 -
 3 files changed, 38 insertions(+), 38 deletions(-)

New commits:
commit 166e2a09052b8adb0525cc3a50b8b7cac0bbe6f1
Author: Noel Grandin 
AuthorDate: Thu Dec 23 19:56:21 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Dec 24 12:45:37 2021 +0100

rename some methods on UcbLockBytes

The Impl suffix means internal methods, but these are part
of the public API of the class

Change-Id: I9c969f3464e0ae78d52f997643f7fb8489b7bd61
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127401
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/unotools/source/ucbhelper/ucblockbytes.cxx 
b/unotools/source/ucbhelper/ucblockbytes.cxx
index da0118a6d60d..2d716f571586 100644
--- a/unotools/source/ucbhelper/ucblockbytes.cxx
+++ b/unotools/source/ucbhelper/ucblockbytes.cxx
@@ -88,13 +88,13 @@ public:
 virtual void SAL_CALL   removeListener ( const Reference 
&/*rxListener*/) override {}
 virtual void SAL_CALL   start() override {}
 virtual void SAL_CALL   terminate() override
-{ m_xLockBytes->terminate_Impl(); }
+{ m_xLockBytes->terminate(); }
 
 // XActiveDataSink.
 virtual void SAL_CALL   setInputStream ( const Reference 
) override
-{ m_xLockBytes->setInputStream_Impl 
(rxInputStream); }
+{ m_xLockBytes->setInputStream(rxInputStream); }
 virtual Reference SAL_CALL getInputStream() override
-{ return m_xLockBytes->getInputStream_Impl(); }
+{ return m_xLockBytes->getInputStream(); }
 };
 
 /**
@@ -115,11 +115,11 @@ public:
 virtual void SAL_CALL   removeListener ( const Reference 
&/*rxListener*/) override {}
 virtual void SAL_CALL   start() override {}
 virtual void SAL_CALL   terminate() override
-{ m_xLockBytes->terminate_Impl(); }
+{ m_xLockBytes->terminate(); }
 
 // XActiveDataStreamer
 virtual void SAL_CALL   setStream( const Reference< XStream >& aStream ) 
override
-{ m_xStream = aStream; 
m_xLockBytes->setStream_Impl( aStream ); }
+{ m_xStream = aStream; m_xLockBytes->setStream( 
aStream ); }
 virtual Reference< XStream > SAL_CALL getStream() override
 { return m_xStream; }
 };
@@ -170,7 +170,7 @@ void SAL_CALL 
UcbPropertiesChangeListener_Impl::propertiesChange ( const Sequenc
 {
 if (rPropChangeEvent.PropertyName == "DocumentHeader")
 {
-m_xLockBytes->SetStreamValid_Impl();
+m_xLockBytes->SetStreamValid();
 }
 }
 }
@@ -655,7 +655,7 @@ static bool UCBOpenContentSync(
 
 if ( !aScheme.equalsIgnoreAsciiCase( "http" ) &&
  !aScheme.equalsIgnoreAsciiCase( "https" ) )
-xLockBytes->SetStreamValid_Impl();
+xLockBytes->SetStreamValid();
 
 Reference< XPropertiesChangeListener > xListener;
 Reference< XPropertiesChangeNotifier > xProps(xContent,UNO_QUERY);
@@ -861,7 +861,7 @@ static bool UCBOpenContentSync_(
 // http protocol must be handled in a special way: during the opening 
process the input stream may change
 // only the last inputstream after notifying the document headers is valid
 if ( !aScheme.equalsIgnoreAsciiCase("http") )
-xLockBytes->SetStreamValid_Impl();
+xLockBytes->SetStreamValid();
 
 Reference< XPropertiesChangeListener > xListener = new 
UcbPropertiesChangeListener_Impl( xLockBytes );
 Reference< XPropertiesChangeNotifier > xProps ( xContent, UNO_QUERY );
@@ -979,23 +979,23 @@ Reference < XInputStream > UcbLockBytes::getInputStream()
 return m_xInputStream;
 }
 
-void UcbLockBytes::setStream_Impl( const Reference& aStream )
+void UcbLockBytes::setStream( const Reference& aStream )
 {
 osl::MutexGuard aGuard( m_aMutex );
 if ( aStream.is() )
 {
 m_xOutputStream = aStream->getOutputStream();
-setInputStream_Impl( aStream->getInputStream(), false );
+setInputStream( aStream->getInputStream(), false );
 m_xSeekable.set( aStream, UNO_QUERY );
 }
 else
 {
 m_xOutputStream.clear();
-setInputStream_Impl( Reference < XInputStream >() );
+setInputStream( Reference < XInputStream >() );
 }
 }
 
-bool UcbLockBytes::setInputStream_Impl( const Reference 
, bool bSetXSeekable )
+bool UcbLockBytes::setInputStream( const Reference 
, bool bSetXSeekable )
 {
 bool bRet = false;
 
@@ -1034,14 +1034,14 @@ bool UcbLockBytes::setInputStream_Impl( const 
Reference 
 return bRet;
 }
 
-void UcbLockBytes::SetStreamValid_Impl()
+void UcbLockBytes::SetStreamValid()
 {
 m_bStreamValid = true;
 if ( 

LibreOffice Tree with columns

2021-12-24 Thread Анна Чернова
Hello!
Please, help me find information, if you can.

I am writing an extension for LibreOffifce.
A tree with columns on my sidebar is needed. (example -
https://doc.qt.io/qt-5/qtwidgets-itemviews-simpletreemodel-example.html)

I found information about Tree Controls and module "tree", e.g. here
https://wiki.openoffice.org/wiki/Treecontrol
https://www.openoffice.org/api/docs/common/ref/com/sun/star/awt/tree/module-ix.html

But I couldn't find anything about writing a *tree with columns*.

There is a quote "You can provide your *own model* which must at least
support the interface com.sun.star.awt.XTreeModel." in the  article "Tree
control", but I also couldn't find any information about providing of my
own models...


[Libreoffice-bugs] [Bug 146382] Can’t open file from Explorer-attached remote location

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146382

Noel Grandin  changed:

   What|Removed |Added

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

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 117073] [META] WebDAV bugs

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=117073
Bug 117073 depends on bug 146382, which changed state.

Bug 146382 Summary: Can’t open file from Explorer-attached remote location
https://bugs.documentfoundation.org/show_bug.cgi?id=146382

   What|Removed |Added

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

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146392] LO build broken due to x509.h

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146392

--- Comment #4 from Roland Kurmann  ---
Building with only --enable-pch is successful.

My commands:

make clean
./autogen.sh CC=clang CXX=clang++ --enable-pch --without-doxygen
--without-system-libxml --without-krb5 --disable-randr --disable-gtk3
--disable-gstreamer-1-0 --without-gssapi --with-theme=breeze --enable-dbgutil
make

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146078] Text starts wiggling/ changing kerning after auto-hyperlink creation (RSID)

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146078

Telesto  changed:

   What|Removed |Added

 CC||caol...@redhat.com

--- Comment #13 from Telesto  ---
@Caolan
You're currently in the possession of active in depth knowledge of glyph
positioning (bug 144862). So you might have some clue what's going on here. 

The wiggle is visually exactly the same as bug 144862, but well a different
trigger. 

I can't tell if commit 6db39dbd7378351f6476f6db25eb7110c9cfb291 being
incomplete/broken (so shifting being caused by multiple text portions) or
something else is intervening (like 6db39dbd7378351f6476f6db25eb7110c9cfb291)

To get things straight: no expectations for a fix nor an implicit requesting to
spend lots of time this. 

My primarily intension to get to the bottom of the problem. Is the primarily
problem in the glyph positioning code or RSID creating text portions (or combo
of both) 

I obviously have to admit this topic being one of my pets. I dislike
dancing/wiggling glyphs /unstable kerning (and there are more people likely me:
bug 103322). And skeptical about floating point being the silver bullet

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146382] Can’t open file from Explorer-attached remote location

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146382

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

https://git.libreoffice.org/core/commit/15bc49228997a5da556961b923bb607642b6

tdf#146382 cant open file from Explorer-attached remote location

It will be available in 7.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] [Bug 146382] Can’t open file from Explorer-attached remote location

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146382

Commit Notification  changed:

   What|Removed |Added

 Whiteboard||target:7.4.0

-- 
You are receiving this mail because:
You are the assignee for the bug.

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

2021-12-24 Thread Noel Grandin (via logerrit)
 tools/qa/cppunit/test_urlobj.cxx |9 +
 tools/source/fsys/urlobj.cxx |3 +++
 2 files changed, 12 insertions(+)

New commits:
commit 15bc49228997a5da556961b923bb607642b6
Author: Noel Grandin 
AuthorDate: Fri Dec 24 09:35:44 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Dec 24 11:27:39 2021 +0100

tdf#146382 cant open file from Explorer-attached remote location

problem spotted by mikekaganski

regression from
commit 0b46361ef84a61100a0274a007062317607d097a
Author: Noel Grandin 
Date:   Sat Nov 20 08:03:49 2021 +0200
tdf#133835 speedup calc autofilter (4)

where I changed the behaviour of parseHostOrNetBiosName from overwriting
the destination buffer to appending to it.

Change-Id: I1bd3c7eade035a867c026364f5e114c1e652ca32
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127352
Tested-by: Gabor Kelemen 
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/tools/qa/cppunit/test_urlobj.cxx b/tools/qa/cppunit/test_urlobj.cxx
index a335701c5cab..ec64b5d66777 100644
--- a/tools/qa/cppunit/test_urlobj.cxx
+++ b/tools/qa/cppunit/test_urlobj.cxx
@@ -311,6 +311,14 @@ namespace tools_urlobj
 obj.GetMainURL(INetURLObject::DecodeMechanism::NONE));
 }
 
+void testTd146382() {
+INetURLObject 
obj("file://share.allotropia.de@SSL/DavWWWRoot/remote.php");
+CPPUNIT_ASSERT(!obj.HasError());
+CPPUNIT_ASSERT_EQUAL(
+
OUString("file://share.allotropia.de@SSL/DavWWWRoot/remote.php"),
+obj.GetMainURL(INetURLObject::DecodeMechanism::NONE));
+}
+
 // Change the following lines only, if you add, remove or rename
 // member functions of the current class,
 // because these macros are need by auto register mechanism.
@@ -326,6 +334,7 @@ namespace tools_urlobj
 CPPUNIT_TEST( testSetName );
 CPPUNIT_TEST( testSetExtension );
 CPPUNIT_TEST( testChangeScheme );
+CPPUNIT_TEST( testTd146382 );
 CPPUNIT_TEST_SUITE_END(  );
 };  // class createPool
 
diff --git a/tools/source/fsys/urlobj.cxx b/tools/source/fsys/urlobj.cxx
index 8c3d2845eab2..7b86fe0f5261 100644
--- a/tools/source/fsys/urlobj.cxx
+++ b/tools/source/fsys/urlobj.cxx
@@ -2880,7 +2880,10 @@ bool INetURLObject::parseHostOrNetBiosName(
 }
 }
 if (pCanonic)
+{
+pCanonic->setLength(nOriginalCanonicLength);
 pCanonic->append(buf);
+}
 }
 else
 {


[Libreoffice-bugs] [Bug 146393] Switch language layout crashes Libreoffice

2021-12-24 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146393

Jan-Marek Glogowski  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEEDINFO
 Ever confirmed|0   |1

--- Comment #5 from Jan-Marek Glogowski  ---
Please still copy the "About LO" information to the bug report as Mike
described in comment 4; that's faster to check then the image.

Then tell us what DE or window manager you are using and describe the steps to
reproduce (STR) you do to "switching layout". 

Neither switching the direct layout nor the IM setting crashes LO master here
for gen/x11 running in KDE. And actually only the Qt / kf5 or the MS Windows
VCL plugin send a SalEvent::InputLanguageChange event, and - according to the
screenshot - you're using neither of them. It might be a bug in the X11 IM
handling.

While a crash backtrace would be nice, without the STR it might be very hard /
impossible to fix.

Info needed:
- copied "About LO" data
- DE or window manager setup info
- STR
- a crash backtrace (optional)

I've changed the report status to NEEDINFO. When you provide the missing
information, please turn it back to UNCONFIRMED.

-- 
You are receiving this mail because:
You are the assignee for the bug.

  1   2   >