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

2021-03-19 Thread Miklos Vajna (via logerrit)
 sw/source/core/inc/UndoTable.hxx |2 +-
 sw/source/core/undo/untbl.cxx|8 ++--
 2 files changed, 7 insertions(+), 3 deletions(-)

New commits:
commit e782937228479054336b564a4c34697a12530469
Author: Miklos Vajna 
AuthorDate: Thu Mar 18 20:59:36 2021 +0100
Commit: Miklos Vajna 
CommitDate: Fri Mar 19 08:08:14 2021 +0100

Related: tdf#133933 sw: fix assertion failure in the SwIndexReg dtor

Make sure that by the time we delete a text node, no SwIndexes point to
us.

Change-Id: I9248a2ebe529e5a0aab0e37bf676e669b9bddb1d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112698
Tested-by: Jenkins
Reviewed-by: Miklos Vajna 

diff --git a/sw/source/core/inc/UndoTable.hxx b/sw/source/core/inc/UndoTable.hxx
index 15bb1dc85bea..472f9fdd65d5 100644
--- a/sw/source/core/inc/UndoTable.hxx
+++ b/sw/source/core/inc/UndoTable.hxx
@@ -273,7 +273,7 @@ class SwUndoTableCpyTable : public SwUndo
 //b6341295: When redlining is active, PrepareRedline has to create the
 //redlining attributes for the new and the old table cell content
 static std::unique_ptr PrepareRedline( SwDoc* pDoc, const 
SwTableBox& rBox,
-const SwPosition& rPos, bool& rJoin, bool bRedo );
+SwPosition& rPos, bool& rJoin, bool bRedo );
 
 public:
 SwUndoTableCpyTable(const SwDoc& rDoc);
diff --git a/sw/source/core/undo/untbl.cxx b/sw/source/core/undo/untbl.cxx
index b15a8db09331..fdc8b25b0631 100644
--- a/sw/source/core/undo/untbl.cxx
+++ b/sw/source/core/undo/untbl.cxx
@@ -2493,7 +2493,7 @@ void SwUndoTableCpyTable::RedoImpl(::sw::UndoRedoContext 
& rContext)
 // Otherwise aInsIdx has been moved during the Undo operation
 if( pEntry->bJoin )
 {
-SwPaM const& rLastPam =
+SwPaM& rLastPam =
 rContext.GetCursorSupplier().GetCurrentShellCursor();
 pUndo = PrepareRedline( &rDoc, rBox, *rLastPam.GetPoint(),
 pEntry->bJoin, true );
@@ -2608,7 +2608,7 @@ void SwUndoTableCpyTable::AddBoxAfter( const SwTableBox& 
rBox, const SwNodeIndex
 // rJoin is true if Redo() is calling and the content has already been merged
 
 std::unique_ptr SwUndoTableCpyTable::PrepareRedline( SwDoc* pDoc, 
const SwTableBox& rBox,
-const SwPosition& rPos, bool& rJoin, bool bRedo )
+SwPosition& rPos, bool& rJoin, bool bRedo )
 {
 std::unique_ptr pUndo;
 // b62341295: Redline for copying tables
@@ -2632,6 +2632,10 @@ std::unique_ptr 
SwUndoTableCpyTable::PrepareRedline( SwDoc* pDoc, const
 if( !bRedo && rPos.nNode.GetNode().GetTextNode() )
 {   // Try to merge, if not called by Redo()
 rJoin = true;
+
+// Park this somewhere else so nothing points to the 
to-be-deleted node.
+rPos.nContent.Assign(pText, 0);
+
 pText->JoinNext();
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: oox import/export of custom shape extrusion

2021-03-19 Thread Miklos Vajna
Hi Regina,

On Thu, Mar 18, 2021 at 04:24:38PM +0100, Regina Henschel 
 wrote:
> I want to work on getting oox import/export of custom shape extrusion.

Great, much appreciated.

> In case a grabBag exists, it will be necessary to merge values from the
> internal "extrusion" property with values from the grabBag on export. In
> case no grabBag exists, writing OOXML from the internal properties is
> needed. In case no internal "extrusion" property exists, the current
> solution of writing the gragBag is OK.
> 
> Should I use separate solutions for the three cases? Advantage would be,
> that the development can be done without interfering the current solution
> for "only grabBag". But a solution for "merge" would be able to solve the
> situation "no grabBag" and "only grabBag" too, so doing only that might
> avoid code duplication.

The motivation bethind grab-bag is to preserve properties from OOXML,
which is not handled by libreoffice core. So in case there is a grab-bag
attached to a shape and you use the UI to modify the shape, it's
perfectly valid to just empty the grab-bag. This is predictable behavior
and you will never have to handle a mix of real document model and
grab-bag for the same shape.

As an example, we store smartart data in the shape grab-bag. But once
you edit the shape text, we empty the grab-bag, so your text edits are
not lost on save (since commit
a01ae07740e5c311fcc37f2ac2e2a0a2a1935920).

Does that help?

Regards,

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


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

2021-03-19 Thread Bartosz Kosiorek (via logerrit)
 emfio/qa/cppunit/emf/EmfImportTest.cxx |   30 +++---
 emfio/source/reader/emfreader.cxx  |   17 ++---
 2 files changed, 21 insertions(+), 26 deletions(-)

New commits:
commit 54a8aefa3372dce3da7cf0ad846d55d1b7b92467
Author: Bartosz Kosiorek 
AuthorDate: Wed Mar 17 18:16:04 2021 +0100
Commit: Bartosz Kosiorek 
CommitDate: Fri Mar 19 09:00:11 2021 +0100

tdf#35986 tdf#140271 EMF Fix line width of CREATEPEN record

According to [MS-EMF] documentation:
"If the pen type in the PenStyle field is
PS_COSMETIC, this value MUST be 0x0001."

Unfortunately based on observation of EMF import,
it seems that it is not true. As a result the implementation
must be partially reversed.

Change-Id: I0c2ec5e26b710e1a12d5196b6c8be4709f26dc4f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112651
Tested-by: Jenkins
Reviewed-by: Bartosz Kosiorek 

diff --git a/emfio/qa/cppunit/emf/EmfImportTest.cxx 
b/emfio/qa/cppunit/emf/EmfImportTest.cxx
index 85450a2e67a3..f587135e1b1e 100644
--- a/emfio/qa/cppunit/emf/EmfImportTest.cxx
+++ b/emfio/qa/cppunit/emf/EmfImportTest.cxx
@@ -312,7 +312,7 @@ void Test::TestEnglishMapMode()
 void Test::TestCreatePen()
 {
 // Check import of EMF image with records: RESTOREDC, SAVEDC, MOVETOEX, 
LINETO, POLYLINE16, EXTTEXTOUTW with DxBuffer
-// The CREATEPEN record is used with PS_COSMETIC line style, which will be 
displayed as solid hairline
+// The CREATEPEN record is used with PS_COSMETIC line style, which 
sometimes will be displayed as solid hairline
 Primitive2DSequence aSequence = 
parseEmf(u"/emfio/qa/cppunit/emf/data/TestCreatePen.emf");
 CPPUNIT_ASSERT_EQUAL(1, static_cast(aSequence.getLength()));
 drawinglayer::Primitive2dXmlDump dumper;
@@ -321,24 +321,24 @@ void Test::TestCreatePen()
 
 assertXPath(pDocument, "/primitive2D/metafile/transform/mask/polypolygon", 
"path", "m0 0h31250v18192h-31250z");
 
-assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke", 3);
-assertXPathContent(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[1]/polygon", "17898,5693 
20172,5693");
-assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[1]/line", "color", 
"#008000");
-assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[1]/line", "width", "3");
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke", 748);
+assertXPathContent(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[1]/polygon", "27875,16523 
27875,1453");
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[1]/line", "color", 
"#ff");
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[1]/line", "width", "6");
 
-assertXPathContent(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[2]/polygon", "17898,6959 
20172,6959");
-assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[2]/line", "color", 
"#80");
-assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[2]/line", "width", "3");
+assertXPathContent(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[2]/polygon", "27975,16453 
27875,16453");
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[2]/line", "color", 
"#ff");
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[2]/line", "width", "6");
 
-assertXPathContent(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[3]/polygon", "17898,7381 
20172,7381");
+assertXPathContent(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[3]/polygon", "27925,16078 
27875,16078");
 assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[3]/line", "color", 
"#ff");
-assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[3]/line", "width", "3");
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonstroke[3]/line", "width", "6");
 
-assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonhairline", 755);
-assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonhairline[10]", "color", "#ff");
-assertXPathContent(pDocument, 
"/primitive2D/metafile/transform/mask/polygonhairline[10]/polygon", 
"27925,14180 27875,14180");
-assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonhairline[100]", "color", 
"#008000");
-assertXPathContent(pDocument, 
"/primitive2D/metafile/transform/mask/polygonhairline[100]/polygon", 
"26100,14414 26050,14414");
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonhairline", 10);
+assertXPath(pDocument, 
"/primitive2D/metafile/transform/mask/polygonhairline[5]", "color", "#008000");
+assertXPathContent(pDocument, 
"/primitive2D/metafile/transform/mask/p

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

2021-03-19 Thread Miklos Vajna (via logerrit)
 sw/CppunitTest_sw_odfimport.mk |2 
 sw/qa/extras/odfimport/data/vmerge-cell-border.odt |binary
 sw/qa/extras/odfimport/odfimport.cxx   |   24 
 sw/source/filter/xml/xmltbli.cxx   |   58 -
 sw/source/filter/xml/xmltbli.hxx   |   10 +++
 5 files changed, 93 insertions(+), 1 deletion(-)

New commits:
commit 1001dbaef4dec2b51c25ed8343bab6910f1219e1
Author: Miklos Vajna 
AuthorDate: Fri Mar 19 08:42:35 2021 +0100
Commit: Miklos Vajna 
CommitDate: Fri Mar 19 09:41:10 2021 +0100

ODT import: handle style name of covered cells

Similar to commit 66ac8e60896f6306bed8fbb34606fd14474f19ce (sw: fix
unwanted long vertical border around vertically merged Word cell,
2021-03-04), but that one was for the DOCX import / rendering.

Change-Id: I394a4f062544a9774b9b40ec757cb37e72220561
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112696
Reviewed-by: Miklos Vajna 
Tested-by: Jenkins

diff --git a/sw/CppunitTest_sw_odfimport.mk b/sw/CppunitTest_sw_odfimport.mk
index 61a1547dd473..dabab5254279 100644
--- a/sw/CppunitTest_sw_odfimport.mk
+++ b/sw/CppunitTest_sw_odfimport.mk
@@ -19,11 +19,13 @@ $(eval $(call 
gb_CppunitTest_add_exception_objects,sw_odfimport, \
 
 $(eval $(call gb_CppunitTest_use_libraries,sw_odfimport, \
 comphelper \
+editeng \
 cppu \
 cppuhelper \
 sal \
svt \
sfx \
+   svl \
sw \
swqahelper \
 test \
diff --git a/sw/qa/extras/odfimport/data/vmerge-cell-border.odt 
b/sw/qa/extras/odfimport/data/vmerge-cell-border.odt
new file mode 100644
index ..bf387bcb183e
Binary files /dev/null and b/sw/qa/extras/odfimport/data/vmerge-cell-border.odt 
differ
diff --git a/sw/qa/extras/odfimport/odfimport.cxx 
b/sw/qa/extras/odfimport/odfimport.cxx
index 6ab87d07a3de..ac7e34a9ebdd 100644
--- a/sw/qa/extras/odfimport/odfimport.cxx
+++ b/sw/qa/extras/odfimport/odfimport.cxx
@@ -43,6 +43,7 @@
 #include 
 
 #include 
+#include 
 
 #include 
 #include 
@@ -54,6 +55,7 @@
 #include 
 #include 
 #include 
+#include 
 
 typedef std::map > 
AllBordersMap;
 typedef std::pair > 
StringSequencePair;
@@ -1158,5 +1160,27 @@ CPPUNIT_TEST_FIXTURE(Test, 
testPasteFirstParaDirectFormat)
  getProperty(xParagraph, "ParaAdjust"));
 }
 
+CPPUNIT_TEST_FIXTURE(Test, testVerticallyMergedCellBorder)
+{
+// Given a document with two cells, vertically merged, when loading the 
document:
+SwDoc* pDoc = createSwDoc(mpTestDocumentPath, "vmerge-cell-border.odt");
+
+// Then make sure that the first cell has a right border while the second 
has no right border:
+SwDocShell* pDocShell = pDoc->GetDocShell();
+SwWrtShell* pWrtShell = pDocShell->GetWrtShell();
+pWrtShell->Down(/*bSelect=*/false, /*nCount=*/1);
+SwShellCursor* pShellCursor = pWrtShell->getShellCursor(/*bBlock=*/false);
+SwStartNode* pA1 = 
pShellCursor->Start()->nNode.GetNode().StartOfSectionNode();
+const SwAttrSet& rA1Set = 
pA1->GetTableBox()->GetFrameFormat()->GetAttrSet();
+CPPUNIT_ASSERT(rA1Set.GetBox().GetRight());
+SwNodeIndex aA2(*pA1->EndOfSectionNode(), 1);
+const SwAttrSet& rA2Set = 
aA2.GetNode().GetTableBox()->GetFrameFormat()->GetAttrSet();
+
+// Without the accompanying fix in place, this test would have failed, as 
the A2 cell also had a
+// right border, even if  
explicitly disabled
+// it.
+CPPUNIT_ASSERT(!rA2Set.GetBox().GetRight());
+}
+
 CPPUNIT_PLUGIN_IMPLEMENT();
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sw/source/filter/xml/xmltbli.cxx b/sw/source/filter/xml/xmltbli.cxx
index e1deda38e2fa..041784f15ef7 100644
--- a/sw/source/filter/xml/xmltbli.cxx
+++ b/sw/source/filter/xml/xmltbli.cxx
@@ -127,6 +127,7 @@ public:
 sal_uInt32 GetRowSpan() const { return nRowSpan; }
 void SetRowSpan( sal_uInt32 nSet ) { nRowSpan = nSet; }
 sal_uInt32 GetColSpan() const { return nColSpan; }
+void SetStyleName(const OUString& rStyleName) { aStyleName = rStyleName; }
 const OUString& GetStyleName() const { return aStyleName; }
 const OUString& GetFormula() const { return sFormula; }
 double GetValue() const { return dValue; }
@@ -333,6 +334,36 @@ public:
 SwXMLImport& GetSwImport() { return 
static_cast(GetImport()); }
 };
 
+/// Handles .
+class SwXMLCoveredTableCellContext : public SvXMLImportContext
+{
+public:
+SwXMLCoveredTableCellContext(SwXMLImport& rImport,
+ const 
Reference& xAttrList,
+ SwXMLTableContext& rTable);
+};
+
+SwXMLCoveredTableCellContext::SwXMLCoveredTableCellContext(
+SwXMLImport& rImport, const Reference& 
xAttrList,
+SwXMLTableContext& rTable)
+: SvXMLImportContext(rImport)
+{
+OUString aStyleName;
+for (auto& rIter : sax_fastparser::castToFastAttributeList(xAttrList))
+{
+switch (rIter.getToken())
+

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

2021-03-19 Thread Michael Stahl (via logerrit)
 include/svl/sigstruct.hxx |   32 +
 svl/source/crypto/cryptosign.cxx  |   16 
 sw/source/core/edit/edfcol.cxx|3 
 xmlsecurity/inc/biginteger.hxx|3 
 xmlsecurity/inc/xmlsignaturehelper.hxx|   12 
 xmlsecurity/inc/xsecctl.hxx   |   14 
 xmlsecurity/source/component/documentdigitalsignatures.cxx|   54 +--
 xmlsecurity/source/dialogs/digitalsignaturesdialog.cxx|   15 
 xmlsecurity/source/helper/documentsignaturehelper.cxx |   63 ++-
 xmlsecurity/source/helper/documentsignaturemanager.cxx|   12 
 xmlsecurity/source/helper/ooxmlsecexporter.cxx|   22 -
 xmlsecurity/source/helper/ooxmlsecparser.cxx  |   22 +
 xmlsecurity/source/helper/pdfsignaturehelper.cxx  |8 
 xmlsecurity/source/helper/xmlsignaturehelper.cxx  |  161 
++
 xmlsecurity/source/helper/xsecctl.cxx |   80 ++--
 xmlsecurity/source/helper/xsecparser.cxx  |  144 

 xmlsecurity/source/helper/xsecsign.cxx|   30 +
 xmlsecurity/source/helper/xsecverify.cxx  |  124 
++-
 xmlsecurity/source/xmlsec/mscrypt/x509certificate_mscryptimpl.cxx |   47 ++
 xmlsecurity/source/xmlsec/mscrypt/xmlsignature_mscryptimpl.cxx|2 
 xmlsecurity/source/xmlsec/nss/x509certificate_nssimpl.cxx |   25 +
 xmlsecurity/source/xmlsec/nss/xmlsignature_nssimpl.cxx|3 
 22 files changed, 678 insertions(+), 214 deletions(-)

New commits:
commit a1cf770c2d7ca3e153e0b1f01ddcc313bc2bed7f
Author: Michael Stahl 
AuthorDate: Thu Feb 25 14:17:48 2021 +0100
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 09:56:51 2021 +0100

xmlsecurity: improve handling of multiple X509Data elements

Combine everything related to a certificate in a new struct X509Data.

The CertDigest is not actually written in the X509Data element but in
xades:Cert, so try to find the matching entry in
XSecController::setX509CertDigest().

There was a confusing interaction with PGP signatures, where ouGpgKeyID
was used for import, but export wrote the value from ouCertDigest
instead - this needed fixing.

The main point of this is enforcing a constraint from xmldsig-core 4.5.4:

  All certificates appearing in an X509Data element MUST relate to the
  validation key by either containing it or being part of a certification
  chain that terminates in a certificate containing the validation key.

Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111254
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
(cherry picked from commit 9e82509b09f5fe2eb77bcdb8fd193c71923abb67)

xmlsecurity: improve handling of multiple certificates per X509Data

It turns out that an X509Data element can contain an arbitrary number of
each of its child elements.

How exactly certificates of an issuer chain may or should be distributed
across multiple X509Data elements isn't terribly obvious.

One thing that is clear is that any element that refers to or contains
one particular certificate has to be a child of the same X509Data
element, although in no particular order, so try to match the 2 such
elements that the parser supports in XSecController::setX509Data().

Presumably the only way it makes sense to have multiple signing
certificates is if they all contain the same key but are signed by
different CAs. This case isn't handled currently; CheckX509Data() will
complain there's not a single chain and validation of the certificates
will fail.

Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111500
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
(cherry picked from commit 5af5ea893bcb8a8eb472ac11133da10e5a604e66)

xmlsecurity: add EqualDistinguishedNames()

Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111545
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
(cherry picked from commit 1d3da3486d827dd5e7a3bf1c7a533f5aa9860e42)

xmlsecurity: avoid exception in DigitalSignaturesDialog::getCertificate()

Fallback to PGP if there's no X509 signing certificate because
CheckX509Data() failed prevents the dialog from popping up.

To avoid confusing the user in this situation, the dialog should
show no certificate, which is already the case.

Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111664
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
(cherry picked from commit 90b725675c2964f4a151d802d9afedd8bc2ae1a7)

xmlsecurity: fix crash in DocumentDigitalSignatures::isAuthorTrusted()

If the argume

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

2021-03-19 Thread Andreas Heinisch (via logerrit)
 basic/qa/basic_coverage/test_string_replace.vb |6 ++
 basic/source/runtime/methods.cxx   |2 +-
 2 files changed, 7 insertions(+), 1 deletion(-)

New commits:
commit d3f425c85d4664069f5b7e5d380a407631070da7
Author: Andreas Heinisch 
AuthorDate: Mon Mar 15 13:41:58 2021 +0100
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 09:58:23 2021 +0100

tdf#141045 - fixed copy paste error in the replace function

Change-Id: Id68670fed89e4cc700c5eea395139914bebdb657
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112509
Tested-by: Jenkins
Reviewed-by: Andreas Heinisch 
(cherry picked from commit ac0b6fb3842201e438950ea99a55ad334f8521ab)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112512
Reviewed-by: Xisco Fauli 
(cherry picked from commit fde29198bd8e345c9a61a9f4d4671a3022a84cf9)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112520
Reviewed-by: Michael Stahl 
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/basic/qa/basic_coverage/test_string_replace.vb 
b/basic/qa/basic_coverage/test_string_replace.vb
index 99eafdba6b14..e2e9ce35962b 100644
--- a/basic/qa/basic_coverage/test_string_replace.vb
+++ b/basic/qa/basic_coverage/test_string_replace.vb
@@ -23,6 +23,12 @@ Function verify_stringReplace() As String
 retStr = Replace("АБВабв", "б", "*")
 TestLog_ASSERT retStr, "А*Ва*в", "case-insensitive non-ASCII: " & retStr
 
+' tdf#141045 - different length of search and replace string. It is 
important
+' that the search string starts with the original string in order to test 
the error.
+' Without the fix in place, the string index calculations result in a 
crash.
+retStr = Replace("a", "abc", "ab")
+TestLog_ASSERT retStr, "a", "different length of search and replace 
string: " & retStr
+
 result = result & Chr$(10) & "Tests passed: " & passCount & Chr$(10) & 
"Tests failed: " & failCount & Chr$(10)
 verify_stringReplace = result
 End Function
diff --git a/basic/source/runtime/methods.cxx b/basic/source/runtime/methods.cxx
index 1fbcdd9680e2..1a598490d68a 100644
--- a/basic/source/runtime/methods.cxx
+++ b/basic/source/runtime/methods.cxx
@@ -1286,7 +1286,7 @@ void SbRtl_Replace(StarBASIC *, SbxArray & rPar, bool)
 const css::lang::Locale& rLocale = 
Application::GetSettings().GetUILanguageTag().getLocale();
 css::uno::Reference < i18n::XCharacterClassification > xCharClass = 
vcl::unohelper::CreateCharacterClassification();
 aSrcStr = xCharClass->toUpper(aSrcStr, 0, aSrcStr.getLength(), 
rLocale);
-aFindStr = xCharClass->toUpper(aFindStr, 0, aSrcStr.getLength(), 
rLocale);
+aFindStr = xCharClass->toUpper(aFindStr, 0, aFindStr.getLength(), 
rLocale);
 }
 
 // Note: the result starts from lStartPos, removing everything to the 
left. See i#94895.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


-> ESC agenda

2021-03-19 Thread Michael Meeks
Hi Lubos, Julian, all,

I'd really like to encourage you not to pre-emptively give up on the
governance that we do have =)

It seems clear that some people object to this change; and it seems
clear that there was not a full discussion that reflected that at the ESC:

On 18/03/2021 18:17, julien2412 wrote:
> No need to come since it's easier for me to write (no schedule 
> constraints for example) and the migration will be done anyway, no 
> need to worry My goal was just to thank a lot Luboš and to
> provide arguments against the migration. Nevertheless, I already knew
> the battle, battle against this censorship (I weigh my words) was
> lost.

Well - if you feel strongly; I'd encourage you to come and make your
point - or for at least someone to speak out.

From my perspective it is -really- important that people can speak up
and say what they think. Ideally in a constructive and as balanced a way
as possible, avoiding overly emotive or simplified constructions - that
makes it easier for everyone to be heard.

I really don't think there is a foregone conclusion.

On 18/03/2021 20:04, Luboš Luňák wrote:
> in order to make the discussion somewhat more constructive, I have
> an alternative proposal on how to resolve the problem.

I'd love you to come and propose that.

I'm also aware that some people are concerned that this turns into a
much wider and more concerning project than it is ie. the slippery slope
argument.

I'm not so convinced about that - but its easy to avoid that by hearing
Thorsten's bigger-picture plan here.

So - thanks for raising this; I'd really like to end the thread here -
and take it to the next ESC meeting.

Miklos - can you put it on the agenda ?

Thanks,

Michael.

-- 
michael.me...@collabora.com <><, GM Collabora Productivity
Hangout: mejme...@gmail.com, Skype: mmeeks
(M) +44 7795 666 147 - timezone usually UK / Europe
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] core.git: Branch 'libreoffice-7-1' - cui/uiconfig

2021-03-19 Thread Caolán McNamara (via logerrit)
 cui/uiconfig/ui/spellingdialog.ui |   12 
 1 file changed, 12 insertions(+)

New commits:
commit 66319fd36564f55feb9669b3f117176f4ef4c939
Author: Caolán McNamara 
AuthorDate: Tue Mar 16 16:07:24 2021 +
Commit: Michael Stahl 
CommitDate: Fri Mar 19 11:12:58 2021 +0100

tdf#141027 set all buttons to the same max width

so changing from "ignore all" to "ignore rule" makes no
difference, which removes the redraw problem with Russian UI

Change-Id: I27c0e7b00a4f77ac5348aeee64bf312afff423cd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112587
Tested-by: Jenkins
Tested-by: Caolán McNamara 
Reviewed-by: Caolán McNamara 
(cherry picked from commit feb6fd1f4213ef114e63a3dd398c2cba6a5e93ea)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112620
Reviewed-by: Michael Stahl 

diff --git a/cui/uiconfig/ui/spellingdialog.ui 
b/cui/uiconfig/ui/spellingdialog.ui
index a12e3a2b6d96..83a838eb0d6e 100644
--- a/cui/uiconfig/ui/spellingdialog.ui
+++ b/cui/uiconfig/ui/spellingdialog.ui
@@ -651,4 +651,16 @@
   
 
   
+  
+
+  
+  
+  
+  
+  
+  
+  
+  
+
+  
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 svx/uiconfig/ui/sidebarstylespanel.ui |   45 --
 1 file changed, 22 insertions(+), 23 deletions(-)

New commits:
commit 5df473002da1d04d5be12b9223e1e132cae53a62
Author: Caolán McNamara 
AuthorDate: Thu Mar 18 17:24:58 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 11:40:37 2021 +0100

drop vexpand from styles panel

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

diff --git a/svx/uiconfig/ui/sidebarstylespanel.ui 
b/svx/uiconfig/ui/sidebarstylespanel.ui
index 2a411ff8d9e4..9c33e7da3d4f 100644
--- a/svx/uiconfig/ui/sidebarstylespanel.ui
+++ b/svx/uiconfig/ui/sidebarstylespanel.ui
@@ -1,49 +1,48 @@
 
-
+
 
   
   
   
 True
-False
+False
 True
 
   
 True
-False
+False
 True
-True
-6
+6
 6
 
   
 True
-False
+False
 True
 vertical
 
   
 True
-True
+True
 True
-icons
-False
+icons
+False
 
   
 True
-False
+False
 True
 
   
 True
-False
+False
 True
-True
-False
+True
+False
 
   
+True
 True
-True
   
 
   
@@ -71,19 +70,19 @@
 
   
 True
-False
+False
 
   
 True
-True
+True
 3
-icons
-False
+icons
+False
 2
 
   
 True
-False
+False
   
   
 True
@@ -93,7 +92,7 @@
 
   
 True
-False
+False
   
   
 True
@@ -103,7 +102,7 @@
 
   
 True
-False
+False
   
   
 True
@@ -126,8 +125,8 @@
 
   
   
-0
-0
+0
+0
   
 
   
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Johnny_M (via logerrit)
 source/text/scalc/02/0801.xhp  |2 +-
 source/text/scalc/main0208.xhp |2 +-
 source/text/shared/02/0802.xhp |2 +-
 source/text/shared/main0208.xhp|2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 5ad095b0204b6f0eb5aab14d1e4a16fce1ba2632
Author: Johnny_M 
AuthorDate: Fri Mar 19 10:46:42 2021 +0100
Commit: Olivier Hallot 
CommitDate: Fri Mar 19 12:03:11 2021 +0100

tdf#132643 Translate German section IDs

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

diff --git a/source/text/scalc/02/0801.xhp 
b/source/text/scalc/02/0801.xhp
index 3f5bb7ec9..39b648223 100644
--- a/source/text/scalc/02/0801.xhp
+++ b/source/text/scalc/02/0801.xhp
@@ -30,7 +30,7 @@
 
 
 
-
+
 
 Position in 
document
 Displays the number of the current sheet and the total 
number of sheets in the spreadsheet.
diff --git a/source/text/scalc/main0208.xhp b/source/text/scalc/main0208.xhp
index f35ccabc5..f06ede5ff 100644
--- a/source/text/scalc/main0208.xhp
+++ b/source/text/scalc/main0208.xhp
@@ -31,7 +31,7 @@
  Status 
Bar
  The 
Status Bar displays information about the current 
sheet.
   
-  
+  
   
   
   
diff --git a/source/text/shared/02/0802.xhp 
b/source/text/shared/02/0802.xhp
index 170639b1b..c58048d4d 100644
--- a/source/text/shared/02/0802.xhp
+++ b/source/text/shared/02/0802.xhp
@@ -28,7 +28,7 @@
   
 
 
-
+
 
 Position in 
Document
 Displays the current cursor position in the 
%PRODUCTNAME Basic document. The row number is specified, then the 
column number.
diff --git a/source/text/shared/main0208.xhp b/source/text/shared/main0208.xhp
index eb8a479d4..a8d22bdc7 100644
--- a/source/text/shared/main0208.xhp
+++ b/source/text/shared/main0208.xhp
@@ -36,7 +36,7 @@
 
 
 
-
+
 
 
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: helpcontent2

2021-03-19 Thread Johnny_M (via logerrit)
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 48feb32280bb229ff8e493381f9d683df8b8e47d
Author: Johnny_M 
AuthorDate: Fri Mar 19 12:03:11 2021 +0100
Commit: Gerrit Code Review 
CommitDate: Fri Mar 19 12:03:11 2021 +0100

Update git submodules

* Update helpcontent2 from branch 'master'
  to 5ad095b0204b6f0eb5aab14d1e4a16fce1ba2632
  - tdf#132643 Translate German section IDs

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

diff --git a/helpcontent2 b/helpcontent2
index 75d047408148..5ad095b0204b 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 75d047408148b9c914b964017738d3b96b528017
+Subproject commit 5ad095b0204b6f0eb5aab14d1e4a16fce1ba2632
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Johnny_M (via logerrit)
 source/text/shared/02/0801.xhp |2 +-
 source/text/shared/main0208.xhp|2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit a24d2d03172ed1a0f6487c075578bde533bff626
Author: Johnny_M 
AuthorDate: Fri Mar 19 10:48:14 2021 +0100
Commit: Olivier Hallot 
CommitDate: Fri Mar 19 12:03:42 2021 +0100

tdf#132643 Translate German section IDs

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

diff --git a/source/text/shared/02/0801.xhp 
b/source/text/shared/02/0801.xhp
index 2b0fbf56a..9813017a8 100644
--- a/source/text/shared/02/0801.xhp
+++ b/source/text/shared/02/0801.xhp
@@ -28,7 +28,7 @@
   
 
 
-
+
 
 Document 
Information
 Displays information about the 
active %PRODUCTNAME Basic document. The 
names of the document, the library, and the module are displayed, separated by 
dots.
diff --git a/source/text/shared/main0208.xhp b/source/text/shared/main0208.xhp
index a8d22bdc7..be505447d 100644
--- a/source/text/shared/main0208.xhp
+++ b/source/text/shared/main0208.xhp
@@ -34,7 +34,7 @@
 Status Bar in $[officename] Basic Documents
 The 
Status Bar displays information about the current $[officename] 
Basic document.
 
-
+
 
 
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: helpcontent2

2021-03-19 Thread Johnny_M (via logerrit)
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 28f2f50510f6d7e667293a32d3cec46bb34a24f7
Author: Johnny_M 
AuthorDate: Fri Mar 19 12:03:42 2021 +0100
Commit: Gerrit Code Review 
CommitDate: Fri Mar 19 12:03:42 2021 +0100

Update git submodules

* Update helpcontent2 from branch 'master'
  to a24d2d03172ed1a0f6487c075578bde533bff626
  - tdf#132643 Translate German section IDs

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

diff --git a/helpcontent2 b/helpcontent2
index 5ad095b0204b..a24d2d03172e 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 5ad095b0204b6f0eb5aab14d1e4a16fce1ba2632
+Subproject commit a24d2d03172ed1a0f6487c075578bde533bff626
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Johnny_M (via logerrit)
 source/text/shared/01/05030100.xhp |4 ++--
 source/text/shared/02/0311.xhp |4 ++--
 source/text/simpress/main0203.xhp  |4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)

New commits:
commit 43ca714789ab82c7561f5b30a56f77d7bffdf963
Author: Johnny_M 
AuthorDate: Fri Mar 19 10:51:40 2021 +0100
Commit: Olivier Hallot 
CommitDate: Fri Mar 19 12:04:33 2021 +0100

tdf#132643 Translate German section IDs

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

diff --git a/source/text/shared/01/05030100.xhp 
b/source/text/shared/01/05030100.xhp
index f8656562c..814c507c9 100644
--- a/source/text/shared/01/05030100.xhp
+++ b/source/text/shared/01/05030100.xhp
@@ -73,7 +73,7 @@
 
 
 
-
+
 
 
 
@@ -85,7 +85,7 @@
 
 
 
-
+
 
 
 
diff --git a/source/text/shared/02/0311.xhp 
b/source/text/shared/02/0311.xhp
index a59a00aea..f620fd361 100644
--- a/source/text/shared/02/0311.xhp
+++ b/source/text/shared/02/0311.xhp
@@ -28,12 +28,12 @@
   
 
 
-
+
 
 Increase 
Spacing
 Click the Increase Spacing icon to 
increase the paragraph spacing above the selected paragraph.
 
-
+
 
   
 
diff --git a/source/text/simpress/main0203.xhp 
b/source/text/simpress/main0203.xhp
index 26505a43c..4024ed6fa 100644
--- a/source/text/simpress/main0203.xhp
+++ b/source/text/simpress/main0203.xhp
@@ -52,8 +52,8 @@
   
   
   
-  
-  
+  
+  
   
   
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: helpcontent2

2021-03-19 Thread Johnny_M (via logerrit)
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 1ca5ce61c9d91eb77da2e085935fc9291b61bf6e
Author: Johnny_M 
AuthorDate: Fri Mar 19 12:04:33 2021 +0100
Commit: Gerrit Code Review 
CommitDate: Fri Mar 19 12:04:33 2021 +0100

Update git submodules

* Update helpcontent2 from branch 'master'
  to 43ca714789ab82c7561f5b30a56f77d7bffdf963
  - tdf#132643 Translate German section IDs

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

diff --git a/helpcontent2 b/helpcontent2
index a24d2d03172e..43ca714789ab 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit a24d2d03172ed1a0f6487c075578bde533bff626
+Subproject commit 43ca714789ab82c7561f5b30a56f77d7bffdf963
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Johnny_M (via logerrit)
 source/text/scalc/main0205.xhp |4 ++--
 source/text/shared/02/0204.xhp |4 ++--
 source/text/simpress/main0203.xhp  |4 ++--
 source/text/swriter/main0220.xhp   |4 ++--
 4 files changed, 8 insertions(+), 8 deletions(-)

New commits:
commit dd35bdfc9a28343688df30e8617b24dab0620cde
Author: Johnny_M 
AuthorDate: Fri Mar 19 10:56:05 2021 +0100
Commit: Olivier Hallot 
CommitDate: Fri Mar 19 12:05:21 2021 +0100

tdf#132643 Translate German section IDs

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

diff --git a/source/text/scalc/main0205.xhp b/source/text/scalc/main0205.xhp
index 1e2003816..2ce760fd2 100644
--- a/source/text/scalc/main0205.xhp
+++ b/source/text/scalc/main0205.xhp
@@ -82,8 +82,8 @@
 
 
 
-
-
+
+
 
 
 
diff --git a/source/text/shared/02/0204.xhp 
b/source/text/shared/02/0204.xhp
index 12e8b3dbc..9be679f1d 100644
--- a/source/text/shared/02/0204.xhp
+++ b/source/text/shared/02/0204.xhp
@@ -28,12 +28,12 @@
   
 
 
-
+
 
 Text running from left to right
 Specifies the horizontal direction of the 
text.
 
-
+
 
   
 
diff --git a/source/text/simpress/main0203.xhp 
b/source/text/simpress/main0203.xhp
index 4024ed6fa..86b1ee602 100644
--- a/source/text/simpress/main0203.xhp
+++ b/source/text/simpress/main0203.xhp
@@ -129,8 +129,8 @@
 
 
 
-  
-  
+  
+  
   
   
 Increase Font Size
diff --git a/source/text/swriter/main0220.xhp b/source/text/swriter/main0220.xhp
index 12cadcde3..1bd738727 100644
--- a/source/text/swriter/main0220.xhp
+++ b/source/text/swriter/main0220.xhp
@@ -60,8 +60,8 @@
 
 
 
-
-
+
+
 
 
 Select 
All
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: helpcontent2

2021-03-19 Thread Johnny_M (via logerrit)
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 1ff26e4787c9e51fa07277c31989176943d46fd1
Author: Johnny_M 
AuthorDate: Fri Mar 19 12:05:21 2021 +0100
Commit: Gerrit Code Review 
CommitDate: Fri Mar 19 12:05:21 2021 +0100

Update git submodules

* Update helpcontent2 from branch 'master'
  to dd35bdfc9a28343688df30e8617b24dab0620cde
  - tdf#132643 Translate German section IDs

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

diff --git a/helpcontent2 b/helpcontent2
index 43ca714789ab..dd35bdfc9a28 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 43ca714789ab82c7561f5b30a56f77d7bffdf963
+Subproject commit dd35bdfc9a28343688df30e8617b24dab0620cde
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Johnny_M (via logerrit)
 source/text/shared/02/14020200.xhp |4 ++--
 source/text/shared/main0214.xhp|4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

New commits:
commit b4fd7e41094ae6e9ca5bdf5d0c86f594ef58271a
Author: Johnny_M 
AuthorDate: Fri Mar 19 11:02:18 2021 +0100
Commit: Olivier Hallot 
CommitDate: Fri Mar 19 12:06:32 2021 +0100

tdf#132643 Translate German section IDs

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

diff --git a/source/text/shared/02/14020200.xhp 
b/source/text/shared/02/14020200.xhp
index 1d40edf3f..e9d93dace 100644
--- a/source/text/shared/02/14020200.xhp
+++ b/source/text/shared/02/14020200.xhp
@@ -28,12 +28,12 @@
   
 
 
-
+
 
 Switch Design View On/Off
 Displays the design view or the SQL view of the 
query.
 
-
+
 
   
 
diff --git a/source/text/shared/main0214.xhp b/source/text/shared/main0214.xhp
index e80b3f9f2..4073f449c 100644
--- a/source/text/shared/main0214.xhp
+++ b/source/text/shared/main0214.xhp
@@ -39,8 +39,8 @@
 
 
 
-
-
+
+
 Add 
Tables
 
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: helpcontent2

2021-03-19 Thread Johnny_M (via logerrit)
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit cfbc3f49e8dd9322390e67a9742d2cc8a9256671
Author: Johnny_M 
AuthorDate: Fri Mar 19 12:06:32 2021 +0100
Commit: Gerrit Code Review 
CommitDate: Fri Mar 19 12:06:32 2021 +0100

Update git submodules

* Update helpcontent2 from branch 'master'
  to b4fd7e41094ae6e9ca5bdf5d0c86f594ef58271a
  - tdf#132643 Translate German section IDs

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

diff --git a/helpcontent2 b/helpcontent2
index dd35bdfc9a28..b4fd7e41094a 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit dd35bdfc9a28343688df30e8617b24dab0620cde
+Subproject commit b4fd7e41094ae6e9ca5bdf5d0c86f594ef58271a
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Johnny_M (via logerrit)
 source/text/shared/00/00040500.xhp  |2 +-
 source/text/shared/01/0503.xhp  |2 +-
 source/text/shared/01/05030300.xhp  |4 ++--
 source/text/shared/02/1301.xhp  |2 +-
 source/text/swriter/01/04120221.xhp |2 +-
 source/text/swriter/01/04120222.xhp |2 +-
 source/text/swriter/01/04120223.xhp |2 +-
 source/text/swriter/01/04120224.xhp |2 +-
 source/text/swriter/01/04120225.xhp |2 +-
 source/text/swriter/01/04120226.xhp |2 +-
 source/text/swriter/01/04120227.xhp |2 +-
 source/text/swriter/main0213.xhp|2 +-
 12 files changed, 13 insertions(+), 13 deletions(-)

New commits:
commit 49b1e322e38412a6c4bce867870691b96719029d
Author: Johnny_M 
AuthorDate: Fri Mar 19 11:14:24 2021 +0100
Commit: Olivier Hallot 
CommitDate: Fri Mar 19 12:08:00 2021 +0100

tdf#132643 Translate German section IDs

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

diff --git a/source/text/shared/00/00040500.xhp 
b/source/text/shared/00/00040500.xhp
index 9000bd53e..81ea09712 100644
--- a/source/text/shared/00/00040500.xhp
+++ b/source/text/shared/00/00040500.xhp
@@ -112,7 +112,7 @@
 Choose Format - Paragraph 
- Indents & Spacing tab.
 Choose View - 
Styles - open context menu of an entry and choose Modify/New - 
Indents & Spacing tab.
 
-
+
 Choose Format - Paragraph 
- Tabs tab.
 Choose View - 
Styles - open context menu of an entry and choose Modify/New - 
Tabs tab.
 Double-click the 
ruler
diff --git a/source/text/shared/01/0503.xhp 
b/source/text/shared/01/0503.xhp
index 9c254c95c..fc56dfdaf 100644
--- a/source/text/shared/01/0503.xhp
+++ b/source/text/shared/01/0503.xhp
@@ -43,7 +43,7 @@
 
 
 
-
+
 
 
 
diff --git a/source/text/shared/01/05030300.xhp 
b/source/text/shared/01/05030300.xhp
index 08a6ced3c..90f6b872b 100644
--- a/source/text/shared/01/05030300.xhp
+++ b/source/text/shared/01/05030300.xhp
@@ -27,7 +27,7 @@
 
 
 
-
+
 formats; tabulators
 fill characters with tabulators
 tab stops;settings
@@ -39,7 +39,7 @@
 
 If you want, 
you can also use the ruler to set the tab positions.
 
-
+
 
 
 Position
diff --git a/source/text/shared/02/1301.xhp 
b/source/text/shared/02/1301.xhp
index 6eba1dadd..bcb170ae0 100644
--- a/source/text/shared/02/1301.xhp
+++ b/source/text/shared/02/1301.xhp
@@ -28,7 +28,7 @@
   
 
 
-
+
 Setting 
Tabs
 On the ruler, 
set the tabs for the current paragraph, or all selected paragraphs, using the 
mouse.
 
diff --git a/source/text/swriter/01/04120221.xhp 
b/source/text/swriter/01/04120221.xhp
index b8ceca9a5..f26ac46e2 100644
--- a/source/text/swriter/01/04120221.xhp
+++ b/source/text/swriter/01/04120221.xhp
@@ -76,7 +76,7 @@
 Inserts the text of the 
chapter heading.
 
 
-
+
 
 
 Tab 
stop (T)
diff --git a/source/text/swriter/01/04120222.xhp 
b/source/text/swriter/01/04120222.xhp
index fac65a8f6..fae15b6d8 100644
--- a/source/text/swriter/01/04120222.xhp
+++ b/source/text/swriter/01/04120222.xhp
@@ -39,7 +39,7 @@
 Level "S" 
refers to the single letter headings that divide the index entries 
alphabetically. To enable these headings, select the Alphabetical 
delimiter check box in the Format area.
 
 
-
+
 
 Chapter Info
 Inserts chapter 
information, such as the chapter heading and number. Select the information 
that you want to display in the Chapter entry 
box.
diff --git a/source/text/swriter/01/04120223.xhp 
b/source/text/swriter/01/04120223.xhp
index 57622c86b..38a7ddce0 100644
--- a/source/text/swriter/01/04120223.xhp
+++ b/source/text/swriter/01/04120223.xhp
@@ -39,7 +39,7 @@
 Illustration 
Indexes only have one index level.
 
 
-
+
 
 
 
diff --git a/source/text/swriter/01/04120224.xhp 
b/source/text/swriter/01/04120224.xhp
index df8e6a1c2..f66b3a2f1 100644
--- a/source/text/swriter/01/04120224.xhp
+++ b/source/text/swriter/01/04120224.xhp
@@ -39,7 +39,7 @@
 An Index of 
Tables only has one index level.
 
 
-
+
 
 
 
diff --git a/source/text/swriter/01/04120225.xhp 
b/source/text/swriter/01/04120225.xhp
index 533a9e14f..914a89829 100644
--- a/source/text/swriter/01/04120225.xhp
+++ b/source/text/swriter/01/04120225.xhp
@@ -39,7 +39,7 @@
 User-defined 
indexes do not support sub-keys.
 
 
-
+
 
 
 
diff --git a/source/text/swriter/01/04120226.xhp 
b/source/text/swriter/01/04120226.xhp
index b52ccf94a..3664b7a0b 100644
--- a/source/text/swriter/01/04120226.xhp
+++ b/source/text/swriter/01/04120226.xhp
@@ -39,7 +39,7 @@
 A Table of 
Objects only has one index level.
 
 
-
+
 
 
 
diff --git a/source/text/swriter/01/04120227.xhp 
b/source/text/swriter/01/04120227.xhp
index e530d0e8a..990daf2d8 100644
--- a/source/text/swriter/01/04120227.xhp
+++ b/source/text/swriter/01/04120227.xhp
@@ -38,7 +38,7 @@
 Type
 The types that 
are displayed depend on the different literature sources.
 
-
+
 Type
 Lists the 
available bibliography entries. To add an entry to the Structure 
l

[Libreoffice-commits] core.git: helpcontent2

2021-03-19 Thread Johnny_M (via logerrit)
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 26c4a732deac00e3d8c1efd60d12beaa3d07271e
Author: Johnny_M 
AuthorDate: Fri Mar 19 12:08:00 2021 +0100
Commit: Gerrit Code Review 
CommitDate: Fri Mar 19 12:08:00 2021 +0100

Update git submodules

* Update helpcontent2 from branch 'master'
  to 49b1e322e38412a6c4bce867870691b96719029d
  - tdf#132643 Translate German section IDs

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

diff --git a/helpcontent2 b/helpcontent2
index b4fd7e41094a..49b1e322e384 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit b4fd7e41094ae6e9ca5bdf5d0c86f594ef58271a
+Subproject commit 49b1e322e38412a6c4bce867870691b96719029d
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Johnny_M (via logerrit)
 source/text/shared/07/0900.xhp |2 +-
 source/text/shared/main0800.xhp|2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 5fd56260ef5af41277d8395dd7ae53bd7a03b147
Author: Johnny_M 
AuthorDate: Fri Mar 19 11:15:56 2021 +0100
Commit: Olivier Hallot 
CommitDate: Fri Mar 19 12:08:41 2021 +0100

tdf#132643 Translate German section IDs

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

diff --git a/source/text/shared/07/0900.xhp 
b/source/text/shared/07/0900.xhp
index 948a6e5b8..8a3630f2e 100644
--- a/source/text/shared/07/0900.xhp
+++ b/source/text/shared/07/0900.xhp
@@ -28,7 +28,7 @@
   
 
 
-
+
 Web Pages
   To create a 
new web page for the Internet, open a new HTML Document by 
choosing File - New.
 
diff --git a/source/text/shared/main0800.xhp b/source/text/shared/main0800.xhp
index e58c8ae06..ffe79351a 100644
--- a/source/text/shared/main0800.xhp
+++ b/source/text/shared/main0800.xhp
@@ -34,6 +34,6 @@
 $[officename] and the Internet
 This section 
provides information on the subject of the Internet. An Internet 
glossary explains the most important terms.
 
-
+
 
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: helpcontent2

2021-03-19 Thread Johnny_M (via logerrit)
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit b1c215aadb2ddcea5d5ec8b97635182300663e5e
Author: Johnny_M 
AuthorDate: Fri Mar 19 12:08:41 2021 +0100
Commit: Gerrit Code Review 
CommitDate: Fri Mar 19 12:08:41 2021 +0100

Update git submodules

* Update helpcontent2 from branch 'master'
  to 5fd56260ef5af41277d8395dd7ae53bd7a03b147
  - tdf#132643 Translate German section IDs

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

diff --git a/helpcontent2 b/helpcontent2
index 49b1e322e384..5fd56260ef5a 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 49b1e322e38412a6c4bce867870691b96719029d
+Subproject commit 5fd56260ef5af41277d8395dd7ae53bd7a03b147
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Johnny_M (via logerrit)
 source/text/shared/main0500.xhp |2 --
 1 file changed, 2 deletions(-)

New commits:
commit 12c12fcb05e61337af68fa7a2af8df756f783a50
Author: Johnny_M 
AuthorDate: Fri Mar 19 11:19:28 2021 +0100
Commit: Olivier Hallot 
CommitDate: Fri Mar 19 12:09:10 2021 +0100

tdf#132643 Translate German section IDs

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

diff --git a/source/text/shared/main0500.xhp b/source/text/shared/main0500.xhp
index bacde3bcc..dc6daf593 100644
--- a/source/text/shared/main0500.xhp
+++ b/source/text/shared/main0500.xhp
@@ -29,10 +29,8 @@
 
 
 
-  
   Glossaries
   This section 
provides a general glossary of technical terms used in $[officename], along 
with a list of Internet terms.
-  
   
   
  
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: helpcontent2

2021-03-19 Thread Johnny_M (via logerrit)
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 86ff911c5991c5874c9022392f5ac398bcb6c999
Author: Johnny_M 
AuthorDate: Fri Mar 19 12:09:10 2021 +0100
Commit: Gerrit Code Review 
CommitDate: Fri Mar 19 12:09:10 2021 +0100

Update git submodules

* Update helpcontent2 from branch 'master'
  to 12c12fcb05e61337af68fa7a2af8df756f783a50
  - tdf#132643 Translate German section IDs

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

diff --git a/helpcontent2 b/helpcontent2
index 5fd56260ef5a..12c12fcb05e6 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 5fd56260ef5af41277d8395dd7ae53bd7a03b147
+Subproject commit 12c12fcb05e61337af68fa7a2af8df756f783a50
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Johnny_M (via logerrit)
 source/text/scalc/02/1801.xhp|2 +-
 source/text/scalc/main0218.xhp   |2 +-
 source/text/sdraw/main0210.xhp   |2 +-
 source/text/shared/00/edit_menu.xhp  |4 ++--
 source/text/shared/01/0206.xhp   |2 +-
 source/text/shared/main0201.xhp  |2 +-
 source/text/simpress/02/1011.xhp |2 +-
 source/text/simpress/main0210.xhp|2 +-
 8 files changed, 9 insertions(+), 9 deletions(-)

New commits:
commit ce121d8e8720a488f4bcefc259ce44c7fa4c16e6
Author: Johnny_M 
AuthorDate: Fri Mar 19 11:28:50 2021 +0100
Commit: Olivier Hallot 
CommitDate: Fri Mar 19 12:10:19 2021 +0100

tdf#132643 Translate German section IDs

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

diff --git a/source/text/scalc/02/1801.xhp 
b/source/text/scalc/02/1801.xhp
index ea9101125..011ae2c75 100644
--- a/source/text/scalc/02/1801.xhp
+++ b/source/text/scalc/02/1801.xhp
@@ -37,7 +37,7 @@
 Click the arrow next to the icon to open the Insert 
toolbar, where you can add graphics and special characters to the 
current sheet.
 
 Tools bar 
icon:
-
+
 
 
 
diff --git a/source/text/scalc/main0218.xhp b/source/text/scalc/main0218.xhp
index 6126a05b8..8466c1329 100644
--- a/source/text/scalc/main0218.xhp
+++ b/source/text/scalc/main0218.xhp
@@ -36,7 +36,7 @@
 Use the Tools bar to access commonly used 
commands.
 
 
-
+
 
 
 
diff --git a/source/text/sdraw/main0210.xhp b/source/text/sdraw/main0210.xhp
index 8402815ae..b66d03ade 100644
--- a/source/text/sdraw/main0210.xhp
+++ b/source/text/sdraw/main0210.xhp
@@ -74,7 +74,7 @@
 
 
 
-
+
 Form Controls
 
 
diff --git a/source/text/shared/00/edit_menu.xhp 
b/source/text/shared/00/edit_menu.xhp
index 937efe4b7..71218306a 100644
--- a/source/text/shared/00/edit_menu.xhp
+++ b/source/text/shared/00/edit_menu.xhp
@@ -102,11 +102,11 @@
 
 
 
-
+
 Choose 
Edit - Paste.
 CommandCtrl+V
 On the 
Standard bar, click
-
+
 
 
 
diff --git a/source/text/shared/01/0206.xhp 
b/source/text/shared/01/0206.xhp
index 2a74e5d8e..27993e859 100644
--- a/source/text/shared/01/0206.xhp
+++ b/source/text/shared/01/0206.xhp
@@ -49,7 +49,7 @@
   Inserts the contents of the clipboard at the location of the 
cursor, and replaces any selected text or objects.UFI: text 
used for icon and menu
 
 
-  
+  
   
 
   Press the Enter key.
diff --git a/source/text/shared/main0201.xhp b/source/text/shared/main0201.xhp
index 649fd3d19..3912bd0a7 100644
--- a/source/text/shared/main0201.xhp
+++ b/source/text/shared/main0201.xhp
@@ -66,7 +66,7 @@
 
 
 
-
+
 
 
 
diff --git a/source/text/simpress/02/1011.xhp 
b/source/text/simpress/02/1011.xhp
index c51206f42..0ae5e1a59 100644
--- a/source/text/simpress/02/1011.xhp
+++ b/source/text/simpress/02/1011.xhp
@@ -28,7 +28,7 @@
 
 Insert
 
-
+
 
 
 
diff --git a/source/text/simpress/main0210.xhp 
b/source/text/simpress/main0210.xhp
index a8b2191ca..a3df9abff 100644
--- a/source/text/simpress/main0210.xhp
+++ b/source/text/simpress/main0210.xhp
@@ -80,7 +80,7 @@
 
 
 
-
+
 Rotate
 This tool is 
used to rotate the object.
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: helpcontent2

2021-03-19 Thread Johnny_M (via logerrit)
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit c0d223f7036263d3e7012d497ea71d4722052927
Author: Johnny_M 
AuthorDate: Fri Mar 19 12:10:19 2021 +0100
Commit: Gerrit Code Review 
CommitDate: Fri Mar 19 12:10:19 2021 +0100

Update git submodules

* Update helpcontent2 from branch 'master'
  to ce121d8e8720a488f4bcefc259ce44c7fa4c16e6
  - tdf#132643 Translate German section IDs

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

diff --git a/helpcontent2 b/helpcontent2
index 12c12fcb05e6..ce121d8e8720 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 12c12fcb05e61337af68fa7a2af8df756f783a50
+Subproject commit ce121d8e8720a488f4bcefc259ce44c7fa4c16e6
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 sfx2/source/sidebar/Deck.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit c797c2bac2dab105856c65a3de4a0dcea629e6f5
Author: Caolán McNamara 
AuthorDate: Thu Mar 18 12:45:36 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 12:11:38 2021 +0100

replace GetParent() with mxParentWindow

which should be the same thing without relying on querying the hierarchy

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

diff --git a/sfx2/source/sidebar/Deck.cxx b/sfx2/source/sidebar/Deck.cxx
index 4b0eb6580d70..4ea279689e8e 100644
--- a/sfx2/source/sidebar/Deck.cxx
+++ b/sfx2/source/sidebar/Deck.cxx
@@ -214,7 +214,7 @@ void Deck::RequestLayout()
 return;
 
 bool bChangeNeeded = false;
-Size aParentSize = GetParent()->GetSizePixel();
+Size aParentSize = mxParentWindow->GetSizePixel();
 
 if (mnMinimalHeight > 0 && (mnMinimalHeight != aParentSize.Height() || 
GetSizePixel().Height() != mnMinimalHeight))
 {
@@ -231,7 +231,7 @@ void Deck::RequestLayout()
 
 if (bChangeNeeded)
 {
-GetParent()->SetSizePixel(aParentSize);
+mxParentWindow->SetSizePixel(aParentSize);
 setPosSizePixel(0, 0, aParentSize.Width(), aParentSize.Height());
 }
 else if (aParentSize != GetSizePixel()) //Sync parent & child sizes
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Arpit Bandejiya license statement

2021-03-19 Thread Arpit Bandejiya
   All of my past & future contributions to LibreOffice may be
   licensed under the MPLv2/LGPLv3+ dual license.


Thanks,
Arpit Bandejiya
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 sc/source/ui/condformat/condformatdlg.cxx |9 +
 sc/source/ui/inc/condformatdlgentry.hxx   |1 +
 2 files changed, 10 insertions(+)

New commits:
commit 426a38ea955742c471c22b4c20f0149ec423f653
Author: Caolán McNamara 
AuthorDate: Thu Mar 18 14:21:22 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 12:59:21 2021 +0100

tdf#118482 set a scroll step of the height of a collapsed entry

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

diff --git a/sc/source/ui/condformat/condformatdlg.cxx 
b/sc/source/ui/condformat/condformatdlg.cxx
index 7a4631289f01..b4759bb3940e 100644
--- a/sc/source/ui/condformat/condformatdlg.cxx
+++ b/sc/source/ui/condformat/condformatdlg.cxx
@@ -174,6 +174,8 @@ void ScCondFormatList::RecalcAll()
 if (mbFrozen)
 return;
 
+int nWheelScroll = SAL_MAX_INT32;
+
 sal_Int32 nIndex = 1;
 for (const auto& item : maEntries)
 {
@@ -181,8 +183,15 @@ void ScCondFormatList::RecalcAll()
 continue;
 item->SetIndex(nIndex);
 item->set_grid_top_attach(nIndex - 1);
+nWheelScroll = std::min(nWheelScroll, item->get_preferred_height());
 ++nIndex;
 }
+
+if (nWheelScroll != SAL_MAX_INT32)
+{
+// tdf#118482 set a scroll step of the height of a collapsed entry
+mxScrollWindow->vadjustment_set_step_increment(nWheelScroll);
+}
 }
 
 IMPL_LINK(ScCondFormatList, ColFormatTypeHdl, weld::ComboBox&, rBox, void)
diff --git a/sc/source/ui/inc/condformatdlgentry.hxx 
b/sc/source/ui/inc/condformatdlgentry.hxx
index bf2d80fba2e0..aed63be47729 100644
--- a/sc/source/ui/inc/condformatdlgentry.hxx
+++ b/sc/source/ui/inc/condformatdlgentry.hxx
@@ -77,6 +77,7 @@ public:
 void Show() { mxGrid->show(); }
 
 void set_grid_top_attach(int nAttach) { 
mxBorder->set_grid_top_attach(nAttach); }
+int get_preferred_height() const { return 
mxBorder->get_preferred_size().Height(); }
 
 void SetPos(const ScAddress& rPos) { maPos = rPos; };
 bool IsSelected() const { return mbActive;}
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sfx2/source sfx2/util

2021-03-19 Thread Noel (via logerrit)
 sfx2/source/notify/globalevents.cxx |   20 ++--
 sfx2/util/sfx.component |3 ++-
 2 files changed, 4 insertions(+), 19 deletions(-)

New commits:
commit 42969ac2c3db42460a9aaa8b51df76f58b360526
Author: Noel 
AuthorDate: Thu Mar 18 11:14:46 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Mar 19 13:23:06 2021 +0100

use single-use attribute for SfxGlobalEvents_Impl

instead of rtl::Instance, which means it will get
cleaned up when UNO shuts down

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

diff --git a/sfx2/source/notify/globalevents.cxx 
b/sfx2/source/notify/globalevents.cxx
index 9ac9e8707565..2176c327243f 100644
--- a/sfx2/source/notify/globalevents.cxx
+++ b/sfx2/source/notify/globalevents.cxx
@@ -519,30 +519,14 @@ TModelList::iterator 
SfxGlobalEvents_Impl::impl_searchDoc(const uno::Reference<
 });
 }
 
-struct Instance {
-explicit Instance(
-css::uno::Reference const & context):
-instance(
-static_cast(new 
SfxGlobalEvents_Impl(context)))
-{}
-
-rtl::Reference instance;
-};
-
-struct Singleton:
-public rtl::StaticWithArg<
-Instance, css::uno::Reference, Singleton>
-{};
-
-}
+} // namespace
 
 extern "C" SAL_DLLPUBLIC_EXPORT css::uno::XInterface *
 com_sun_star_comp_sfx2_GlobalEventBroadcaster_get_implementation(
 css::uno::XComponentContext *context,
 css::uno::Sequence const &)
 {
-return cppu::acquire(static_cast(
-Singleton::get(context).instance.get()));
+return cppu::acquire(new SfxGlobalEvents_Impl(context));
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sfx2/util/sfx.component b/sfx2/util/sfx.component
index 314b8c18ecc9..a8f4b7104991 100644
--- a/sfx2/util/sfx.component
+++ b/sfx2/util/sfx.component
@@ -70,7 +70,8 @@
 
   
   
+  
constructor="com_sun_star_comp_sfx2_GlobalEventBroadcaster_get_implementation"
+  single-instance="true">
 
 
   
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sfx2/source sfx2/util

2021-03-19 Thread Noel (via logerrit)
 sfx2/source/appl/shutdownicon.cxx |   21 +
 sfx2/util/sfx.component   |3 ++-
 2 files changed, 3 insertions(+), 21 deletions(-)

New commits:
commit 896bed62eff679e354e46635f5659fe0caa650fe
Author: Noel 
AuthorDate: Thu Mar 18 11:16:27 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Mar 19 13:58:40 2021 +0100

use single-use attribute for ShutdownIcon

instead of rtl::Instance, which means it will get
cleaned up when UNO shuts down

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

diff --git a/sfx2/source/appl/shutdownicon.cxx 
b/sfx2/source/appl/shutdownicon.cxx
index 40febc9b422a..fcebfe809d1e 100644
--- a/sfx2/source/appl/shutdownicon.cxx
+++ b/sfx2/source/appl/shutdownicon.cxx
@@ -706,31 +706,12 @@ css::uno::Any SAL_CALL 
ShutdownIcon::getFastPropertyValue( ::sal_Int32 nHandle )
 return aValue;
 }
 
-namespace {
-
-struct Instance {
-explicit Instance(
-css::uno::Reference const & context):
-instance(static_cast(new ShutdownIcon(context)))
-{}
-
-rtl::Reference instance;
-};
-
-struct Singleton:
-public rtl::StaticWithArg<
-Instance, css::uno::Reference, Singleton>
-{};
-
-}
-
 extern "C" SAL_DLLPUBLIC_EXPORT css::uno::XInterface *
 com_sun_star_comp_desktop_QuickstartWrapper_get_implementation(
 css::uno::XComponentContext *context,
 css::uno::Sequence const &)
 {
-return cppu::acquire(static_cast(
-Singleton::get(context).instance.get()));
+return cppu::acquire(new ShutdownIcon(context));
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sfx2/util/sfx.component b/sfx2/util/sfx.component
index a8f4b7104991..a7c9930e0ec8 100644
--- a/sfx2/util/sfx.component
+++ b/sfx2/util/sfx.component
@@ -33,7 +33,8 @@
 
   
   
+  
constructor="com_sun_star_comp_desktop_QuickstartWrapper_get_implementation"
+  single-instance="true">
 
 
   
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Alternative master/main branch proposal (Re: ANN: renaming of master branch to "main" for core repository and submodules (dictionaries, help, translations))

2021-03-19 Thread Kohei Yoshida
On Thu, 2021-03-18 at 21:04 +0100, Luboš Luňák wrote:
> Therefore, I propose that the decision to rename the master branch
> is postponed for as long as the git project does not take a
> definitive stance on it. That stance may take the form of e.g. the
> git project making a statement on it or changing their default and
> using it. Our decision can be then based on this input and may e.g.
> take the form of simply taking the technical decision to do what git
> does.

+1

I support this approach too.

Kohei

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


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

2021-03-19 Thread Johnny_M (via logerrit)
 source/text/scalc/main0208.xhp |2 +-
 source/text/shared/02/2002.xhp |2 +-
 source/text/simpress/main0206.xhp  |2 +-
 source/text/swriter/main0208.xhp   |2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

New commits:
commit c04755eb8c7beac1bf8d0a967d05a78fee715ce5
Author: Johnny_M 
AuthorDate: Fri Mar 19 11:00:30 2021 +0100
Commit: Olivier Hallot 
CommitDate: Fri Mar 19 14:20:03 2021 +0100

tdf#132643 Translate German section IDs

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

diff --git a/source/text/scalc/main0208.xhp b/source/text/scalc/main0208.xhp
index f06ede5ff..cc4d70d5c 100644
--- a/source/text/scalc/main0208.xhp
+++ b/source/text/scalc/main0208.xhp
@@ -32,7 +32,7 @@
  The 
Status Bar displays information about the current 
sheet.
   
   
-  
+  
   
   
   
diff --git a/source/text/shared/02/2002.xhp 
b/source/text/shared/02/2002.xhp
index 0d2b97538..d9f606950 100644
--- a/source/text/shared/02/2002.xhp
+++ b/source/text/shared/02/2002.xhp
@@ -25,7 +25,7 @@
   
 
 
-
+
 page styles;editing/applying with 
statusbar
 
 
diff --git a/source/text/simpress/main0206.xhp 
b/source/text/simpress/main0206.xhp
index 9893a7ae8..104ee8df0 100644
--- a/source/text/simpress/main0206.xhp
+++ b/source/text/simpress/main0206.xhp
@@ -40,6 +40,6 @@
   
   
   
-  
+  
  
 
diff --git a/source/text/swriter/main0208.xhp b/source/text/swriter/main0208.xhp
index ba693253a..18e5222f4 100644
--- a/source/text/swriter/main0208.xhp
+++ b/source/text/swriter/main0208.xhp
@@ -33,7 +33,7 @@
 
 
 
-
+
 
 
 Languagei80434
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: helpcontent2

2021-03-19 Thread Johnny_M (via logerrit)
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit e250f17d9653042c3edcb4e55167159746d8ffae
Author: Johnny_M 
AuthorDate: Fri Mar 19 14:20:03 2021 +0100
Commit: Gerrit Code Review 
CommitDate: Fri Mar 19 14:20:03 2021 +0100

Update git submodules

* Update helpcontent2 from branch 'master'
  to c04755eb8c7beac1bf8d0a967d05a78fee715ce5
  - tdf#132643 Translate German section IDs

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

diff --git a/helpcontent2 b/helpcontent2
index ce121d8e8720..c04755eb8c7b 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit ce121d8e8720a488f4bcefc259ce44c7fa4c16e6
+Subproject commit c04755eb8c7beac1bf8d0a967d05a78fee715ce5
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: oox import/export of custom shape extrusion

2021-03-19 Thread Regina Henschel

Hi Miklos,

Miklos Vajna schrieb am 19.03.2021 um 08:18:

Hi Regina,

On Thu, Mar 18, 2021 at 04:24:38PM +0100, Regina Henschel 
 wrote:

I want to work on getting oox import/export of custom shape extrusion.


Great, much appreciated.


In case a grabBag exists, it will be necessary to merge values from the
internal "extrusion" property with values from the grabBag on export. In
case no grabBag exists, writing OOXML from the internal properties is
needed. In case no internal "extrusion" property exists, the current
solution of writing the gragBag is OK.

Should I use separate solutions for the three cases? Advantage would be,
that the development can be done without interfering the current solution
for "only grabBag". But a solution for "merge" would be able to solve the
situation "no grabBag" and "only grabBag" too, so doing only that might
avoid code duplication.


The motivation bethind grab-bag is to preserve properties from OOXML,
which is not handled by libreoffice core. So in case there is a grab-bag
attached to a shape and you use the UI to modify the shape, it's
perfectly valid to just empty the grab-bag. This is predictable behavior
and you will never have to handle a mix of real document model and
grab-bag for the same shape.

As an example, we store smartart data in the shape grab-bag. But once
you edit the shape text, we empty the grab-bag, so your text edits are
not lost on save (since commit
a01ae07740e5c311fcc37f2ac2e2a0a2a1935920).

Does that help?


It would indeed make the structure much easier, if only the situations 
"no grabBag" and "only grabBag" need to be handled. So that is fine for me.


OOX has some 3D properties, which do not exist in ODF and/or cannot be 
rendered by LO. Those would be lost in a PPTX - LO - PPTX roundtrip. Of 
cause they will be lost in PPTX - LO - ODF anyway. I would not mind 
loosing them, but what do your customers expect?


Such properties are for sure property 'Bevel', color of light (used in 
presets "sunset" and "freeze" for example) and property 'Contour'. I 
have not yet investigated, what is possible in ODF in regard to property 
'Material' and which of the Material presets LO would be able to render 
although no UI exists.


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


Re: oox import/export of custom shape extrusion

2021-03-19 Thread Miklos Vajna
Hi Regina,

On Fri, Mar 19, 2021 at 02:20:54PM +0100, Regina Henschel 
 wrote:
> It would indeed make the structure much easier, if only the situations "no
> grabBag" and "only grabBag" need to be handled. So that is fine for me.
> 
> OOX has some 3D properties, which do not exist in ODF and/or cannot be
> rendered by LO. Those would be lost in a PPTX - LO - PPTX roundtrip. Of
> cause they will be lost in PPTX - LO - ODF anyway.

The grab-bag format is only in the memory, so you don't have to care
about stability / compatibility there (just keep import and export in
sync). If something works for LO shapes, those can be properly
imported/exported. If the shape is edited, the grab-bag can be emptied.
And finally if a property is not something an LO shape / ODF can handle,
it can stay in grab-bag.

That way you would never have conflicting information in the grab-bag
and in the real document model on export. Would this approach work?

Regards,

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


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

2021-03-19 Thread Bayram Çiçek (via logerrit)
 desktop/source/deployment/manager/dp_extensionmanager.cxx |   59 +++---
 1 file changed, 33 insertions(+), 26 deletions(-)

New commits:
commit 7d3addde836bbd8d990a363562310b237213527a
Author: Bayram Çiçek 
AuthorDate: Sun Mar 14 14:39:14 2021 +0300
Commit: Mike Kaganski 
CommitDate: Fri Mar 19 14:37:42 2021 +0100

tdf#122406: only restart when user accepted the license agreement

In Tools > Extension Manager, LO requires restart after
declining the license agreement on the *first* try.
Don't need to restart LO because no change made.

- Current location of fireModified() function doesn't
respect this decision.

- fireModified() has moved to proper location.


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

diff --git a/desktop/source/deployment/manager/dp_extensionmanager.cxx 
b/desktop/source/deployment/manager/dp_extensionmanager.cxx
index 63882cefdfd3..68a6fe08579d 100644
--- a/desktop/source/deployment/manager/dp_extensionmanager.cxx
+++ b/desktop/source/deployment/manager/dp_extensionmanager.cxx
@@ -722,6 +722,39 @@ Reference 
ExtensionManager::addExtension(
 sNewExtensionIdentifier, sNewExtensionFileName,
 bUserDisabled2, false, xAbortChannel,
 Reference());
+
+// if reached this section,
+// this means that either the licensedialog.ui didn't 
popup,
+// or user accepted the license agreement. otherwise
+// no need to call fireModified() because user declined
+// the license agreement therefore no change made.
+try
+{
+fireModified();
+
+}catch ( const css::deployment::DeploymentException& ) {
+throw;
+} catch ( const ucb::CommandFailedException & ) {
+throw;
+} catch ( const ucb::CommandAbortedException & ) {
+throw;
+} catch (const lang::IllegalArgumentException &) {
+throw;
+} catch (const uno::RuntimeException &) {
+throw;
+} catch (const uno::Exception &) {
+uno::Any excOccurred = ::cppu::getCaughtException();
+css::deployment::DeploymentException exc(
+"Extension Manager: Exception on fireModified() "
+"in the scope of 'if (failedPrereq == 0)'",
+static_cast(this), excOccurred);
+throw exc;
+} catch (...) {
+throw uno::RuntimeException(
+"Extension Manager: RuntimeException on 
fireModified() "
+"in the scope of 'if (failedPrereq == 0)'",
+static_cast(this));
+}
 }
 else
 {
@@ -780,32 +813,6 @@ Reference 
ExtensionManager::addExtension(
 }
 } // leaving the guarded section (getMutex())
 
-try
-{
-fireModified();
-
-}catch ( const css::deployment::DeploymentException& ) {
-throw;
-} catch ( const ucb::CommandFailedException & ) {
-throw;
-} catch ( const ucb::CommandAbortedException & ) {
-throw;
-} catch (const lang::IllegalArgumentException &) {
-throw;
-} catch (const uno::RuntimeException &) {
-throw;
-} catch (const uno::Exception &) {
-uno::Any excOccurred = ::cppu::getCaughtException();
-css::deployment::DeploymentException exc(
-"Extension Manager: exception in doChecksForAddExtension",
-static_cast(this), excOccurred);
-throw exc;
-} catch (...) {
-throw uno::RuntimeException(
-"Extension Manager: unexpected exception in 
doChecksForAddExtension",
-static_cast(this));
-}
-
 return xNewExtension;
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: android/README

2021-03-19 Thread Michael Weghorn (via logerrit)
 android/README |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit b0d1ad82e33868683edd7a1e50f516f38d0634b0
Author: Michael Weghorn 
AuthorDate: Fri Mar 19 10:07:12 2021 +0100
Commit: Michael Weghorn 
CommitDate: Fri Mar 19 14:48:39 2021 +0100

android: Update obj path in README

... according to the changes done in

commit 0dffc65236fbacf98047d6dbfc82b4efe7dd959b
Date:   Fri Jun 5 08:59:26 2020 -0400

android: fix the build output directory

When compiling in a different build output directory,
the "liblo-native-code.so" file is created in the source directory 
and
it fails compiling the "online" project

Change-Id: I886231034bbe3937748d9b5ef56239aea3cf7b8c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112702
Tested-by: Jenkins
Reviewed-by: Michael Weghorn 

diff --git a/android/README b/android/README
index 5b27ceb8e971..cfce3ddaa942 100644
--- a/android/README
+++ b/android/README
@@ -241,7 +241,7 @@ Using lldb from within Android Studio is more comfortable 
though and works like
 - make sure you select the right build variant (strippedUIDebug is what 
you want)
 - use Run|Edit Configurations to create a new configuration of type 
"Android Native"
 - on tab "General" pick module "source"
-- on tab "Native Debugger" add android/source/obj/local/ to
+- on tab "Native Debugger" add android/obj/local/ to
   the Symbol directories
 - on the LLDB startup commands tab add
   "command script import /path/to/solenv/lldb/libreoffice/LO.py"
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Seth Chaiklin (via logerrit)
 source/text/shared/00/00040500.xhp |5 +++--
 source/text/shared/01/0605.xhp |2 +-
 2 files changed, 4 insertions(+), 3 deletions(-)

New commits:
commit 4ceb9e1c9bc4a9961ece57131f8d9db569998568
Author: Seth Chaiklin 
AuthorDate: Fri Mar 19 14:22:12 2021 +0100
Commit: Seth Chaiklin 
CommitDate: Fri Mar 19 15:03:30 2021 +0100

tdf#139667 access and apply corrections to Bullets and Numbering

   ( shared/00/00040500.xhp )
+ add context menu access
* repair incorrect icon and label

   ( shared/01/0605.xhp )
 + add that the dialog can also be applied to selected
   paragraphs.

Change-Id: I052f7cae83859ab1a2051cc58a0355f33c66097b
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/112722
Reviewed-by: Seth Chaiklin 
Tested-by: Jenkins

diff --git a/source/text/shared/00/00040500.xhp 
b/source/text/shared/00/00040500.xhp
index 81ea09712..3845475be 100644
--- a/source/text/shared/00/00040500.xhp
+++ b/source/text/shared/00/00040500.xhp
@@ -262,15 +262,16 @@
 Open 
the context menu of the 3D object, choose 3D Effects - Material 
tab.
 
 Choose Format - Bullets 
and Numbering.
+Right-click a 
paragraph, choose List - Bullets and Numbering.
 On Formatting bar, 
click
 
 
   
 
-  Icon bullet list
+  Bullets and Numbering dialog Icon
 
 
-  Bullets On/Off
+  Bullets and 
Numbering
 
   
 
diff --git a/source/text/shared/01/0605.xhp 
b/source/text/shared/01/0605.xhp
index 08910dd29..78405abb8 100644
--- a/source/text/shared/01/0605.xhp
+++ b/source/text/shared/01/0605.xhp
@@ -32,7 +32,7 @@
 
 
 Bullets and 
Numbering
-Adds numbering or bullets to the current 
paragraph, and lets you edit format of the numbering or bullets.
+Adds numbering or bullets to the current 
paragraph or to selected paragraphs, and lets you edit format of the numbering 
or bullets.
 
 
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: helpcontent2

2021-03-19 Thread Seth Chaiklin (via logerrit)
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 6bc79df7e0171452e912789cf0962831c44fdc97
Author: Seth Chaiklin 
AuthorDate: Fri Mar 19 15:03:30 2021 +0100
Commit: Gerrit Code Review 
CommitDate: Fri Mar 19 15:03:30 2021 +0100

Update git submodules

* Update helpcontent2 from branch 'master'
  to 4ceb9e1c9bc4a9961ece57131f8d9db569998568
  - tdf#139667 access and apply corrections to Bullets and Numbering

   ( shared/00/00040500.xhp )
+ add context menu access
* repair incorrect icon and label

   ( shared/01/0605.xhp )
 + add that the dialog can also be applied to selected
   paragraphs.

Change-Id: I052f7cae83859ab1a2051cc58a0355f33c66097b
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/112722
Reviewed-by: Seth Chaiklin 
Tested-by: Jenkins

diff --git a/helpcontent2 b/helpcontent2
index c04755eb8c7b..4ceb9e1c9bc4 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit c04755eb8c7beac1bf8d0a967d05a78fee715ce5
+Subproject commit 4ceb9e1c9bc4a9961ece57131f8d9db569998568
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Seth Chaiklin (via logerrit)
 source/text/swriter/02/add_to_list.xhp |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 4703fa8a90baedcf23cb5739507a46d4b03418f1
Author: Seth Chaiklin 
AuthorDate: Fri Mar 19 15:04:04 2021 +0100
Commit: Seth Chaiklin 
CommitDate: Fri Mar 19 15:06:58 2021 +0100

correction to access command

Change-Id: I2a05689b8dd050e6851e38b4291a25a78282d8a1
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/112723
Reviewed-by: Seth Chaiklin 
Tested-by: Jenkins

diff --git a/source/text/swriter/02/add_to_list.xhp 
b/source/text/swriter/02/add_to_list.xhp
index 69ba5a3d4..ff4306569 100644
--- a/source/text/swriter/02/add_to_list.xhp
+++ b/source/text/swriter/02/add_to_list.xhp
@@ -26,7 +26,7 @@
 
 
 Choose Format - Lists - Add to 
List.
-Right-click a 
paragraph, choose Lists - Add to List.
+Right-click a 
paragraph, choose List - Add to List.
 On Bullets and Numbering bar, click Add 
to List icon.
 
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: helpcontent2

2021-03-19 Thread Seth Chaiklin (via logerrit)
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 6aee002aa3094ebaa863e79ff3c3b1c6f5ef3c76
Author: Seth Chaiklin 
AuthorDate: Fri Mar 19 15:06:58 2021 +0100
Commit: Gerrit Code Review 
CommitDate: Fri Mar 19 15:06:58 2021 +0100

Update git submodules

* Update helpcontent2 from branch 'master'
  to 4703fa8a90baedcf23cb5739507a46d4b03418f1
  - correction to access command

Change-Id: I2a05689b8dd050e6851e38b4291a25a78282d8a1
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/112723
Reviewed-by: Seth Chaiklin 
Tested-by: Jenkins

diff --git a/helpcontent2 b/helpcontent2
index 4ceb9e1c9bc4..4703fa8a90ba 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 4ceb9e1c9bc4a9961ece57131f8d9db569998568
+Subproject commit 4703fa8a90baedcf23cb5739507a46d4b03418f1
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Tomaž Vajngerl (via logerrit)
 vcl/qa/cppunit/png/PngFilterTest.cxx   |   10 --
 vcl/qa/cppunit/png/data/color-rect-8bit-RGB-interlaced.png |binary
 2 files changed, 8 insertions(+), 2 deletions(-)

New commits:
commit 26d2356f64dc24b654f4afdb7d4b701d0a09e1e8
Author: Tomaž Vajngerl 
AuthorDate: Fri Mar 19 20:49:50 2021 +0900
Commit: Tomaž Vajngerl 
CommitDate: Fri Mar 19 16:25:43 2021 +0100

vcl: add interlaced test case for PngImageReader

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

diff --git a/vcl/qa/cppunit/png/PngFilterTest.cxx 
b/vcl/qa/cppunit/png/PngFilterTest.cxx
index 3b02e1135504..c167c4c9c636 100644
--- a/vcl/qa/cppunit/png/PngFilterTest.cxx
+++ b/vcl/qa/cppunit/png/PngFilterTest.cxx
@@ -97,8 +97,14 @@ void PngFilterTest::testPng()
 }
 }
 }
-for (const OUString& aFileName :
- { OUString("color-rect-8bit-RGB.png"), 
OUString("color-rect-4bit-pal.png") })
+
+OUString aFilenames[] = {
+OUString("color-rect-8bit-RGB.png"),
+OUString("color-rect-8bit-RGB-interlaced.png"),
+OUString("color-rect-4bit-pal.png"),
+};
+
+for (const OUString& aFileName : aFilenames)
 {
 SvFileStream aFileStream(getFullUrl(aFileName), StreamMode::READ);
 
diff --git a/vcl/qa/cppunit/png/data/color-rect-8bit-RGB-interlaced.png 
b/vcl/qa/cppunit/png/data/color-rect-8bit-RGB-interlaced.png
new file mode 100644
index ..17ca9a350d9c
Binary files /dev/null and 
b/vcl/qa/cppunit/png/data/color-rect-8bit-RGB-interlaced.png differ
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: oox import/export of custom shape extrusion

2021-03-19 Thread Regina Henschel

Hi Miklos,

Miklos Vajna schrieb am 19.03.2021 um 14:36:

Hi Regina,

On Fri, Mar 19, 2021 at 02:20:54PM +0100, Regina Henschel 
 wrote:

It would indeed make the structure much easier, if only the situations "no
grabBag" and "only grabBag" need to be handled. So that is fine for me.

OOX has some 3D properties, which do not exist in ODF and/or cannot be
rendered by LO. Those would be lost in a PPTX - LO - PPTX roundtrip. Of
cause they will be lost in PPTX - LO - ODF anyway.


The grab-bag format is only in the memory, so you don't have to care
about stability / compatibility there (just keep import and export in
sync). If something works for LO shapes, those can be properly
imported/exported.


If the shape is edited, the grab-bag can be emptied.

I thought to only put something into the grab-bag on import, if LO 
cannot handle it and it is needed in export. So removing something from 
the grab-bag should not be necessary. That means import has to be done 
parallel to export.
Removing something in the grab-bag when shape is edited, would require 
changes in places, which are not related to import/export. I think, that 
will be complicate and dangerous.



And finally if a property is not something an LO shape / ODF can handle,
it can stay in grab-bag.


Exactly that is what I have called "merge". The export happens in
DrawingML::WriteShape3DEffects
https://opengrok.libreoffice.org/xref/core/oox/source/export/drawingml.cxx?r=68bc6a5d#4342
It means, that I have to put those values, which are set by LO and 
therefore not in the grab-bag, into the sequences aEffectProps, 
aLightRigProps and aShape3DProps there.




That way you would never have conflicting information in the grab-bag
and in the real document model on export. Would this approach work?


If I not only implement export but handle import at the same time it 
should work. I think, I will start and sometime in April you can see 
some concrete code in Gerrit.


Thank you for your thoughts so far.

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


[Libreoffice-commits] core.git: include/cppuhelper

2021-03-19 Thread Stephan Bergmann (via logerrit)
 include/cppuhelper/implbase.hxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 13ea8a2f245fd07056e9a86ed3860109950470ef
Author: Stephan Bergmann 
AuthorDate: Fri Mar 19 14:04:00 2021 +0100
Commit: Stephan Bergmann 
CommitDate: Fri Mar 19 16:32:56 2021 +0100

Fix typo in comment

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

diff --git a/include/cppuhelper/implbase.hxx b/include/cppuhelper/implbase.hxx
index 02bfb4ce7eef..95c2b0fc74f6 100644
--- a/include/cppuhelper/implbase.hxx
+++ b/include/cppuhelper/implbase.hxx
@@ -40,7 +40,7 @@
 // is unlikely ever be a better choice than WeakImplHelper, so all their
 // existing uses are probably confused and should use WeakImplHelper instead.
 //
-// Replacements for WeakAggImplHelperN and AggImplInheritanceHelper1 have
+// Replacements for WeakAggImplHelperN and AggImplInheritanceHelperN have
 // deliberately been left out, as the underlying aggregation mechanism is known
 // broken in general and should not be used.
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Alternative master/main branch proposal

2021-03-19 Thread Alan Frank
I'm not yet an active contributor, but hope it's okay to share my 2c 
worth.


I think that a change to a word with dozens of meanings, based on the 
offensiveness of one, is an overreaction.  I am reminded of when "leg" 
and "occupy" were taboo.


I am also fine with the "wait and see" proposal.


 Original Message 
Subject: Re: Alternative master/main branch proposal (Re: ANN: renaming 
of master branch to "main" for core repository and submodules 
(dictionaries, help, translations))

Date: 19.03.2021 02:07
From: Michael Weghorn 
To: libreoffice@lists.freedesktop.org

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


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

2021-03-19 Thread Tibor Nagy (via logerrit)
 oox/source/export/shapes.cxx|8 +++-
 sd/qa/unit/data/pptx/tdf131905.pptx |binary
 sd/qa/unit/export-tests-ooxml2.cxx  |   28 
 3 files changed, 35 insertions(+), 1 deletion(-)

New commits:
commit 48f6ce530f3546570eabdf398388185ab988636b
Author: Tibor Nagy 
AuthorDate: Mon Mar 8 16:07:59 2021 +0100
Commit: László Németh 
CommitDate: Fri Mar 19 16:55:03 2021 +0100

tdf#131905 PPTX table export: fix vertical alignment

Table cells lost their vertical alignment.

Change-Id: Id5ca22550b5613566d7ea2a93e8a36fdca31860d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112171
Tested-by: László Németh 
Reviewed-by: László Németh 

diff --git a/oox/source/export/shapes.cxx b/oox/source/export/shapes.cxx
index 13c52d22c56e..e889901c477c 100644
--- a/oox/source/export/shapes.cxx
+++ b/oox/source/export/shapes.cxx
@@ -1780,6 +1780,8 @@ void ShapeExport::WriteTable( const Reference< XShape >& 
rXShape  )
 void ShapeExport::WriteTableCellProperties(const Reference< XPropertySet>& 
xCellPropSet)
 {
 sal_Int32 nLeftMargin(0), nRightMargin(0);
+TextVerticalAdjust eVerticalAlignment;
+const char* sVerticalAlignment;
 
 Any aLeftMargin = xCellPropSet->getPropertyValue("TextLeftDistance");
 aLeftMargin >>= nLeftMargin;
@@ -1787,7 +1789,11 @@ void ShapeExport::WriteTableCellProperties(const 
Reference< XPropertySet>& xCell
 Any aRightMargin = xCellPropSet->getPropertyValue("TextRightDistance");
 aRightMargin >>= nRightMargin;
 
-mpFS->startElementNS(XML_a, XML_tcPr,
+Any aVerticalAlignment = 
xCellPropSet->getPropertyValue("TextVerticalAdjust");
+aVerticalAlignment >>= eVerticalAlignment;
+sVerticalAlignment = GetTextVerticalAdjust(eVerticalAlignment);
+
+mpFS->startElementNS(XML_a, XML_tcPr, XML_anchor, sVerticalAlignment,
 XML_marL, 
sax_fastparser::UseIf(OString::number(oox::drawingml::convertHmmToEmu(nLeftMargin)),
 nLeftMargin > 0),
 XML_marR, 
sax_fastparser::UseIf(OString::number(oox::drawingml::convertHmmToEmu(nRightMargin)),
 nRightMargin > 0));
 
diff --git a/sd/qa/unit/data/pptx/tdf131905.pptx 
b/sd/qa/unit/data/pptx/tdf131905.pptx
new file mode 100644
index ..4d203510f093
Binary files /dev/null and b/sd/qa/unit/data/pptx/tdf131905.pptx differ
diff --git a/sd/qa/unit/export-tests-ooxml2.cxx 
b/sd/qa/unit/export-tests-ooxml2.cxx
index 9523d5718a46..3d8e93b8b003 100644
--- a/sd/qa/unit/export-tests-ooxml2.cxx
+++ b/sd/qa/unit/export-tests-ooxml2.cxx
@@ -87,6 +87,7 @@ static void assertMotionPath(const OUString &rStr1, const 
OUString &rStr2, const
 class SdOOXMLExportTest2 : public SdModelTestBaseXML
 {
 public:
+void testTdf131905();
 void testTdf93883();
 void testTdf91378();
 void testBnc822341();
@@ -211,6 +212,7 @@ public:
 
 CPPUNIT_TEST_SUITE(SdOOXMLExportTest2);
 
+CPPUNIT_TEST(testTdf131905);
 CPPUNIT_TEST(testTdf93883);
 CPPUNIT_TEST(testTdf91378);
 CPPUNIT_TEST(testBnc822341);
@@ -366,6 +368,32 @@ public:
 
 };
 
+void SdOOXMLExportTest2::testTdf131905()
+{
+::sd::DrawDocShellRef xDocShRef
+= 
loadURL(m_directories.getURLFromSrc(u"/sd/qa/unit/data/pptx/tdf131905.pptx"), 
PPTX);
+utl::TempFile tempFile;
+xDocShRef = saveAndReload(xDocShRef.get(), PPTX, &tempFile);
+
+xmlDocUniquePtr pXmlDocContent = parseExport(tempFile, 
"ppt/slides/slide1.xml");
+assertXPath(
+pXmlDocContent,
+
"/p:sld/p:cSld/p:spTree/p:graphicFrame/a:graphic/a:graphicData/a:tbl/a:tr[1]/a:tc/a:tcPr",
+"anchor", "t");
+
+assertXPath(
+pXmlDocContent,
+
"/p:sld/p:cSld/p:spTree/p:graphicFrame/a:graphic/a:graphicData/a:tbl/a:tr[2]/a:tc/a:tcPr",
+"anchor", "ctr");
+
+assertXPath(
+pXmlDocContent,
+
"/p:sld/p:cSld/p:spTree/p:graphicFrame/a:graphic/a:graphicData/a:tbl/a:tr[3]/a:tc/a:tcPr",
+"anchor", "b");
+
+xDocShRef->DoClose();
+}
+
 void SdOOXMLExportTest2::testTdf93883()
 {
 ::sd::DrawDocShellRef xDocShRef = loadURL( 
m_directories.getURLFromSrc(u"/sd/qa/unit/data/odp/tdf93883.odp"), ODP);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: framework/source framework/util

2021-03-19 Thread Noel (via logerrit)
 framework/source/uifactory/windowcontentfactorymanager.cxx |   19 -
 framework/util/fwk.component   |3 +-
 2 files changed, 3 insertions(+), 19 deletions(-)

New commits:
commit 22e4f7a348f82fae7eefae5b67de3d829dceadcb
Author: Noel 
AuthorDate: Thu Mar 18 11:17:57 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Mar 19 17:36:29 2021 +0100

use single-use attribute for WindowContentFactoryManager

instead of rtl::Instance, which means it will get
cleaned up when UNO shuts down

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

diff --git a/framework/source/uifactory/windowcontentfactorymanager.cxx 
b/framework/source/uifactory/windowcontentfactorymanager.cxx
index 996fae5f41a1..0d2991c80b7e 100644
--- a/framework/source/uifactory/windowcontentfactorymanager.cxx
+++ b/framework/source/uifactory/windowcontentfactorymanager.cxx
@@ -193,22 +193,6 @@ uno::Reference< uno::XInterface > SAL_CALL 
WindowContentFactoryManager::createIn
 return xWindow;
 }
 
-struct Instance {
-explicit Instance(
-css::uno::Reference const & context):
-instance(static_cast(
-new WindowContentFactoryManager(context)))
-{
-}
-
-css::uno::Reference instance;
-};
-
-struct Singleton:
-public rtl::StaticWithArg<
-Instance, css::uno::Reference, Singleton>
-{};
-
 }
 
 extern "C" SAL_DLLPUBLIC_EXPORT css::uno::XInterface *
@@ -216,8 +200,7 @@ 
com_sun_star_comp_framework_WindowContentFactoryManager_get_implementation(
 css::uno::XComponentContext *context,
 css::uno::Sequence const &)
 {
-return cppu::acquire(static_cast(
-Singleton::get(context).instance.get()));
+return cppu::acquire(new WindowContentFactoryManager(context));
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/framework/util/fwk.component b/framework/util/fwk.component
index e056e8a9aeb6..d6ca2652d3cf 100644
--- a/framework/util/fwk.component
+++ b/framework/util/fwk.component
@@ -175,7 +175,8 @@
 
   
   
+  
constructor="com_sun_star_comp_framework_WindowContentFactoryManager_get_implementation"
+  single-instance="true">
 
 
   
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: framework/source framework/util

2021-03-19 Thread Noel (via logerrit)
 framework/source/uifactory/uielementfactorymanager.cxx |   19 -
 framework/util/fwk.component   |3 +-
 2 files changed, 3 insertions(+), 19 deletions(-)

New commits:
commit f5784794f4cecd4fd57a803191b544f80f7ba02e
Author: Noel 
AuthorDate: Thu Mar 18 11:19:21 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Mar 19 17:37:07 2021 +0100

use single-use attribute for UIElementFactoryManager

instead of rtl::Instance, which means it will get
cleaned up when UNO shuts down

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

diff --git a/framework/source/uifactory/uielementfactorymanager.cxx 
b/framework/source/uifactory/uielementfactorymanager.cxx
index 02d5435bab80..b1567724ff06 100644
--- a/framework/source/uifactory/uielementfactorymanager.cxx
+++ b/framework/source/uifactory/uielementfactorymanager.cxx
@@ -542,22 +542,6 @@ void SAL_CALL UIElementFactoryManager::deregisterFactory( 
const OUString& aType,
 // SAFE
 }
 
-struct Instance {
-explicit Instance(
-css::uno::Reference const & context):
-instance(static_cast(
-new UIElementFactoryManager(context)))
-{
-}
-
-css::uno::Reference instance;
-};
-
-struct Singleton:
-public rtl::StaticWithArg<
-Instance, css::uno::Reference, Singleton>
-{};
-
 }
 
 extern "C" SAL_DLLPUBLIC_EXPORT css::uno::XInterface *
@@ -565,8 +549,7 @@ 
com_sun_star_comp_framework_UIElementFactoryManager_get_implementation(
 css::uno::XComponentContext *context,
 css::uno::Sequence const &)
 {
-return cppu::acquire(static_cast(
-Singleton::get(context).instance.get()));
+return cppu::acquire(new UIElementFactoryManager(context));
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/framework/util/fwk.component b/framework/util/fwk.component
index d6ca2652d3cf..9f050373c603 100644
--- a/framework/util/fwk.component
+++ b/framework/util/fwk.component
@@ -166,7 +166,8 @@
 
   
   
+  
constructor="com_sun_star_comp_framework_UIElementFactoryManager_get_implementation"
+  single-instance="true">
 
 
   
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Crash test update: VectorGraphicData::ensureSequenceAndRange

2021-03-19 Thread Caolán McNamara
On Thu, 2021-02-18 at 16:44 +0900, Tomaž Vajngerl wrote:
> On Sun, Feb 14, 2021 at 6:21 AM Caolán McNamara  
> > reproducible for me:
> > $ ./instdir/program/soffice.bin --headless --convert-to 
> > odg:output.odg
> > ./tdf118346-1.odg
> > 
> > crashes in VectorGraphicData::ensureSequenceAndRange in some odd way
> > during assigning to maSequence
> > 
> Sorry, I missed this mail. I'll take a look.

What I seem to see is recursive reduceGraphicMemory calls and
https://gerrit.libreoffice.org/c/core/+/112772 works for me on this
example document

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


New Defects reported by Coverity Scan for LibreOffice

2021-03-19 Thread scan-admin
Hi,

Please find the latest report on new defect(s) introduced to LibreOffice found 
with Coverity Scan.

385 new defect(s) introduced to LibreOffice found with Coverity Scan.
24 defect(s), reported by Coverity Scan earlier, were marked fixed in the 
recent build analyzed by Coverity Scan.

New defect(s) Reported-by: Coverity Scan
Showing 20 of 385 defect(s)


** CID 1474403:  Error handling issues  (CHECKED_RETURN)
/sw/source/core/undo/rolbck.cxx: 152 in 
SwHistorySetFormat::dumpAsXml(_xmlTextWriter *) const()



*** CID 1474403:  Error handling issues  (CHECKED_RETURN)
/sw/source/core/undo/rolbck.cxx: 152 in 
SwHistorySetFormat::dumpAsXml(_xmlTextWriter *) const()
146 return aResult;
147 }
148 
149 void SwHistorySetFormat::dumpAsXml(xmlTextWriterPtr pWriter) const
150 {
151 xmlTextWriterStartElement(pWriter, BAD_CAST("SwHistorySetFormat"));
>>> CID 1474403:  Error handling issues  (CHECKED_RETURN)
>>> Calling "xmlTextWriterWriteAttribute(pWriter, (xmlChar 
>>> *)"m_nNodeIndex", (xmlChar *)rtl::OString::number(this->m_nNodeIndex, 
>>> 10).getStr())" without checking return value. This library function may 
>>> fail and return an error code.
152 xmlTextWriterWriteAttribute(pWriter, BAD_CAST("m_nNodeIndex"),
153 
BAD_CAST(OString::number(m_nNodeIndex).getStr()));
154 SwHistoryHint::dumpAsXml(pWriter);
155 
156 if (m_pAttr)
157 {

** CID 1474402:  Error handling issues  (CHECKED_RETURN)
/drawinglayer/source/dumper/XShapeDumper.cxx: 236 in 
::dumpFillTransparenceGradientNameAsAttribute(std::basic_string_view>, _xmlTextWriter *)()



*** CID 1474402:  Error handling issues  (CHECKED_RETURN)
/drawinglayer/source/dumper/XShapeDumper.cxx: 236 in 
::dumpFillTransparenceGradientNameAsAttribute(std::basic_string_view>, _xmlTextWriter *)()
230 {
231 xmlTextWriterWriteFormatAttribute(xmlWriter, 
BAD_CAST("fillTransparence"), "%" SAL_PRIdINT32, aTransparence);
232 }
233 
234 void dumpFillTransparenceGradientNameAsAttribute(std::u16string_view 
sTranspGradName, xmlTextWriterPtr xmlWriter)
235 {
>>> CID 1474402:  Error handling issues  (CHECKED_RETURN)
>>> Calling "xmlTextWriterWriteFormatAttribute(xmlWriter, (xmlChar 
>>> *)"fillTransparenceGradientName", "%s", 
>>> rtl::OString(rtl::OUStringToOString(sTranspGradName, 76, 1382U)).getStr())" 
>>> without checking return value. This library function may fail and return an 
>>> error code.
236 xmlTextWriterWriteFormatAttribute(xmlWriter, 
BAD_CAST("fillTransparenceGradientName"), "%s",
237 OUStringToOString(sTranspGradName, 
RTL_TEXTENCODING_UTF8).getStr());
238 }
239 
240 //because there's more awt::Gradient properties to dump
241 void dumpGradientProperty(const awt::Gradient& rGradient, 
xmlTextWriterPtr xmlWriter)

** CID 1474401:  Error handling issues  (CHECKED_RETURN)
/svl/source/undo/undo.cxx: 1367 in SfxListUndoAction::dumpAsXml(_xmlTextWriter 
*) const()



*** CID 1474401:  Error handling issues  (CHECKED_RETURN)
/svl/source/undo/undo.cxx: 1367 in SfxListUndoAction::dumpAsXml(_xmlTextWriter 
*) const()
1361 return !maUndoActions.empty() && 
maUndoActions[maUndoActions.size()-1].pAction->Merge( pNextAction );
1362 }
1363 
1364 void SfxListUndoAction::dumpAsXml(xmlTextWriterPtr pWriter) const
1365 {
1366 xmlTextWriterStartElement(pWriter, BAD_CAST("SfxListUndoAction"));
>>> CID 1474401:  Error handling issues  (CHECKED_RETURN)
>>> Calling "xmlTextWriterWriteAttribute(pWriter, (xmlChar *)"size", 
>>> (xmlChar *)rtl::OString::number(this->maUndoActions.size(), 10).getStr())" 
>>> without checking return value. This library function may fail and return an 
>>> error code.
1367 xmlTextWriterWriteAttribute(pWriter, BAD_CAST("size"), 
BAD_CAST(OString::number(maUndoActions.size()).getStr()));
1368 SfxUndoAction::dumpAsXml(pWriter);
1369 
1370 for (size_t i = 0; i < maUndoActions.size(); ++i)
1371 maUndoActions[i].pAction->dumpAsXml(pWriter);
1372 

** CID 1474400:  Error handling issues  (CHECKED_RETURN)
/svl/source/items/stritem.cxx: 31 in SfxStringItem::dumpAsXml(_xmlTextWriter *) 
const()



*** CID 1474400:  Error handling issues  (CHECKED_RETURN)
/svl/source/items/stritem.cxx: 31 in SfxStringItem::dumpAsXml(_xmlTextWriter *) 
const()
25 {
26 return new SfxStringItem(*this);
27 }
28 
29 void SfxStringItem::dumpAsXml(xmlTextWriterPtr pWriter) const
30 {
>>> CID 1474

Viswaas Loganadhan Prabunathan license statement

2021-03-19 Thread Viswaas Loganadhan Prabunathan
Hello,

All of my past & future contributions to LibreOffice maybe
licensed under the MPLv2/LGPLv3+ dual license.

Thanks & Regards,
Viswaas L Prabunathan
B.S. Comp Sci & Eng | UC Davis
___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice


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

2021-03-19 Thread Armin Le Grand (Allotropia) (via logerrit)
 vcl/unx/gtk3/gtk3gtkframe.cxx |7 ++-
 1 file changed, 6 insertions(+), 1 deletion(-)

New commits:
commit cc1690b9752b3fef3d541de0d26b7c7681615767
Author: Armin Le Grand (Allotropia) 
AuthorDate: Wed Mar 17 18:31:59 2021 +0100
Commit: Armin Le Grand 
CommitDate: Fri Mar 19 19:14:43 2021 +0100

tdf#131031 take MenuBar into account on size change

in GtkSalFrame::SetPosSize when setting local vars for
maGeometry.nWidth/nHeight the implicitely existing gtk
native menu bar has to be taken into account. This can
be done by extracting the values for it and re-apply
to the new values.

Change-Id: Ic0e76ecfd49fab7dbf330f3e26ab4a6ad3267b35
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112654
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
Reviewed-by: Armin Le Grand 

diff --git a/vcl/unx/gtk3/gtk3gtkframe.cxx b/vcl/unx/gtk3/gtk3gtkframe.cxx
index bf47097af565..e52ff192869f 100644
--- a/vcl/unx/gtk3/gtk3gtkframe.cxx
+++ b/vcl/unx/gtk3/gtk3gtkframe.cxx
@@ -1553,8 +1553,13 @@ void GtkSalFrame::SetPosSize( tools::Long nX, 
tools::Long nY, tools::Long nWidth
 {
 m_bDefaultSize = false;
 
+// tdf#131031 Just setting maGeometry.nWidth/nHeight will delete
+// the evtl. implicitely existing space at top for the gtk native 
MenuBar,
+// will make the Window too big and the StatusBar at the bottom vanish
+// and thus breaks the fix in tdf#130841.
+const int nImplicitMenuBarHeight(m_pSalMenu ? 
m_pSalMenu->GetMenuBarHeight() : 0);
 maGeometry.nWidth = nWidth;
-maGeometry.nHeight = nHeight;
+maGeometry.nHeight = nHeight - nImplicitMenuBarHeight;
 
 if( isChild( false ) )
 widget_set_size_request(nWidth, nHeight);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Opinions are like languages: Everyone has (at least) one

2021-03-19 Thread John

I hope we can be careful how strongly we word sentiments like:


Most of us are not even native English speakers


We wouldn't want people to feel like their opinion doesn't count. If you
make use of English in any capacity (and if you're reading this you do),
then it's reasonable for you to have an opinion about this tool you use.

The fact that my native language is the (ironically-termed) lingua
franca here and so many places is a privilege I have due to nothing more
than the circumstances of my birth. It does not grant me special
authority to tell you which words you should use or what they should
mean to you. If anything, it should encourage me to show charity and
make room for those who felt forced "to learn a foreign language before
they can express themselves electronically". If this conversation were
in any other language I would be, at best, severely struggling.

On a related note, there's a certain irony here:


to get lectured on English by a German


If we were weighing people's opinion based on their native language,
which I advise against, German-speakers would fare pretty well since our
languages are so closely related. In this conversation, for example,
"master" and "Meister" share the same origin and are practically the
same word. In fact, they show up in adages in both our languages where
they refer to mastering a skill: "Jack of all trades, master of none"
and "Übung macht den Meister.".

But that doesn't mean a speaker of a more distant language like Russian
or Chinese can't have a valid observation.

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


[Libreoffice-commits] core.git: 2 commits - framework/source framework/util

2021-03-19 Thread Noel (via logerrit)
 framework/source/uiconfiguration/moduleuicfgsupplier.cxx  |   19 --
 framework/source/uiconfiguration/windowstateconfiguration.cxx |   19 --
 framework/util/fwk.component  |6 ++-
 3 files changed, 6 insertions(+), 38 deletions(-)

New commits:
commit ccef00e160439f1836ab27cd13cbe6aa8f9e5146
Author: Noel 
AuthorDate: Thu Mar 18 11:27:23 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Mar 19 20:04:25 2021 +0100

use single-use attribute for WindowStateConfiguration

instead of rtl::Instance, which means it will get
cleaned up when UNO shuts down

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

diff --git a/framework/source/uiconfiguration/windowstateconfiguration.cxx 
b/framework/source/uiconfiguration/windowstateconfiguration.cxx
index ba8d0c15a43e..31b08076e953 100644
--- a/framework/source/uiconfiguration/windowstateconfiguration.cxx
+++ b/framework/source/uiconfiguration/windowstateconfiguration.cxx
@@ -1383,22 +1383,6 @@ sal_Bool SAL_CALL WindowStateConfiguration::hasElements()
 return true;
 }
 
-struct Instance {
-explicit Instance(
-css::uno::Reference const & context):
-instance(static_cast(
-new WindowStateConfiguration(context)))
-{
-}
-
-css::uno::Reference instance;
-};
-
-struct Singleton:
-public rtl::StaticWithArg<
-Instance, css::uno::Reference, Singleton>
-{};
-
 }
 
 extern "C" SAL_DLLPUBLIC_EXPORT css::uno::XInterface *
@@ -1406,8 +1390,7 @@ 
com_sun_star_comp_framework_WindowStateConfiguration_get_implementation(
 css::uno::XComponentContext *context,
 css::uno::Sequence const &)
 {
-return cppu::acquire(static_cast(
-Singleton::get(context).instance.get()));
+return cppu::acquire(new WindowStateConfiguration(context));
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/framework/util/fwk.component b/framework/util/fwk.component
index efc79a108e5d..f3a0cfcf1edd 100644
--- a/framework/util/fwk.component
+++ b/framework/util/fwk.component
@@ -183,7 +183,8 @@
 
   
   
+  
constructor="com_sun_star_comp_framework_WindowStateConfiguration_get_implementation"
+  single-instance="true">
 
 
   
commit 2167edad3b1f79e304a072b4d74eb1705cfe66b4
Author: Noel 
AuthorDate: Thu Mar 18 11:31:28 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Mar 19 20:04:18 2021 +0100

use single-use attribute for ModuleUIConfigurationManagerSupplier

instead of rtl::Instance, which means it will get
cleaned up when UNO shuts down

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

diff --git a/framework/source/uiconfiguration/moduleuicfgsupplier.cxx 
b/framework/source/uiconfiguration/moduleuicfgsupplier.cxx
index 31d61d8d0ad5..f28c2afedddf 100644
--- a/framework/source/uiconfiguration/moduleuicfgsupplier.cxx
+++ b/framework/source/uiconfiguration/moduleuicfgsupplier.cxx
@@ -171,22 +171,6 @@ Reference< XUIConfigurationManager > SAL_CALL 
ModuleUIConfigurationManagerSuppli
 return pIter->second;
 }
 
-struct Instance {
-explicit Instance(
-css::uno::Reference const & context):
-instance(static_cast(
-new ModuleUIConfigurationManagerSupplier(context)))
-{
-}
-
-css::uno::Reference instance;
-};
-
-struct Singleton:
-public rtl::StaticWithArg<
-Instance, css::uno::Reference, Singleton>
-{};
-
 }
 
 extern "C" SAL_DLLPUBLIC_EXPORT css::uno::XInterface *
@@ -194,8 +178,7 @@ 
com_sun_star_comp_framework_ModuleUIConfigurationManagerSupplier_get_implementat
 css::uno::XComponentContext *context,
 css::uno::Sequence const &)
 {
-return cppu::acquire(static_cast(
-Singleton::get(context).instance.get()));
+return cppu::acquire(new ModuleUIConfigurationManagerSupplier(context));
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/framework/util/fwk.component b/framework/util/fwk.component
index 9f050373c603..efc79a108e5d 100644
--- a/framework/util/fwk.component
+++ b/framework/util/fwk.component
@@ -91,7 +91,8 @@
 
   
   
+  
constructor="com_sun_star_comp_framework_ModuleUIConfigurationManagerSupplier_get_implementation"
+  single-instance="true">
 
 
   
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: framework/source framework/util

2021-03-19 Thread Noel (via logerrit)
 framework/source/uifactory/uicontrollerfactory.cxx |   21 +
 framework/util/fwk.component   |3 ++-
 2 files changed, 3 insertions(+), 21 deletions(-)

New commits:
commit 109312f4223a986444c589eca51eac7cdeb5ff48
Author: Noel 
AuthorDate: Thu Mar 18 11:24:37 2021 +0200
Commit: Noel Grandin 
CommitDate: Fri Mar 19 20:04:43 2021 +0100

use single-use attribute for PopupMenuControllerFactory

instead of rtl::Instance, which means it will get
cleaned up when UNO shuts down

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

diff --git a/framework/source/uifactory/uicontrollerfactory.cxx 
b/framework/source/uifactory/uicontrollerfactory.cxx
index 619b66d7c420..5f81f80be026 100644
--- a/framework/source/uifactory/uicontrollerfactory.cxx
+++ b/framework/source/uifactory/uicontrollerfactory.cxx
@@ -261,24 +261,6 @@ PopupMenuControllerFactory::PopupMenuControllerFactory( 
const Reference< XCompon
 {
 }
 
-struct PopupMenuControllerFactoryInstance {
-explicit PopupMenuControllerFactoryInstance(
-css::uno::Reference const & context):
-instance(static_cast(
-new PopupMenuControllerFactory(context)))
-{
-}
-
-css::uno::Reference instance;
-};
-
-struct PopupMenuControllerFactorySingleton:
-public rtl::StaticWithArg<
-PopupMenuControllerFactoryInstance,
-css::uno::Reference,
-PopupMenuControllerFactorySingleton>
-{};
-
 class ToolbarControllerFactory :  public UIControllerFactory
 {
 public:
@@ -376,8 +358,7 @@ 
com_sun_star_comp_framework_PopupMenuControllerFactory_get_implementation(
 css::uno::XComponentContext *context,
 css::uno::Sequence const &)
 {
-return cppu::acquire(static_cast(
-PopupMenuControllerFactorySingleton::get(context).instance.get()));
+return cppu::acquire(new PopupMenuControllerFactory(context));
 }
 
 extern "C" SAL_DLLPUBLIC_EXPORT css::uno::XInterface *
diff --git a/framework/util/fwk.component b/framework/util/fwk.component
index f3a0cfcf1edd..2d71213468d6 100644
--- a/framework/util/fwk.component
+++ b/framework/util/fwk.component
@@ -110,7 +110,8 @@
 
   
   
+  
constructor="com_sun_star_comp_framework_PopupMenuControllerFactory_get_implementation"
+  single-instance="true">
 
 
   
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Tamás Zolnai (via logerrit)
 sd/qa/unit/tiledrendering/tiledrendering.cxx |6 +-
 sfx2/source/control/bindings.cxx |   23 +++
 2 files changed, 28 insertions(+), 1 deletion(-)

New commits:
commit 69bcdb4cffe2c3a12e215b6cf5d99f18e49a5973
Author: Tamás Zolnai 
AuthorDate: Mon Mar 15 17:10:10 2021 +0100
Commit: Tamás Zolnai 
CommitDate: Fri Mar 19 20:40:21 2021 +0100

lok: switch to the correct view before any UI update is done via 
SfxBindings.

Change-Id: I541e53b4219340233ed5a4bec407ded5adf6a841
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112538
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Tamás Zolnai 

diff --git a/sd/qa/unit/tiledrendering/tiledrendering.cxx 
b/sd/qa/unit/tiledrendering/tiledrendering.cxx
index 79163b0cea67..5c4cb06f0790 100644
--- a/sd/qa/unit/tiledrendering/tiledrendering.cxx
+++ b/sd/qa/unit/tiledrendering/tiledrendering.cxx
@@ -1002,6 +1002,7 @@ void SdTiledRenderingTest::testViewCursorParts()
 SfxLokHelper::createView();
 
pXImpressDocument->initializeForTiledRendering(uno::Sequence());
 ViewCallback aView2;
+int nView2 = SfxLokHelper::getView();
 
 // Select the shape in the second view.
 sd::ViewShell* pViewShell = 
pXImpressDocument->GetDocShell()->GetViewShell();
@@ -1015,6 +1016,7 @@ void SdTiledRenderingTest::testViewCursorParts()
 pView->UnmarkAllObj(pView->GetSdrPageView());
 
 // Now switch to the second part in the second view.
+SfxLokHelper::setView(nView2);
 pXImpressDocument->setPart(1);
 aView2.m_nPart = 1;
 aView1.m_bGraphicViewSelectionInvalidated = false;
@@ -1055,10 +1057,10 @@ void SdTiledRenderingTest::testCursorViews()
 // second view as well, even if the second view was created after begin
 // text edit in the first view.
 ViewCallback aView2;
+SfxLokHelper::setView(nView1);
 // This failed: the second view didn't get a lock notification, even if the
 // first view already started text edit.
 CPPUNIT_ASSERT(aView2.m_bViewLock);
-SfxLokHelper::setView(nView1);
 aView2.m_bTilesInvalidated = false;
 pXImpressDocument->postKeyEvent(LOK_KEYEVENT_KEYINPUT, 'x', 0);
 pXImpressDocument->postKeyEvent(LOK_KEYEVENT_KEYUP, 'x', 0);
@@ -1185,6 +1187,8 @@ void 
SdTiledRenderingTest::testCursorVisibility_MultiView()
 pXImpressDocument->postMouseEvent(LOK_MOUSEEVENT_MOUSEBUTTONDOWN,
   centerX, centerY,
   2, MOUSE_LEFT, 0);
+
+SfxLokHelper::setView(nView1);
 pXImpressDocument->postMouseEvent(LOK_MOUSEEVENT_MOUSEBUTTONUP,
   centerX, centerY,
   2, MOUSE_LEFT, 0);
diff --git a/sfx2/source/control/bindings.cxx b/sfx2/source/control/bindings.cxx
index 1d6fd0d62441..6be545d382bf 100644
--- a/sfx2/source/control/bindings.cxx
+++ b/sfx2/source/control/bindings.cxx
@@ -55,6 +55,9 @@
 #include 
 #include 
 #include 
+#include 
+#include 
+#include 
 
 #include 
 #include 
@@ -1240,6 +1243,26 @@ bool SfxBindings::NextJob_Impl(Timer const * pTimer)
 
 SfxApplication *pSfxApp = SfxGetpApp();
 
+// Switch to the right view for state update.
+if (comphelper::LibreOfficeKit::isActive())
+{
+if( pDispatcher )
+{
+SfxViewFrame* pFrame = pDispatcher->GetFrame();
+if (pFrame)
+{
+vcl::Window* pFrameWin = pFrame->GetWindow().GetFrameWindow();
+if (pFrameWin && pFrameWin->GetLOKNotifier()) {
+const SfxViewShell* pView = dynamic_cast(pFrameWin->GetLOKNotifier());
+if (pView)
+{
+SfxLokHelper::setView(pView->GetViewShellId().get());
+}
+}
+}
+}
+}
+
 if( pDispatcher )
 pDispatcher->Update_Impl();
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 vcl/unx/generic/printer/cpdmgr.cxx |   49 ++---
 1 file changed, 25 insertions(+), 24 deletions(-)

New commits:
commit 236f3a8e60e05147a37f294774b0c07d40aff36f
Author: Caolán McNamara 
AuthorDate: Fri Mar 19 14:11:45 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 21:01:42 2021 +0100

cid#1473818 Use after free

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

diff --git a/vcl/unx/generic/printer/cpdmgr.cxx 
b/vcl/unx/generic/printer/cpdmgr.cxx
index bebd568330c5..0789ae7e1ca7 100644
--- a/vcl/unx/generic/printer/cpdmgr.cxx
+++ b/vcl/unx/generic/printer/cpdmgr.cxx
@@ -43,11 +43,11 @@ void CPDManager::onNameAcquired (GDBusConnection 
*connection,
  gpointer user_data)
 {
 gchar* contents;
-GDBusNodeInfo *introspection_data;
-
 // Get Interface for introspection
-g_file_get_contents (FRONTEND_INTERFACE, &contents, nullptr, nullptr);
-introspection_data = g_dbus_node_info_new_for_xml (contents, nullptr);
+if (!g_file_get_contents (FRONTEND_INTERFACE, &contents, nullptr, nullptr))
+return;
+
+GDBusNodeInfo *introspection_data = g_dbus_node_info_new_for_xml 
(contents, nullptr);
 
 g_dbus_connection_register_object (connection,
"/org/libreoffice/PrintDialog",
@@ -63,28 +63,29 @@ void CPDManager::onNameAcquired (GDBusConnection 
*connection,
 std::vector> backends = 
current->getTempBackends();
 for (auto const& backend : backends)
 {
-GDBusProxy *proxy;
 // Get Interface for introspection
-g_file_get_contents (BACKEND_INTERFACE, &contents, nullptr, nullptr);
-introspection_data = g_dbus_node_info_new_for_xml (contents, nullptr);
-proxy = g_dbus_proxy_new_sync (connection,
-   G_DBUS_PROXY_FLAGS_NONE,
-   introspection_data->interfaces[0],
-   backend.first.c_str(),
-   backend.second,
-   "org.openprinting.PrintBackend",
-   nullptr,
-   nullptr);
+if (g_file_get_contents(BACKEND_INTERFACE, &contents, nullptr, 
nullptr))
+{
+introspection_data = g_dbus_node_info_new_for_xml (contents, 
nullptr);
+GDBusProxy *proxy = g_dbus_proxy_new_sync (connection,
+   G_DBUS_PROXY_FLAGS_NONE,
+   introspection_data->interfaces[0],
+   backend.first.c_str(),
+   backend.second,
+   "org.openprinting.PrintBackend",
+   nullptr,
+   nullptr);
+g_assert (proxy != nullptr);
+g_dbus_proxy_call(proxy, "ActivateBackend",
+  nullptr,
+  G_DBUS_CALL_FLAGS_NONE,
+  -1, nullptr, nullptr, nullptr);
+
+g_free(contents);
+g_object_unref(proxy);
+g_dbus_node_info_unref(introspection_data);
+}
 g_free(backend.second);
-g_assert (proxy != nullptr);
-g_dbus_proxy_call(proxy, "ActivateBackend",
-  nullptr,
-  G_DBUS_CALL_FLAGS_NONE,
-  -1, nullptr, nullptr, nullptr);
-
-g_free(contents);
-g_object_unref(proxy);
-g_dbus_node_info_unref(introspection_data);
 }
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 sfx2/source/sidebar/SidebarController.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit b3c122fe936339f789945aa88fb26b099246ac37
Author: Caolán McNamara 
AuthorDate: Fri Mar 19 11:06:40 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 21:02:01 2021 +0100

Show is always called with true in this case

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

diff --git a/sfx2/source/sidebar/SidebarController.cxx 
b/sfx2/source/sidebar/SidebarController.cxx
index b8df3b61e046..40b48b790782 100644
--- a/sfx2/source/sidebar/SidebarController.cxx
+++ b/sfx2/source/sidebar/SidebarController.cxx
@@ -1378,7 +1378,7 @@ void SidebarController::UpdateDeckOpenState()
 
 mbIsDeckOpen = *mbIsDeckRequestedOpen;
 if (*mbIsDeckOpen && mpCurrentDeck)
-mpCurrentDeck->Show(*mbIsDeckOpen);
+mpCurrentDeck->Show();
 NotifyResize();
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Tünde Tóth (via logerrit)
 sc/source/ui/cctrl/dpcontrol.cxx |6 --
 sw/uiconfig/swriter/ui/navigatorpanel.ui |5 +
 2 files changed, 9 insertions(+), 2 deletions(-)

New commits:
commit aad8b94ed6494e19611a779aed74adca931d14aa
Author: Tünde Tóth 
AuthorDate: Fri Mar 12 11:33:54 2021 +0100
Commit: László Németh 
CommitDate: Fri Mar 19 21:02:42 2021 +0100

tdf#140955 sc UI: highlight active autofilter dropdown

arrow by inverting its colors, as suggested by Heiko Tietze.

The small difference in the active and inactive arrow shapes
(a dot in the corner) is much more visible this way.

Change-Id: I784fcb809549b22930037d28e007c2a42dcff337
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112378
Tested-by: László Németh 
Reviewed-by: László Németh 

diff --git a/sc/source/ui/cctrl/dpcontrol.cxx b/sc/source/ui/cctrl/dpcontrol.cxx
index d7dd13dd5c4e..0419b785b712 100644
--- a/sc/source/ui/cctrl/dpcontrol.cxx
+++ b/sc/source/ui/cctrl/dpcontrol.cxx
@@ -175,12 +175,14 @@ void ScDPFieldButton::drawPopupButton()
 
 // Background & outer black border
 mpOutDev->SetLineColor(COL_BLACK);
-Color aBackgroundColor = mbPopupPressed ? mpStyle->GetShadowColor() : 
mpStyle->GetFaceColor();
+Color aBackgroundColor
+= mbHasHiddenMember ? mpStyle->GetHighlightColor()
+: mbPopupPressed ? mpStyle->GetShadowColor() : 
mpStyle->GetFaceColor();
 mpOutDev->SetFillColor(aBackgroundColor);
 mpOutDev->DrawRect(tools::Rectangle(aPos, aSize));
 
 // the arrowhead
-Color aArrowColor = mbHasHiddenMember ? mpStyle->GetHighlightLinkColor() : 
mpStyle->GetButtonTextColor();
+Color aArrowColor = mbHasHiddenMember ? mpStyle->GetHighlightTextColor() : 
mpStyle->GetButtonTextColor();
 mpOutDev->SetLineColor(aArrowColor);
 mpOutDev->SetFillColor(aArrowColor);
 
commit c3e703d115981cf9a7e9a2be493eec2ef3f3b95e
Author: Caolán McNamara 
AuthorDate: Fri Mar 19 13:11:15 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 21:02:26 2021 +0100

don't hide toolbar item widgets if the pane shrinks

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

diff --git a/sw/uiconfig/swriter/ui/navigatorpanel.ui 
b/sw/uiconfig/swriter/ui/navigatorpanel.ui
index e8aa7fdc66fe..936df34e1d52 100644
--- a/sw/uiconfig/swriter/ui/navigatorpanel.ui
+++ b/sw/uiconfig/swriter/ui/navigatorpanel.ui
@@ -250,6 +250,7 @@
 True
 vertical
 
+  
   
 True
 False
@@ -258,6 +259,7 @@
 True
 True
 icons
+False
 
   
 True
@@ -287,6 +289,7 @@
 True
 True
 icons
+False
 
   
 True
@@ -331,6 +334,7 @@
 True
 True
 icons
+False
 
   
 True
@@ -363,6 +367,7 @@
 True
 True
 icons
+False
 
   
 True
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 sfx2/source/sidebar/SidebarController.cxx |4 
 1 file changed, 4 deletions(-)

New commits:
commit 3f0995d665a892bbc9d4cd01842da145bac8f488
Author: Caolán McNamara 
AuthorDate: Fri Mar 19 13:07:29 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 21:03:22 2021 +0100

shouldn't have to force a redraw for desktop either

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

diff --git a/sfx2/source/sidebar/SidebarController.cxx 
b/sfx2/source/sidebar/SidebarController.cxx
index 40b48b790782..f839fd98e0a9 100644
--- a/sfx2/source/sidebar/SidebarController.cxx
+++ b/sfx2/source/sidebar/SidebarController.cxx
@@ -828,10 +828,6 @@ void SidebarController::SwitchToDeck (
 msCurrentDeckId = rDeckDescriptor.msId;
 }
 
-// invisible with LOK, so avoid invalidations
-if (!comphelper::LibreOfficeKit::isActive())
-mpTabBar->Invalidate();
-
 mpTabBar->HighlightDeck(msCurrentDeckId);
 
 // Determine the panels to display in the deck.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 vcl/unx/gtk3/gtk3gtkinst.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 303c1433005044e3a818490d65385febc8eb6030
Author: Caolán McNamara 
AuthorDate: Fri Mar 19 14:15:45 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 21:03:47 2021 +0100

cid#1474031 silence Double free

coverity has learned about GObject apparently

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

diff --git a/vcl/unx/gtk3/gtk3gtkinst.cxx b/vcl/unx/gtk3/gtk3gtkinst.cxx
index fc958c780f18..6e5e5b48895d 100644
--- a/vcl/unx/gtk3/gtk3gtkinst.cxx
+++ b/vcl/unx/gtk3/gtk3gtkinst.cxx
@@ -3137,6 +3137,7 @@ public:
 gtk_widget_destroy(m_pMouseEventBox);
 
 gtk_container_add(GTK_CONTAINER(pParent), m_pWidget);
+// coverity[freed_arg : FALSE] - this does not free m_pWidget, it 
is reffed by pParent
 g_object_unref(m_pWidget);
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 sw/source/core/edit/edfcol.cxx |6 --
 1 file changed, 4 insertions(+), 2 deletions(-)

New commits:
commit caf97c68afc35a8c9a60dd86bf7df205e635bf53
Author: Caolán McNamara 
AuthorDate: Fri Mar 19 14:21:20 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 21:04:13 2021 +0100

cid#1473777 silence Using invalid iterator

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

diff --git a/sw/source/core/edit/edfcol.cxx b/sw/source/core/edit/edfcol.cxx
index ed6e90b12ede..bcff20b8b243 100644
--- a/sw/source/core/edit/edfcol.cxx
+++ b/sw/source/core/edit/edfcol.cxx
@@ -1946,6 +1946,7 @@ void 
SwEditShell::RestoreMetadataFieldsAndValidateParagraphSignatures()
 std::vector aResults;
 if (!sFieldNames.isEmpty())
 {
+assert(it != aStatements.end() && "can only be non-empty if it 
was valid");
 // Order the fields
 sal_Int32 nIndex = 0;
 do
@@ -1955,8 +1956,9 @@ void 
SwEditShell::RestoreMetadataFieldsAndValidateParagraphSignatures()
 break;
 
 const auto it2 = aStatements.find(sCurFieldName);
-const OUString sName = (it2 != aStatements.end() ? 
it->first : sBlank);
-const OUString sValue = (it2 != aStatements.end() ? 
it->second : sBlank);
+bool bStatementFound = it2 != aStatements.end();
+const OUString sName = bStatementFound ? it->first : 
sBlank;
+const OUString sValue = bStatementFound ? it->second : 
sBlank;
 
 if (aKeyCreator.isMarkingTextKey(sName))
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 vcl/source/helper/canvastools.cxx |1 -
 1 file changed, 1 deletion(-)

New commits:
commit 2f30b6af9ed727a967947b870b56689046f4a745
Author: Caolán McNamara 
AuthorDate: Fri Mar 19 14:24:39 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 21:04:41 2021 +0100

cid#1474156 Logically dead code

nAlphaDepth can only be 0, 1 or 8

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

diff --git a/vcl/source/helper/canvastools.cxx 
b/vcl/source/helper/canvastools.cxx
index 54dd76ae7651..8e837ac069e8 100644
--- a/vcl/source/helper/canvastools.cxx
+++ b/vcl/source/helper/canvastools.cxx
@@ -272,7 +272,6 @@ namespace vcl::unotools
   vcl::PixelFormat::N24_BPP;
 auto eAlphaPixelFormat =
 ( nAlphaDepth <= 1 ) ? vcl::PixelFormat::N1_BPP :
-( nAlphaDepth <= 4 ) ? vcl::PixelFormat::N4_BPP :
vcl::PixelFormat::N8_BPP;
 
 ::Bitmap aBitmap( aPixelSize,
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 sfx2/source/sidebar/DeckLayouter.cxx |   26 +++---
 1 file changed, 11 insertions(+), 15 deletions(-)

New commits:
commit bd7e258bac69f3b9f9e87128aacb989d23e24d4f
Author: Caolán McNamara 
AuthorDate: Fri Mar 19 14:35:03 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 21:05:10 2021 +0100

cid#1473847 Logically dead code

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

diff --git a/sfx2/source/sidebar/DeckLayouter.cxx 
b/sfx2/source/sidebar/DeckLayouter.cxx
index 36d6357ef305..3752e7df8fe4 100644
--- a/sfx2/source/sidebar/DeckLayouter.cxx
+++ b/sfx2/source/sidebar/DeckLayouter.cxx
@@ -192,16 +192,21 @@ void LayoutPanels (
 //   size:
 //   Use the unmodified preferred height for all panels.
 
-LayoutMode eMode (MinimumOrLarger);
+LayoutMode eMode(MinimumOrLarger);
 if (bShowVerticalScrollBar)
+{
 eMode = Preferred;
-else if (nTotalPreferredHeight <= nAvailableHeight)
-eMode = PreferredOrLarger;
-else
-eMode = MinimumOrLarger;
 
-if (eMode != Preferred)
+const sal_Int32 nContentHeight(nTotalPreferredHeight + 
nTotalDecorationHeight);
+SetupVerticalScrollBar(rVerticalScrollBar, nContentHeight, 
aBox.GetHeight());
+}
+else
 {
+if (nTotalPreferredHeight <= nAvailableHeight)
+eMode = PreferredOrLarger;
+else
+eMode = MinimumOrLarger;
+
 const sal_Int32 nTotalHeight (eMode==MinimumOrLarger ? 
nTotalMinimumHeight : nTotalPreferredHeight);
 
 DistributeHeights(
@@ -211,15 +216,6 @@ void LayoutPanels (
 eMode==MinimumOrLarger);
 }
 
-if (bShowVerticalScrollBar)
-{
-const sal_Int32 nContentHeight(
-eMode==Preferred
-? nTotalPreferredHeight + nTotalDecorationHeight
-: aBox.GetHeight());
-SetupVerticalScrollBar(rVerticalScrollBar, nContentHeight, 
aBox.GetHeight());
-}
-
 const sal_Int32 nUsedHeight(PlacePanels(rLayoutItems, eMode));
 rMinimalHeight = nUsedHeight;
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: chart2/source drawinglayer/source editeng/source sc/source sd/qa sd/source sfx2/source svl/source svx/source sw/qa sw/source tools/source writerfilter/source

2021-03-19 Thread Caolán McNamara (via logerrit)
 chart2/source/model/main/ChartModel.cxx|6 
 chart2/source/view/main/ChartView.cxx  |6 
 chart2/source/view/main/DrawModelWrapper.cxx   |6 
 drawinglayer/source/dumper/EnhancedShapeDumper.cxx |  338 +++---
 drawinglayer/source/dumper/XShapeDumper.cxx|  484 ++---
 editeng/source/editeng/editattr.cxx|8 
 editeng/source/editeng/editdoc.cxx |   24 -
 editeng/source/editeng/editobj.cxx |   28 -
 editeng/source/items/frmitems.cxx  |  102 ++--
 editeng/source/items/numitem.cxx   |   32 -
 editeng/source/items/paraitem.cxx  |   12 
 editeng/source/items/textitem.cxx  |   84 +--
 editeng/source/outliner/outliner.cxx   |8 
 editeng/source/outliner/outlobj.cxx|6 
 editeng/source/outliner/paralist.cxx   |   14 
 sc/source/core/data/attrib.cxx |   36 -
 sc/source/ui/view/gridwin_dbgutil.cxx  |   18 
 sd/qa/unit/tiledrendering/tiledrendering.cxx   |4 
 sd/source/core/drawdoc.cxx |   10 
 sd/source/core/sdpage2.cxx |6 
 sfx2/source/view/viewsh.cxx|8 
 svl/source/items/cenumitm.cxx  |8 
 svl/source/items/intitem.cxx   |   24 -
 svl/source/items/itempool.cxx  |4 
 svl/source/items/itemset.cxx   |4 
 svl/source/items/poolitem.cxx  |   23 
 svl/source/items/stritem.cxx   |8 
 svl/source/undo/undo.cxx   |   40 -
 svx/source/items/e3ditem.cxx   |   12 
 svx/source/sdr/properties/defaultproperties.cxx|4 
 svx/source/sdr/properties/properties.cxx   |4 
 svx/source/svdraw/svdattr.cxx  |   34 -
 svx/source/svdraw/svdmodel.cxx |6 
 svx/source/svdraw/svdobj.cxx   |   18 
 svx/source/svdraw/svdogrp.cxx  |6 
 svx/source/svdraw/svdpage.cxx  |8 
 svx/source/svdraw/svdtext.cxx  |4 
 svx/source/table/cell.cxx  |8 
 svx/source/table/svdotable.cxx |   10 
 svx/source/table/tablelayouter.cxx |   22 
 svx/source/table/tablemodel.cxx|4 
 svx/source/xoutdev/xattr.cxx   |   38 -
 svx/source/xoutdev/xattr2.cxx  |   32 -
 svx/source/xoutdev/xattrbmp.cxx|6 
 sw/qa/core/uwriter.cxx |4 
 sw/qa/unit/swmodeltestbase.cxx |4 
 sw/source/core/attr/fmtwrapinfluenceonobjpos.cxx   |   10 
 sw/source/core/bastyp/swrect.cxx   |   12 
 sw/source/core/crsr/bookmrk.cxx|   34 -
 sw/source/core/crsr/crsrsh.cxx |8 
 sw/source/core/crsr/pam.cxx|   20 
 sw/source/core/doc/DocumentSettingManager.cxx  |  340 +++---
 sw/source/core/doc/docbm.cxx   |8 
 sw/source/core/doc/docfmt.cxx  |   26 -
 sw/source/core/doc/docredln.cxx|   36 -
 sw/source/core/doc/fmtcol.cxx  |   10 
 sw/source/core/doc/number.cxx  |   14 
 sw/source/core/docnode/node.cxx|   52 +-
 sw/source/core/docnode/nodes.cxx   |4 
 sw/source/core/docnode/section.cxx |   10 
 sw/source/core/fields/authfld.cxx  |   34 -
 sw/source/core/fields/docufld.cxx  |   12 
 sw/source/core/fields/fldbas.cxx   |   26 -
 sw/source/core/fields/usrfld.cxx   |   16 
 sw/source/core/layout/atrfrm.cxx   |  174 +++
 sw/source/core/layout/dumpfilter.cxx   |4 
 sw/source/core/layout/pagedesc.cxx |   32 -
 sw/source/core/layout/tabfrm.cxx   |4 
 sw/source/core/ole/ndole.cxx   |   20 
 sw/source/core/para/paratr.cxx |8 
 sw/source/core/text/xmldump.cxx|  186 
 sw/source/core/txtnode/atrfld.cxx  |8 
 sw/source/core/txtnode/atrftn.cxx  |   22 
 sw/source/core/txtnode/chrfmt.cxx  |   11 
 sw/source/core/txtnode/fmtatr2.cxx |8 
 sw/source/core/txtnode/ndtxt.cxx   |   28 -
 sw/source/core/txtnode/txatbase.cxx|   18 
 sw/source/core/undo/docundo.cxx|8 
 sw/source/core/undo/rolbck.cxx |   30 -
 sw/source/core/undo/undel.cxx  |6 
 sw/source/core/undo/undobj.cxx |6 
 sw/source/core/undo/undobj1.cxx 

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

2021-03-19 Thread Caolán McNamara (via logerrit)
 vcl/unx/gtk3/gtk3gtkinst.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 9e0b747a51fe30c58296c5350214e98d7ecb80a8
Author: Caolán McNamara 
AuthorDate: Fri Mar 19 15:13:12 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 21:07:22 2021 +0100

cid#1473912 silence Use after free

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

diff --git a/vcl/unx/gtk3/gtk3gtkinst.cxx b/vcl/unx/gtk3/gtk3gtkinst.cxx
index 6e5e5b48895d..52c06df10452 100644
--- a/vcl/unx/gtk3/gtk3gtkinst.cxx
+++ b/vcl/unx/gtk3/gtk3gtkinst.cxx
@@ -6368,6 +6368,7 @@ private:
 g_object_ref(m_pNotebook);
 gtk_container_remove(GTK_CONTAINER(pParent), GTK_WIDGET(m_pNotebook));
 gtk_box_pack_start(m_pOverFlowBox, GTK_WIDGET(m_pNotebook), true, 
true, 0);
+// coverity[freed_arg : FALSE] - this does not free m_pNotebook , it 
is reffed by pParent
 g_object_unref(m_pNotebook);
 gtk_widget_show(GTK_WIDGET(m_pOverFlowBox));
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 svx/source/tbxctrls/StylesPreviewWindow.cxx |3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

New commits:
commit b41bb50749cee0281ecd19bafc2bb82c671eeeba
Author: Caolán McNamara 
AuthorDate: Fri Mar 19 15:15:18 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 21:07:59 2021 +0100

cid#1473993 Uninitialized pointer read

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

diff --git a/svx/source/tbxctrls/StylesPreviewWindow.cxx 
b/svx/source/tbxctrls/StylesPreviewWindow.cxx
index 59e86b7d2bd6..08d812af6493 100644
--- a/svx/source/tbxctrls/StylesPreviewWindow.cxx
+++ b/svx/source/tbxctrls/StylesPreviewWindow.cxx
@@ -78,8 +78,7 @@ 
StylePoolChangeListener::StylePoolChangeListener(StylesPreviewWindow_Base* pPrev
 {
 SfxObjectShell* pDocShell = SfxObjectShell::Current();
 
-if (pDocShell)
-m_pStyleSheetPool = pDocShell->GetStyleSheetPool();
+m_pStyleSheetPool = pDocShell ? pDocShell->GetStyleSheetPool() : nullptr;
 
 if (m_pStyleSheetPool)
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 vcl/unx/gtk3/gtk3gtkinst.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 754aea8115291304a7ca57396c32cbc516ccb843
Author: Caolán McNamara 
AuthorDate: Fri Mar 19 15:17:29 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 21:08:23 2021 +0100

cid#1474020 silence Use after free

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

diff --git a/vcl/unx/gtk3/gtk3gtkinst.cxx b/vcl/unx/gtk3/gtk3gtkinst.cxx
index 52c06df10452..745934c192c2 100644
--- a/vcl/unx/gtk3/gtk3gtkinst.cxx
+++ b/vcl/unx/gtk3/gtk3gtkinst.cxx
@@ -1882,6 +1882,7 @@ namespace
 
 gtk_container_add(GTK_CONTAINER(pReplacement), pWidget);
 
+// coverity[freed_arg : FALSE] - this does not free pWidget, it is 
reffed by pReplacement
 g_object_unref(pWidget);
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 vcl/unx/gtk3/gtk3gtkinst.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 401105d4fc9c5a1746b4d887554b169e023a06c1
Author: Caolán McNamara 
AuthorDate: Fri Mar 19 15:22:04 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 21:08:42 2021 +0100

cid#1474327 silence Use after free

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

diff --git a/vcl/unx/gtk3/gtk3gtkinst.cxx b/vcl/unx/gtk3/gtk3gtkinst.cxx
index 745934c192c2..c9fe5288ae2f 100644
--- a/vcl/unx/gtk3/gtk3gtkinst.cxx
+++ b/vcl/unx/gtk3/gtk3gtkinst.cxx
@@ -6136,6 +6136,7 @@ public:
 
 gtk_container_add(GTK_CONTAINER(m_pScrolledWindow), m_pOrigViewport);
 g_object_unref(m_pOrigViewport);
+// coverity[freed_arg : FALSE] - this does not free m_pOrigViewport, 
it is reffed by m_pScrolledWindow
 gtk_container_add(GTK_CONTAINER(m_pOrigViewport), pChild);
 g_object_unref(pChild);
 gtk_widget_destroy(pViewport);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 sd/source/ui/app/scalectrl.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit b7676c2fa79d035fd8746b6cf9099c94d0c120ec
Author: Caolán McNamara 
AuthorDate: Fri Mar 19 15:28:34 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 21:56:38 2021 +0100

cid#1474113 Out-of-bounds read

order of Precedence is % before -

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

diff --git a/sd/source/ui/app/scalectrl.cxx b/sd/source/ui/app/scalectrl.cxx
index e1826957b03e..6ab069f8a1e4 100644
--- a/sd/source/ui/app/scalectrl.cxx
+++ b/sd/source/ui/app/scalectrl.cxx
@@ -91,7 +91,7 @@ void SdScaleControl::Command(const CommandEvent& rCEvt)
 if (i > 11)
 nX = 1;
 else
-nX = aTable[12 - i % 12];
+nX = aTable[(12 - i) % 12];
 if (i > 11)
 nY = aTable[i % 12];
 else
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 svx/source/form/fmview.cxx |   34 ++
 1 file changed, 18 insertions(+), 16 deletions(-)

New commits:
commit 229558c0bf257e4e559cc1b84bd2918b04c68305
Author: Caolán McNamara 
AuthorDate: Fri Mar 19 15:32:53 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 21:57:04 2021 +0100

cid#1474166 Deference null return value

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

diff --git a/svx/source/form/fmview.cxx b/svx/source/form/fmview.cxx
index feb1e43f2f8e..9adf7d830cf6 100644
--- a/svx/source/form/fmview.cxx
+++ b/svx/source/form/fmview.cxx
@@ -499,24 +499,26 @@ bool FmFormView::KeyInput(const KeyEvent& rKEvt, 
vcl::Window* pWin)
 // tdf#139804 Allow selecting form controls with Alt-
 if (rKeyCode.IsMod2() && rKeyCode.GetCode())
 {
-FmFormPage* pCurPage = GetCurPage();
-for (size_t a = 0; a < pCurPage->GetObjCount(); ++a)
+if (FmFormPage* pCurPage = GetCurPage())
 {
-SdrObject* pObj = pCurPage->GetObj(a);
-FmFormObj* pFormObject = FmFormObj::GetFormObject(pObj);
-if (!pFormObject)
-continue;
-
-Reference xControl = 
pFormObject->GetUnoControl(*this, *pWin);
-if (!xControl.is())
-continue;
-const vcl::I18nHelper& rI18nHelper = 
Application::GetSettings().GetUILocaleI18nHelper();
-VclPtr pWindow = 
VCLUnoHelper::GetWindow(xControl->getPeer());
-if (rI18nHelper.MatchMnemonic(pWindow->GetText(), 
rKEvt.GetCharCode()))
+for (size_t a = 0; a < pCurPage->GetObjCount(); ++a)
 {
-pWindow->GrabFocus();
-bDone = true;
-break;
+SdrObject* pObj = pCurPage->GetObj(a);
+FmFormObj* pFormObject = FmFormObj::GetFormObject(pObj);
+if (!pFormObject)
+continue;
+
+Reference xControl = 
pFormObject->GetUnoControl(*this, *pWin);
+if (!xControl.is())
+continue;
+const vcl::I18nHelper& rI18nHelper = 
Application::GetSettings().GetUILocaleI18nHelper();
+VclPtr pWindow = 
VCLUnoHelper::GetWindow(xControl->getPeer());
+if (rI18nHelper.MatchMnemonic(pWindow->GetText(), 
rKEvt.GetCharCode()))
+{
+pWindow->GrabFocus();
+bDone = true;
+break;
+}
 }
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Caolán McNamara (via logerrit)
 ucb/source/ucp/gio/gio_content.cxx |5 +
 1 file changed, 5 insertions(+)

New commits:
commit b4b0fedaee69fd69aaf1e686f22c2ce3606ef52b
Author: Caolán McNamara 
AuthorDate: Fri Mar 19 15:39:01 2021 +
Commit: Caolán McNamara 
CommitDate: Fri Mar 19 21:57:26 2021 +0100

cid#1473830 Resource leak

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

diff --git a/ucb/source/ucp/gio/gio_content.cxx 
b/ucb/source/ucp/gio/gio_content.cxx
index cfce60270c27..ed8eedbadf41 100644
--- a/ucb/source/ucp/gio/gio_content.cxx
+++ b/ucb/source/ucp/gio/gio_content.cxx
@@ -790,6 +790,8 @@ css::uno::Sequence< css::uno::Any > 
Content::setPropertyValues(
 else
 mpInfo = pNewInfo;
 
+pNewInfo = nullptr;
+
 if (mpFile) //Discard and refetch
 {
 g_object_unref(mpFile);
@@ -801,6 +803,9 @@ css::uno::Sequence< css::uno::Any > 
Content::setPropertyValues(
 notifyPropertiesChange( aChanges );
 }
 
+if (pNewInfo)
+g_object_unref(pNewInfo);
+
 return aRet;
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-6.4' - configure.ac

2021-03-19 Thread Andras Timar (via logerrit)
 configure.ac |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 03685f913d6aa46b8152feb15d3b873b9cc793a7
Author: Andras Timar 
AuthorDate: Fri Mar 19 22:26:29 2021 +0100
Commit: Andras Timar 
CommitDate: Fri Mar 19 22:26:29 2021 +0100

Bump version to 6.4-31

Change-Id: Iff12253d56f3316bff798410d7b90e695821d652

diff --git a/configure.ac b/configure.ac
index b3b8162b9eb3..bd98318047d8 100644
--- a/configure.ac
+++ b/configure.ac
@@ -9,7 +9,7 @@ dnl in order to create a configure script.
 # several non-alphanumeric characters, those are split off and used only for 
the
 # ABOUTBOXPRODUCTVERSIONSUFFIX in openoffice.lst. Why that is necessary, no 
idea.
 
-AC_INIT([Collabora Office],[6.4.10.30],[],[],[https://collaboraoffice.com/])
+AC_INIT([Collabora Office],[6.4.10.31],[],[],[https://collaboraoffice.com/])
 
 dnl libnumbertext needs autoconf 2.68, but that can pick up autoconf268 just 
fine if it is installed
 dnl whereas aclocal (as run by autogen.sh) insists on using autoconf and fails 
hard
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] dictionaries.git: Changes to 'refs/tags/cp-6.4-31'

2021-03-19 Thread Andras Timar (via logerrit)
Tag 'cp-6.4-31' created by Andras Timar  at 
2021-03-19 21:32 +

cp-6.4-31

Changes since co-6.4-20:
Andras Timar (1):
  update Danish dictionary

---
 da_DK/README_da_DK.txt |4 
 da_DK/da_DK.aff| 3401 
 da_DK/da_DK.dic|288016 
+++--
 da_DK/description.xml  |9 
 4 files changed, 139245 insertions(+), 152185 deletions(-)
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] help.git: Changes to 'refs/tags/cp-6.4-31'

2021-03-19 Thread Andras Timar (via logerrit)
Tag 'cp-6.4-31' created by Andras Timar  at 
2021-03-19 21:32 +

cp-6.4-31

Changes since cp-6.4-branch-point-11:
---
 0 files changed
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] translations.git: Changes to 'refs/tags/cp-6.4-31'

2021-03-19 Thread Andras Timar (via logerrit)
Tag 'cp-6.4-31' created by Andras Timar  at 
2021-03-19 21:32 +

cp-6.4-31

Changes since cp-6.4-23:
Andras Timar (1):
  Translation update

---
 source/ar/cui/messages.po  |  130 
 source/ar/dictionaries/da_DK.po|   24 
 source/ar/officecfg/registry/data/org/openoffice/Office/UI.po  |   24 
 source/ar/svtools/messages.po  |   44 
 source/ar/svx/messages.po  |  502 -
 source/ar/sw/messages.po   |  405 
 source/as/cui/messages.po  |  106 
 source/as/dictionaries/da_DK.po|   18 
 source/as/officecfg/registry/data/org/openoffice/Office/UI.po  |   12 
 source/as/svtools/messages.po  |   42 
 source/as/svx/messages.po  |  472 
 source/as/sw/messages.po   |  385 
 source/ast/cui/messages.po |  108 
 source/ast/dictionaries/da_DK.po   |   18 
 source/ast/officecfg/registry/data/org/openoffice/Office/UI.po |   18 
 source/ast/svtools/messages.po |   42 
 source/ast/svx/messages.po |  485 -
 source/ast/sw/messages.po  |  387 
 source/bg/cui/messages.po  |  106 
 source/bg/dictionaries/da_DK.po|   24 
 source/bg/officecfg/registry/data/org/openoffice/Office/UI.po  |   20 
 source/bg/svtools/messages.po  |   42 
 source/bg/svx/messages.po  |  465 
 source/bg/sw/messages.po   |  385 
 source/bn-IN/cui/messages.po   |  106 
 source/bn-IN/dictionaries/da_DK.po |   18 
 source/bn-IN/officecfg/registry/data/org/openoffice/Office/UI.po   |   12 
 source/bn-IN/svtools/messages.po   |   42 
 source/bn-IN/svx/messages.po   |  482 
 source/bn-IN/sw/messages.po|  386 
 source/br/cui/messages.po  |  106 
 source/br/dictionaries/da_DK.po|   18 
 source/br/officecfg/registry/data/org/openoffice/Office/UI.po  |   12 
 source/br/svtools/messages.po  |   42 
 source/br/svx/messages.po  |  487 -
 source/br/sw/messages.po   |  385 
 source/ca-valencia/cui/messages.po |  106 
 source/ca-valencia/dictionaries/da_DK.po   |   18 
 source/ca-valencia/officecfg/registry/data/org/openoffice/Office/UI.po |   12 
 source/ca-valencia/svtools/messages.po |   42 
 source/ca-valencia/svx/messages.po |  471 
 source/ca-valencia/sw/messages.po  |  385 
 source/ca/cui/messages.po  |  106 
 source/ca/dictionaries/da_DK.po|   24 
 source/ca/officecfg/registry/data/org/openoffice/Office/UI.po  |   20 
 source/ca/svtools/messages.po  |   42 
 source/ca/svx/messages.po  |  471 
 source/ca/sw/messages.po   |  385 
 source/cs/cui/messages.po  |  106 
 source/cs/dictionaries/da_DK.po|   24 
 source/cs/officecfg/registry/data/org/openoffice/Office/UI.po  |   20 
 source/cs/svtools/messages.po  |   42 
 source/cs/svx/messages.po  |  465 
 source/cs/sw/messages.po   |  385 
 source/cy/cui/messages.po  |  106 
 source/cy/dictionaries/da_DK.po|   24 
 source/cy/officecfg/registry/data/org/openoffice/Office/UI.po  |   20 
 source/cy/svtools/messages.po  |   42 
 source/cy/svx/messages.po  |  467 
 source/cy/sw/messages.po   |  385 
 source/da/cui/messages.po 

[Libreoffice-commits] core.git: Changes to 'refs/tags/cp-6.4-31'

2021-03-19 Thread Andras Timar (via logerrit)
Tag 'cp-6.4-31' created by Andras Timar  at 
2021-03-19 21:32 +

cp-6.4-31

Changes since cp-6.4-30-1:
---
 0 files changed
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2021-03-19 Thread Luboš Luňák (via logerrit)
 solenv/gbuild/PrecompiledHeaders.mk |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 43f4769ae537310a6fe6a1edfbc6687cc26fd996
Author: Luboš Luňák 
AuthorDate: Fri Mar 19 16:28:09 2021 +0100
Commit: Luboš Luňák 
CommitDate: Fri Mar 19 23:07:14 2021 +0100

VCL_INTERNALS #define doesn't matter for reused PCH

It's only used by VCL, so system PCH doesn't depend on it.

Change-Id: If7a459d2d4c31d370e2b0e7fe57fcf68db72ed3e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112764
Tested-by: Jenkins
Reviewed-by: Luboš Luňák 

diff --git a/solenv/gbuild/PrecompiledHeaders.mk 
b/solenv/gbuild/PrecompiledHeaders.mk
index 0a07d6588436..0d7ccf6610fb 100644
--- a/solenv/gbuild/PrecompiledHeaders.mk
+++ b/solenv/gbuild/PrecompiledHeaders.mk
@@ -141,6 +141,7 @@ gb_PrecompiledHeader_ignore_flags_system := \
 -DSK_USER_CONFIG_HEADER=% \
 -DSKIA_DLL \
 -DGLM_FORCE_CTOR_INIT \
+-DVCL_INTERNALS \
 -include $(SRCDIR)/pch/inc/clangfix.hxx \
 
 # Probably also update pch/inc/clangfix.hxx if you extend the list.
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


Re: Unit Tests failing when built with clang 12

2021-03-19 Thread Luke Benes
The UITest_calc_tests9 failure with clang 12/13  was fixed by 
https://cgit.freedesktop.org/libreoffice/core/commit/?id=9ccbf716ba16
Fix null-pointer-use

Thanks Stephan!


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


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

2021-03-19 Thread Noel Grandin (via logerrit)
 include/svx/unoshape.hxx|7 +--
 svx/source/unodraw/unoshap2.cxx |5 -
 2 files changed, 1 insertion(+), 11 deletions(-)

New commits:
commit e8f9830f48ba08b3b949d8b0fd1d33aad0f5b0e9
Author: Noel Grandin 
AuthorDate: Fri Mar 19 15:37:24 2021 +0200
Commit: Noel Grandin 
CommitDate: Sat Mar 20 07:47:23 2021 +0100

SvxShape: no need to inherit from WeakAggImplHelper1 twice

once here, and once in one of the superclasses

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

diff --git a/include/svx/unoshape.hxx b/include/svx/unoshape.hxx
index dc0d94a80f9c..844c2aac23bb 100644
--- a/include/svx/unoshape.hxx
+++ b/include/svx/unoshape.hxx
@@ -794,11 +794,7 @@ public:
 /***
 *  *
 ***/
-typedef ::cppu::WeakAggImplHelper1<
-css::drawing::XEnhancedCustomShapeDefaulter
-> SvxShape_UnoImplHelper1;
-
-class SVXCORE_DLLPUBLIC SvxCustomShape final : public SvxShapeText, public 
SvxShape_UnoImplHelper1
+class SVXCORE_DLLPUBLIC SvxCustomShape final : public SvxShapeText, public 
css::drawing::XEnhancedCustomShapeDefaulter
 {
 protected:
 using SvxUnoTextRangeBase::setPropertyValue;
@@ -827,7 +823,6 @@ public:
 void SAL_CALL setPropertyValue( const OUString& aPropertyName, const 
css::uno::Any& aValue ) override;
 
 // XTypeProvider
-virtual css::uno::Sequence< css::uno::Type > SAL_CALL getTypes(  ) 
override;
 virtual css::uno::Sequence< sal_Int8 > SAL_CALL getImplementationId(  ) 
override;
 
 //XEnhancedCustomShapeDefaulter
diff --git a/svx/source/unodraw/unoshap2.cxx b/svx/source/unodraw/unoshap2.cxx
index 2c4487b4cec6..56d1c1569479 100644
--- a/svx/source/unodraw/unoshap2.cxx
+++ b/svx/source/unodraw/unoshap2.cxx
@@ -1582,11 +1582,6 @@ uno::Any SAL_CALL SvxCustomShape::queryAggregation( 
const uno::Type & rType )
 return aReturn;
 }
 
-uno::Sequence< uno::Type > SAL_CALL SvxCustomShape::getTypes()
-{
-return SvxShapeText::getTypes();
-}
-
 uno::Sequence< sal_Int8 > SAL_CALL SvxCustomShape::getImplementationId()
 {
 return css::uno::Sequence();
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: framework/source framework/util

2021-03-19 Thread Noel (via logerrit)
 framework/source/uifactory/uicontrollerfactory.cxx |   21 +
 framework/util/fwk.component   |3 ++-
 2 files changed, 3 insertions(+), 21 deletions(-)

New commits:
commit ea004488f6d55eaebe58c595ac53392bf936b5e4
Author: Noel 
AuthorDate: Thu Mar 18 11:23:01 2021 +0200
Commit: Noel Grandin 
CommitDate: Sat Mar 20 07:52:33 2021 +0100

use single-use attribute for ToolbarControllerFactory

instead of rtl::Instance, which means it will get
cleaned up when UNO shuts down

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

diff --git a/framework/source/uifactory/uicontrollerfactory.cxx 
b/framework/source/uifactory/uicontrollerfactory.cxx
index 5f81f80be026..493d1d5522d0 100644
--- a/framework/source/uifactory/uicontrollerfactory.cxx
+++ b/framework/source/uifactory/uicontrollerfactory.cxx
@@ -288,24 +288,6 @@ ToolbarControllerFactory::ToolbarControllerFactory( const 
Reference< XComponentC
 {
 }
 
-struct ToolbarControllerFactoryInstance {
-explicit ToolbarControllerFactoryInstance(
-css::uno::Reference const & context):
-instance(static_cast(
-new ToolbarControllerFactory(context)))
-{
-}
-
-css::uno::Reference instance;
-};
-
-struct ToolbarControllerFactorySingleton:
-public rtl::StaticWithArg<
-ToolbarControllerFactoryInstance,
-css::uno::Reference,
-ToolbarControllerFactorySingleton>
-{};
-
 class StatusbarControllerFactory :  public UIControllerFactory
 {
 public:
@@ -366,8 +348,7 @@ 
com_sun_star_comp_framework_ToolBarControllerFactory_get_implementation(
 css::uno::XComponentContext *context,
 css::uno::Sequence const &)
 {
-return cppu::acquire(static_cast(
-ToolbarControllerFactorySingleton::get(context).instance.get()));
+return cppu::acquire(new ToolbarControllerFactory(context));
 }
 
 extern "C" SAL_DLLPUBLIC_EXPORT css::uno::XInterface *
diff --git a/framework/util/fwk.component b/framework/util/fwk.component
index 2d71213468d6..80af7267b774 100644
--- a/framework/util/fwk.component
+++ b/framework/util/fwk.component
@@ -145,7 +145,8 @@
 
   
   
+  
constructor="com_sun_star_comp_framework_ToolBarControllerFactory_get_implementation"
+  single-instance="true">
 
 
   
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits