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

2020-11-27 Thread Noel (via logerrit)
 sw/source/filter/xml/xmltbli.cxx |  128 ---
 sw/source/filter/xml/xmltbli.hxx |4 -
 2 files changed, 54 insertions(+), 78 deletions(-)

New commits:
commit e46be5018d78097f9cd75a36992a2af88652fe4c
Author: Noel 
AuthorDate: Fri Nov 27 15:27:33 2020 +0200
Commit: Noel Grandin 
CommitDate: Sat Nov 28 08:49:59 2020 +0100

fastparser in SwXMLTableContext

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

diff --git a/sw/source/filter/xml/xmltbli.cxx b/sw/source/filter/xml/xmltbli.cxx
index e3061074329d..e26da2ab50b0 100644
--- a/sw/source/filter/xml/xmltbli.cxx
+++ b/sw/source/filter/xml/xmltbli.cxx
@@ -714,8 +714,8 @@ class SwXMLTableColContext_Impl : public SvXMLImportContext
 public:
 
 SwXMLTableColContext_Impl(
-SwXMLImport& rImport, sal_uInt16 nPrfx, const OUString& rLName,
-const Reference< xml::sax::XAttributeList > & xAttrList,
+SwXMLImport& rImport,
+const Reference< xml::sax::XFastAttributeList > & xAttrList,
 SwXMLTableContext *pTable );
 
 SwXMLImport& GetSwImport() { return 
static_cast(GetImport()); }
@@ -724,45 +724,43 @@ public:
 }
 
 SwXMLTableColContext_Impl::SwXMLTableColContext_Impl(
-SwXMLImport& rImport, sal_uInt16 nPrfx, const OUString& rLName,
-const Reference< xml::sax::XAttributeList > & xAttrList,
+SwXMLImport& rImport,
+const Reference< xml::sax::XFastAttributeList > & xAttrList,
 SwXMLTableContext *pTable ) :
-SvXMLImportContext( rImport, nPrfx, rLName ),
+SvXMLImportContext( rImport ),
 xMyTable( pTable )
 {
 sal_uInt32 nColRep = 1;
 OUString aStyleName, aDfltCellStyleName;
 
-sal_Int16 nAttrCount = xAttrList.is() ? xAttrList->getLength() : 0;
-for( sal_Int16 i=0; i < nAttrCount; i++ )
+for( auto &aIter : sax_fastparser::castToFastAttributeList( xAttrList ) )
 {
-const OUString& rAttrName = xAttrList->getNameByIndex( i );
-
-OUString aLocalName;
-const sal_uInt16 nPrefix =
-GetImport().GetNamespaceMap().GetKeyByAttrName( rAttrName,
-&aLocalName );
-const OUString& rValue = xAttrList->getValueByIndex( i );
-if( XML_NAMESPACE_TABLE == nPrefix )
+OUString sValue = aIter.toString();
+switch (aIter.getToken())
 {
-if( IsXMLToken( aLocalName, XML_STYLE_NAME ) )
-aStyleName = rValue;
-else if( IsXMLToken( aLocalName, XML_NUMBER_COLUMNS_REPEATED ) )
+case XML_ELEMENT(TABLE, XML_STYLE_NAME):
+aStyleName = sValue;
+break;
+case XML_ELEMENT(TABLE, XML_NUMBER_COLUMNS_REPEATED):
 {
-nColRep = static_cast(std::max(1, 
rValue.toInt32()));
+nColRep = static_cast(std::max(1, 
sValue.toInt32()));
 if (nColRep > 256)
 {
 SAL_INFO("sw.xml", "ignoring huge 
table:number-columns-repeated " << nColRep);
 nColRep = 1;
 }
+break;
 }
-else if( IsXMLToken( aLocalName, XML_DEFAULT_CELL_STYLE_NAME ) )
-aDfltCellStyleName = rValue;
-}
-else if ( (XML_NAMESPACE_XML == nPrefix) &&
- IsXMLToken( aLocalName, XML_ID ) )
-{
-//FIXME where to put this??? columns do not actually exist in writer...
+case XML_ELEMENT(TABLE, XML_DEFAULT_CELL_STYLE_NAME):
+aDfltCellStyleName = sValue;
+break;
+case XML_ELEMENT(XML, XML_ID):
+{
+//FIXME where to put this??? columns do not actually exist in 
writer...
+break;
+}
+default:
+XMLOFF_WARN_UNKNOWN("sw", aIter);
 }
 }
 
@@ -803,13 +801,11 @@ class SwXMLTableColsContext_Impl : public 
SvXMLImportContext
 public:
 
 SwXMLTableColsContext_Impl(
-SwXMLImport& rImport, sal_uInt16 nPrfx,
-const OUString& rLName,
+SwXMLImport& rImport,
 SwXMLTableContext *pTable );
 
-virtual SvXMLImportContextRef CreateChildContext(
-sal_uInt16 nPrefix, const OUString& rLocalName,
-const Reference< xml::sax::XAttributeList > & xAttrList ) override;
+virtual css::uno::Reference< css::xml::sax::XFastContextHandler > SAL_CALL 
createFastChildContext(
+sal_Int32 Element, const css::uno::Reference< 
css::xml::sax::XFastAttributeList > & xAttrList ) override;
 
 SwXMLImport& GetSwImport() { return 
static_cast(GetImport()); }
 };
@@ -817,26 +813,23 @@ public:
 }
 
 SwXMLTableColsContext_Impl::SwXMLTableColsContext_Impl(
-SwXMLImport& rImport, sal_uInt16 nPrfx, const OUString& rLNam

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

2020-11-27 Thread Noel (via logerrit)
 sc/source/filter/xml/xmlconti.cxx |   28 
 sc/source/filter/xml/xmlconti.hxx |4 
 2 files changed, 32 deletions(-)

New commits:
commit 19382197332ee94b4725160267e987b447b49245
Author: Noel 
AuthorDate: Fri Nov 27 15:12:34 2020 +0200
Commit: Noel Grandin 
CommitDate: Sat Nov 28 08:33:19 2020 +0100

fastparser in ScXMLContentContext

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

diff --git a/sc/source/filter/xml/xmlconti.cxx 
b/sc/source/filter/xml/xmlconti.cxx
index 1c5d55292fdc..871cabf888be 100644
--- a/sc/source/filter/xml/xmlconti.cxx
+++ b/sc/source/filter/xml/xmlconti.cxx
@@ -46,34 +46,6 @@ ScXMLContentContext::~ScXMLContentContext()
 {
 }
 
-SvXMLImportContextRef ScXMLContentContext::CreateChildContext( sal_uInt16 
nPrefix,
-const OUString& rLName,
-const 
css::uno::Reference& xAttrList )
-{
-if ((nPrefix == XML_NAMESPACE_TEXT) && IsXMLToken(rLName, XML_S))
-{
-sal_Int32 nRepeat(0);
-sal_Int16 nAttrCount = xAttrList.is() ? xAttrList->getLength() : 0;
-for( sal_Int16 i=0; i < nAttrCount; ++i )
-{
-const OUString& sAttrName(xAttrList->getNameByIndex( i ));
-const OUString& sAttrValue(xAttrList->getValueByIndex( i ));
-OUString aLocalName;
-sal_uInt16 nPrfx = 
GetScImport().GetNamespaceMap().GetKeyByAttrName(
-sAttrName, &aLocalName );
-if ((nPrfx == XML_NAMESPACE_TEXT) && IsXMLToken(aLocalName, XML_C))
-nRepeat = sAttrValue.toInt32();
-}
-if (nRepeat)
-for (sal_Int32 j = 0; j < nRepeat; ++j)
-sValue.append(' ');
-else
-sValue.append(' ');
-}
-
-return new SvXMLImportContext( GetImport(), nPrefix, rLName );
-}
-
 css::uno::Reference< css::xml::sax::XFastContextHandler > 
ScXMLContentContext::createFastChildContext(
 sal_Int32 nElement, const css::uno::Reference< 
css::xml::sax::XFastAttributeList >& xAttrList )
 {
diff --git a/sc/source/filter/xml/xmlconti.hxx 
b/sc/source/filter/xml/xmlconti.hxx
index 7fd2cb0274db..2eb568a27e24 100644
--- a/sc/source/filter/xml/xmlconti.hxx
+++ b/sc/source/filter/xml/xmlconti.hxx
@@ -36,10 +36,6 @@ public:
 
 virtual ~ScXMLContentContext() override;
 
-virtual SvXMLImportContextRef CreateChildContext( sal_uInt16 nPrefix,
- const OUString& rLocalName,
- const 
css::uno::Reference& xAttrList ) override;
-
 virtual css::uno::Reference< css::xml::sax::XFastContextHandler > SAL_CALL 
createFastChildContext(
 sal_Int32 nElement, const css::uno::Reference< 
css::xml::sax::XFastAttributeList >& xAttrList ) override;
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Noel (via logerrit)
 xmloff/source/draw/ximpstyl.cxx |   23 +++
 xmloff/source/draw/ximpstyl.hxx |5 -
 2 files changed, 7 insertions(+), 21 deletions(-)

New commits:
commit abdf72c6dc8e021de29459f129e39d6ff279c05c
Author: Noel 
AuthorDate: Fri Nov 27 15:03:56 2020 +0200
Commit: Noel Grandin 
CommitDate: Sat Nov 28 07:46:35 2020 +0100

fastparser in SdXMLMasterStylesContext

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

diff --git a/xmloff/source/draw/ximpstyl.cxx b/xmloff/source/draw/ximpstyl.cxx
index 8cf89f450484..3a25c4c65027 100644
--- a/xmloff/source/draw/ximpstyl.cxx
+++ b/xmloff/source/draw/ximpstyl.cxx
@@ -1307,7 +1307,11 @@ css::uno::Reference< css::xml::sax::XFastContextHandler 
> SdXMLMasterStylesConte
 sal_Int32 nElement,
 const css::uno::Reference< css::xml::sax::XFastAttributeList >& xAttrList )
 {
-if( nElement == XML_ELEMENT(STYLE, XML_MASTER_PAGE) )
+if( nElement == XML_ELEMENT(DRAW, XML_LAYER_SET) )
+{
+return new SdXMLLayerSetContext( GetImport() );
+}
+else if( nElement == XML_ELEMENT(STYLE, XML_MASTER_PAGE) )
 {
 // style:masterpage inside office:styles context
 uno::Reference< drawing::XDrawPage > xNewMasterPage;
@@ -1357,24 +1361,11 @@ css::uno::Reference< css::xml::sax::XFastContextHandler 
> SdXMLMasterStylesConte
 }
 }
 }
+else
+XMLOFF_WARN_UNKNOWN_ELEMENT("xmloff", nElement);
 return nullptr;
 }
 
-SvXMLImportContextRef SdXMLMasterStylesContext::CreateChildContext(
-sal_uInt16 nPrefix,
-const OUString& rLocalName,
-const uno::Reference< xml::sax::XAttributeList >& /*xAttrList*/)
-{
-SvXMLImportContextRef xContext;
-
-if( (nPrefix == XML_NAMESPACE_DRAW) && IsXMLToken( rLocalName, 
XML_LAYER_SET ) )
-{
-xContext = new SdXMLLayerSetContext( GetImport() );
-}
-
-return xContext;
-}
-
 SdXMLHeaderFooterDeclContext::SdXMLHeaderFooterDeclContext(SvXMLImport& 
rImport,
 const css::uno::Reference< css::xml::sax::XFastAttributeList >& xAttrList)
 : SvXMLStyleContext( rImport )
diff --git a/xmloff/source/draw/ximpstyl.hxx b/xmloff/source/draw/ximpstyl.hxx
index a7eee862783c..cebb2f93c3bf 100644
--- a/xmloff/source/draw/ximpstyl.hxx
+++ b/xmloff/source/draw/ximpstyl.hxx
@@ -224,11 +224,6 @@ public:
 
 virtual css::uno::Reference< css::xml::sax::XFastContextHandler > SAL_CALL 
createFastChildContext(
 sal_Int32 nElement, const css::uno::Reference< 
css::xml::sax::XFastAttributeList >& AttrList ) override;
-
-virtual SvXMLImportContextRef CreateChildContext(
-sal_uInt16 nPrefix,
-const OUString& rLocalName,
-const css::uno::Reference< css::xml::sax::XAttributeList >& xAttrList 
) override;
 };
 
 // ,  and 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Noel (via logerrit)
 xmloff/source/text/XMLFootnoteBodyImportContext.cxx |7 +--
 xmloff/source/text/XMLFootnoteBodyImportContext.hxx |6 ---
 xmloff/source/text/XMLFootnoteImportContext.cxx |   38 +++-
 xmloff/source/text/XMLFootnoteImportContext.hxx |6 +--
 4 files changed, 19 insertions(+), 38 deletions(-)

New commits:
commit 818be5ed71d6c4eb515c907add4179100eed6072
Author: Noel 
AuthorDate: Fri Nov 27 14:56:19 2020 +0200
Commit: Noel Grandin 
CommitDate: Sat Nov 28 06:53:45 2020 +0100

fastparser in XMLFootnoteBodyImportContext

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

diff --git a/xmloff/source/text/XMLFootnoteBodyImportContext.cxx 
b/xmloff/source/text/XMLFootnoteBodyImportContext.cxx
index dc560f624c10..8b0268318355 100644
--- a/xmloff/source/text/XMLFootnoteBodyImportContext.cxx
+++ b/xmloff/source/text/XMLFootnoteBodyImportContext.cxx
@@ -28,11 +28,8 @@ using ::com::sun::star::uno::Reference;
 using ::com::sun::star::xml::sax::XAttributeList;
 
 
-XMLFootnoteBodyImportContext::XMLFootnoteBodyImportContext(
-SvXMLImport& rImport,
-sal_uInt16 nPrfx,
-const OUString& rLocalName ) :
-SvXMLImportContext(rImport, nPrfx, rLocalName)
+XMLFootnoteBodyImportContext::XMLFootnoteBodyImportContext( SvXMLImport& 
rImport ) :
+SvXMLImportContext(rImport)
 {
 }
 
diff --git a/xmloff/source/text/XMLFootnoteBodyImportContext.hxx 
b/xmloff/source/text/XMLFootnoteBodyImportContext.hxx
index 647867a5cdcf..19007558899e 100644
--- a/xmloff/source/text/XMLFootnoteBodyImportContext.hxx
+++ b/xmloff/source/text/XMLFootnoteBodyImportContext.hxx
@@ -37,11 +37,7 @@ class XMLFootnoteBodyImportContext : public 
SvXMLImportContext
 
 public:
 
-
-XMLFootnoteBodyImportContext(
-SvXMLImport& rImport,
-sal_uInt16 nPrfx,
-const OUString& rLocalName );
+XMLFootnoteBodyImportContext( SvXMLImport& rImport );
 
 protected:
 
diff --git a/xmloff/source/text/XMLFootnoteImportContext.cxx 
b/xmloff/source/text/XMLFootnoteImportContext.cxx
index 645e910c8a36..95a30097675e 100644
--- a/xmloff/source/text/XMLFootnoteImportContext.cxx
+++ b/xmloff/source/text/XMLFootnoteImportContext.cxx
@@ -34,6 +34,7 @@
 #include 
 #include 
 #include 
+#include 
 
 
 using namespace ::com::sun::star::uno;
@@ -164,49 +165,38 @@ void XMLFootnoteImportContext::endFastElement(sal_Int32 )
 }
 }
 
-SvXMLImportContextRef XMLFootnoteImportContext::CreateChildContext(
-sal_uInt16 p_nPrefix,
-const OUString& rLocalName,
-const Reference & xAttrList )
+css::uno::Reference< css::xml::sax::XFastContextHandler > 
XMLFootnoteImportContext::createFastChildContext(
+sal_Int32 nElement, const css::uno::Reference< 
css::xml::sax::XFastAttributeList >& xAttrList )
 {
 SvXMLImportContextRef xContext;
 
-static const SvXMLTokenMap aTokenMap(aFootnoteChildTokenMap);
-
-switch(aTokenMap.Get(p_nPrefix, rLocalName))
+switch(nElement)
 {
-case XML_TOK_FTN_NOTE_CITATION:
+case XML_ELEMENT(TEXT, XML_NOTE_CITATION):
 {
 // little hack: we only care for one attribute of the citation
 //  element. We handle that here, and then return a
 //  default context.
-sal_Int16 nLength = xAttrList->getLength();
-for(sal_Int16 nAttr = 0; nAttr < nLength; nAttr++)
+for( auto& aIter : 
sax_fastparser::castToFastAttributeList(xAttrList) )
 {
-OUString sLocalName;
-sal_uInt16 nPrefix = GetImport().GetNamespaceMap().
-GetKeyByAttrName( xAttrList->getNameByIndex(nAttr),
-  &sLocalName );
-
-if ( (nPrefix == XML_NAMESPACE_TEXT) &&
- IsXMLToken( sLocalName, XML_LABEL ) )
-{
-xFootnote->setLabel(xAttrList->getValueByIndex(nAttr));
-}
+if ( aIter.getToken() == XML_ELEMENT(TEXT, XML_LABEL) )
+xFootnote->setLabel(aIter.toString());
 }
 
 // ignore content: return default context
 break;
 }
 
-case XML_TOK_FTN_NOTE_BODY:
+case XML_ELEMENT(TEXT, XML_NOTE_BODY):
 // return footnote body
-xContext = new XMLFootnoteBodyImportContext(GetImport(),
-p_nPrefix, rLocalName);
+xContext = new XMLFootnoteBodyImportContext(GetImport());
 break;
+
+default:
+XMLOFF_WARN_UNKNOWN_ELEMENT("xmloff", nElement);
 }
 
-return xContext;
+return xContext.get();
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/xmloff/source/text/XMLFootnoteImportContext.hxx 
b/xmloff/source/text/XMLFootnoteImportContext.hxx
index 72ad330c876

[Libreoffice-commits] core.git: Branch 'libreoffice-7-1' - icon-themes/elementary icon-themes/elementary_svg

2020-11-27 Thread Rizal Muttaqin (via logerrit)
 icon-themes/elementary/cmd/32/addressbooksource.png  |binary
 icon-themes/elementary/cmd/32/bmpmask.png|binary
 icon-themes/elementary/cmd/32/decrementlevel.png |binary
 icon-themes/elementary/cmd/32/decrementsublevels.png |binary
 icon-themes/elementary/cmd/32/downsearch.png |binary
 icon-themes/elementary/cmd/32/duplicatepage.png  |binary
 icon-themes/elementary/cmd/32/firstpage.png  |binary
 icon-themes/elementary/cmd/32/firstrecord.png|binary
 icon-themes/elementary/cmd/32/incrementlevel.png |binary
 icon-themes/elementary/cmd/32/incrementsublevels.png |binary
 icon-themes/elementary/cmd/32/lastpage.png   |binary
 icon-themes/elementary/cmd/32/lastrecord.png |binary
 icon-themes/elementary/cmd/32/movedown.png   |binary
 icon-themes/elementary/cmd/32/movedownsubitems.png   |binary
 icon-themes/elementary/cmd/32/movepagefirst.png  |binary
 icon-themes/elementary/cmd/32/movepagelast.png   |binary
 icon-themes/elementary/cmd/32/moveup.png |binary
 icon-themes/elementary/cmd/32/moveupsubitems.png |binary
 icon-themes/elementary/cmd/32/nextpage.png   |binary
 icon-themes/elementary/cmd/32/nextrecord.png |binary
 icon-themes/elementary/cmd/32/previouspage.png   |binary
 icon-themes/elementary/cmd/32/prevrecord.png |binary
 icon-themes/elementary/cmd/32/upsearch.png   |binary
 icon-themes/elementary/cmd/lc_addressbooksource.png  |binary
 icon-themes/elementary/cmd/lc_bmpmask.png|binary
 icon-themes/elementary/cmd/lc_decrementlevel.png |binary
 icon-themes/elementary/cmd/lc_decrementsublevels.png |binary
 icon-themes/elementary/cmd/lc_downsearch.png |binary
 icon-themes/elementary/cmd/lc_firstpage.png  |binary
 icon-themes/elementary/cmd/lc_firstrecord.png|binary
 icon-themes/elementary/cmd/lc_incrementlevel.png |binary
 icon-themes/elementary/cmd/lc_incrementsublevels.png |binary
 icon-themes/elementary/cmd/lc_lastpage.png   |binary
 icon-themes/elementary/cmd/lc_lastrecord.png |binary
 icon-themes/elementary/cmd/lc_movedown.png   |binary
 icon-themes/elementary/cmd/lc_movedownsubitems.png   |binary
 icon-themes/elementary/cmd/lc_movepagefirst.png  |binary
 icon-themes/elementary/cmd/lc_movepagelast.png   |binary
 icon-themes/elementary/cmd/lc_moveup.png |binary
 icon-themes/elementary/cmd/lc_moveupsubitems.png |binary
 icon-themes/elementary/cmd/lc_nextpage.png   |binary
 icon-themes/elementary/cmd/lc_nextrecord.png |binary
 icon-themes/elementary/cmd/lc_previouspage.png   |binary
 icon-themes/elementary/cmd/lc_prevrecord.png |binary
 icon-themes/elementary/cmd/lc_upsearch.png   |binary
 icon-themes/elementary/cmd/sc_addressbooksource.png  |binary
 icon-themes/elementary/cmd/sc_bmpmask.png|binary
 icon-themes/elementary/cmd/sc_decrementlevel.png |binary
 icon-themes/elementary/cmd/sc_decrementsublevels.png |binary
 icon-themes/elementary/cmd/sc_downsearch.png |binary
 icon-themes/elementary/cmd/sc_firstpage.png  |binary
 icon-themes/elementary/cmd/sc_firstrecord.png|binary
 icon-themes/elementary/cmd/sc_incrementlevel.png |binary
 icon-themes/elementary/cmd/sc_incrementsublevels.png |binary
 icon-themes/elementary/cmd/sc_lastpage.png   |binary
 icon-themes/elementary/cmd/sc_lastrecord.png |binary
 icon-themes/elementary/cmd/sc_movedown.png   |binary
 icon-themes/elementary/cmd/sc_movedownsubitems.png   |binary
 icon-themes/elementary/cmd/sc_movepagefirst.png  |binary
 icon-themes/elementary/cmd/sc_movepagelast.png   |binary
 icon-themes/elementary/cmd/sc_moveup.png |binary
 icon-themes/elementary/cmd/sc_moveupsubitems.png |binary
 icon-themes/elementary/cmd/sc_nextpage.png   |binary
 icon-themes/elementary/cmd/sc_nextrecord.png |binary
 icon-themes/elementary/cmd/sc_previouspage.png   |binary
 icon-themes/elementary/cmd/sc_prevrecord.png |binary
 icon-themes/elementary/cmd/sc_upsearch.png   |binary
 icon-themes/elementary/links.txt |   87 +--
 icon-themes/elementary_svg/cmd/32/addressbooksource.svg  |2 
 icon-themes/elementary_svg/cmd/32/bmpmask.svg|2 
 icon-themes/elementary_svg/cmd/32/decrementlevel.svg |1 
 icon-themes/elementary_svg/cmd/32/decrementsublevels.svg |2 
 icon-themes/elementary_svg/cmd/32/downsearch.svg |2 
 icon-themes/elementary_svg/cmd/32/duplicatepage.svg  |2 
 icon-themes/elementary_svg/cmd/32/firstpage.svg  |1 
 icon-themes/elementary_svg/cmd/32

[Libreoffice-commits] core.git: icon-themes/elementary icon-themes/elementary_svg

2020-11-27 Thread Rizal Muttaqin (via logerrit)
 icon-themes/elementary/cmd/32/addressbooksource.png  |binary
 icon-themes/elementary/cmd/32/bmpmask.png|binary
 icon-themes/elementary/cmd/32/decrementlevel.png |binary
 icon-themes/elementary/cmd/32/decrementsublevels.png |binary
 icon-themes/elementary/cmd/32/downsearch.png |binary
 icon-themes/elementary/cmd/32/duplicatepage.png  |binary
 icon-themes/elementary/cmd/32/firstpage.png  |binary
 icon-themes/elementary/cmd/32/firstrecord.png|binary
 icon-themes/elementary/cmd/32/incrementlevel.png |binary
 icon-themes/elementary/cmd/32/incrementsublevels.png |binary
 icon-themes/elementary/cmd/32/lastpage.png   |binary
 icon-themes/elementary/cmd/32/lastrecord.png |binary
 icon-themes/elementary/cmd/32/movedown.png   |binary
 icon-themes/elementary/cmd/32/movedownsubitems.png   |binary
 icon-themes/elementary/cmd/32/movepagefirst.png  |binary
 icon-themes/elementary/cmd/32/movepagelast.png   |binary
 icon-themes/elementary/cmd/32/moveup.png |binary
 icon-themes/elementary/cmd/32/moveupsubitems.png |binary
 icon-themes/elementary/cmd/32/nextpage.png   |binary
 icon-themes/elementary/cmd/32/nextrecord.png |binary
 icon-themes/elementary/cmd/32/previouspage.png   |binary
 icon-themes/elementary/cmd/32/prevrecord.png |binary
 icon-themes/elementary/cmd/32/upsearch.png   |binary
 icon-themes/elementary/cmd/lc_addressbooksource.png  |binary
 icon-themes/elementary/cmd/lc_bmpmask.png|binary
 icon-themes/elementary/cmd/lc_decrementlevel.png |binary
 icon-themes/elementary/cmd/lc_decrementsublevels.png |binary
 icon-themes/elementary/cmd/lc_downsearch.png |binary
 icon-themes/elementary/cmd/lc_firstpage.png  |binary
 icon-themes/elementary/cmd/lc_firstrecord.png|binary
 icon-themes/elementary/cmd/lc_incrementlevel.png |binary
 icon-themes/elementary/cmd/lc_incrementsublevels.png |binary
 icon-themes/elementary/cmd/lc_lastpage.png   |binary
 icon-themes/elementary/cmd/lc_lastrecord.png |binary
 icon-themes/elementary/cmd/lc_movedown.png   |binary
 icon-themes/elementary/cmd/lc_movedownsubitems.png   |binary
 icon-themes/elementary/cmd/lc_movepagefirst.png  |binary
 icon-themes/elementary/cmd/lc_movepagelast.png   |binary
 icon-themes/elementary/cmd/lc_moveup.png |binary
 icon-themes/elementary/cmd/lc_moveupsubitems.png |binary
 icon-themes/elementary/cmd/lc_nextpage.png   |binary
 icon-themes/elementary/cmd/lc_nextrecord.png |binary
 icon-themes/elementary/cmd/lc_previouspage.png   |binary
 icon-themes/elementary/cmd/lc_prevrecord.png |binary
 icon-themes/elementary/cmd/lc_upsearch.png   |binary
 icon-themes/elementary/cmd/sc_addressbooksource.png  |binary
 icon-themes/elementary/cmd/sc_bmpmask.png|binary
 icon-themes/elementary/cmd/sc_decrementlevel.png |binary
 icon-themes/elementary/cmd/sc_decrementsublevels.png |binary
 icon-themes/elementary/cmd/sc_downsearch.png |binary
 icon-themes/elementary/cmd/sc_firstpage.png  |binary
 icon-themes/elementary/cmd/sc_firstrecord.png|binary
 icon-themes/elementary/cmd/sc_incrementlevel.png |binary
 icon-themes/elementary/cmd/sc_incrementsublevels.png |binary
 icon-themes/elementary/cmd/sc_lastpage.png   |binary
 icon-themes/elementary/cmd/sc_lastrecord.png |binary
 icon-themes/elementary/cmd/sc_movedown.png   |binary
 icon-themes/elementary/cmd/sc_movedownsubitems.png   |binary
 icon-themes/elementary/cmd/sc_movepagefirst.png  |binary
 icon-themes/elementary/cmd/sc_movepagelast.png   |binary
 icon-themes/elementary/cmd/sc_moveup.png |binary
 icon-themes/elementary/cmd/sc_moveupsubitems.png |binary
 icon-themes/elementary/cmd/sc_nextpage.png   |binary
 icon-themes/elementary/cmd/sc_nextrecord.png |binary
 icon-themes/elementary/cmd/sc_previouspage.png   |binary
 icon-themes/elementary/cmd/sc_prevrecord.png |binary
 icon-themes/elementary/cmd/sc_upsearch.png   |binary
 icon-themes/elementary/links.txt |   87 +--
 icon-themes/elementary_svg/cmd/32/addressbooksource.svg  |2 
 icon-themes/elementary_svg/cmd/32/bmpmask.svg|2 
 icon-themes/elementary_svg/cmd/32/decrementlevel.svg |1 
 icon-themes/elementary_svg/cmd/32/decrementsublevels.svg |2 
 icon-themes/elementary_svg/cmd/32/downsearch.svg |2 
 icon-themes/elementary_svg/cmd/32/duplicatepage.svg  |2 
 icon-themes/elementary_svg/cmd/32/firstpage.svg  |1 
 icon-themes/elementary_svg/cmd/32

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

2020-11-27 Thread Julien Nabet (via logerrit)
 connectivity/source/commontools/TTableHelper.cxx |5 +
 1 file changed, 5 insertions(+)

New commits:
commit 5dae252e76da9c95049ba8124422a1f7b2f10596
Author: Julien Nabet 
AuthorDate: Thu Nov 26 23:12:46 2020 +0100
Commit: Julien Nabet 
CommitDate: Fri Nov 27 23:01:48 2020 +0100

tdf#137745: crash, when deleting tables and changed relationship isn't 
changed

bt here:
https://bugs.documentfoundation.org/attachment.cgi?id=166782

some explanations here:
https://bugs.documentfoundation.org/show_bug.cgi?id=137745#c8

Change-Id: I1d9dc8d2e3ad7e3e36c4687cb9b2c5990f2d2c8d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106715
(cherry picked from commit 4335810b00abb9b00a9d81caa5ffe09a3ea927fd)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106742
Reviewed-by: Lionel Mamane 
Tested-by: Jenkins

diff --git a/connectivity/source/commontools/TTableHelper.cxx 
b/connectivity/source/commontools/TTableHelper.cxx
index e86af9673052..03724d669704 100644
--- a/connectivity/source/commontools/TTableHelper.cxx
+++ b/connectivity/source/commontools/TTableHelper.cxx
@@ -72,6 +72,11 @@ public:
 }
 virtual void SAL_CALL elementRemoved( const 
css::container::ContainerEvent& Event ) override
 {
+// tdf#137745, perhaps connectivity::OTableHelper::disposing() has 
been called
+// which called OTableContainerListener::clear(), so m_pComponent may 
be null
+if (m_pComponent == nullptr)
+return;
+
 OUString sName;
 Event.Accessor  >>= sName;
 if ( m_aRefNames.find(sName) != m_aRefNames.end() )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-7-1' - cui/inc extras/source vcl/Package_tipoftheday.mk

2020-11-27 Thread andreas kainz (via logerrit)
 cui/inc/tipoftheday.hrc  |8 
 extras/source/tipoftheday/formdocuments.png  |binary
 extras/source/tipoftheday/masterdocument.png |binary
 extras/source/tipoftheday/printnote.png  |binary
 extras/source/tipoftheday/statusbar.png  |binary
 vcl/Package_tipoftheday.mk   |4 
 6 files changed, 8 insertions(+), 4 deletions(-)

New commits:
commit dca40f028a6e050ed990a7604c09653af09e5d13
Author: andreas kainz 
AuthorDate: Fri Nov 27 08:54:43 2020 +0100
Commit: Andreas Kainz 
CommitDate: Fri Nov 27 22:43:52 2020 +0100

tdf#127294 add ToD Previews

Change-Id: Icd049b0ae313e4c3095975a98f8765372fa22abf
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106719
Tested-by: Jenkins
Reviewed-by: Andreas Kainz 
(cherry picked from commit c12bc5757c021f8ba3f802b26805964a11aaa71d)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106733

diff --git a/cui/inc/tipoftheday.hrc b/cui/inc/tipoftheday.hrc
index f5d06db9657e..95806eb9bff1 100644
--- a/cui/inc/tipoftheday.hrc
+++ b/cui/inc/tipoftheday.hrc
@@ -47,13 +47,13 @@ const std::tuple 
TIPOFTHEDAY_STRINGARRAY[] =
 {
  { NC_("RID_CUI_TIPOFTHEDAY", "%PRODUCTNAME offers a variety of user 
interface options to make you feel at home"), ".uno:ToolbarModeUI", 
"toolbarmode.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "Need to allow changes to parts of a 
read-only document in Writer? Insert frames or sections that can authorize 
changes."), 
"https://help.libreoffice.org/%PRODUCTVERSION/%LANGUAGENAME/text/swriter/guide/section_edit.html";,
 "tipoftheday_w.png"}, //local help missing
- { NC_("RID_CUI_TIPOFTHEDAY", "To print the notes of your slides go to 
File ▸ Print ▸ Impress tab and select Notes under Document ▸ Type."), "", 
"tipoftheday_i.png"},
+ { NC_("RID_CUI_TIPOFTHEDAY", "To print the notes of your slides go to 
File ▸ Print ▸ Impress tab and select Notes under Document ▸ Type."), "", 
"printnote.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "To start temporarily with a fresh user 
profile, or to restore a non-working %PRODUCTNAME, use Help ▸ Restart in Safe 
Mode."), "svx/ui/safemodedialog/SafeModeDialog", ""}, 
//https://help.libreoffice.org/%PRODUCTVERSION/%LANGUAGENAME/text/shared/01/profile_safe_mode.html
  { NC_("RID_CUI_TIPOFTHEDAY", "Writing a book? %PRODUCTNAME master 
document lets you manage large documents as a container for individual 
%PRODUCTNAME Writer files."), 
"https://help.libreoffice.org/%PRODUCTVERSION/%LANGUAGENAME/text/swriter/guide/globaldoc.html";,
 "tipoftheday_w.png"}, //local help missing
  { NC_("RID_CUI_TIPOFTHEDAY", "Create editable Hybrid PDFs with 
%PRODUCTNAME."), 
"https://wiki.documentfoundation.org/Documentation/HowTo/CreateAHybridPDF";, ""},
  { NC_("RID_CUI_TIPOFTHEDAY", "Explore the ten different functions in the 
status bar (at the bottom of the document window). Place the cursor over each 
field for an explanation. If not visible, use View ▸ Status Bar."), "", 
"tipoftheday_w.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "Want to sum a cell through several sheets? 
Refer to the range of sheets e.g. =SUM(Sheet1.A1:Sheet3.A1)."), "", 
"tipoftheday_c.png"},
- { NC_("RID_CUI_TIPOFTHEDAY", "Create fillable form documents (even PDFs) 
with %PRODUCTNAME."), 
"https://www.techrepublic.com/article/how-to-create-interactive-pdfs-with-libreoffice";,
 ""},
+ { NC_("RID_CUI_TIPOFTHEDAY", "Create fillable form documents (even PDFs) 
with %PRODUCTNAME."), 
"https://www.techrepublic.com/article/how-to-create-interactive-pdfs-with-libreoffice";,
 "formdocuments.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "Cannot see all the text in a cell? Expand 
the input line in the formula bar and you can scroll."), "", 
"tipoftheday_c.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "Optimize your table layout with Table ▸ 
Size ▸ Distribute Rows / Columns Evenly."), "", "tipoftheday_w.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "Find all expressions in brackets per Edit ▸ 
Find and Replace ▸ Find ▸ \\([^)]+\\) (check “Regular expressions”)"), "", 
"tipoftheday_w.png"},
@@ -66,7 +66,7 @@ const std::tuple 
TIPOFTHEDAY_STRINGARRAY[] =
  { NC_("RID_CUI_TIPOFTHEDAY", "You can restart the slide show after a 
pause specified at Slide Show ▸ Slide Show Settings ▸ Loop and repeat."), "", 
"tipoftheday_i.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "To distribute some text in multi-columns 
select the text and apply Format ▸ Columns."), "", "tipoftheday_w.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "Use View ▸ Value Highlighting to display 
cell contents in colors: Text/black, Formulas/green, Numbers/blue, Protected 
cells/grey background."), 
"https://help.libreoffice.org/%PRODUCTVERSION/%LANGUAGENAME/text/scalc/01/0308.html";,
 "tipoftheday_c.png"}, //local help missing
- { NC_("RID_CUI_TIPOFTHEDAY", "Create different master pages in a 
presentation template: View ▸ Master Slide and Slide ▸ New Master (or per 
toolbar or right click in slide p

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

2020-11-27 Thread andreas kainz (via logerrit)
 sw/uiconfig/swriter/ui/notebookbar.ui |   51 +++---
 1 file changed, 41 insertions(+), 10 deletions(-)

New commits:
commit 9ffec868f9f42eaca7282e70fa0fd85182831777
Author: andreas kainz 
AuthorDate: Fri Nov 27 09:39:52 2020 +0100
Commit: Andreas Kainz 
CommitDate: Fri Nov 27 22:43:26 2020 +0100

writer notebookbar update for 7.1

Change-Id: Id754c59acd5a548fb96b085e63d1e8f39805ca46
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106721
Tested-by: Jenkins
Reviewed-by: Andreas Kainz 
(cherry picked from commit c8ca030367180330359ad3b0c6e0fcd8759741e8)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106738

diff --git a/sw/uiconfig/swriter/ui/notebookbar.ui 
b/sw/uiconfig/swriter/ui/notebookbar.ui
index b2db8be91554..3a5e5b195fba 100644
--- a/sw/uiconfig/swriter/ui/notebookbar.ui
+++ b/sw/uiconfig/swriter/ui/notebookbar.ui
@@ -3365,6 +3365,8 @@
   
 True
 True
+center
+True
 both-horiz
 False
 
@@ -3389,6 +3391,8 @@
   
 True
 True
+center
+True
 both-horiz
 False
 
@@ -3442,10 +3446,13 @@
   
 True
 False
+center
+True
 
   
 True
 True
+center
 True
 
   
@@ -3470,6 +3477,7 @@
   
 True
 True
+center
 5
 
   
@@ -3493,6 +3501,7 @@
   
 True
 True
+center
 icons
 False
 
@@ -3536,10 +3545,12 @@
 True
 False
 center
+True
 
   
 True
 True
+center
 icons
 False
 
@@ -3608,6 +3619,7 @@
 True
 False
 end
+center
 5
 5
 True
@@ -3624,6 +3636,7 @@
   
 True
 True
+center
 icons
 False
 
@@ -3659,6 +3672,7 @@
   
 True
 False
+center
 5
 5
 vertical
@@ -3674,6 +3688,7 @@
   
 True
 True
+center
 icons
 False
 
@@ -3741,8 +3756,6 @@
   
 True
 False
-center
-True
 
   
 True
@@ -3769,10 +3782,12 @@
 True
 False
 center
+True
 
   
 True
   

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

2020-11-27 Thread andreas kainz (via logerrit)
 sd/uiconfig/sdraw/ui/notebookbar.ui|   12 ++--
 sd/uiconfig/simpress/ui/notebookbar.ui |   49 -
 2 files changed, 47 insertions(+), 14 deletions(-)

New commits:
commit 9273b8bb8dee8ee627d59efa610b18aa4a4ebc4c
Author: andreas kainz 
AuthorDate: Fri Nov 27 10:10:56 2020 +0100
Commit: Andreas Kainz 
CommitDate: Fri Nov 27 22:42:58 2020 +0100

impress/draw notebookbar update for 7.1

Change-Id: I85b8fcc5cdc58752d1a9079046e2be26e64a3b74
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106724
Tested-by: Jenkins
Reviewed-by: Andreas Kainz 
(cherry picked from commit 364e440e024a4af6a0bdd354ae2bd4bcdd9536c6)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106744

diff --git a/sd/uiconfig/sdraw/ui/notebookbar.ui 
b/sd/uiconfig/sdraw/ui/notebookbar.ui
index 918e487e7ed3..512ddb512df7 100644
--- a/sd/uiconfig/sdraw/ui/notebookbar.ui
+++ b/sd/uiconfig/sdraw/ui/notebookbar.ui
@@ -11852,7 +11852,6 @@
 False
 
   
-130
 True
 False
 center
@@ -11877,12 +11876,11 @@
 both-horiz
 False
 
-  
-130
+  
 True
 False
 center
-.uno:GrafTransparence
+.uno:GrafContrast
   
   
 False
@@ -11933,12 +11931,11 @@
 both-horiz
 False
 
-  
-130
+  
 True
 False
 center
-.uno:GrafContrast
+.uno:GrafTransparence
   
   
 False
@@ -11960,7 +11957,6 @@
 False
 
   
-130
 True
 False
 center
diff --git a/sd/uiconfig/simpress/ui/notebookbar.ui 
b/sd/uiconfig/simpress/ui/notebookbar.ui
index 72d4edb0c0fb..318f7c61fd67 100644
--- a/sd/uiconfig/simpress/ui/notebookbar.ui
+++ b/sd/uiconfig/simpress/ui/notebookbar.ui
@@ -3540,6 +3540,8 @@
   
 True
 True
+center
+True
 both-horiz
 False
 
@@ -3564,6 +3566,8 @@
   
 True
 True
+center
+True
 both-horiz
 False
 
@@ -3617,10 +3621,13 @@
   
 True
 False
+center
+True
 
   
 True
 True
+center
 True
 
   
@@ -3645,6 +3652,7 @@
   
 True
 True
+center
 5
 
   
@@ -3669,6 +3677,7 @@
 True
 True
 end
+center
 icons
 False
 
@@ -3712,6 +3721,7 @@
 True
 False
 center
+True

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

2020-11-27 Thread andreas kainz (via logerrit)
 sc/uiconfig/scalc/ui/notebookbar.ui |   71 +++-
 1 file changed, 63 insertions(+), 8 deletions(-)

New commits:
commit 50df08ba0b1a54710af4c868d457ee15b865da4b
Author: andreas kainz 
AuthorDate: Fri Nov 27 10:03:37 2020 +0100
Commit: Andreas Kainz 
CommitDate: Fri Nov 27 21:36:31 2020 +0100

calc notebookbar update for 7.1

Change-Id: I9b9e09c1ce0681e9a17ca79677d03200b1fc536d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106723
Tested-by: Jenkins
Reviewed-by: Andreas Kainz 
(cherry picked from commit 3fbda6517e4ee345966873d2930c226b890f33e9)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106739

diff --git a/sc/uiconfig/scalc/ui/notebookbar.ui 
b/sc/uiconfig/scalc/ui/notebookbar.ui
index f34610b1fc7b..ba90157318e1 100644
--- a/sc/uiconfig/scalc/ui/notebookbar.ui
+++ b/sc/uiconfig/scalc/ui/notebookbar.ui
@@ -3229,6 +3229,8 @@
   
 True
 True
+center
+True
 both-horiz
 False
 
@@ -3253,6 +3255,8 @@
   
 True
 True
+center
+True
 both-horiz
 False
 
@@ -3312,10 +3316,12 @@
   
 True
 False
+True
 
   
 True
 True
+center
 True
 
   
@@ -3340,6 +3346,7 @@
   
 True
 True
+center
 5
 5
 
@@ -3364,6 +3371,7 @@
   
 True
 True
+center
 icons
 False
 
@@ -3407,10 +3415,12 @@
 True
 False
 center
+True
 
   
 True
 True
+center
 icons
 False
 
@@ -3484,6 +3494,7 @@
   
 True
 True
+center
 icons
 False
 
@@ -3534,6 +3545,7 @@
   
 True
 True
+center
 icons
 False
 
@@ -3639,10 +3651,12 @@
 True
 False
 center
+True
 
   
 True
 True
+center
 icons
 False
 
@@ -3697,10 +3711,12 @@
 True
 False
 center
+True
 
   
 True
 True
+center
 icons
 False
 
@@ -3784,6 +3800

[Libreoffice-commits] core.git: oox/inc oox/source solenv/clang-format

2020-11-27 Thread Miklos Vajna (via logerrit)
 oox/inc/drawingml/effectpropertiescontext.hxx|5 +
 oox/source/drawingml/effectpropertiescontext.cxx |2 +-
 oox/source/drawingml/shapepropertiescontext.cxx  |2 +-
 oox/source/drawingml/themeelementscontext.cxx|2 +-
 solenv/clang-format/excludelist  |2 +-
 5 files changed, 5 insertions(+), 8 deletions(-)

New commits:
commit f2e1685e489c054ea10afc0df52451c74da10cc0
Author: Miklos Vajna 
AuthorDate: Fri Nov 27 16:58:38 2020 +0100
Commit: Miklos Vajna 
CommitDate: Fri Nov 27 21:33:33 2020 +0100

oox: make effectpropertiescontext available to everyone in oox/

I plan to use this from oox/source/drawingml/table/tablecontext.cxx for
tdf#129961.

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

diff --git a/oox/source/drawingml/effectpropertiescontext.hxx 
b/oox/inc/drawingml/effectpropertiescontext.hxx
similarity index 85%
rename from oox/source/drawingml/effectpropertiescontext.hxx
rename to oox/inc/drawingml/effectpropertiescontext.hxx
index ce7b5b3ef363..09ec2f02ec80 100644
--- a/oox/source/drawingml/effectpropertiescontext.hxx
+++ b/oox/inc/drawingml/effectpropertiescontext.hxx
@@ -7,8 +7,7 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
 
-#ifndef INCLUDED_OOX_DRAWINGML_EFFECTPROPERTIESCONTEXT_HXX
-#define INCLUDED_OOX_DRAWINGML_EFFECTPROPERTIESCONTEXT_HXX
+#pragma once
 
 #include 
 
@@ -35,6 +34,4 @@ private:
 
 }
 
-#endif // INCLUDED_OOX_DRAWINGML_EFFECTPROPERTIESCONTEXT_HXX
-
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/oox/source/drawingml/effectpropertiescontext.cxx 
b/oox/source/drawingml/effectpropertiescontext.cxx
index f287c897d891..b7cb03bc08b6 100644
--- a/oox/source/drawingml/effectpropertiescontext.cxx
+++ b/oox/source/drawingml/effectpropertiescontext.cxx
@@ -7,7 +7,7 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
 
-#include "effectpropertiescontext.hxx"
+#include 
 #include "effectproperties.hxx"
 #include 
 #include 
diff --git a/oox/source/drawingml/shapepropertiescontext.cxx 
b/oox/source/drawingml/shapepropertiescontext.cxx
index 4591cb834489..574e0793c89a 100644
--- a/oox/source/drawingml/shapepropertiescontext.cxx
+++ b/oox/source/drawingml/shapepropertiescontext.cxx
@@ -24,7 +24,7 @@
 #include 
 #include 
 #include 
-#include "effectpropertiescontext.hxx"
+#include 
 #include 
 #include 
 #include 
diff --git a/oox/source/drawingml/themeelementscontext.cxx 
b/oox/source/drawingml/themeelementscontext.cxx
index e5f84e5a02a0..6fbd62e91d18 100644
--- a/oox/source/drawingml/themeelementscontext.cxx
+++ b/oox/source/drawingml/themeelementscontext.cxx
@@ -27,7 +27,7 @@
 #include 
 #include 
 #include "effectproperties.hxx"
-#include "effectpropertiescontext.hxx"
+#include 
 #include 
 #include 
 
diff --git a/solenv/clang-format/excludelist b/solenv/clang-format/excludelist
index c7eee25f0f8c..5249848a75d8 100644
--- a/solenv/clang-format/excludelist
+++ b/solenv/clang-format/excludelist
@@ -7185,6 +7185,7 @@ oox/inc/drawingml/clrschemecontext.hxx
 oox/inc/drawingml/colorchoicecontext.hxx
 oox/inc/drawingml/customshapegeometry.hxx
 oox/inc/drawingml/customshapeproperties.hxx
+oox/inc/drawingml/effectpropertiescontext.hxx
 oox/inc/drawingml/embeddedwavaudiofile.hxx
 oox/inc/drawingml/fillproperties.hxx
 oox/inc/drawingml/graphicproperties.hxx
@@ -7303,7 +7304,6 @@ oox/source/drawingml/drawingmltypes.cxx
 oox/source/drawingml/effectproperties.cxx
 oox/source/drawingml/effectproperties.hxx
 oox/source/drawingml/effectpropertiescontext.cxx
-oox/source/drawingml/effectpropertiescontext.hxx
 oox/source/drawingml/fillproperties.cxx
 oox/source/drawingml/graphicshapecontext.cxx
 oox/source/drawingml/guidcontext.cxx
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread andreas kainz (via logerrit)
 sd/uiconfig/sdraw/ui/notebookbar.ui|   12 ++--
 sd/uiconfig/simpress/ui/notebookbar.ui |   49 -
 2 files changed, 47 insertions(+), 14 deletions(-)

New commits:
commit 364e440e024a4af6a0bdd354ae2bd4bcdd9536c6
Author: andreas kainz 
AuthorDate: Fri Nov 27 10:10:56 2020 +0100
Commit: Andreas Kainz 
CommitDate: Fri Nov 27 21:29:32 2020 +0100

impress/draw notebookbar update for 7.1

Change-Id: I85b8fcc5cdc58752d1a9079046e2be26e64a3b74
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106724
Tested-by: Jenkins
Reviewed-by: Andreas Kainz 

diff --git a/sd/uiconfig/sdraw/ui/notebookbar.ui 
b/sd/uiconfig/sdraw/ui/notebookbar.ui
index 918e487e7ed3..512ddb512df7 100644
--- a/sd/uiconfig/sdraw/ui/notebookbar.ui
+++ b/sd/uiconfig/sdraw/ui/notebookbar.ui
@@ -11852,7 +11852,6 @@
 False
 
   
-130
 True
 False
 center
@@ -11877,12 +11876,11 @@
 both-horiz
 False
 
-  
-130
+  
 True
 False
 center
-.uno:GrafTransparence
+.uno:GrafContrast
   
   
 False
@@ -11933,12 +11931,11 @@
 both-horiz
 False
 
-  
-130
+  
 True
 False
 center
-.uno:GrafContrast
+.uno:GrafTransparence
   
   
 False
@@ -11960,7 +11957,6 @@
 False
 
   
-130
 True
 False
 center
diff --git a/sd/uiconfig/simpress/ui/notebookbar.ui 
b/sd/uiconfig/simpress/ui/notebookbar.ui
index 72d4edb0c0fb..318f7c61fd67 100644
--- a/sd/uiconfig/simpress/ui/notebookbar.ui
+++ b/sd/uiconfig/simpress/ui/notebookbar.ui
@@ -3540,6 +3540,8 @@
   
 True
 True
+center
+True
 both-horiz
 False
 
@@ -3564,6 +3566,8 @@
   
 True
 True
+center
+True
 both-horiz
 False
 
@@ -3617,10 +3621,13 @@
   
 True
 False
+center
+True
 
   
 True
 True
+center
 True
 
   
@@ -3645,6 +3652,7 @@
   
 True
 True
+center
 5
 
   
@@ -3669,6 +3677,7 @@
 True
 True
 end
+center
 icons
 False
 
@@ -3712,6 +3721,7 @@
 True
 False
 center
+True
 
   
 True
@@ -3801,6 +3811,7 @@
   

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

2020-11-27 Thread Caolán McNamara (via logerrit)
 vcl/inc/salgdi.hxx  |4 +++-
 vcl/source/gdi/salgdilayout.cxx |   28 
 2 files changed, 27 insertions(+), 5 deletions(-)

New commits:
commit 0c5f58114cea0fbb7c638726b1ce6bc214fea504
Author: Caolán McNamara 
AuthorDate: Fri Nov 27 11:47:35 2020 +
Commit: Caolán McNamara 
CommitDate: Fri Nov 27 21:03:47 2020 +0100

with RTL UI the LTR scrollbar in 'gen' calc is rendered incorrectly

this is noticable since...

commit 4deadc3c78949c18bb886eb1f66caa8f3cd7a2df
Date:   Fri Sep 25 13:30:11 2020 +0200

disentangle AA and B2D use in VCL drawing

but is probably a problem since around...

commit b5f081e1ac14f60497f62a27be86b07b0baa42f7
Date:   Thu Aug 30 23:41:36 2018 +0200

Support RTL layout in VCL using Matrices

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

diff --git a/vcl/inc/salgdi.hxx b/vcl/inc/salgdi.hxx
index 9fa687123bfa..6ed360105d24 100644
--- a/vcl/inc/salgdi.hxx
+++ b/vcl/inc/salgdi.hxx
@@ -596,7 +596,9 @@ private:
 
 // for buffering the Mirror-Matrix, see ::getMirror
 basegfx::B2DHomMatrix   m_aLastMirror;
-tools::Longm_aLastMirrorW;
+tools::Long m_aLastMirrorW;
+tools::Long m_nLastMirrorDeviceLTRButBiDiRtlTranslate;
+boolm_bLastMirrorDeviceLTRButBiDiRtlSet;
 
 protected:
 /// flags which hold the SetAntialiasing() value from OutputDevice
diff --git a/vcl/source/gdi/salgdilayout.cxx b/vcl/source/gdi/salgdilayout.cxx
index 89089d9755a2..5638cb9bb856 100644
--- a/vcl/source/gdi/salgdilayout.cxx
+++ b/vcl/source/gdi/salgdilayout.cxx
@@ -58,6 +58,8 @@ SalGraphics::SalGraphics()
 :   m_nLayout( SalLayoutFlags::NONE ),
 m_aLastMirror(),
 m_aLastMirrorW(0),
+m_nLastMirrorDeviceLTRButBiDiRtlTranslate(0),
+m_bLastMirrorDeviceLTRButBiDiRtlSet(false),
 m_bAntiAlias(false)
 {
 // read global RTL settings
@@ -316,14 +318,33 @@ const basegfx::B2DHomMatrix& SalGraphics::getMirror( 
const OutputDevice* i_pOutD
 const tools::Long w = GetDeviceWidth(i_pOutDev);
 SAL_WARN_IF( !w, "vcl", "missing graphics width" );
 
-if(w != m_aLastMirrorW)
+const bool bMirrorDeviceLTRButBiDiRtlSet = i_pOutDev && 
!i_pOutDev->IsRTLEnabled();
+tools::Long nMirrorDeviceLTRButBiDiRtlTranslate(0);
+if (bMirrorDeviceLTRButBiDiRtlSet)
+nMirrorDeviceLTRButBiDiRtlTranslate = w - 
i_pOutDev->GetOutputWidthPixel() - (2 * i_pOutDev->GetOutOffXPixel());
+
+// if the device width, or mirror state of the device changed, then 
m_aLastMirror is invalid
+bool bLastMirrorValid = w == m_aLastMirrorW && 
bMirrorDeviceLTRButBiDiRtlSet == m_bLastMirrorDeviceLTRButBiDiRtlSet;
+if (bLastMirrorValid && bMirrorDeviceLTRButBiDiRtlSet)
+{
+// if the device is in in the unusual mode of a LTR device, but layout 
flags of SalLayoutFlags::BiDiRtl are
+// in use, then the m_aLastMirror is invalid if the distance it should 
translate has changed
+bLastMirrorValid = nMirrorDeviceLTRButBiDiRtlTranslate == 
m_nLastMirrorDeviceLTRButBiDiRtlTranslate;
+}
+
+if (!bLastMirrorValid)
 {
 const_cast(this)->m_aLastMirrorW = w;
+const_cast(this)->m_bLastMirrorDeviceLTRButBiDiRtlSet = 
bMirrorDeviceLTRButBiDiRtlSet;
+
const_cast(this)->m_nLastMirrorDeviceLTRButBiDiRtlTranslate = 
nMirrorDeviceLTRButBiDiRtlTranslate;
 
 if(w)
 {
-if(nullptr != i_pOutDev && !i_pOutDev->IsRTLEnabled())
+if (bMirrorDeviceLTRButBiDiRtlSet)
 {
+/* This path gets exercised in calc's RTL UI (e.g. 
SAL_RTL_ENABLED=1)
+   with its LTR horizontal scrollbar */
+
 // Original code was (removed here already 
pOutDevRef->i_pOutDev):
 //  // mirror this window back
 //  double devX = 
w-i_pOutDev->GetOutputWidthPixel()-i_pOutDev->GetOutOffXPixel();   // 
re-mirrored mnOutOffX
@@ -332,8 +353,7 @@ const basegfx::B2DHomMatrix& SalGraphics::getMirror( const 
OutputDevice* i_pOutD
 // that this works as before, but I have reduced this (by 
re-placing and re-formatting) to
 // a simple translation:
 const_cast(this)->m_aLastMirror = 
basegfx::utils::createTranslateB2DHomMatrix(
-w - i_pOutDev->GetOutputWidthPixel() - (2 * 
i_pOutDev->GetOutOffXPixel()),
-0.0);
+nMirrorDeviceLTRButBiDiRtlTranslate, 0.0);
 }
 else
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Julien Nabet (via logerrit)
 connectivity/source/commontools/TTableHelper.cxx |5 +
 1 file changed, 5 insertions(+)

New commits:
commit 4335810b00abb9b00a9d81caa5ffe09a3ea927fd
Author: Julien Nabet 
AuthorDate: Thu Nov 26 23:12:46 2020 +0100
Commit: Lionel Mamane 
CommitDate: Fri Nov 27 19:08:09 2020 +0100

tdf#137745: crash, when deleting tables and changed relationship isn't 
changed

bt here:
https://bugs.documentfoundation.org/attachment.cgi?id=166782

some explanations here:
https://bugs.documentfoundation.org/show_bug.cgi?id=137745#c8

Change-Id: I1d9dc8d2e3ad7e3e36c4687cb9b2c5990f2d2c8d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106715
Tested-by: Jenkins
Reviewed-by: Lionel Mamane 

diff --git a/connectivity/source/commontools/TTableHelper.cxx 
b/connectivity/source/commontools/TTableHelper.cxx
index e86af9673052..03724d669704 100644
--- a/connectivity/source/commontools/TTableHelper.cxx
+++ b/connectivity/source/commontools/TTableHelper.cxx
@@ -72,6 +72,11 @@ public:
 }
 virtual void SAL_CALL elementRemoved( const 
css::container::ContainerEvent& Event ) override
 {
+// tdf#137745, perhaps connectivity::OTableHelper::disposing() has 
been called
+// which called OTableContainerListener::clear(), so m_pComponent may 
be null
+if (m_pComponent == nullptr)
+return;
+
 OUString sName;
 Event.Accessor  >>= sName;
 if ( m_aRefNames.find(sName) != m_aRefNames.end() )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-6.2' - download.lst external/pdfium RepositoryExternal.mk svx/source vcl/qa

2020-11-27 Thread Miklos Vajna (via logerrit)
 RepositoryExternal.mk |1 
 download.lst  |4 -
 external/pdfium/Library_pdfium.mk |  116 +-
 external/pdfium/UnpackedTarball_pdfium.mk |   13 ++-
 external/pdfium/build.patch.1 |  110 
 external/pdfium/c++20-comparison.patch|   13 +++
 external/pdfium/ubsan.patch   |   26 +++---
 external/pdfium/visibility.patch.1|   30 ---
 external/pdfium/windows7.patch.1  |   34 
 svx/source/svdraw/svdpdf.cxx  |   31 
 vcl/qa/cppunit/pdfexport/pdfexport.cxx|6 -
 11 files changed, 162 insertions(+), 222 deletions(-)

New commits:
commit d38a0c077ff47de61519f5e0edf65c844651e1c5
Author: Miklos Vajna 
AuthorDate: Mon Nov 19 09:03:40 2018 +0100
Commit: Miklos Vajna 
CommitDate: Fri Nov 27 18:32:36 2020 +0100

external: update pdfium from 3550 to 4137

This is a combination of 9 commits, which brings pdfium to the same
version as cp-6.4 (ignoring recent changes).

This is the 1st commit message:

external: update pdfium to 3613

(cherry picked from commit ec11c1aee04eacb00d94a6359f959b990ddb6923)

This is the commit message #2:

external: update pdfium to 3667

(cherry picked from commit 2044475c8cb33b76591aa6de77dd43a0bf9f5145)

Conflicts:
solenv/flatpak-manifest.in

This is the commit message #3:

external: update pdfium to 3730

(cherry picked from commit 8743247493ba90098e3e32cf30de0e8995569852)

This is the commit message #4:

pdfium: avoid problems with SetForm using WIN32_LEAN_AND_MEAN

So that it does not get defined to SetFormA() or SetFormW() and still
requires no patching.

(cherry picked from commit 66c29fd202f22a36edbb929ddcc1f1cadb0a6e8f)

This is the commit message #5:

external: update pdfium to 3794

(cherry picked from commit 3dbe66b7895a412ad7ad9aede4be383489d805de)

Conflicts:
external/pdfium/Library_pdfium.mk

This is the commit message #6:

external: update pdfium to 3849

(cherry picked from commit 0ee0ca3036629b69bf20b448d74991fd133f08ac)

Conflicts:
external/pdfium/inc/pch/precompiled_pdfium.hxx

This is the commit message #7:

external: update pdfium to 3896

(cherry picked from commit 735af14843eab3e75ac9ed6f0773ce7bb3241c8a)

Conflicts:
external/pdfium/inc/pch/precompiled_pdfium.hxx
solenv/flatpak-manifest.in

This is the commit message #8:

external: update pdfium to 3963

Also simplify visibility.patch.1.

(cherry picked from commit 71cb2705af38df7f382014fb68f43bed98abf9b4)

Conflicts:
solenv/flatpak-manifest.in
svx/source/svdraw/svdpdf.cxx

This is the commit message #9:

external: update pdfium from 3963 to 4137

This is a combination of 6 commits, which brings pdfium to the same
version as libreoffice-7-0.

(cherry picked from commit 2cd3ddad396043c8c1af2e03bd1c53db084ccbf0)

Conflicts:
external/pdfium/inc/pch/precompiled_pdfium.hxx
solenv/flatpak-manifest.in
vcl/qa/cppunit/pdfexport/pdfexport.cxx
vcl/source/pdf/PDFiumLibrary.cxx

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

diff --git a/RepositoryExternal.mk b/RepositoryExternal.mk
index def5fa9a0e72..5db8ff9b9f06 100644
--- a/RepositoryExternal.mk
+++ b/RepositoryExternal.mk
@@ -4192,6 +4192,7 @@ ifneq ($(ENABLE_PDFIUM),)
 define gb_LinkTarget__use_pdfium
 $(call gb_LinkTarget_set_include,$(1),\
-I$(call gb_UnpackedTarball_get_dir,pdfium)/public \
+   -DCOMPONENT_BUILD \
$$(INCLUDE) \
 )
 $(call gb_LinkTarget_use_libraries,$(1),pdfium)
diff --git a/download.lst b/download.lst
index a265d508d677..5b30906cad23 100644
--- a/download.lst
+++ b/download.lst
@@ -222,8 +222,8 @@ export OWNCLOUD_ANDROID_LIB_SHA256SUM := 
b18b3e3ef7fae6a79b62f2bb43cc47a5346b633
 export OWNCLOUD_ANDROID_LIB_TARBALL := 
owncloud-android-library-0.9.4-no-binary-deps.tar.gz
 export PAGEMAKER_SHA256SUM := 
66adacd705a7d19895e08eac46d1e851332adf2e736c566bef1164e7a442519d
 export PAGEMAKER_TARBALL := libpagemaker-0.0.4.tar.xz
-export PDFIUM_SHA256SUM := 
572460f7f9e2f86d022a9c6a82f1e2ded6c3c29ba352d4b9fac60b87e2159679
-export PDFIUM_TARBALL := pdfium-3550.tar.bz2
+export PDFIUM_SHA256SUM := 
9a2f9bddca935a263f06c81003483473a525ccd0f4e517bc75fceb914d4c54b6
+export PDFIUM_TARBALL := pdfium-4137.tar.bz2
 export PIXMAN_SHA256SUM := 
21b6b249b51c6800dc9553b65106e1e37d0e25df942c90531d4c3997aa20a88e
 export PIXMAN_TARBALL := e80ebae4da01e77f68744319f01d52a3-pixman-0.34

[Libreoffice-commits] core.git: sal/rtl

2020-11-27 Thread Eike Rathke (via logerrit)
 sal/rtl/math.cxx |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 1a0f9a9c56e7b7952b813b3efd34f9be6c853957
Author: Eike Rathke 
AuthorDate: Fri Nov 27 13:26:26 2020 +0100
Commit: Eike Rathke 
CommitDate: Fri Nov 27 17:24:35 2020 +0100

Consistently use RTL_CONSTASCII_LENGTH(), tdf#136272 follow-up

The mix with SAL_N_ELEMENTS() was confusing and even wrong in one
case.

Change-Id: Ife73342b0efc01ed4e76e217d372fc32500c9b2c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106764
Reviewed-by: Eike Rathke 
Tested-by: Jenkins

diff --git a/sal/rtl/math.cxx b/sal/rtl/math.cxx
index 81fa0cf925b7..6ed4906270e0 100644
--- a/sal/rtl/math.cxx
+++ b/sal/rtl/math.cxx
@@ -292,7 +292,7 @@ void doubleToString(typename T::String ** pResult,
 constexpr char pDig[] = "7976931348623157";
 constexpr char pRou[] = "8087931459623267"; // the only up-carry 
is 80
 static_assert(SAL_N_ELEMENTS(pDig) == SAL_N_ELEMENTS(pRou), "digit 
count mismatch");
-constexpr sal_Int32 nDig2 = SAL_N_ELEMENTS(pRou) - 2;
+constexpr sal_Int32 nDig2 = RTL_CONSTASCII_LENGTH(pRou) - 2;
 sal_Int32 nCapacity = RTL_CONSTASCII_LENGTH(pRou) + 8;  // + "-1.E+308"
 const char pSlot[5][2][3] =
 { // rounded, not
@@ -314,7 +314,7 @@ void doubleToString(typename T::String ** pResult,
 T::appendAscii(pResult, pResultCapacity, &nResultOffset,
RTL_CONSTASCII_STRINGPARAM("-"));
 
-nDecPlaces = std::clamp( nDecPlaces, 0, 
SAL_N_ELEMENTS(pRou));
+nDecPlaces = std::clamp( nDecPlaces, 0, 
RTL_CONSTASCII_LENGTH(pRou));
 if (nDecPlaces == 0)
 {
 T::appendAscii(pResult, pResultCapacity, &nResultOffset,
@@ -342,7 +342,7 @@ void doubleToString(typename T::String ** pResult,
 const size_t nSlot = ((fValue < fB3) ? 4 : ((fValue < fB2) ? 3
 : ((fValue < fB1) ? 2 : ((fValue < DBL_MAX) ? 1 : 
0;
 
-T::appendAscii(pResult, pResultCapacity, &nResultOffset, pDig, 
nDecPlaces - 1);
+T::appendAscii(pResult, pResultCapacity, &nResultOffset, pDig, 
nDecPlaces);
 T::appendAscii(pResult, pResultCapacity, &nResultOffset, 
pSlot[nSlot][nDec-1], nDec);
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Xisco Fauli (via logerrit)
 sc/qa/unit/data/ods/tdf138507.ods  |binary
 sc/qa/unit/subsequent_filters-test.cxx |   14 ++
 2 files changed, 14 insertions(+)

New commits:
commit 33e0221cac467d32debe4fab6592517b0c1c195e
Author: Xisco Fauli 
AuthorDate: Thu Nov 26 21:16:43 2020 +0100
Commit: Xisco Fauli 
CommitDate: Fri Nov 27 15:36:08 2020 +0100

tdf#138507: sc_subsequent_filters: Add unittest

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

diff --git a/sc/qa/unit/data/ods/tdf138507.ods 
b/sc/qa/unit/data/ods/tdf138507.ods
new file mode 100644
index ..5b83a0dccd5a
Binary files /dev/null and b/sc/qa/unit/data/ods/tdf138507.ods differ
diff --git a/sc/qa/unit/subsequent_filters-test.cxx 
b/sc/qa/unit/subsequent_filters-test.cxx
index 2323ee50b4c5..9c087549ec3e 100644
--- a/sc/qa/unit/subsequent_filters-test.cxx
+++ b/sc/qa/unit/subsequent_filters-test.cxx
@@ -248,6 +248,7 @@ public:
 void testTdf132278();
 void testTdf130959();
 void testTdf129410();
+void testTdf138507();
 void testTdf131380();
 void testTdf129681();
 void testTdf111974XLSM();
@@ -427,6 +428,7 @@ public:
 CPPUNIT_TEST(testTdf132278);
 CPPUNIT_TEST(testTdf130959);
 CPPUNIT_TEST(testTdf129410);
+CPPUNIT_TEST(testTdf138507);
 CPPUNIT_TEST(testTdf131380);
 CPPUNIT_TEST(testTdf129681);
 CPPUNIT_TEST(testTdf111974XLSM);
@@ -3737,6 +3739,18 @@ void ScFiltersTest::testTdf129410()
 xDocSh->DoClose();
 }
 
+void ScFiltersTest::testTdf138507()
+{
+// Would fail to load without the fix
+ScDocShellRef xDocSh = loadDoc("tdf138507.", FORMAT_ODS);
+CPPUNIT_ASSERT_MESSAGE("Failed to open doc", xDocSh.is());
+
+ScDocument& rDoc = xDocSh->GetDocument();
+rDoc.CalcAll(); // perform hard re-calculation.
+
+xDocSh->DoClose();
+}
+
 void ScFiltersTest::testTdf131380()
 {
 ScDocShellRef xDocSh = loadDoc("tdf131380.", FORMAT_XLSX);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Caolán McNamara (via logerrit)
 vcl/inc/salgdi.hxx  |4 +++-
 vcl/source/gdi/salgdilayout.cxx |   28 
 2 files changed, 27 insertions(+), 5 deletions(-)

New commits:
commit 7f876fd0d49527fef03d0248f776143e150feb33
Author: Caolán McNamara 
AuthorDate: Fri Nov 27 11:47:35 2020 +
Commit: Caolán McNamara 
CommitDate: Fri Nov 27 15:32:56 2020 +0100

with RTL UI the LRT scrollbar in 'gen' calc is rendered incorrectly

this is noticable since...

commit 4deadc3c78949c18bb886eb1f66caa8f3cd7a2df
Date:   Fri Sep 25 13:30:11 2020 +0200

disentangle AA and B2D use in VCL drawing

but is probably a problem since around...

commit b5f081e1ac14f60497f62a27be86b07b0baa42f7
Date:   Thu Aug 30 23:41:36 2018 +0200

Support RTL layout in VCL using Matrices

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

diff --git a/vcl/inc/salgdi.hxx b/vcl/inc/salgdi.hxx
index 9fa687123bfa..6ed360105d24 100644
--- a/vcl/inc/salgdi.hxx
+++ b/vcl/inc/salgdi.hxx
@@ -596,7 +596,9 @@ private:
 
 // for buffering the Mirror-Matrix, see ::getMirror
 basegfx::B2DHomMatrix   m_aLastMirror;
-tools::Longm_aLastMirrorW;
+tools::Long m_aLastMirrorW;
+tools::Long m_nLastMirrorDeviceLTRButBiDiRtlTranslate;
+boolm_bLastMirrorDeviceLTRButBiDiRtlSet;
 
 protected:
 /// flags which hold the SetAntialiasing() value from OutputDevice
diff --git a/vcl/source/gdi/salgdilayout.cxx b/vcl/source/gdi/salgdilayout.cxx
index 89089d9755a2..5638cb9bb856 100644
--- a/vcl/source/gdi/salgdilayout.cxx
+++ b/vcl/source/gdi/salgdilayout.cxx
@@ -58,6 +58,8 @@ SalGraphics::SalGraphics()
 :   m_nLayout( SalLayoutFlags::NONE ),
 m_aLastMirror(),
 m_aLastMirrorW(0),
+m_nLastMirrorDeviceLTRButBiDiRtlTranslate(0),
+m_bLastMirrorDeviceLTRButBiDiRtlSet(false),
 m_bAntiAlias(false)
 {
 // read global RTL settings
@@ -316,14 +318,33 @@ const basegfx::B2DHomMatrix& SalGraphics::getMirror( 
const OutputDevice* i_pOutD
 const tools::Long w = GetDeviceWidth(i_pOutDev);
 SAL_WARN_IF( !w, "vcl", "missing graphics width" );
 
-if(w != m_aLastMirrorW)
+const bool bMirrorDeviceLTRButBiDiRtlSet = i_pOutDev && 
!i_pOutDev->IsRTLEnabled();
+tools::Long nMirrorDeviceLTRButBiDiRtlTranslate(0);
+if (bMirrorDeviceLTRButBiDiRtlSet)
+nMirrorDeviceLTRButBiDiRtlTranslate = w - 
i_pOutDev->GetOutputWidthPixel() - (2 * i_pOutDev->GetOutOffXPixel());
+
+// if the device width, or mirror state of the device changed, then 
m_aLastMirror is invalid
+bool bLastMirrorValid = w == m_aLastMirrorW && 
bMirrorDeviceLTRButBiDiRtlSet == m_bLastMirrorDeviceLTRButBiDiRtlSet;
+if (bLastMirrorValid && bMirrorDeviceLTRButBiDiRtlSet)
+{
+// if the device is in in the unusual mode of a LTR device, but layout 
flags of SalLayoutFlags::BiDiRtl are
+// in use, then the m_aLastMirror is invalid if the distance it should 
translate has changed
+bLastMirrorValid = nMirrorDeviceLTRButBiDiRtlTranslate == 
m_nLastMirrorDeviceLTRButBiDiRtlTranslate;
+}
+
+if (!bLastMirrorValid)
 {
 const_cast(this)->m_aLastMirrorW = w;
+const_cast(this)->m_bLastMirrorDeviceLTRButBiDiRtlSet = 
bMirrorDeviceLTRButBiDiRtlSet;
+
const_cast(this)->m_nLastMirrorDeviceLTRButBiDiRtlTranslate = 
nMirrorDeviceLTRButBiDiRtlTranslate;
 
 if(w)
 {
-if(nullptr != i_pOutDev && !i_pOutDev->IsRTLEnabled())
+if (bMirrorDeviceLTRButBiDiRtlSet)
 {
+/* This path gets exercised in calc's RTL UI (e.g. 
SAL_RTL_ENABLED=1)
+   with its LTR horizontal scrollbar */
+
 // Original code was (removed here already 
pOutDevRef->i_pOutDev):
 //  // mirror this window back
 //  double devX = 
w-i_pOutDev->GetOutputWidthPixel()-i_pOutDev->GetOutOffXPixel();   // 
re-mirrored mnOutOffX
@@ -332,8 +353,7 @@ const basegfx::B2DHomMatrix& SalGraphics::getMirror( const 
OutputDevice* i_pOutD
 // that this works as before, but I have reduced this (by 
re-placing and re-formatting) to
 // a simple translation:
 const_cast(this)->m_aLastMirror = 
basegfx::utils::createTranslateB2DHomMatrix(
-w - i_pOutDev->GetOutputWidthPixel() - (2 * 
i_pOutDev->GetOutOffXPixel()),
-0.0);
+nMirrorDeviceLTRButBiDiRtlTranslate, 0.0);
 }
 else
 {
___
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' - desktop/source

2020-11-27 Thread Mert Tumer (via logerrit)
 desktop/source/lib/init.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 455cd8fe00d1c756da1bb50a50bac68a1ad645ef
Author: Mert Tumer 
AuthorDate: Fri Nov 27 16:56:53 2020 +0300
Commit: Mert Tumer 
CommitDate: Fri Nov 27 15:31:59 2020 +0100

Fix unmodified pdf does forced-save

This is a problem when dealing with the large
pdfs which take long amount of time to save.
Adding or removing annotations do change the state of
IsModified()

Change-Id: I872cb3aec5188986cc13c6659e1bb741a6870409
Signed-off-by: Mert Tumer 
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106778
Reviewed-by: Michael Meeks 
Tested-by: Jenkins CollaboraOffice 

diff --git a/desktop/source/lib/init.cxx b/desktop/source/lib/init.cxx
index b37c85ffbc63..c9d5e7d1d489 100644
--- a/desktop/source/lib/init.cxx
+++ b/desktop/source/lib/init.cxx
@@ -3841,7 +3841,7 @@ static void doc_postUnoCommand(LibreOfficeKitDocument* 
pThis, const char* pComma
 {
 // Check if saving a PDF file
 OUString aMimeType = lcl_getCurrentDocumentMimeType(pDocument);
-if (aMimeType == "application/pdf")
+if (pDocSh->IsModified() && aMimeType == "application/pdf")
 {
 // If we have a PDF file (for saving annotations for example), we 
need
 // to run save-as to the same file as the opened document. Plain 
save
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-6.2' - include/vcl vcl/source xmlsecurity/inc xmlsecurity/qa xmlsecurity/source xmlsecurity/workben

2020-11-27 Thread Miklos Vajna (via logerrit)
 include/vcl/filter/PDFiumLibrary.hxx|2 
 include/vcl/filter/pdfdocument.hxx  |6 +
 vcl/source/filter/ipdf/pdfdocument.cxx  |   82 ++--
 vcl/source/pdf/PDFiumLibrary.cxx|   12 +-
 xmlsecurity/inc/pdfio/pdfdocument.hxx   |2 
 xmlsecurity/qa/unit/pdfsigning/data/bad-cert-p1.pdf |binary
 xmlsecurity/qa/unit/pdfsigning/pdfsigning.cxx   |   27 +-
 xmlsecurity/source/helper/pdfsignaturehelper.cxx|5 -
 xmlsecurity/source/pdfio/pdfdocument.cxx|   18 ++--
 xmlsecurity/workben/pdfverify.cxx   |3 
 10 files changed, 131 insertions(+), 26 deletions(-)

New commits:
commit 9fda2bf1da07f89c1e8613d2b92ea423f68d0c6e
Author: Miklos Vajna 
AuthorDate: Mon Oct 19 16:50:07 2020 +0200
Commit: Miklos Vajna 
CommitDate: Fri Nov 27 15:31:09 2020 +0100

xmlsecurity: handle MDP permission during PDF verify

(cherry picked from commit 586f6abee92af3cdabdce034b607b9a046ed3946)

Conflicts:
include/vcl/filter/PDFiumLibrary.hxx
vcl/source/filter/ipdf/pdfdocument.cxx
vcl/source/pdf/PDFiumLibrary.cxx
xmlsecurity/qa/unit/pdfsigning/pdfsigning.cxx
xmlsecurity/source/helper/pdfsignaturehelper.cxx

(cherry picked from commit 00479937dc071246cc27f33fd6397668448a7ed9)

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

diff --git a/include/vcl/filter/PDFiumLibrary.hxx 
b/include/vcl/filter/PDFiumLibrary.hxx
index 7e8ef4e3ad48..33a982b7bf51 100644
--- a/include/vcl/filter/PDFiumLibrary.hxx
+++ b/include/vcl/filter/PDFiumLibrary.hxx
@@ -58,7 +58,7 @@ public:
 FPDF_PAGE getPointer() { return mpPage; }
 
 /// Get bitmap checksum of the page, without annotations/commenting.
-BitmapChecksum getChecksum();
+BitmapChecksum getChecksum(int nMDPPerm);
 };
 
 struct PDFiumLibrary : public rtl::StaticWithInit, 
PDFiumLibrary>
diff --git a/include/vcl/filter/pdfdocument.hxx 
b/include/vcl/filter/pdfdocument.hxx
index 56debd53e8c7..06a2618a94ab 100644
--- a/include/vcl/filter/pdfdocument.hxx
+++ b/include/vcl/filter/pdfdocument.hxx
@@ -375,6 +375,7 @@ public:
 size_t GetObjectOffset(size_t nIndex) const;
 const std::vector>& GetElements();
 std::vector GetPages();
+PDFObjectElement* GetCatalog();
 /// Remember the end location of an EOF token.
 void PushBackEOF(size_t nOffset);
 /// Look up object based on object number, possibly by parsing object 
streams.
@@ -400,6 +401,11 @@ public:
 bool Write(SvStream& rStream);
 /// Get a list of signatures embedded into this document.
 std::vector GetSignatureWidgets();
+/**
+ * Get the value of the "modification detection and prevention" permission:
+ * Valid values are 1, 2 and 3: only 3 allows annotations after signing.
+ */
+int GetMDPPerm();
 /// Remove the nth signature from read document in the edit buffer.
 bool RemoveSignature(size_t nPosition);
 /// Get byte offsets of the end of incremental updates.
diff --git a/vcl/source/filter/ipdf/pdfdocument.cxx 
b/vcl/source/filter/ipdf/pdfdocument.cxx
index 655450e28aa0..b668cb2d654a 100644
--- a/vcl/source/filter/ipdf/pdfdocument.cxx
+++ b/vcl/source/filter/ipdf/pdfdocument.cxx
@@ -1857,10 +1857,8 @@ static void visitPages(PDFObjectElement* pPages, 
std::vector&
 pPages->setVisiting(false);
 }
 
-std::vector PDFDocument::GetPages()
+PDFObjectElement* PDFDocument::GetCatalog()
 {
-std::vector aRet;
-
 PDFReferenceElement* pRoot = nullptr;
 
 PDFTrailerElement* pTrailer = nullptr;
@@ -1880,11 +1878,18 @@ std::vector PDFDocument::GetPages()
 
 if (!pRoot)
 {
-SAL_WARN("vcl.filter", "PDFDocument::GetPages: trailer has no Root 
key");
-return aRet;
+SAL_WARN("vcl.filter", "PDFDocument::GetCatalog: trailer has no Root 
key");
+return nullptr;
 }
 
-PDFObjectElement* pCatalog = pRoot->LookupObject();
+return pRoot->LookupObject();
+}
+
+std::vector PDFDocument::GetPages()
+{
+std::vector aRet;
+
+PDFObjectElement* pCatalog = GetCatalog();
 if (!pCatalog)
 {
 SAL_WARN("vcl.filter", "PDFDocument::GetPages: trailer has no 
catalog");
@@ -1957,6 +1962,71 @@ std::vector 
PDFDocument::GetSignatureWidgets()
 return aRet;
 }
 
+int PDFDocument::GetMDPPerm()
+{
+int nRet = 3;
+
+std::vector aSignatures = GetSignatureWidgets();
+if (aSignatures.empty())
+{
+return nRet;
+}
+
+for (const auto& pSignature : aSignatures)
+{
+vcl::filter::PDFObjectElement* pSig = pSignature->LookupObject("V");
+if (!pSig)
+{
+SAL_WARN("vcl.filter", "PDFDocument::GetMDPPerm: can't find 
signature object");
+continue;
+}
+
+auto pReference 

[Libreoffice-commits] core.git: helpcontent2

2020-11-27 Thread Seth Chaiklin (via logerrit)
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit e94c60090efe417c2eaf540428bce93b2cc22d60
Author: Seth Chaiklin 
AuthorDate: Fri Nov 27 15:00:53 2020 +0100
Commit: Gerrit Code Review 
CommitDate: Fri Nov 27 15:00:53 2020 +0100

Update git submodules

* Update helpcontent2 from branch 'master'
  to f7bffa21c346dc8ce9db41e8753bef8d033968d9
  - tdf#137680 update Options - View help page, including Skia

  + Add "Shortcut" label and description
  + Add "from" label
  * "User Interface" --> "Icon Size"
  * change OpenGL to "Skia" and add sys switch to
  only show for WIN
  * Re-organize order of presentation of controls to
  correspond to dialog box
  - remove sys switch for screen font antialiasing
  - remove inline sys switch for Win and its content
 about icon size
  (Thanks to Ilmari Lauhakangas for help about sys differences
   and Luboš Luňák for explanations about Skia and hardware
   acceleration.)

Change-Id: I4eca72fb3e2a6aa312dc0b06d0a060df453cc3ac
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/106140
Tested-by: Jenkins
Reviewed-by: Luboš Luňák 
Reviewed-by: Seth Chaiklin 

diff --git a/helpcontent2 b/helpcontent2
index e95e2a6d94ac..f7bffa21c346 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit e95e2a6d94ac5c08d3821f3da198a7982b7d0af8
+Subproject commit f7bffa21c346dc8ce9db41e8753bef8d033968d9
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Seth Chaiklin (via logerrit)
 source/text/shared/optionen/01010800.xhp |   89 +++
 1 file changed, 45 insertions(+), 44 deletions(-)

New commits:
commit f7bffa21c346dc8ce9db41e8753bef8d033968d9
Author: Seth Chaiklin 
AuthorDate: Fri Nov 20 00:53:43 2020 +0100
Commit: Seth Chaiklin 
CommitDate: Fri Nov 27 15:00:53 2020 +0100

tdf#137680 update Options - View help page, including Skia

  + Add "Shortcut" label and description
  + Add "from" label
  * "User Interface" --> "Icon Size"
  * change OpenGL to "Skia" and add sys switch to
  only show for WIN
  * Re-organize order of presentation of controls to
  correspond to dialog box
  - remove sys switch for screen font antialiasing
  - remove inline sys switch for Win and its content
 about icon size
  (Thanks to Ilmari Lauhakangas for help about sys differences
   and Luboš Luňák for explanations about Skia and hardware
   acceleration.)

Change-Id: I4eca72fb3e2a6aa312dc0b06d0a060df453cc3ac
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/106140
Tested-by: Jenkins
Reviewed-by: Luboš Luňák 
Reviewed-by: Seth Chaiklin 

diff --git a/source/text/shared/optionen/01010800.xhp 
b/source/text/shared/optionen/01010800.xhp
index 414c02147..2849884b7 100644
--- a/source/text/shared/optionen/01010800.xhp
+++ b/source/text/shared/optionen/01010800.xhp
@@ -26,8 +26,6 @@
 
 
 
-
-
 
 
   views; defaults
@@ -40,12 +38,11 @@
   font lists
   font name box
   mouse; positioning
+  mouse; middle button
   clipboard; selection clipboard
   selection clipboard
-  OpenGL;settings
-  OpenGL;blacklist
-  OpenGL;whitelist
-  OpenGL;graphics output
+  Skia;settings
+  Skia;graphics output
   notebook bar;icon size
 
 
@@ -60,42 +57,41 @@
 
 
 
-User Interface
-
+  
+
+Icon style
+Specifies the icon 
style for icons in toolbars and dialogs.
 
-Toolbar icon size
-Specifies the display size of toolbar 
icons. The 
Automatic option uses the font size settings of your operating 
system for menus.
-
-
+Icon Size
+
 
-Sidebar icon size
-Specifies the 
display size of sidebar icons.
+Toolbar
+Specifies the display size of toolbar 
icons.
 
 
-Notebook bar icon size
+Notebookbar
 Specifies 
the display size of notebook 
bar icons.
-
 
-Icon style
-Specifies the icon 
style for icons in toolbars and dialogs.
-
-
-
+
+Sidebar
+Specifies the 
display size of sidebar icons.
 
-Screen font antialiasing
-Select to smooth the screen appearance of 
text.
-
-Enter the smallest font size to apply 
antialiasing.
-
-
+Visibility
+
+
+Menu icons
+Displays icons next to the corresponding 
menu items. Select from "Automatic", "Hide" and "Show". "Automatic" displays 
icons according to system settings and themes.
+
+Shortcuts
+Displays shortcut keys 
next to corresponding menu items. Select from "Automatic", "Hide", and "Show". 
"Automatic" displays shortcut keys according to system settings.
 
 Mouse
 
-Mouse positioning
+Positioning
 Specifies if and how the mouse pointer will 
be positioned in newly opened dialogs.
 
 
-Middle mouse button
+Middle button
 Defines the function of the middle mouse 
button.
 
 
@@ -154,10 +150,9 @@
   
 
 
-
-Graphics output
-
 Press Shift+Ctrl+R to restore or 
refresh the view of the current document.
+Graphics Output
+
 
 
 Use hardware acceleration
@@ -167,23 +162,29 @@
 Use Anti-Aliasing
 When supported, you can 
enable and disable anti-aliasing of graphics. With anti-aliasing enabled, the 
display of most graphical objects looks smoother and with less 
artifacts.
 
-Use OpenGL for all rendering (on restart)
-Use the high 
performance Open Graphics Library (OpenGL) to render all visual elements of the application, 
including windows, menus, toolbars and icons. OpenGL uses the computer 
graphics device to accelerate the graphics rendering. If the device is 
blacklisted (see below) this option will not be effective.
-
-Force OpenGL even if blacklisted (on 
restart)
-Forces the use of 
OpenGL even if the graphics device is blacklisted. A device is 
blacklisted when it is buggy or may render graphics with poor 
quality.
-
-Menu
-
-
-Icons in menus
-Displays icons next to the corresponding 
menu items. Select from "Automatic", "Hide" and "Show". "Automatic" displays 
icons according to system settings and themes.
+Only WIN gets Skia controls in 7.1
+
+
+Use Skia for all rendering (on restart)
+Use the high 
performance Skia graphics engine to render all visual elements of the 
application, including windows, menus, toolbars and icons. Skia uses 
the computer graphics device to accelerate the graphics rendering.
+
+Force Skia software rendering (on restart)
+Select this 
control to disable Skia's use of the computer graphics 
device.
+During installation, 
%PRODUCTNAME chooses the best options for configuring the Skia graphics engine. 
Normally you do not need to change the default settings if Skia is 
enabled.
+If you experience 
graphical display problems (e.g., with t

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

2020-11-27 Thread Stephan Bergmann (via logerrit)
 vcl/inc/win/salframe.h  |4 ++--
 vcl/win/window/salframe.cxx |4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

New commits:
commit f4a021a196e8f71b92d9e6b4021a99526cb5cc72
Author: Stephan Bergmann 
AuthorDate: Fri Nov 27 11:15:07 2020 +0100
Commit: Stephan Bergmann 
CommitDate: Fri Nov 27 14:27:21 2020 +0100

WinSalFrame::mnWidth/Height should presumably be of type LONG

...as they are only ever set to LO-/HIWORD values cast to a signed integer 
type

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

diff --git a/vcl/inc/win/salframe.h b/vcl/inc/win/salframe.h
index 05f907dc66b9..5068ecd987aa 100644
--- a/vcl/inc/win/salframe.h
+++ b/vcl/inc/win/salframe.h
@@ -41,8 +41,8 @@ public:
 SystemEnvData   maSysData;  // system data
 SalFrameState   maState = {};   // frame state
 int mnShowState;// show state
-longmnWidth;// client width in pixeln
-longmnHeight;   // client height in pixeln
+LONGmnWidth;// client width in pixeln
+LONGmnHeight;   // client height in pixeln
 int mnMinWidth; // min. client width in 
pixeln
 int mnMinHeight;// min. client height in 
pixeln
 int mnMaxWidth; // max. client width in 
pixeln
diff --git a/vcl/win/window/salframe.cxx b/vcl/win/window/salframe.cxx
index 34a569be39ab..3454ebf885c8 100644
--- a/vcl/win/window/salframe.cxx
+++ b/vcl/win/window/salframe.cxx
@@ -4025,8 +4025,8 @@ static void ImplHandleSizeMsg( HWND hWnd, WPARAM wParam, 
LPARAM lParam )
 {
 UpdateFrameGeometry( hWnd, pFrame );
 
-pFrame->mnWidth  = static_cast(LOWORD(lParam));
-pFrame->mnHeight = static_cast(HIWORD(lParam));
+pFrame->mnWidth  = static_cast(LOWORD(lParam));
+pFrame->mnHeight = static_cast(HIWORD(lParam));
 // save state
 ImplSaveFrameState( pFrame );
 // Call Hdl
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Stephan Bergmann (via logerrit)
 vcl/win/gdi/DWriteTextRenderer.cxx |8 
 1 file changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 092ec538045f443dce3e45ff73064eb46d0a4fc0
Author: Stephan Bergmann 
AuthorDate: Fri Nov 27 11:10:56 2020 +0100
Commit: Stephan Bergmann 
CommitDate: Fri Nov 27 14:26:58 2020 +0100

DWRITE_GLYPH_METRICS members are of INT32 and UINT32 types

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

diff --git a/vcl/win/gdi/DWriteTextRenderer.cxx 
b/vcl/win/gdi/DWriteTextRenderer.cxx
index 185925ae7967..abf9be7d5f86 100644
--- a/vcl/win/gdi/DWriteTextRenderer.cxx
+++ b/vcl/win/gdi/DWriteTextRenderer.cxx
@@ -349,10 +349,10 @@ std::vector 
D2DWriteTextOutRenderer::GetGlyphInkBoxes(uint16_t
 auto pOut = aOut.begin();
 for (auto &m : metrics)
 {
-const long left  = m.leftSideBearing,
-   top   = m.topSideBearing - m.verticalOriginY,
-   right = m.advanceWidth - m.rightSideBearing,
-   bottom = INT32(m.advanceHeight) - m.verticalOriginY - 
m.bottomSideBearing;
+const auto left  = m.leftSideBearing;
+const auto top   = m.topSideBearing - m.verticalOriginY;
+const auto right = m.advanceWidth - m.rightSideBearing;
+const auto bottom = INT32(m.advanceHeight) - m.verticalOriginY - 
m.bottomSideBearing;
 
 // Scale to screen space.
 pOut->SetLeft( std::floor(left * mlfEmHeight / 
aFontMetrics.designUnitsPerEm) );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Stephan Bergmann (via logerrit)
 vcl/win/gdi/salnativewidgets-luna.cxx |8 
 1 file changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 4ca0a15af2a1bd9386f5a0d93e9616cf1d818535
Author: Stephan Bergmann 
AuthorDate: Fri Nov 27 11:07:53 2020 +0100
Commit: Stephan Bergmann 
CommitDate: Fri Nov 27 14:26:31 2020 +0100

RECT members are of type LONG

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

diff --git a/vcl/win/gdi/salnativewidgets-luna.cxx 
b/vcl/win/gdi/salnativewidgets-luna.cxx
index 793b061e0e5c..9b50c7405d52 100644
--- a/vcl/win/gdi/salnativewidgets-luna.cxx
+++ b/vcl/win/gdi/salnativewidgets-luna.cxx
@@ -437,10 +437,10 @@ static void impl_drawAeroToolbar( HDC hDC, RECT rc, bool 
bHorizontal )
 {
 if ( rc.top == 0 && bHorizontal )
 {
-const long GRADIENT_HEIGHT = 32;
+const int GRADIENT_HEIGHT = 32;
 
-long gradient_break = rc.top;
-long gradient_bottom = rc.bottom - 1;
+LONG gradient_break = rc.top;
+LONG gradient_bottom = rc.bottom - 1;
 GRADIENT_RECT g_rect[1] = { { 0, 1 } };
 
 // very slow gradient at the top (if we have space for that)
@@ -478,7 +478,7 @@ static void impl_drawAeroToolbar( HDC hDC, RECT rc, bool 
bHorizontal )
 // when done for the vertical ones too
 if ( bHorizontal )
 {
-long from_x, from_y, to_x, to_y;
+LONG from_x, from_y, to_x, to_y;
 
 from_x = rc.left;
 to_x = rc.right;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Tor Lillqvist (via logerrit)
Tag 'cp-6.4-15' created by Andras Timar  at 
2020-11-27 13:09 +

cp-6.4-15

Changes since co-6.4-15:
Tor Lillqvist (1):
  tdf#138481: Trust the break iterator character data in a bundled ICU

---
 i18npool/CustomTarget_breakiterator.mk |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
---
___
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-15'

2020-11-27 Thread Andras Timar (via logerrit)
Tag 'cp-6.4-15' created by Andras Timar  at 
2020-11-27 13:09 +

cp-6.4-15

Changes since cp-6.4-9-1:
---
 0 files changed
---
___
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-15'

2020-11-27 Thread Andras Timar (via logerrit)
Tag 'cp-6.4-15' created by Andras Timar  at 
2020-11-27 13:09 +

cp-6.4-15

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] dictionaries.git: Changes to 'refs/tags/cp-6.4-15'

2020-11-27 Thread Andras Timar (via logerrit)
Tag 'cp-6.4-15' created by Andras Timar  at 
2020-11-27 13:09 +

cp-6.4-15

Changes since cp-6.4-7:
Andras Timar (1):
  more/better suggestions for Spanish

---
 es/es_ANY.aff |3 +++
 es/es_ANY.dic |   20 ++--
 2 files changed, 13 insertions(+), 10 deletions(-)
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Stephan Bergmann (via logerrit)
 vcl/win/gdi/salnativewidgets-luna.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 25aad15042f45682211ab318dc0e2c004d61c72c
Author: Stephan Bergmann 
AuthorDate: Fri Nov 27 11:05:57 2020 +0100
Commit: Stephan Bergmann 
CommitDate: Fri Nov 27 14:03:35 2020 +0100

LOGFONTW::lfHeight is of type LONG

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

diff --git a/vcl/win/gdi/salnativewidgets-luna.cxx 
b/vcl/win/gdi/salnativewidgets-luna.cxx
index 95b95861586f..793b061e0e5c 100644
--- a/vcl/win/gdi/salnativewidgets-luna.cxx
+++ b/vcl/win/gdi/salnativewidgets-luna.cxx
@@ -1425,7 +1425,7 @@ bool WinSalGraphics::getNativeControlRegion(  ControlType 
nType,
 aNonClientMetrics.cbSize = sizeof( aNonClientMetrics );
 if ( SystemParametersInfoW( SPI_GETNONCLIENTMETRICS, sizeof( 
aNonClientMetrics ), &aNonClientMetrics, 0 ) )
 {
-long nFontHeight = aNonClientMetrics.lfMessageFont.lfHeight;
+LONG nFontHeight = aNonClientMetrics.lfMessageFont.lfHeight;
 if( nFontHeight < 0 )
 nFontHeight = -nFontHeight;
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread andreas kainz (via logerrit)
 sc/uiconfig/scalc/ui/notebookbar.ui |   71 +++-
 1 file changed, 63 insertions(+), 8 deletions(-)

New commits:
commit 3fbda6517e4ee345966873d2930c226b890f33e9
Author: andreas kainz 
AuthorDate: Fri Nov 27 10:03:37 2020 +0100
Commit: Andreas Kainz 
CommitDate: Fri Nov 27 13:57:09 2020 +0100

calc notebookbar update for 7.1

Change-Id: I9b9e09c1ce0681e9a17ca79677d03200b1fc536d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106723
Tested-by: Jenkins
Reviewed-by: Andreas Kainz 

diff --git a/sc/uiconfig/scalc/ui/notebookbar.ui 
b/sc/uiconfig/scalc/ui/notebookbar.ui
index f34610b1fc7b..ba90157318e1 100644
--- a/sc/uiconfig/scalc/ui/notebookbar.ui
+++ b/sc/uiconfig/scalc/ui/notebookbar.ui
@@ -3229,6 +3229,8 @@
   
 True
 True
+center
+True
 both-horiz
 False
 
@@ -3253,6 +3255,8 @@
   
 True
 True
+center
+True
 both-horiz
 False
 
@@ -3312,10 +3316,12 @@
   
 True
 False
+True
 
   
 True
 True
+center
 True
 
   
@@ -3340,6 +3346,7 @@
   
 True
 True
+center
 5
 5
 
@@ -3364,6 +3371,7 @@
   
 True
 True
+center
 icons
 False
 
@@ -3407,10 +3415,12 @@
 True
 False
 center
+True
 
   
 True
 True
+center
 icons
 False
 
@@ -3484,6 +3494,7 @@
   
 True
 True
+center
 icons
 False
 
@@ -3534,6 +3545,7 @@
   
 True
 True
+center
 icons
 False
 
@@ -3639,10 +3651,12 @@
 True
 False
 center
+True
 
   
 True
 True
+center
 icons
 False
 
@@ -3697,10 +3711,12 @@
 True
 False
 center
+True
 
   
 True
 True
+center
 icons
 False
 
@@ -3784,6 +3800,7 @@
 True
 False
 center
+ 

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

2020-11-27 Thread Stephan Bergmann (via logerrit)
 vcl/win/gdi/salprn.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit f813a5d68af4e856a260cc1b6ec8813ed99db3bf
Author: Stephan Bergmann 
AuthorDate: Fri Nov 27 10:56:25 2020 +0100
Commit: Stephan Bergmann 
CommitDate: Fri Nov 27 13:49:14 2020 +0100

DocumentPropertiesW returns LONG

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

diff --git a/vcl/win/gdi/salprn.cxx b/vcl/win/gdi/salprn.cxx
index 0deb814880e4..e7a6936364f5 100644
--- a/vcl/win/gdi/salprn.cxx
+++ b/vcl/win/gdi/salprn.cxx
@@ -281,7 +281,7 @@ static bool ImplTestSalJobSetup( WinSalInfoPrinter const * 
pPrinter,
 BYTE const * pDriverData = reinterpret_cast(pSalDriverData) + pSalDriverData->mnDriverOffset;
 pDevModeW = reinterpret_cast(pDriverData);
 
-long nSysJobSize = -1;
+LONG nSysJobSize = -1;
 if( pPrinter && pDevModeW )
 {
 // just too many driver crashes in that area -> check the 
dmSpecVersion and dmDriverVersion fields always !!!
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Stephan Bergmann (via logerrit)
 vcl/win/gdi/salnativewidgets-luna.cxx |   12 ++--
 vcl/win/window/salframe.cxx   |2 +-
 2 files changed, 7 insertions(+), 7 deletions(-)

New commits:
commit e99897ff2854c1f37bbbe3cc7bf8738495403f6e
Author: Stephan Bergmann 
AuthorDate: Fri Nov 27 11:01:35 2020 +0100
Commit: Stephan Bergmann 
CommitDate: Fri Nov 27 13:48:47 2020 +0100

loplugin:toolslong (Library_vclplug_win)

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

diff --git a/vcl/win/gdi/salnativewidgets-luna.cxx 
b/vcl/win/gdi/salnativewidgets-luna.cxx
index 68a18b020733..95b95861586f 100644
--- a/vcl/win/gdi/salnativewidgets-luna.cxx
+++ b/vcl/win/gdi/salnativewidgets-luna.cxx
@@ -1008,7 +1008,7 @@ static bool ImplDrawNativeControl( HDC hDC, HTHEME 
hTheme, RECT rc,
 if( vsAPI.GetThemeBackgroundContentRect( hTheme, hDC, PP_BAR, iState, 
&rc, &aProgressRect) != S_OK )
 return false;
 
-long nProgressWidth = aValue.getNumericVal();
+tools::Long nProgressWidth = aValue.getNumericVal();
 nProgressWidth *= (aProgressRect.right - aProgressRect.left);
 nProgressWidth /= (rc.right - rc.left);
 if( AllSettings::GetLayoutRTL() )
@@ -1028,13 +1028,13 @@ static bool ImplDrawNativeControl( HDC hDC, HTHEME 
hTheme, RECT rc,
 RECT aTRect = rc;
 if( nPart == ControlPart::TrackHorzArea )
 {
-long nH = aTrackRect.GetHeight();
+tools::Long nH = aTrackRect.GetHeight();
 aTRect.top += (rc.bottom - rc.top - nH)/2;
 aTRect.bottom = aTRect.top + nH;
 }
 else
 {
-long nW = aTrackRect.GetWidth();
+tools::Long nW = aTrackRect.GetWidth();
 aTRect.left += (rc.right - rc.left - nW)/2;
 aTRect.right = aTRect.left + nW;
 }
@@ -1138,7 +1138,7 @@ static bool ImplDrawNativeControl( HDC hDC, HTHEME 
hTheme, RECT rc,
 tools::Rectangle aRect( ImplGetThemeRect( hTheme, hDC,
 MENU_POPUPSEPARATOR, 0, tools::Rectangle( rc.left, rc.top, 
rc.right, rc.bottom ) ) );
 // center the separator inside the passed rectangle
-long nDY = ((rc.bottom - rc.top + 1) - aRect.GetHeight()) / 2;
+auto const nDY = ((rc.bottom - rc.top + 1) - 
aRect.GetHeight()) / 2;
 rc.top += nDY;
 rc.bottom = rc.top+aRect.GetHeight()-1;
 return ImplDrawTheme( hTheme, hDC, MENU_POPUPSEPARATOR, 0, rc, 
aCaption );
@@ -1478,7 +1478,7 @@ bool WinSalGraphics::getNativeControlRegion(  ControlType 
nType,
 tools::Rectangle aThumbRect = ImplGetThemeRect( hTheme, hDC, 
iPart, iState, tools::Rectangle() );
 if( nPart == ControlPart::ThumbHorz )
 {
-long nW = aThumbRect.GetWidth();
+tools::Long nW = aThumbRect.GetWidth();
 tools::Rectangle aRect( rControlRegion );
 aRect.SetRight( aRect.Left() + nW - 1 );
 rNativeContentRegion = aRect;
@@ -1486,7 +1486,7 @@ bool WinSalGraphics::getNativeControlRegion(  ControlType 
nType,
 }
 else
 {
-long nH = aThumbRect.GetHeight();
+tools::Long nH = aThumbRect.GetHeight();
 tools::Rectangle aRect( rControlRegion );
 aRect.SetBottom( aRect.Top() + nH - 1 );
 rNativeContentRegion = aRect;
diff --git a/vcl/win/window/salframe.cxx b/vcl/win/window/salframe.cxx
index e016430611ad..34a569be39ab 100644
--- a/vcl/win/window/salframe.cxx
+++ b/vcl/win/window/salframe.cxx
@@ -2580,7 +2580,7 @@ static void ImplSalUpdateStyleFontW( HDC hDC, const 
LOGFONTW& rLogFont, vcl::Fon
 
 static tools::Long ImplW2I( const wchar_t* pStr )
 {
-longn = 0;
+tools::Long n = 0;
 int nSign = 1;
 
 if ( *pStr == '-' )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-6.2' - include/vcl vcl/source xmlsecurity/Library_xmlsecurity.mk xmlsecurity/qa xmlsecurity/source xmlsecurity/workben

2020-11-27 Thread Miklos Vajna (via logerrit)
 include/vcl/filter/PDFiumLibrary.hxx   |   
30 
 vcl/source/pdf/PDFiumLibrary.cxx   |   
32 
 xmlsecurity/Library_xmlsecurity.mk |   
 5 
 xmlsecurity/qa/unit/signing/data/hide-and-replace-shadow-file-signed-2.pdf 
|binary
 xmlsecurity/qa/unit/signing/signing.cxx|   
18 ++
 xmlsecurity/source/pdfio/pdfdocument.cxx   |   
70 ++
 xmlsecurity/workben/pdfverify.cxx  |   
 5 
 7 files changed, 157 insertions(+), 3 deletions(-)

New commits:
commit 9d27b59f64a1c04a27cf5fa0c91245a74b063bdf
Author: Miklos Vajna 
AuthorDate: Fri Sep 4 17:17:48 2020 +0200
Commit: Miklos Vajna 
CommitDate: Fri Nov 27 13:38:20 2020 +0100

xmlsecurity: pdf incremental updates that are non-commenting are invalid

I.e. it's OK to add incremental updates for annotation/commenting
purposes and that doesn't invalite existing signatures. Everything else
does.

(cherry picked from commit 61834cd574568613f0b0a2ee099a60fa5a8d9804)

Conflicts:
include/vcl/filter/PDFiumLibrary.hxx
vcl/source/pdf/PDFiumLibrary.cxx
xmlsecurity/qa/unit/signing/signing.cxx

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

diff --git a/include/vcl/filter/PDFiumLibrary.hxx 
b/include/vcl/filter/PDFiumLibrary.hxx
index bc7912c17e81..7e8ef4e3ad48 100644
--- a/include/vcl/filter/PDFiumLibrary.hxx
+++ b/include/vcl/filter/PDFiumLibrary.hxx
@@ -17,6 +17,9 @@
 #include 
 #include 
 #include 
+#include 
+
+#include 
 
 namespace vcl::pdf
 {
@@ -31,6 +34,33 @@ public:
 ~PDFium();
 };
 
+class VCL_DLLPUBLIC PDFiumPage final
+{
+private:
+FPDF_PAGE mpPage;
+
+private:
+PDFiumPage(const PDFiumPage&) = delete;
+PDFiumPage& operator=(const PDFiumPage&) = delete;
+
+public:
+PDFiumPage(FPDF_PAGE pPage)
+: mpPage(pPage)
+{
+}
+
+~PDFiumPage()
+{
+if (mpPage)
+FPDF_ClosePage(mpPage);
+}
+
+FPDF_PAGE getPointer() { return mpPage; }
+
+/// Get bitmap checksum of the page, without annotations/commenting.
+BitmapChecksum getChecksum();
+};
+
 struct PDFiumLibrary : public rtl::StaticWithInit, 
PDFiumLibrary>
 {
 std::shared_ptr operator()() { return std::make_shared(); }
diff --git a/vcl/source/pdf/PDFiumLibrary.cxx b/vcl/source/pdf/PDFiumLibrary.cxx
index 604807524bf9..5b4c56a3cef3 100644
--- a/vcl/source/pdf/PDFiumLibrary.cxx
+++ b/vcl/source/pdf/PDFiumLibrary.cxx
@@ -15,6 +15,10 @@
 #include 
 #include 
 
+#include 
+
+#include 
+
 namespace vcl::pdf
 {
 PDFium::PDFium()
@@ -29,6 +33,34 @@ PDFium::PDFium()
 
 PDFium::~PDFium() { FPDF_DestroyLibrary(); }
 
+BitmapChecksum PDFiumPage::getChecksum()
+{
+size_t nPageWidth = FPDF_GetPageWidth(mpPage);
+size_t nPageHeight = FPDF_GetPageHeight(mpPage);
+FPDF_BITMAP pPdfBitmap = FPDFBitmap_Create(nPageWidth, nPageHeight, 
/*alpha=*/1);
+if (!pPdfBitmap)
+{
+return 0;
+}
+
+// Intentionally not using FPDF_ANNOT here, annotations/commenting is OK 
to not affect the
+// checksum, signature verification wants this.
+FPDF_RenderPageBitmap(pPdfBitmap, mpPage, /*start_x=*/0, /*start_y=*/0, 
nPageWidth, nPageHeight,
+  /*rotate=*/0, /*flags=*/0);
+Bitmap aBitmap(Size(nPageWidth, nPageHeight), 24);
+{
+BitmapScopedWriteAccess pWriteAccess(aBitmap);
+const auto pPdfBuffer = 
static_cast(FPDFBitmap_GetBuffer(pPdfBitmap));
+const int nStride = FPDFBitmap_GetStride(pPdfBitmap);
+for (size_t nRow = 0; nRow < nPageHeight; ++nRow)
+{
+ConstScanline pPdfLine = pPdfBuffer + (nStride * nRow);
+pWriteAccess->CopyScanline(nRow, pPdfLine, 
ScanlineFormat::N32BitTcBgra, nStride);
+}
+}
+return aBitmap.GetChecksum();
+}
+
 } // end vcl::pdf
 
 #endif // HAVE_FEATURE_PDFIUM
diff --git a/xmlsecurity/Library_xmlsecurity.mk 
b/xmlsecurity/Library_xmlsecurity.mk
index 9a65dd2152a9..0ad912d5e0cc 100644
--- a/xmlsecurity/Library_xmlsecurity.mk
+++ b/xmlsecurity/Library_xmlsecurity.mk
@@ -20,7 +20,10 @@ $(eval $(call gb_Library_add_defs,xmlsecurity,\
 -DXMLSECURITY_DLLIMPLEMENTATION \
 ))
 
-$(eval $(call gb_Library_use_externals,xmlsecurity,boost_headers))
+$(eval $(call gb_Library_use_externals,xmlsecurity,\
+   boost_headers \
+   $(if $(filter PDFIUM,$(BUILD_TYPE)),pdfium) \
+))
 
 $(eval $(call 
gb_Library_set_precompiled_header,xmlsecurity,$(SRCDIR)/xmlsecurity/inc/pch/precompiled_xmlsecurity))
 
diff --git 
a/xmlsecurity/qa/unit/signing/data/hide-and-replace-shadow-file-signed-2.pdf 
b/xmlsecurity/qa/unit/signing/data/hide-and-replace-shadow-file-signed-2.pdf
n

[Libreoffice-commits] core.git: Branch 'libreoffice-7-0' - solenv/flatpak-manifest.in

2020-11-27 Thread Stephan Bergmann (via logerrit)
 solenv/flatpak-manifest.in |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 4743b3153f4a191f9430ff962033b39526206fb1
Author: Stephan Bergmann 
AuthorDate: Fri Nov 27 08:22:33 2020 +0100
Commit: Caolán McNamara 
CommitDate: Fri Nov 27 13:25:44 2020 +0100

Adapt solenv/flatpak-manifest.in to312a33b7636334f6ce3b6d1702bc5d3e45215601

..."Update language-subtag-registry to 2020-09-29"

Change-Id: Idea4df8f6a920cbceb37d5156417f6979d7a4fff
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106718
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann 
(cherry picked from commit 25bb4c31a68acd4526176580be72a9d4d626d013)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106731
Reviewed-by: Caolán McNamara 

diff --git a/solenv/flatpak-manifest.in b/solenv/flatpak-manifest.in
index 0719073025ed..1da85580037d 100644
--- a/solenv/flatpak-manifest.in
+++ b/solenv/flatpak-manifest.in
@@ -255,10 +255,10 @@
 "dest-filename": 
"external/tarballs/libfreehand-0.1.2.tar.xz"
 },
 {
-"url": 
"https://dev-www.libreoffice.org/src/language-subtag-registry-2020-04-01.tar.bz2";,
-"sha256": 
"fb1ee0dabfd956a445fbe9f351e86a52767808558f20f4256e67fbbb3768e9da",
+"url": 
"https://dev-www.libreoffice.org/src/language-subtag-registry-2020-09-29.tar.bz2";,
+"sha256": 
"cbe9fca811a37056560aab73e9fc9d3522b46b6785cb02db165f521bf42c230f",
 "type": "file",
-"dest-filename": 
"external/tarballs/language-subtag-registry-2020-04-01.tar.bz2"
+"dest-filename": 
"external/tarballs/language-subtag-registry-2020-09-29.tar.bz2"
 },
 {
 "url": 
"https://dev-www.libreoffice.org/src/liblangtag-0.6.2.tar.bz2";,
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Stephan Bergmann (via logerrit)
 vcl/win/gdi/salprn.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 8a27fbebc5bcb69a1b2664a5ffa31e5d8bb7d225
Author: Stephan Bergmann 
AuthorDate: Fri Nov 27 10:54:48 2020 +0100
Commit: Stephan Bergmann 
CommitDate: Fri Nov 27 13:20:14 2020 +0100

GetLastError returns DWORD

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

diff --git a/vcl/win/gdi/salprn.cxx b/vcl/win/gdi/salprn.cxx
index d552d17ed96b..0deb814880e4 100644
--- a/vcl/win/gdi/salprn.cxx
+++ b/vcl/win/gdi/salprn.cxx
@@ -1509,7 +1509,7 @@ bool WinSalPrinter::StartJob( const OUString* pFileName,
 
 if ( nRet <= 0 )
 {
-long nError = GetLastError();
+DWORD nError = GetLastError();
 if ( (nRet == SP_USERABORT) || (nRet == SP_APPABORT) || (nError == 
ERROR_PRINT_CANCELLED) || (nError == ERROR_CANCELLED) )
 mnError = SalPrinterError::Abort;
 else
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Stephan Bergmann (via logerrit)
 shell/source/win32/spsupp/registrar.cxx |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit a63087f58090418b6a7c8b046698b372ddc903cf
Author: Stephan Bergmann 
AuthorDate: Fri Nov 27 10:52:12 2020 +0100
Commit: Stephan Bergmann 
CommitDate: Fri Nov 27 13:19:24 2020 +0100

RegCreateKeyExW etc. return LSTATUS

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

diff --git a/shell/source/win32/spsupp/registrar.cxx 
b/shell/source/win32/spsupp/registrar.cxx
index 008c2ac59c82..1d647f3a556c 100644
--- a/shell/source/win32/spsupp/registrar.cxx
+++ b/shell/source/win32/spsupp/registrar.cxx
@@ -16,7 +16,7 @@ namespace {
 HRESULT RegRead(HKEY hRootKey, const wchar_t* subKey, const wchar_t* 
valName, wchar_t* valData, size_t cchData)
 {
 HKEY hKey;
-long iRetVal = RegCreateKeyExW(
+LSTATUS iRetVal = RegCreateKeyExW(
 hRootKey,
 subKey,
 0,
@@ -43,7 +43,7 @@ namespace {
 HRESULT RegWrite(HKEY hRootKey, const wchar_t* subKey, const wchar_t* 
valName, const wchar_t* valData, HKEY *hKeyResult = nullptr)
 {
 HKEY hKey;
-long iRetVal = RegCreateKeyExW(
+LSTATUS iRetVal = RegCreateKeyExW(
 hRootKey,
 subKey,
 0,
@@ -72,7 +72,7 @@ namespace {
 
 HRESULT RegDel(HKEY hRootKey, const wchar_t* subKey)
 {
-long iRetVal = RegDeleteKeyW(hRootKey, subKey);
+LSTATUS iRetVal = RegDeleteKeyW(hRootKey, subKey);
 return HRESULT_FROM_WIN32(iRetVal);
 }
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Stephan Bergmann (via logerrit)
 shell/source/win32/shlxthandler/util/utilities.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit d52ea2baa0885d49f3b5eaa74d782315d01d36ea
Author: Stephan Bergmann 
AuthorDate: Fri Nov 27 10:49:33 2020 +0100
Commit: Stephan Bergmann 
CommitDate: Fri Nov 27 13:17:49 2020 +0100

GetShortPathNameW returns DWORD

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

diff --git a/shell/source/win32/shlxthandler/util/utilities.cxx 
b/shell/source/win32/shlxthandler/util/utilities.cxx
index 80bd6a1cb4fa..489474b53baf 100644
--- a/shell/source/win32/shlxthandler/util/utilities.cxx
+++ b/shell/source/win32/shlxthandler/util/utilities.cxx
@@ -121,7 +121,7 @@ bool HasOnlySpaces(const std::wstring& String)
 std::wstring getShortPathName( const std::wstring& aLongName )
 {
 std::wstring shortName = aLongName;
-long length= GetShortPathNameW( aLongName.c_str(), nullptr, 0 
);
+DWORDlength= GetShortPathNameW( aLongName.c_str(), nullptr, 0 
);
 
 if ( length != 0 )
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Stephan Bergmann (via logerrit)
 shell/inc/columninfo.hxx|4 ++--
 shell/inc/global.hxx|2 +-
 shell/inc/infotips.hxx  |4 ++--
 shell/inc/propertyhdl.hxx   |8 
 shell/inc/propsheets.hxx|4 ++--
 shell/inc/spsupp/COMOpenDocuments.hpp   |4 ++--
 shell/inc/spsupp/COMRefCounted.hpp  |2 +-
 shell/inc/spsupp/spsuppClassFactory.hpp |8 
 shell/inc/thumbviewer.hxx   |4 ++--
 shell/source/win32/shlxthandler/classfactory.cxx|4 ++--
 shell/source/win32/shlxthandler/classfactory.hxx|4 ++--
 shell/source/win32/shlxthandler/columninfo/columninfo.cxx   |4 ++--
 shell/source/win32/shlxthandler/infotips/infotips.cxx   |4 ++--
 shell/source/win32/shlxthandler/ooofilt/ooofilt.hxx |6 +++---
 shell/source/win32/shlxthandler/prophdl/propertyhdl.cxx |   10 +-
 shell/source/win32/shlxthandler/propsheets/propsheets.cxx   |8 
 shell/source/win32/shlxthandler/shlxthdl.cxx|2 +-
 shell/source/win32/shlxthandler/thumbviewer/thumbviewer.cxx |6 +++---
 shell/source/win32/spsupp/COMOpenDocuments.cxx  |4 ++--
 shell/source/win32/spsupp/spsuppClassFactory.cxx|4 ++--
 vcl/win/dtrans/source.cxx   |2 +-
 vcl/win/dtrans/source.hxx   |2 +-
 22 files changed, 50 insertions(+), 50 deletions(-)

New commits:
commit 2a78e9ddafcfa644e771bb2fc47d5ca23f46af59
Author: Stephan Bergmann 
AuthorDate: Fri Nov 27 10:38:05 2020 +0100
Commit: Stephan Bergmann 
CommitDate: Fri Nov 27 13:16:46 2020 +0100

Windows InterlockedIncrement/Decrement use LONG

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

diff --git a/shell/inc/columninfo.hxx b/shell/inc/columninfo.hxx
index 6a6710a65ac6..de122ad1f361 100644
--- a/shell/inc/columninfo.hxx
+++ b/shell/inc/columninfo.hxx
@@ -26,7 +26,7 @@
 class CColumnInfo : public IColumnProvider
 {
 public:
-CColumnInfo(long RefCnt = 1);
+CColumnInfo(LONG RefCnt = 1);
 virtual ~CColumnInfo();
 
 
@@ -53,7 +53,7 @@ public:
 LPCSHCOLUMNID pscid, LPCSHCOLUMNDATA pscd, VARIANT *pvarData) override;
 
 private:
-longm_RefCnt;
+LONGm_RefCnt;
 };
 
 #endif
diff --git a/shell/inc/global.hxx b/shell/inc/global.hxx
index fb812bcf8140..a729d59301e8 100644
--- a/shell/inc/global.hxx
+++ b/shell/inc/global.hxx
@@ -25,7 +25,7 @@
 #endif
 #include 
 
-extern long g_DllRefCnt;
+extern LONG g_DllRefCnt;
 
 #endif
 
diff --git a/shell/inc/infotips.hxx b/shell/inc/infotips.hxx
index ac9218a71762..5a1128cb7d61 100644
--- a/shell/inc/infotips.hxx
+++ b/shell/inc/infotips.hxx
@@ -28,7 +28,7 @@
 class CInfoTip : public IQueryInfo, public IPersistFile
 {
 public:
-CInfoTip(long RefCnt = 1);
+CInfoTip(LONG RefCnt = 1);
 virtual ~CInfoTip();
 
 
@@ -78,7 +78,7 @@ public:
 /* [out] */ LPOLESTR __RPC_FAR *ppszFileName) override;
 
 private:
-longm_RefCnt;
+LONGm_RefCnt;
 Filepath_char_t m_szFileName[MAX_PATH];
 std::wstringm_FileNameOnly;
 };
diff --git a/shell/inc/propertyhdl.hxx b/shell/inc/propertyhdl.hxx
index 725885525de6..44c848e403cc 100644
--- a/shell/inc/propertyhdl.hxx
+++ b/shell/inc/propertyhdl.hxx
@@ -33,7 +33,7 @@ class CPropertyHdl : public IPropertyStore,
  public IInitializeWithStream
 {
 public:
- CPropertyHdl(long RefCnt = 1);
+ CPropertyHdl(LONG RefCnt = 1);
 virtual ~CPropertyHdl();
 
 
@@ -68,7 +68,7 @@ private:
 voidLoadProperties( CMetaInfoReader *pMetaInfoReader );
 
 private:
-long m_RefCnt;
+LONG m_RefCnt;
 IPropertyStoreCache* m_pCache;
 };
 
@@ -99,9 +99,9 @@ public:
 static bool IsLocked();
 
 private:
-long  m_RefCnt;
+LONG  m_RefCnt;
 CLSID m_Clsid;
-static long  s_ServerLocks;
+static LONG  s_ServerLocks;
 };
 
 #endif
diff --git a/shell/inc/propsheets.hxx b/shell/inc/propsheets.hxx
index 2c3883555d1e..68e28192922d 100644
--- a/shell/inc/propsheets.hxx
+++ b/shell/inc/propsheets.hxx
@@ -30,7 +30,7 @@
 class CPropertySheet : public IShellExtInit, public IShellPropSheetExt
 {
 public:
-CPropertySheet(long RefCnt = 1);
+CPropertySheet(LONG RefCnt = 1);
 
 virtual ~CPropertySheet();
 
@@ -74,7 +74,7 @@ private:
 void InitPropPageStatistics(HWND hwnd, LPPROPSHEETPAGE lppsp);
 
 private:
-long m_RefCnt;
+LONG m_RefCnt;
 Filepath_char_t m_szFileName[MAX_PATH];
 };
 
diff --git a/shell/inc/spsupp/COMOpenDocuments.hpp 
b/shell/inc/spsupp/COMOpenDocuments.hpp
index 21b378d895e6..

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

2020-11-27 Thread Stephan Bergmann (via logerrit)
 extensions/source/ole/unoconversionutilities.hxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit a93b47c0a788771cc761f37e610985cbeb971a1e
Author: Stephan Bergmann 
AuthorDate: Fri Nov 27 10:27:48 2020 +0100
Commit: Stephan Bergmann 
CommitDate: Fri Nov 27 13:15:59 2020 +0100

CComVariant.lVal is of type LONG

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

diff --git a/extensions/source/ole/unoconversionutilities.hxx 
b/extensions/source/ole/unoconversionutilities.hxx
index 32788abc4ad5..449941b202d9 100644
--- a/extensions/source/ole/unoconversionutilities.hxx
+++ b/extensions/source/ole/unoconversionutilities.hxx
@@ -1957,7 +1957,7 @@ void 
UnoConversionUtilities::dispatchExObject2Sequence( const VARIANTARG* pva
 if( FAILED( VariantChangeType( &result, &result, 0, VT_I4)))
 throw BridgeRuntimeError("[automation bridge] 
UnoConversionUtilities::dispatchExObject2Sequence \n"
  "Conversion of dispatch object to 
Sequence failed!");
-long length= result.lVal;
+LONG length= result.lVal;
 
 result.Clear();
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Balazs Varga (via logerrit)
 chart2/qa/extras/chart2export.cxx  |   20 +++-
 chart2/qa/extras/chart2import.cxx  |2 --
 chart2/qa/extras/data/ods/tdf72776.ods |binary
 svx/source/svdraw/svdoashp.cxx |6 ++
 svx/source/unodraw/unoshtxt.cxx|5 -
 sw/qa/extras/layout/layout2.cxx|9 -
 6 files changed, 33 insertions(+), 9 deletions(-)

New commits:
commit 71b66b0039819f38c935b4eb5d5951ceaf6e8468
Author: Balazs Varga 
AuthorDate: Tue Oct 27 21:19:36 2020 +0100
Commit: László Németh 
CommitDate: Fri Nov 27 13:09:33 2020 +0100

tdf#138307 Chart import: fix lost custom shape text

Do not set empty text to SdrOutliner, if it is already set.

Note: Also fix the missing custom shape problem in
tdf#72776 and tdf#93641. The wrong position (X:0, Y:0) of
these shapes is another problem.

Regression from commit 6f62a5c4ee2c1b7654c7fcaa618fb495e0d32f0a
(tdf#128985: ODP: Style text directions revert to RTL on save and re-open)

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

diff --git a/chart2/qa/extras/chart2export.cxx 
b/chart2/qa/extras/chart2export.cxx
index a39b36234862..d92e06cdc01c 100644
--- a/chart2/qa/extras/chart2export.cxx
+++ b/chart2/qa/extras/chart2export.cxx
@@ -184,6 +184,7 @@ public:
 void testTdf137917();
 void testTdf138204();
 void testTdf138181();
+void testCustomShapeText();
 
 CPPUNIT_TEST_SUITE(Chart2ExportTest);
 CPPUNIT_TEST(testErrorBarXLSX);
@@ -330,6 +331,7 @@ public:
 CPPUNIT_TEST(testTdf137917);
 CPPUNIT_TEST(testTdf138204);
 CPPUNIT_TEST(testTdf138181);
+CPPUNIT_TEST(testCustomShapeText);
 
 CPPUNIT_TEST_SUITE_END();
 
@@ -3016,7 +3018,7 @@ void Chart2ExportTest::testTdf138181()
 {
 load("/chart2/qa/extras/data/xlsx/", "piechart_deleted_legendentry.xlsx");
 Reference xChartDoc(getChartDocFromSheet(0, 
mxComponent),
-   UNO_QUERY_THROW);
+UNO_QUERY_THROW);
 Reference xDrawPageSupplier(xChartDoc, 
UNO_QUERY_THROW);
 Reference xDrawPage(xDrawPageSupplier->getDrawPage(), 
UNO_SET_THROW);
 Reference xShapes(xDrawPage->getByIndex(0), 
UNO_QUERY_THROW);
@@ -3038,6 +3040,22 @@ void Chart2ExportTest::testTdf138181()
 CPPUNIT_ASSERT(xLegendEntry3.is());
 }
 
+void Chart2ExportTest::testCustomShapeText()
+{
+load("/chart2/qa/extras/data/ods/", "tdf72776.ods");
+reload("calc8");
+Reference xChartDoc(getChartDocFromSheet(0, 
mxComponent),
+UNO_QUERY_THROW);
+// test that the text of custom shape exists inside the chart
+Reference xDrawPageSupplier(xChartDoc, 
UNO_QUERY_THROW);
+Reference xDrawPage(xDrawPageSupplier->getDrawPage(), 
UNO_SET_THROW);
+Reference xCustomShape(xDrawPage->getByIndex(1), 
UNO_QUERY_THROW);
+CPPUNIT_ASSERT(xCustomShape.is());
+
+Reference< text::XText > xRange(xCustomShape, uno::UNO_QUERY_THROW);
+CPPUNIT_ASSERT(!xRange->getString().isEmpty());
+}
+
 CPPUNIT_TEST_SUITE_REGISTRATION(Chart2ExportTest);
 
 CPPUNIT_PLUGIN_IMPLEMENT();
diff --git a/chart2/qa/extras/chart2import.cxx 
b/chart2/qa/extras/chart2import.cxx
index 490b2ce2698d..cb8c08c0697e 100644
--- a/chart2/qa/extras/chart2import.cxx
+++ b/chart2/qa/extras/chart2import.cxx
@@ -30,8 +30,6 @@
 #include 
 #include 
 #include 
-#include 
-#include 
 #include 
 
 #include 
diff --git a/chart2/qa/extras/data/ods/tdf72776.ods 
b/chart2/qa/extras/data/ods/tdf72776.ods
new file mode 100644
index ..4ddad64a5ef5
Binary files /dev/null and b/chart2/qa/extras/data/ods/tdf72776.ods differ
diff --git a/svx/source/svdraw/svdoashp.cxx b/svx/source/svdraw/svdoashp.cxx
index 29461df169a5..35812345b27b 100644
--- a/svx/source/svdraw/svdoashp.cxx
+++ b/svx/source/svdraw/svdoashp.cxx
@@ -2162,10 +2162,8 @@ void SdrObjCustomShape::SetVerticalWriting( bool 
bVertical )
 
 DBG_ASSERT( pOutlinerParaObject, "SdrTextObj::SetVerticalWriting() without 
OutlinerParaObject!" );
 
-if( !pOutlinerParaObject )
-return;
-
-if(pOutlinerParaObject->IsVertical() == bVertical)
+if( !pOutlinerParaObject ||
+(pOutlinerParaObject->IsVertical() == bVertical) )
 return;
 
 // get item settings
diff --git a/svx/source/unodraw/unoshtxt.cxx b/svx/source/unodraw/unoshtxt.cxx
index e0ade5699832..7f2a8af49606 100644
--- a/svx/source/unodraw/unoshtxt.cxx
+++ b/svx/source/unodraw/unoshtxt.cxx
@@ -545,7 +545,10 @@ SvxTextForwarder* 
SvxTextEditSourceImpl::GetBackgroundTextForwarder()
 
 if( pOutlinerParaObject && ( bOwnParaObj || 
!mpObject->IsEmptyPresObj() || 
mpObject->getSdrPageFromSdrObject()->IsMasterPage() ) )
 {
-mpOutliner->SetText( *pOutlinerParaObject );
+// tdf#72776: do not set empty text to SdrOutliner, if it is 
already set
+if( !mpOutliner->GetEditEngine()

[Libreoffice-commits] core.git: Branch 'distro/vector/vector-5.4' - vcl/CppunitTest_vcl_graphic_test.mk vcl/Module_vcl.mk vcl/qa vcl/source

2020-11-27 Thread Miklos Vajna (via logerrit)
 vcl/CppunitTest_vcl_graphic_test.mk |   48 +++
 vcl/Module_vcl.mk   |1 
 vcl/qa/cppunit/GraphicTest.cxx  |   90 
 vcl/qa/cppunit/data/roundtrip.wmf   |binary
 vcl/source/filter/graphicfilter.cxx |   31 ++--
 5 files changed, 165 insertions(+), 5 deletions(-)

New commits:
commit 66db3307b75ce21767328a00371a8683207ddc99
Author: Miklos Vajna 
AuthorDate: Wed Nov 25 18:01:08 2020 +0100
Commit: Miklos Vajna 
CommitDate: Fri Nov 27 13:07:40 2020 +0100

WMF export: just write the source data as is for from-WMF graphics

This was already working on master since commit
5868745db74ae930edb0058490076d82aaeafbe9 (emfplus: make VectorFormats
Emf/Wmf/Svg work, 2017-06-12), but a matching testcase was missing.

[ And on this branch, fix the actual problem by picking the small
relevant subset of the above commit. ]

(cherry picked from commit 6bb0e09e2423ae00e06e6b7ae2c5a0af6ca100a1)

Conflicts:
vcl/qa/cppunit/GraphicTest.cxx

Change-Id: I7661cd5e66d13b1750f16c0c423c4b69420577a0

diff --git a/vcl/CppunitTest_vcl_graphic_test.mk 
b/vcl/CppunitTest_vcl_graphic_test.mk
new file mode 100644
index ..98eaae8800e7
--- /dev/null
+++ b/vcl/CppunitTest_vcl_graphic_test.mk
@@ -0,0 +1,48 @@
+# -*- Mode: makefile-gmake; tab-width: 4; indent-tabs-mode: t -*-
+#
+# This file is part of the LibreOffice project.
+#
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+#
+
+$(eval $(call gb_CppunitTest_CppunitTest,vcl_graphic_test))
+
+$(eval $(call gb_CppunitTest_add_exception_objects,vcl_graphic_test, \
+vcl/qa/cppunit/GraphicTest \
+))
+
+$(eval $(call gb_CppunitTest_use_externals,vcl_graphic_test,\
+   boost_headers \
+   glm_headers \
+))
+
+$(eval $(call gb_CppunitTest_set_include,vcl_graphic_test,\
+$$(INCLUDE) \
+-I$(SRCDIR)/vcl/inc \
+))
+
+$(eval $(call gb_CppunitTest_use_libraries,vcl_graphic_test, \
+   comphelper \
+   cppu \
+   cppuhelper \
+   sal \
+svt \
+   test \
+   tl \
+   unotest \
+   vcl \
+   utl \
+))
+
+$(eval $(call gb_CppunitTest_use_sdk_api,vcl_graphic_test))
+$(eval $(call gb_CppunitTest_use_ure,vcl_graphic_test))
+$(eval $(call gb_CppunitTest_use_vcl,vcl_graphic_test))
+$(eval $(call gb_CppunitTest_use_rdb,vcl_graphic_test,services))
+$(eval $(call gb_CppunitTest_use_configuration,vcl_graphic_test))
+
+# we need to explicitly depend on Library_gie because it's dynamically loaded 
for .gif
+$(call gb_CppunitTest_get_target,vcl_graphic_test) : $(call 
gb_Library_get_target,gie)
+
+# vim: set noet sw=4 ts=4:
diff --git a/vcl/Module_vcl.mk b/vcl/Module_vcl.mk
index a5e4bf5d68d2..b74bc2e67208 100644
--- a/vcl/Module_vcl.mk
+++ b/vcl/Module_vcl.mk
@@ -154,6 +154,7 @@ $(eval $(call gb_Module_add_check_targets,vcl,\
CppunitTest_vcl_svm_test \
CppunitTest_vcl_pdfexport \
 CppunitTest_vcl_errorhandler \
+   CppunitTest_vcl_graphic_test \
 ))
 
 
diff --git a/vcl/qa/cppunit/GraphicTest.cxx b/vcl/qa/cppunit/GraphicTest.cxx
new file mode 100644
index ..d71a85ebf07b
--- /dev/null
+++ b/vcl/qa/cppunit/GraphicTest.cxx
@@ -0,0 +1,90 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ */
+
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+
+#include 
+
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+
+using namespace css;
+
+namespace
+{
+class GraphicTest : public CppUnit::TestFixture
+{
+public:
+~GraphicTest();
+
+private:
+void testWMFRoundtrip();
+
+CPPUNIT_TEST_SUITE(GraphicTest);
+CPPUNIT_TEST(testWMFRoundtrip);
+CPPUNIT_TEST_SUITE_END();
+};
+
+GraphicTest::~GraphicTest()
+{
+}
+
+void GraphicTest::testWMFRoundtrip()
+{
+// Load a WMF file.
+test::Directories aDirectories;
+OUString aURL = 
aDirectories.getURLFromSrc("vcl/qa/cppunit/data/roundtrip.wmf");
+SvFileStream aStream(aURL, StreamMode::READ);
+sal_uInt64 nPos = aStream.Tell();
+aStream.Seek(STREAM_SEEK_TO_END);
+sal_uInt64 nExpectedSize = aStream.Tell();
+aStream.Seek(nPos);
+GraphicFilter& rGraphicFilter = GraphicFilter::GetGraphicFilter();
+Graphic aGraphic;
+rGraphicFilter.ImportGraphic(aGraphic, OUString(), aStream);
+
+// Save as WMF.
+utl::TempFile aTempFile;
+aTempFile.EnableKillingFile();
+sal_uInt16 nFormat = 
rGraphicFilter.GetExportFormatNumberForShortName(u"WMF");
+SvStream& rOutStream = *aTempFile.GetStr

[Libreoffice-commits] core.git: Branch 'libreoffice-7-1' - i18npool/CustomTarget_breakiterator.mk i18npool/source

2020-11-27 Thread Tor Lillqvist (via logerrit)
 i18npool/CustomTarget_breakiterator.mk  |1 
 i18npool/source/breakiterator/data/char.txt |  119 
 2 files changed, 120 deletions(-)

New commits:
commit 27efeeb7a80c84dbe243f2301f1dbf10cff7ce8d
Author: Tor Lillqvist 
AuthorDate: Thu Nov 26 15:47:26 2020 +0200
Commit: Caolán McNamara 
CommitDate: Fri Nov 27 12:59:41 2020 +0100

tdf#138481: Trust the built-in break iterator character data in ICU

Don't use our own char.txt.

Quite possibly we wouldn't need to use the other data in the
i18npool/source/breakiterator/data folder either. See the README file
there for the sad details.

Change-Id: I82923ae76407fdd3fa3642d818d43427fe4f5591
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106632
Tested-by: Jenkins
Reviewed-by: Tor Lillqvist 
(cherry picked from commit 8578a1c9d167c19f1d8038fac5946b4b3cae305e)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106688
Reviewed-by: Caolán McNamara 

diff --git a/i18npool/CustomTarget_breakiterator.mk 
b/i18npool/CustomTarget_breakiterator.mk
index dee46a31f0b1..e6d658f01405 100644
--- a/i18npool/CustomTarget_breakiterator.mk
+++ b/i18npool/CustomTarget_breakiterator.mk
@@ -51,7 +51,6 @@ endif
 
 i18npool_BRKTXTS := \
 $(if $(i18npool_breakiterator_want_in),char_in.brk) \
-char.brk \
 count_word.brk \
 $(call gb_Helper_optional_locale,he,dict_word_he.brk) \
 $(call gb_Helper_optional_locale,hu,dict_word_hu.brk) \
diff --git a/i18npool/source/breakiterator/data/char.txt 
b/i18npool/source/breakiterator/data/char.txt
deleted file mode 100644
index 3b9f801a7ace..
--- a/i18npool/source/breakiterator/data/char.txt
+++ /dev/null
@@ -1,119 +0,0 @@
-#
-#   Copyright (C) 2002-2009, International Business Machines Corporation and 
others.
-#   All Rights Reserved.
-#
-#   file:  char.txt 
-#
-#   ICU Character Break Rules, also known as Grapheme Cluster Boundaries
-#  See Unicode Standard Annex #29.
-#  These rules are based on TR29 Revision 13, for Unicode Version 5.1
-#   Modifications to SpacingMark and Prepend by M. Hosken.
-#
-
-#
-#  Character Class Definitions.
-#
-$CR  = [\p{Grapheme_Cluster_Break = CR}];
-$LF  = [\p{Grapheme_Cluster_Break = LF}];
-$Control = [\p{Grapheme_Cluster_Break = Control}];
-$Prepend = [\p{Grapheme_Cluster_Break = Prepend}];
-$Extend  = [\p{Grapheme_Cluster_Break = Extend}];
-$SpacingMark = [\p{Grapheme_Cluster_Break = SpacingMark}];
-# True Indic wants to move by syllables. Break up SpacingMark. This based on 
Unicode 6.0 data
-# In effect it is [\p{Grapheme_Cluster_Break = SpacingMark} - \u0E30 \u0E32 
\u0E45 \u0EB0 \u0EB2 \u102B \u102C \u1038
-#   \u1062-\u1064 \u1067-\u106D \u1083 \u1087-\u108C \u108F \u109A-\u109C 
\u19B0-\u19B4 \u19B8-\u19C0 \u19C8 \u19C9
-#   \u1A61 \u1A63 \u1A64 \u1BE7 \u1BEA-\u1BEC \u1BEE \u1BF2 \u1BF3 \uAA7B
-$IndicSpacing = [\u0903 \u093B \u093E-\u0940 \u0949-\u094C \u094E \u094F 
\u0982 \u0983 \u09BF \u09C0 \u09C7 \u09C8 \u09CB \u09CC \u0A03 \u0A3E-\u0A40 
\u0A83 \u0ABE-\u0AC0 \u0AC9 \u0ACB \u0ACC \u0B02 \u0B03 \u0B40 \u0B47 \u0B48 
\u0B4B-\u0B4C \u0BBF \u0BC1 \u0BC2 \u0BC6-\u0BC8 \u0BCA-\u0BCC \u0C01-\u0C03 
\u0C41-\u0C44 \u0C82 \u0C83 \u0CBE \u0CC0 \u0CC1 \u0CC3 \u0CC4 \u0CC7 \u0CC8 
\u0CCA \u0CCB \u0D02 \u0D03 \u0D3F \u0D40 \u0D46-\u0D48 \u0D4A-\u0D4C \u0D82 
\u0D83 \u0DD0 \u0DD1 \u0DD8-\u0DDE \u0F3E \u0F3F \u0F7F \u1923-\u1926 
\u1929-\u192B \u1930 \u1931 \u1933-\u1938 \u1A19-\u1A1B \u1B04 \u1B35 \u1B3B 
\u1B3D-\u1B41 \u1B43 \u1B44 \u1B82 \u1BA1 \u1BA6 \u1BA7 \u1BAA \u1C24-\u1C2B 
\u1C34 \u1C35 \u1CE1 \u1CF2 \uA880 \uA881 \uA8B4-\uA8C3 \uA952 \uA953 \uA983 
\uA9B4 \uA9B5 \uA9BA \uA9BB \uA9BD-\uA9C0 \uAA2F \uAA30 \uAA33 \uAA34 \uABE3 
\uABE4 \uABE6 \uABE7 \uABE9 \uABEA \uABEC \U00011000 \U00011002 \U00011082 
\U000110B0-\U000110B2 \U000110B7 \U000100B8 \U0001D166 \U0001D16D];
-# SEAsian (Thai, Lao, Burmese, Tai Lue, Tai Tham, Batak) are cluster based not 
syllable based
-$SEASpacing = [\u0E33 \u0EB3 \u1031 \u103B \u103C \u1056 \u1057 \u1084 \u17B6 
\u17BE-\u17C5 \u17C7 \u17C8 \u1A55 \u1A57 \u1A6D-\u1A72 \uA823 \uA824 \uA827 
\uAA4D];
-$BengaliLetter = [\u0985-\u09B9 \u09CE \u09DC-\u09E1 \u09F0-\u09F1];
-$BengaliSignVirama = \u09CD;
-$GujaratiLetter = [\u0A85-\u0A8C \u0A8F-\u0A90 \u0A93-\u0AB9 \u0AE0-\u0AE1];
-$GujaratiSignVirama = \u0ACD;
-$DevanagariLetter = [\u0904-\u0939 \u0958-\u0961 \u0972-\u097F];
-$DevanagariSignVirama = \u094D;
-$KannadaLetter = [\u0C85-\u0CB9 \u0CDE-\u0CE1];
-$KannadaSignVirama = \u0CCD;
-$MalayalamLetter = [\u0D05-\u0D39 \u0D60-\u0D61 \u0D7A-\u0D7F];
-$MalayalamSignVirama = \u0D4D;
-$OdiaLetter = [\u0B05-\u0B39 \u0B5C-\u0B61 \u0B71];
-$OdiaSignVirama = \u0B4D;
-$GurmukhiLetter = [\u0A05-\u0A39 \u0A59-\u0A5E];
-$GurmukhiSignVirama = \u0A4D;
-$TamilKa = \u0B95;
-$TamilSignVirama = \u0BCD;
-$TamilSsa = \u0BB7;
-$TeluguLetter = [\u0C05-\u0C39 \u0C58-\u0C61];
-$TeluguSignVirama = \u0C4D;
-
-#
-# Korean Syll

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

2020-11-27 Thread Michael Stahl (via logerrit)
 sw/source/core/doc/docbm.cxx |7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)

New commits:
commit fd596591b385a8b03459a7dd3f79e65a59f530f4
Author: Michael Stahl 
AuthorDate: Thu Nov 26 19:55:27 2020 +0100
Commit: Michael Stahl 
CommitDate: Fri Nov 27 12:50:19 2020 +0100

sw_fieldmarkhide: fix getFieldMarkFor() to return innermost field

... like it's supposed to; if it finds the outer field and then the next
field is inside but ends *before* the rPos then the loop is erroneously
terminated.

Change-Id: Ic5e469d1c05cecd7feca275dc1689af2c40567f3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106710
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
(cherry picked from commit 6a11f94ab366e1d736e97e31e908e86aa5d8b533)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106689

diff --git a/sw/source/core/doc/docbm.cxx b/sw/source/core/doc/docbm.cxx
index 406685c58691..c71178e1f005 100644
--- a/sw/source/core/doc/docbm.cxx
+++ b/sw/source/core/doc/docbm.cxx
@@ -1405,10 +1405,11 @@ namespace sw::mark
 return nullptr;
 auto pFieldmark(*itFieldmark);
 for ( ; itFieldmark != m_vFieldmarks.end()
-&& (**itFieldmark).IsCoveringPosition(rPos); ++itFieldmark)
+&& (**itFieldmark).GetMarkStart() <= rPos; ++itFieldmark)
 {   // find the innermost fieldmark
-if (pFieldmark->GetMarkStart() < (**itFieldmark).GetMarkStart()
-|| (**itFieldmark).GetMarkEnd() < pFieldmark->GetMarkEnd())
+if (rPos < (**itFieldmark).GetMarkEnd()
+&& (pFieldmark->GetMarkStart() < (**itFieldmark).GetMarkStart()
+|| (**itFieldmark).GetMarkEnd() < 
pFieldmark->GetMarkEnd()))
 {
 pFieldmark = *itFieldmark;
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-7-1' - vcl/inc

2020-11-27 Thread Noel Grandin (via logerrit)
 vcl/inc/bitmap/impoctree.hxx |   12 ++--
 1 file changed, 6 insertions(+), 6 deletions(-)

New commits:
commit ace0af37f2f79561cb52d695e3aa00eab3e2f547
Author: Noel Grandin 
AuthorDate: Thu Nov 26 20:10:43 2020 +0200
Commit: Noel Grandin 
CommitDate: Fri Nov 27 12:30:53 2020 +0100

tdf#138486 Artifacts in some impress templates

fields need to be signed

regression from
commit 548d77d0c06f7088dd3eb408797aa1fc1d7eb277
Author: Noel 
Date:   Fri Nov 13 10:23:36 2020 +0200
tools::Long->sal_uInt16 in ImpErrorQuad

Change-Id: Ia2a70d55d9af2ea3712ace3394c0de3931bee95c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106709
Tested-by: Jenkins
Reviewed-by: Noel Grandin 
(cherry picked from commit 31f2b4b23e5d16c1b04035f08ebbb11c1abf62e3)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106735

diff --git a/vcl/inc/bitmap/impoctree.hxx b/vcl/inc/bitmap/impoctree.hxx
index 7c464dbb9758..b77e964f4f79 100644
--- a/vcl/inc/bitmap/impoctree.hxx
+++ b/vcl/inc/bitmap/impoctree.hxx
@@ -24,9 +24,9 @@
 
 class ImpErrorQuad
 {
-sal_uInt32 nRed;
-sal_uInt32 nGreen;
-sal_uInt32 nBlue;
+sal_Int16 nRed;
+sal_Int16 nGreen;
+sal_Int16 nBlue;
 
 public:
 ImpErrorQuad()
@@ -100,9 +100,9 @@ inline void ImpErrorQuad::ImplAddColorError7(const 
ImpErrorQuad& rErrQuad)
 
 inline BitmapColor ImpErrorQuad::ImplGetColor()
 {
-return BitmapColor(std::clamp(nRed, 0, 8160) >> 5,
-   std::clamp(nGreen, 0, 8160) >> 5,
-   std::clamp(nBlue, 0, 8160) >> 5);
+return BitmapColor(std::clamp(nRed, sal_Int16(0), sal_Int16(8160)) >> 5,
+   std::clamp(nGreen, sal_Int16(0), sal_Int16(8160)) >> 5,
+   std::clamp(nBlue, sal_Int16(0), sal_Int16(8160)) >> 5);
 }
 
 #endif // INCLUDED_VCL_INC_IMPOCTREE_HXX
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Justin Luth (via logerrit)
 sw/qa/extras/globalfilter/data/tdf138345_charstyle_highlight.odt |binary
 sw/qa/extras/globalfilter/globalfilter.cxx   |   46 
++
 sw/source/filter/ww8/ww8atr.cxx  |7 +
 3 files changed, 53 insertions(+)

New commits:
commit 5951a867b87cbb6968c1e2059f2cf461d11c
Author: Justin Luth 
AuthorDate: Thu Nov 19 19:58:16 2020 +0300
Commit: Miklos Vajna 
CommitDate: Fri Nov 27 12:15:18 2020 +0100

tdf#138345 sw ms export: Char highlight: no export to char-style

MS Word ignores w:highlight in a character style.
It only accepts it as direct formatting or in para-styles.

So never export the character background as w:highlight,
but always as w:shd for character styles.

Change-Id: I7c9f4f00d957a8add848746051d3c4b1522d1520
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106182
Tested-by: Jenkins
Reviewed-by: Justin Luth 
Reviewed-by: Miklos Vajna 

diff --git a/sw/qa/extras/globalfilter/data/tdf138345_charstyle_highlight.odt 
b/sw/qa/extras/globalfilter/data/tdf138345_charstyle_highlight.odt
new file mode 100644
index ..6886ed8f71c6
Binary files /dev/null and 
b/sw/qa/extras/globalfilter/data/tdf138345_charstyle_highlight.odt differ
diff --git a/sw/qa/extras/globalfilter/globalfilter.cxx 
b/sw/qa/extras/globalfilter/globalfilter.cxx
index a2ef4d318ca6..bb9521823783 100644
--- a/sw/qa/extras/globalfilter/globalfilter.cxx
+++ b/sw/qa/extras/globalfilter/globalfilter.cxx
@@ -48,6 +48,7 @@ public:
 void testCharHighlight();
 void testCharHighlightODF();
 void testCharHighlightBody();
+void testCharStyleHighlight();
 void testMSCharBackgroundEditing();
 void testCharBackgroundToHighlighting();
 #if !defined(_WIN32)
@@ -574,16 +575,61 @@ void Test::testCharHighlightBody()
 }
 }
 
+void Test::testCharStyleHighlight()
+{
+// MS Word has two kind of character backgrounds called character shading 
and highlighting.
+// However, their character style can only accept shading. It ignores the 
highlighting value.
+
+const OUString aFilterNames[] = {
+"Rich Text Format",
+"MS Word 97",
+"Office Open XML Text",
+};
+
+for (OUString const & rFilterName : aFilterNames)
+{
+if (mxComponent.is())
+mxComponent->dispose();
+mxComponent = 
loadFromDesktop(m_directories.getURLFromSrc("/sw/qa/extras/globalfilter/data/tdf138345_charstyle_highlight.odt"),
+  "com.sun.star.text.TextDocument");
+
+const OString sFailedMessage = OStringLiteral("Failed on filter: ") + 
rFilterName.toUtf8();
+
+// Export the document and import again for a check
+uno::Reference xStorable(mxComponent, 
uno::UNO_QUERY);
+
+utl::MediaDescriptor aMediaDescriptor;
+aMediaDescriptor["FilterName"] <<= rFilterName;
+
+utl::TempFile aTempFile;
+aTempFile.EnableKillingFile();
+xStorable->storeToURL(aTempFile.GetURL(), 
aMediaDescriptor.getAsConstPropertyValueList());
+uno::Reference< lang::XComponent > xComponent(xStorable, 
uno::UNO_QUERY);
+xComponent->dispose();
+mxComponent = loadFromDesktop(aTempFile.GetURL(), 
"com.sun.star.text.TextDocument");
+
+uno::Reference xCharStyle;
+getStyles("CharacterStyles")->getByName("charBackground") >>= 
xCharStyle;
+const sal_Int32 nBackColor(0xFFDBB6); //orange-y
+
+// Always export character style's background colour as shading, never 
as highlighting.
+CPPUNIT_ASSERT_EQUAL_MESSAGE(sFailedMessage.getStr(), 
static_cast(COL_TRANSPARENT), 
getProperty(xCharStyle,"CharHighlight"));
+CPPUNIT_ASSERT_EQUAL_MESSAGE(sFailedMessage.getStr(), nBackColor, 
getProperty(xCharStyle,"CharBackColor"));
+}
+}
+
 void Test::testCharHighlight()
 {
 SvtFilterOptions& rOpt = SvtFilterOptions::Get();
 rOpt.SetCharBackground2Shading();
 
 testCharHighlightBody();
+testCharStyleHighlight();
 
 rOpt.SetCharBackground2Highlighting();
 
 testCharHighlightBody();
+testCharStyleHighlight();
 }
 
 void Test::testCharHighlightODF()
diff --git a/sw/source/filter/ww8/ww8atr.cxx b/sw/source/filter/ww8/ww8atr.cxx
index 0da5f1fb4c1c..ef5ea8b38ae6 100644
--- a/sw/source/filter/ww8/ww8atr.cxx
+++ b/sw/source/filter/ww8/ww8atr.cxx
@@ -5556,6 +5556,13 @@ void AttributeOutputBase::CharBackgroundBase( const 
SvxBrushItem& rBrush )
 bool bConvertToShading = 
SvtFilterOptions::Get().IsCharBackground2Shading();
 bool bHasShadingMarker = false;
 
+// MS Word doesn't support highlight in character styles. Always export 
those as shading.
+if ( !bConvertToShading && GetExport().m_bStyDef )
+{
+const SwFormat* pFormat = dynamic_cast( 
GetExport().m_pOutFormatNode );
+bConvertToShading = pFormat && pFormat->Which() == RES_CHRFMT;
+}
+
 // Check shading marker
 const SfxPoolItem* pItem = GetE

[Libreoffice-commits] core.git: Branch 'distro/collabora/cp-6.2' - vcl/CppunitTest_vcl_filter_ipdf.mk vcl/Module_vcl.mk vcl/qa vcl/source

2020-11-27 Thread Miklos Vajna (via logerrit)
 vcl/CppunitTest_vcl_filter_ipdf.mk  |   49 +
 vcl/Module_vcl.mk   |1 
 vcl/qa/cppunit/filter/ipdf/data/dict-array-dict.pdf |   55 +++
 vcl/qa/cppunit/filter/ipdf/ipdf.cxx |  100 
 vcl/source/filter/ipdf/pdfdocument.cxx  |   13 ++
 5 files changed, 216 insertions(+), 2 deletions(-)

New commits:
commit 82b9c35264b62b16dd02837a64580ea5be6a2844
Author: Miklos Vajna 
AuthorDate: Fri Oct 16 18:15:21 2020 +0200
Commit: Miklos Vajna 
CommitDate: Fri Nov 27 12:13:04 2020 +0100

vcl pdf tokenizer: fix handling of dict -> array -> dict tokens

Needed to be able to parse the /Reference key of signatures.

(cherry picked from commit 056c1284d6a68525002c54bef10834cc135385db)

Conflicts:
vcl/qa/cppunit/filter/ipdf/ipdf.cxx

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

diff --git a/vcl/CppunitTest_vcl_filter_ipdf.mk 
b/vcl/CppunitTest_vcl_filter_ipdf.mk
new file mode 100644
index ..403836ac781a
--- /dev/null
+++ b/vcl/CppunitTest_vcl_filter_ipdf.mk
@@ -0,0 +1,49 @@
+# -*- Mode: makefile-gmake; tab-width: 4; indent-tabs-mode: t -*-
+#*
+#
+# This file is part of the LibreOffice project.
+#
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+#
+#*
+
+$(eval $(call gb_CppunitTest_CppunitTest,vcl_filter_ipdf))
+
+$(eval $(call gb_CppunitTest_use_externals,vcl_filter_ipdf,\
+   boost_headers \
+   pdfium \
+))
+
+$(eval $(call gb_CppunitTest_add_exception_objects,vcl_filter_ipdf, \
+vcl/qa/cppunit/filter/ipdf/ipdf \
+))
+
+$(eval $(call gb_CppunitTest_use_libraries,vcl_filter_ipdf, \
+comphelper \
+cppu \
+sal \
+sfx \
+svx \
+test \
+tl \
+unotest \
+utl \
+vcl \
+))
+
+$(eval $(call gb_CppunitTest_use_sdk_api,vcl_filter_ipdf))
+
+$(eval $(call gb_CppunitTest_use_ure,vcl_filter_ipdf))
+$(eval $(call gb_CppunitTest_use_vcl,vcl_filter_ipdf))
+
+$(eval $(call gb_CppunitTest_use_rdb,vcl_filter_ipdf,services))
+
+$(eval $(call gb_CppunitTest_use_custom_headers,vcl_filter_ipdf,\
+   officecfg/registry \
+))
+
+$(eval $(call gb_CppunitTest_use_configuration,vcl_filter_ipdf))
+
+# vim: set noet sw=4 ts=4:
diff --git a/vcl/Module_vcl.mk b/vcl/Module_vcl.mk
index 737f8a101b8d..a83097a765e0 100644
--- a/vcl/Module_vcl.mk
+++ b/vcl/Module_vcl.mk
@@ -219,6 +219,7 @@ $(eval $(call gb_Module_add_check_targets,vcl,\
 ifneq (,$(filter PDFIUM,$(BUILD_TYPE)))
 $(eval $(call gb_Module_add_check_targets,vcl,\
CppunitTest_vcl_pdfexport \
+   CppunitTest_vcl_filter_ipdf \
 ))
 endif
 
diff --git a/vcl/qa/cppunit/filter/ipdf/data/dict-array-dict.pdf 
b/vcl/qa/cppunit/filter/ipdf/data/dict-array-dict.pdf
new file mode 100644
index ..73de3117b9a6
--- /dev/null
+++ b/vcl/qa/cppunit/filter/ipdf/data/dict-array-dict.pdf
@@ -0,0 +1,55 @@
+%PDF-1.7
+%���
+1 0 obj <<
+  /Type /Catalog
+  /Pages 2 0 R
+>>
+endobj
+2 0 obj <<
+  /Type /Pages
+  /MediaBox [0 0 200 300]
+  /Count 1
+  /Kids [3 0 R]
+>>
+endobj
+3 0 obj <<
+  /Type /Page
+  /Parent 2 0 R
+  /Contents 4 0 R
+  /Key[<>]
+>>
+endobj
+4 0 obj <<
+  /Length 188
+>>
+stream
+q
+0 0 0 rg
+0 290 10 10 re B*
+10 150 50 30 re B*
+0 0 1 rg
+190 290 10 10 re B*
+70 232 50 30 re B*
+0 1 0 rg
+190 0 10 10 re B*
+130 150 50 30 re B*
+1 0 0 rg
+0 0 10 10 re B*
+70 67 50 30 re B*
+Q
+endstream
+endobj
+xref
+0 5
+00 65535 f 
+15 0 n 
+68 0 n 
+000157 0 n 
+000251 0 n 
+trailer <<
+  /Root 1 0 R
+  /Size 5
+>>
+startxref
+491
+%%EOF
diff --git a/vcl/qa/cppunit/filter/ipdf/ipdf.cxx 
b/vcl/qa/cppunit/filter/ipdf/ipdf.cxx
new file mode 100644
index ..2a3357322b50
--- /dev/null
+++ b/vcl/qa/cppunit/filter/ipdf/ipdf.cxx
@@ -0,0 +1,100 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ */
+
+#include 
+#include 
+
+#include 
+#include 
+
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+
+using namespace ::com::sun::star;
+
+namespace
+{
+char const DATA_DIRECTORY[] = "/vcl/qa/cppunit/filter/ipdf/data/";
+}
+
+/// Covers vcl/source/filter/ipdf/ fixes

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

2020-11-27 Thread Noel Grandin (via logerrit)
 xmloff/source/forms/elementimport.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 274cb276b2542b782988b1ced7347f2e8ce76c26
Author: Noel Grandin 
AuthorDate: Thu Nov 26 19:49:19 2020 +0200
Commit: Xisco Fauli 
CommitDate: Fri Nov 27 11:55:48 2020 +0100

tdf#138507 error in loading form

regression from
commit 3de38e95561ab7ca114d9f3307702ba89c4e3e9a
Author: Noel Grandin 
Date:   Tue Nov 10 19:20:06 2020 +0200
use fastparser in forms

Change-Id: I995677d4e381e8229e7f052cf54a134bdf49d378
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106708
Tested-by: Jenkins
Reviewed-by: Noel Grandin 
(cherry picked from commit 3faf503b1a389affcff1bcbd63630963f634e486)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106734
Reviewed-by: Xisco Fauli 

diff --git a/xmloff/source/forms/elementimport.cxx 
b/xmloff/source/forms/elementimport.cxx
index 0b14b3155956..896aaf3cd3cc 100644
--- a/xmloff/source/forms/elementimport.cxx
+++ b/xmloff/source/forms/elementimport.cxx
@@ -140,7 +140,7 @@ namespace xmloff
 {
 ENTER_LOG_CONTEXT( "xmloff::OElementImport - importing one element" );
 
-const OUString sControlImplementation = _rxAttrList->getValue( 
XML_ELEMENT(FORM, XML_CONTROL_IMPLEMENTATION) );
+const OUString sControlImplementation = _rxAttrList->getOptionalValue( 
XML_ELEMENT(FORM, XML_CONTROL_IMPLEMENTATION) );
 
 // retrieve the service name
 if ( !sControlImplementation.isEmpty() )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Szymon Kłos (via logerrit)
 include/vcl/toolkit/dialog.hxx   |2 ++
 vcl/jsdialog/executor.cxx|   12 
 vcl/jsdialog/jsdialogbuilder.cxx |3 +++
 vcl/source/window/dialog.cxx |7 +++
 4 files changed, 24 insertions(+)

New commits:
commit 60ec1659924be692de2985d2b1a5363976db6ca1
Author: Szymon Kłos 
AuthorDate: Thu Nov 19 13:50:30 2020 +0100
Commit: Szymon Kłos 
CommitDate: Fri Nov 27 11:42:54 2020 +0100

jsdialog: dump dialog title

Change-Id: Id4c9f336039f8a0b8d1d43fdff4852f32e2c7ae6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106707
Tested-by: Jenkins
Reviewed-by: Szymon Kłos 

diff --git a/include/vcl/toolkit/dialog.hxx b/include/vcl/toolkit/dialog.hxx
index 3c519d0a5004..1d3064784bd4 100644
--- a/include/vcl/toolkit/dialog.hxx
+++ b/include/vcl/toolkit/dialog.hxx
@@ -154,6 +154,8 @@ public:
 voidset_default_response(int nResponse);
 int get_default_response() const;
 vcl::Window*get_widget_for_response(int nResponse);
+
+voidDumpAsPropertyTree(tools::JsonWriter& rJsonWriter) 
override;
 };
 
 #endif // INCLUDED_VCL_DIALOG_HXX
diff --git a/vcl/source/window/dialog.cxx b/vcl/source/window/dialog.cxx
index 5618af225d4c..c6aed4909b87 100644
--- a/vcl/source/window/dialog.cxx
+++ b/vcl/source/window/dialog.cxx
@@ -60,6 +60,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include 
 #include 
@@ -1614,4 +1615,10 @@ void TopLevelWindowLocker::decBusy()
 m_aBusyStack.pop();
 }
 
+void Dialog::DumpAsPropertyTree(tools::JsonWriter& rJsonWriter)
+{
+SystemWindow::DumpAsPropertyTree(rJsonWriter);
+rJsonWriter.put("title", GetText());
+}
+
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
commit b6d634d74c0231173d6b43357b7cd3cecb76407d
Author: Szymon Kłos 
AuthorDate: Wed Nov 18 12:14:35 2020 +0100
Commit: Szymon Kłos 
CommitDate: Fri Nov 27 11:42:38 2020 +0100

jsdialog: remember dialog instance & handle close

Change-Id: I1b10d12edfa7ffca1061f50b5219baae1ac3caeb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106706
Tested-by: Jenkins
Reviewed-by: Szymon Kłos 

diff --git a/vcl/jsdialog/executor.cxx b/vcl/jsdialog/executor.cxx
index 13ffc1a33e06..83d19fd7b657 100644
--- a/vcl/jsdialog/executor.cxx
+++ b/vcl/jsdialog/executor.cxx
@@ -242,6 +242,18 @@ bool ExecuteAction(sal_uInt64 nWindowId, const OString& 
rWidget, StringMap& rDat
 }
 }
 }
+else if (sControlType == "dialog")
+{
+auto pDialog = dynamic_cast(pWidget);
+if (pDialog)
+{
+if (sAction == "close")
+{
+pDialog->response(RET_CANCEL);
+return true;
+}
+}
+}
 }
 
 return false;
diff --git a/vcl/jsdialog/jsdialogbuilder.cxx b/vcl/jsdialog/jsdialogbuilder.cxx
index 58c8c64d1108..13b438022e7e 100644
--- a/vcl/jsdialog/jsdialogbuilder.cxx
+++ b/vcl/jsdialog/jsdialogbuilder.cxx
@@ -365,6 +365,9 @@ std::unique_ptr 
JSInstanceBuilder::weld_dialog(const OString& id)
 std::unique_ptr pRet(pDialog ? new 
JSDialog(m_aOwnedToplevel, m_aOwnedToplevel,
   pDialog, this, 
false, m_sTypeOfJSON)
: nullptr);
+
+RememberWidget("__DIALOG__", pRet.get());
+
 if (pDialog)
 {
 assert(!m_aOwnedToplevel && "only one toplevel per .ui allowed");
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Szymon Kłos (via logerrit)
 vcl/inc/jsdialog/jsdialogbuilder.hxx |   55 +++
 vcl/jsdialog/executor.cxx|   21 +
 vcl/jsdialog/jsdialogbuilder.cxx |  123 +++
 3 files changed, 199 insertions(+)

New commits:
commit 3000f3ee1ac63deff58fa5725a6f6c2f05aec9bb
Author: Szymon Kłos 
AuthorDate: Wed Nov 18 10:12:38 2020 +0100
Commit: Szymon Kłos 
CommitDate: Fri Nov 27 11:42:16 2020 +0100

jsdialog:  drag and drop support for TreeView

Change-Id: I67e2eb986b48591b7f758bbb5c1d72b6a322d4d8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106705
Tested-by: Jenkins
Reviewed-by: Szymon Kłos 

diff --git a/vcl/inc/jsdialog/jsdialogbuilder.hxx 
b/vcl/inc/jsdialog/jsdialogbuilder.hxx
index 4b9e042960f4..ab7c038648d3 100644
--- a/vcl/inc/jsdialog/jsdialogbuilder.hxx
+++ b/vcl/inc/jsdialog/jsdialogbuilder.hxx
@@ -19,6 +19,11 @@
 #include 
 #include 
 
+#include 
+#include 
+#include 
+#include 
+
 class ToolBox;
 class ComboBox;
 class VclMultiLineEdit;
@@ -58,6 +63,40 @@ public:
 void notifyDialogState(bool bForce = false);
 };
 
+class JSDropTarget final
+: public cppu::WeakComponentImplHelper
+{
+osl::Mutex m_aMutex;
+
std::vector> 
m_aListeners;
+
+public:
+JSDropTarget();
+
+// XInitialization
+virtual void SAL_CALL initialize(const css::uno::Sequence& 
rArgs) override;
+
+// XDropTarget
+virtual void SAL_CALL addDropTargetListener(
+const 
css::uno::Reference&) override;
+virtual void SAL_CALL removeDropTargetListener(
+const 
css::uno::Reference&) override;
+virtual sal_Bool SAL_CALL isActive() override;
+virtual void SAL_CALL setActive(sal_Bool active) override;
+virtual sal_Int8 SAL_CALL getDefaultActions() override;
+virtual void SAL_CALL setDefaultActions(sal_Int8 actions) override;
+
+OUString SAL_CALL getImplementationName() override;
+
+sal_Bool SAL_CALL supportsService(OUString const& ServiceName) override;
+
+css::uno::Sequence SAL_CALL getSupportedServiceNames() override;
+
+void fire_drop(const css::datatransfer::dnd::DropTargetDropEvent& dtde);
+
+void fire_dragEnter(const 
css::datatransfer::dnd::DropTargetDragEnterEvent& dtde);
+};
+
 class JSInstanceBuilder : public SalInstanceBuilder
 {
 sal_uInt64 m_nWindowId;
@@ -128,6 +167,9 @@ private:
 template 
 class JSWidget : public BaseInstanceClass, public JSDialogSender
 {
+protected:
+rtl::Reference m_xDropTarget;
+
 public:
 JSWidget(VclPtr aNotifierWindow, VclPtr 
aContentWindow,
  VclClass* pObject, SalInstanceBuilder* pBuilder, bool 
bTakeOwnership,
@@ -155,6 +197,14 @@ public:
 BaseInstanceClass::set_sensitive(sensitive);
 notifyDialogState();
 }
+
+virtual css::uno::Reference 
get_drop_target() override
+{
+if (!m_xDropTarget)
+m_xDropTarget.set(new JSDropTarget);
+
+return m_xDropTarget.get();
+}
 };
 
 class JSDialog : public JSWidget
@@ -309,6 +359,11 @@ public:
 using SalInstanceTreeView::select;
 /// pos is used differently here, it defines how many steps of iterator we 
need to perform to take entry
 virtual void select(int pos) override;
+
+virtual weld::TreeView* get_drag_source() const override;
+
+void drag_start();
+void drag_end();
 };
 
 class JSExpander : public JSWidget
diff --git a/vcl/jsdialog/executor.cxx b/vcl/jsdialog/executor.cxx
index 24e3eb1f14d3..13ffc1a33e06 100644
--- a/vcl/jsdialog/executor.cxx
+++ b/vcl/jsdialog/executor.cxx
@@ -206,6 +206,27 @@ bool ExecuteAction(sal_uInt64 nWindowId, const OString& 
rWidget, StringMap& rDat
 
 pTreeView->select(nRow);
 LOKTrigger::trigger_row_activated(*pTreeView);
+return true;
+}
+else if (sAction == "dragstart")
+{
+OString nRowString
+= OUStringToOString(rData["data"], 
RTL_TEXTENCODING_ASCII_US);
+int nRow = std::atoi(nRowString.getStr());
+
+pTreeView->select(nRow);
+
+JSTreeView* pJSTreeView = 
dynamic_cast(pTreeView);
+if (pJSTreeView)
+pJSTreeView->drag_start();
+return true;
+}
+else if (sAction == "dragend")
+{
+JSTreeView* pJSTreeView = 
dynamic_cast(pTreeView);
+if (pJSTreeView)
+pJSTreeView->drag_end();
+return true;
 }
 }
 }
diff --git a/vcl/jsdialog/jsdialogbuilder.cxx b/vcl/jsdialog/jsdialogbuilder.cxx
index 4fb351b5439f..58c8c64d1108 100644
--- a/vcl/jsdialog/jsdialogbuilder.cxx
+++ b/vcl/jsdialog/jsdialogbuilder.cxx
@@ -24,6 +24,7 @@
 #include 
 #include 
 #include 
+#include 
 
 JSDialogNotifyIdle::JSDialogNotifyIdle(VclPtr aNotifierWindow,
 

[Libreoffice-commits] core.git: Branch 'libreoffice-7-1' - solenv/flatpak-manifest.in

2020-11-27 Thread Stephan Bergmann (via logerrit)
 solenv/flatpak-manifest.in |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit bbb5b5ea98a8f77c3dc501d4f7e497bd3a9fbd3c
Author: Stephan Bergmann 
AuthorDate: Fri Nov 27 08:22:33 2020 +0100
Commit: Stephan Bergmann 
CommitDate: Fri Nov 27 11:18:12 2020 +0100

Adapt solenv/flatpak-manifest.in to312a33b7636334f6ce3b6d1702bc5d3e45215601

..."Update language-subtag-registry to 2020-09-29"

Change-Id: Idea4df8f6a920cbceb37d5156417f6979d7a4fff
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106718
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann 
(cherry picked from commit 25bb4c31a68acd4526176580be72a9d4d626d013)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106730

diff --git a/solenv/flatpak-manifest.in b/solenv/flatpak-manifest.in
index d1529d4768f1..b4e31e620a03 100644
--- a/solenv/flatpak-manifest.in
+++ b/solenv/flatpak-manifest.in
@@ -255,10 +255,10 @@
 "dest-filename": 
"external/tarballs/libfreehand-0.1.2.tar.xz"
 },
 {
-"url": 
"https://dev-www.libreoffice.org/src/language-subtag-registry-2020-04-01.tar.bz2";,
-"sha256": 
"fb1ee0dabfd956a445fbe9f351e86a52767808558f20f4256e67fbbb3768e9da",
+"url": 
"https://dev-www.libreoffice.org/src/language-subtag-registry-2020-09-29.tar.bz2";,
+"sha256": 
"cbe9fca811a37056560aab73e9fc9d3522b46b6785cb02db165f521bf42c230f",
 "type": "file",
-"dest-filename": 
"external/tarballs/language-subtag-registry-2020-04-01.tar.bz2"
+"dest-filename": 
"external/tarballs/language-subtag-registry-2020-09-29.tar.bz2"
 },
 {
 "url": 
"https://dev-www.libreoffice.org/src/liblangtag-0.6.2.tar.bz2";,
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Gabor Kelemen (via logerrit)
 sc/IwyuFilter_sc.yaml|  207 ++-
 sc/inc/NumberFormatControl.hxx   |1 
 sc/inc/dpdimsave.hxx |2 
 sc/inc/dpfilteredcache.hxx   |1 
 sc/inc/dpoutput.hxx  |2 
 sc/inc/dpresfilter.hxx   |1 
 sc/inc/formularesult.hxx |1 
 sc/inc/global.hxx|2 
 sc/inc/inputopt.hxx  |1 
 sc/inc/rechead.hxx   |1 
 sc/inc/sc.hrc|1 
 sc/inc/scmod.hxx |1 
 sc/inc/sheetlimits.hxx   |1 
 sc/inc/userdat.hxx   |1 
 sc/qa/extras/anchor.cxx  |1 
 sc/qa/unit/copy_paste_test.cxx   |1 
 sc/qa/unit/filters-test.cxx  |1 
 sc/qa/unit/helper/debughelper.hxx|2 
 sc/qa/unit/jumbosheets-test.cxx  |2 
 sc/qa/unit/scshapetest.cxx   |3 
 sc/qa/unit/subsequent_export-test.cxx|3 
 sc/qa/unit/subsequent_filters-test.cxx   |2 
 sc/qa/unit/tiledrendering/tiledrendering.cxx |4 
 sc/qa/unit/ucalc_sort.cxx|1 
 sc/qa/unit/uicalc/uicalc.cxx |7 
 sc/source/core/data/column2.cxx  |1 
 sc/source/core/data/docpool.cxx  |1 
 sc/source/core/data/documen2.cxx |1 
 sc/source/core/data/documen3.cxx |1 
 sc/source/core/data/documen7.cxx |1 
 sc/source/core/data/documen8.cxx |2 
 sc/source/core/data/document.cxx |6 
 sc/source/core/data/document10.cxx   |1 
 sc/source/core/data/dptabsrc.cxx |2 
 sc/source/core/data/drwlayer.cxx |2 
 sc/source/core/data/formulacell.cxx  |1 
 sc/source/core/data/markarr.cxx  |3 
 sc/source/core/data/markmulti.cxx|2 
 sc/source/core/data/poolhelp.cxx |1 
 sc/source/core/data/segmenttree.cxx  |1 
 sc/source/core/data/table4.cxx   |1 
 sc/source/core/data/validat.cxx  |2 
 sc/source/core/inc/arraysumfunctor.hxx   |4 
 sc/source/core/inc/grouptokenconverter.hxx   |1 
 sc/source/core/inc/interpre.hxx  |1 
 sc/source/core/opencl/opinlinefun_math.hxx   |2 
 sc/source/core/tool/addincol.cxx |4 
 sc/source/core/tool/addinhelpid.cxx  |1 
 sc/source/core/tool/addinlis.cxx |1 
 sc/source/core/tool/adiasync.cxx |2 
 sc/source/core/tool/appoptio.cxx |4 
 sc/source/core/tool/autoform.cxx |9 
 sc/source/core/tool/calcconfig.cxx   |5 
 sc/source/core/tool/callform.cxx |3 
 sc/source/core/tool/cellform.cxx |2 
 sc/source/core/tool/cellkeytranslator.cxx|1 
 sc/source/core/tool/chartarr.cxx |4 
 sc/source/core/tool/chartlock.cxx|1 
 sc/source/core/tool/chartpos.cxx |1 
 sc/source/core/tool/chgtrack.cxx |8 
 sc/source/core/tool/chgviset.cxx |1 
 sc/source/core/tool/compiler.cxx |4 
 sc/source/core/tool/consoli.cxx  |1 
 sc/source/core/tool/dbdata.cxx   |1 
 sc/source/core/tool/defaultsoptions.cxx  |4 
 sc/source/core/tool/detdata.cxx  |1 
 sc/source/core/tool/detfunc.cxx  |7 
 sc/source/core/tool/docoptio.cxx |3 
 sc/source/core/tool/doubleref.cxx|1 
 sc/source/core/tool/editutil.cxx |4 
 sc/source/core/tool/formulagroup.cxx |8 
 sc/source/core/tool/formulaopt.cxx   |1 
 sc/source/core/tool/inputopt.cxx |2 
 sc/source/core/tool/interpr1.cxx |5 
 sc/source/core/tool/interpr2.cxx |   10 
 sc/source/core/tool/interpr3.cxx 

[Libreoffice-commits] core.git: sal/rtl

2020-11-27 Thread Eike Rathke (via logerrit)
 sal/rtl/math.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit b8404ae521a9c2d183d4e076a7884627ba353e4b
Author: Eike Rathke 
AuthorDate: Fri Nov 27 10:50:49 2020 +0100
Commit: Eike Rathke 
CommitDate: Fri Nov 27 10:51:29 2020 +0100

Fix comment

Change-Id: I2ae6e3cadc0f182c4798e5d33b0c7f07fbcbbff6

diff --git a/sal/rtl/math.cxx b/sal/rtl/math.cxx
index 7d46c754b433..81fa0cf925b7 100644
--- a/sal/rtl/math.cxx
+++ b/sal/rtl/math.cxx
@@ -338,7 +338,7 @@ void doubleToString(typename T::String ** pResult,
 {
 const sal_Int32 nDec = nDecPlaces - nDig2;
 nDecPlaces -= nDec;
-// nDec-1 is also offset into slot, rounded(-1=0) or not(-2=1)
+// nDec-1 is also offset into slot, rounded(1-1=0) or 
not(2-1=1)
 const size_t nSlot = ((fValue < fB3) ? 4 : ((fValue < fB2) ? 3
 : ((fValue < fB1) ? 2 : ((fValue < DBL_MAX) ? 1 : 
0;
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Heiko Tietze (via logerrit)
 sc/source/ui/inc/inscodlg.hxx|   15 
 sc/source/ui/miscdlgs/inscodlg.cxx   |  136 +++---
 sc/uiconfig/scalc/ui/pastespecial.ui |  723 +++
 3 files changed, 484 insertions(+), 390 deletions(-)

New commits:
commit 4c0c77f10c3967c720cb0511dc26986a5b003f33
Author: Heiko Tietze 
AuthorDate: Tue Nov 24 17:14:08 2020 +0100
Commit: Heiko Tietze 
CommitDate: Fri Nov 27 10:39:55 2020 +0100

Resolves tdf#134802 - Improved "Paste Special" dialog box

* New design
* Paste Format Only added
* Code clean-up

Change-Id: Id414b5f0d4cc27ffd861c2272d755f7ccbd06d64
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106540
Tested-by: Jenkins
Reviewed-by: Heiko Tietze 

diff --git a/sc/source/ui/inc/inscodlg.hxx b/sc/source/ui/inc/inscodlg.hxx
index 9f23194c66ae..7cb2ce637f34 100644
--- a/sc/source/ui/inc/inscodlg.hxx
+++ b/sc/source/ui/inc/inscodlg.hxx
@@ -34,6 +34,7 @@ public:
 
 InsertDeleteFlags GetInsContentsCmdBits() const;
 ScPasteFunc   GetFormulaCmdBits() const;
+
 boolIsSkipEmptyCells() const;
 boolIsTranspose() const;
 boolIsLink() const;
@@ -50,11 +51,10 @@ private:
 bool  bChangeTrack;
 bool  bMoveDownDisabled;
 bool  bMoveRightDisabled;
-bool  bUsedShortCut;
-
-InsertDeleteFlags nShortCutInsContentsCmdBits;
-bool  bShortCutTranspose;
-
+void SetInsContentsCmdBits(const InsertDeleteFlags eFlags);
+void SetFormulaCmdBits(const ScPasteFunc eFlags);
+void SetCellCmdFlags(const InsCellCmd eFlags);
+void SetContentsFlags(const InsertContentsFlags eFlags);
 
 std::unique_ptrmxBtnInsAll;
 std::unique_ptrmxBtnInsStrings;
@@ -82,12 +82,15 @@ private:
 std::unique_ptr  mxBtnShortCutPasteValuesOnly;
 std::unique_ptr  mxBtnShortCutPasteValuesFormats;
 std::unique_ptr  mxBtnShortCutPasteTranspose;
+std::unique_ptr  mxBtnShortCutPasteFormats;
+
+std::unique_ptr mxImmediately;
 
 static bool bPreviousAllCheck;
 static InsertDeleteFlags nPreviousChecks;
 static InsertContentsFlags nPreviousChecks2;
 static ScPasteFunc  nPreviousFormulaChecks;
-static sal_uInt16   nPreviousMoveMode;  // enum InsCellCmd
+static InsCellCmd   nPreviousMoveMode;
 
 void DisableChecks( bool bInsAllChecked );
 void TestModes();
diff --git a/sc/source/ui/miscdlgs/inscodlg.cxx 
b/sc/source/ui/miscdlgs/inscodlg.cxx
index d3edb5855c34..772f207a4afe 100644
--- a/sc/source/ui/miscdlgs/inscodlg.cxx
+++ b/sc/source/ui/miscdlgs/inscodlg.cxx
@@ -21,11 +21,10 @@
 
 #include 
 
-bool   ScInsertContentsDlg::bPreviousAllCheck = false;
 InsertDeleteFlags ScInsertContentsDlg::nPreviousChecks   = 
InsertDeleteFlags::VALUE | InsertDeleteFlags::DATETIME | 
InsertDeleteFlags::STRING;
 ScPasteFunc  ScInsertContentsDlg::nPreviousFormulaChecks = ScPasteFunc::NONE;
 InsertContentsFlags ScInsertContentsDlg::nPreviousChecks2 = 
InsertContentsFlags::NONE;
-sal_uInt16 ScInsertContentsDlg::nPreviousMoveMode = INS_NONE;   // enum 
InsCellCmd
+InsCellCmd ScInsertContentsDlg::nPreviousMoveMode = InsCellCmd::INS_NONE;
 
 ScInsertContentsDlg::ScInsertContentsDlg(weld::Window* pParent,
  const OUString* pStrTitle )
@@ -35,9 +34,6 @@ ScInsertContentsDlg::ScInsertContentsDlg(weld::Window* 
pParent,
 , bChangeTrack(false)
 , bMoveDownDisabled(false)
 , bMoveRightDisabled(false)
-, bUsedShortCut(false)
-, nShortCutInsContentsCmdBits(InsertDeleteFlags::NONE )
-, bShortCutTranspose(false)
 , mxBtnInsAll(m_xBuilder->weld_check_button("paste_all"))
 , mxBtnInsStrings(m_xBuilder->weld_check_button("text"))
 , mxBtnInsNumbers(m_xBuilder->weld_check_button("numbers"))
@@ -60,47 +56,25 @@ ScInsertContentsDlg::ScInsertContentsDlg(weld::Window* 
pParent,
 , 
mxBtnShortCutPasteValuesOnly(m_xBuilder->weld_button("paste_values_only"))
 , 
mxBtnShortCutPasteValuesFormats(m_xBuilder->weld_button("paste_values_formats"))
 , mxBtnShortCutPasteTranspose(m_xBuilder->weld_button("paste_transpose"))
+, mxBtnShortCutPasteFormats(m_xBuilder->weld_button("paste_formats"))
+, mxImmediately(m_xBuilder->weld_check_button("cbImmediately"))
 {
 if (pStrTitle)
 m_xDialog->set_title(*pStrTitle);
 
-mxBtnInsAll->set_active( ScInsertContentsDlg::bPreviousAllCheck );
-mxBtnInsStrings->set_active( bool(InsertDeleteFlags::STRING & 
ScInsertContentsDlg::nPreviousChecks) );
-mxBtnInsNumbers->set_active( bool(InsertDeleteFlags::VALUE & 
ScInsertContentsDlg::nPreviousChecks) );
-mxBtnInsDateTime->set_active( bool(InsertDeleteFlags::DATETIME & 
ScInsertContentsDlg::nPreviousChecks) );
-mxBtnInsFormulas->set_active( bool(InsertDeleteFlags::FORMULA & 
ScInsertContentsDlg::nPreviousChecks) );
-mxBtnInsNotes->set_active( bool(InsertDeleteFlags::N

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

2020-11-27 Thread andreas kainz (via logerrit)
 sw/uiconfig/swriter/ui/notebookbar.ui |   51 +++---
 1 file changed, 41 insertions(+), 10 deletions(-)

New commits:
commit c8ca030367180330359ad3b0c6e0fcd8759741e8
Author: andreas kainz 
AuthorDate: Fri Nov 27 09:39:52 2020 +0100
Commit: Andreas Kainz 
CommitDate: Fri Nov 27 10:36:15 2020 +0100

writer notebookbar update for 7.2

Change-Id: Id754c59acd5a548fb96b085e63d1e8f39805ca46
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106721
Tested-by: Jenkins
Reviewed-by: Andreas Kainz 

diff --git a/sw/uiconfig/swriter/ui/notebookbar.ui 
b/sw/uiconfig/swriter/ui/notebookbar.ui
index b2db8be91554..3a5e5b195fba 100644
--- a/sw/uiconfig/swriter/ui/notebookbar.ui
+++ b/sw/uiconfig/swriter/ui/notebookbar.ui
@@ -3365,6 +3365,8 @@
   
 True
 True
+center
+True
 both-horiz
 False
 
@@ -3389,6 +3391,8 @@
   
 True
 True
+center
+True
 both-horiz
 False
 
@@ -3442,10 +3446,13 @@
   
 True
 False
+center
+True
 
   
 True
 True
+center
 True
 
   
@@ -3470,6 +3477,7 @@
   
 True
 True
+center
 5
 
   
@@ -3493,6 +3501,7 @@
   
 True
 True
+center
 icons
 False
 
@@ -3536,10 +3545,12 @@
 True
 False
 center
+True
 
   
 True
 True
+center
 icons
 False
 
@@ -3608,6 +3619,7 @@
 True
 False
 end
+center
 5
 5
 True
@@ -3624,6 +3636,7 @@
   
 True
 True
+center
 icons
 False
 
@@ -3659,6 +3672,7 @@
   
 True
 False
+center
 5
 5
 vertical
@@ -3674,6 +3688,7 @@
   
 True
 True
+center
 icons
 False
 
@@ -3741,8 +3756,6 @@
   
 True
 False
-center
-True
 
   
 True
@@ -3769,10 +3782,12 @@
 True
 False
 center
+True
 
   
 True
 True
+center
 icons
   

Re: FastSaxParser ans XFastParser

2020-11-27 Thread Michael Stahl

On 27.11.20 01:20, Regina Henschel wrote:
The API has an interface XDTDHandler in com::sun::star::xml::sax with 
the methods "notationDecl" and "unparsedEntityDecl". An OpenGrok search 
leads to /core/sax/source/expatwrap/sax_expat.cxx. So it seems you need 
to implement a class which supports this interface. But I'm still a 
novice in many aspects and it would be better a professional developer 
gives you ideas for further steps.


true, but
* FastParser doesn't use XDTDHandler, it's only used with the "old" 
expatwrap parser that isn't "fast".
* XDTDHandler is only for unparsed entities, whereas here internal 
entities would be required and those are always parsed entities


there isn't any UNO API for internal entities.

the internal entities should be handled by libxml2 itself, the problem 
is that the DTD isn't read (it's not bundled?) so they aren't declared.


so i guess you could either figure out how to get this DTD bundled (note 
that there are different versions of MathML, and that you should ask 
your local Debian Developer what they think about its license before 
bundling it), or to provide these entities as built-in via some hook as 
Noel proposed in https://gerrit.libreoffice.org/c/core/+/106630.

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


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

2020-11-27 Thread Anshu (via logerrit)
 basic/qa/vba_tests/cint.vb  |4 
 basic/qa/vba_tests/isnumeric.vb |4 
 basic/source/sbx/sbxscan.cxx|4 +++-
 3 files changed, 11 insertions(+), 1 deletion(-)

New commits:
commit ceb69f85c2c77827642be47aa14cb15ca87f683a
Author: Anshu 
AuthorDate: Wed Nov 25 09:19:31 2020 +0530
Commit: Mike Kaganski 
CommitDate: Fri Nov 27 10:14:56 2020 +0100

tdf#136801 : IsNumeric("+2") and CInt("+2") return correct values

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

diff --git a/basic/qa/vba_tests/cint.vb b/basic/qa/vba_tests/cint.vb
index 6c1d53c93c61..a178a0f97d62 100644
--- a/basic/qa/vba_tests/cint.vb
+++ b/basic/qa/vba_tests/cint.vb
@@ -73,6 +73,10 @@ REMTestLog_ASSERT nr1 = nr2, "the return CInt is: " & nr1
 nr1 = CInt("&H754")
 TestLog_ASSERT nr1 = nr2, "the return CInt is: " & nr1
 
+nr2 = 21
+nr1 = CInt("+21")
+TestLog_ASSERT nr1 = nr2, "the return CInt is:" & nr1
+
 result = result & Chr$(10) & "Tests passed: " & passCount & Chr$(10) & 
"Tests failed: " & failCount & Chr$(10)
 verify_testCInt = result
 
diff --git a/basic/qa/vba_tests/isnumeric.vb b/basic/qa/vba_tests/isnumeric.vb
index 2383f4bfe299..511a51bd9106 100644
--- a/basic/qa/vba_tests/isnumeric.vb
+++ b/basic/qa/vba_tests/isnumeric.vb
@@ -51,6 +51,10 @@ remTestLog_ASSERT date1 = date2, "the return IsNumeric 
is: " & date1
 date1 = IsNumeric("123")
 TestLog_ASSERT date1 = date2, "the return IsNumeric is: " & date1
 
+date2 = True
+date1 = IsNumeric("+123")
+TestLog_ASSERT date1 = date2, "the return IsNumeric is: " & date1
+
 result = result & Chr$(10) & "Tests passed: " & passCount & Chr$(10) & 
"Tests failed: " & failCount & Chr$(10)
 verify_testIsNumeric = result
 
diff --git a/basic/source/sbx/sbxscan.cxx b/basic/source/sbx/sbxscan.cxx
index d2f5c91e3483..de3d50dbed38 100644
--- a/basic/source/sbx/sbxscan.cxx
+++ b/basic/source/sbx/sbxscan.cxx
@@ -108,7 +108,9 @@ ErrCode ImpScan( const OUString& rWSrc, double& nVal, 
SbxDataType& rType,
 SbxDataType eScanType = SbxSINGLE;
 while( *p == ' ' || *p == '\t' )
 p++;
-if( *p == '-' )
+if (*p == '+')
+p++;
+else if( *p == '-' )
 {
 p++;
 bMinus = true;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: solenv/flatpak-manifest.in

2020-11-27 Thread Stephan Bergmann (via logerrit)
 solenv/flatpak-manifest.in |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 25bb4c31a68acd4526176580be72a9d4d626d013
Author: Stephan Bergmann 
AuthorDate: Fri Nov 27 08:22:33 2020 +0100
Commit: Stephan Bergmann 
CommitDate: Fri Nov 27 09:48:53 2020 +0100

Adapt solenv/flatpak-manifest.in to312a33b7636334f6ce3b6d1702bc5d3e45215601

..."Update language-subtag-registry to 2020-09-29"

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

diff --git a/solenv/flatpak-manifest.in b/solenv/flatpak-manifest.in
index 66ced179cb17..fd8d63d1797e 100644
--- a/solenv/flatpak-manifest.in
+++ b/solenv/flatpak-manifest.in
@@ -255,10 +255,10 @@
 "dest-filename": 
"external/tarballs/libfreehand-0.1.2.tar.xz"
 },
 {
-"url": 
"https://dev-www.libreoffice.org/src/language-subtag-registry-2020-04-01.tar.bz2";,
-"sha256": 
"fb1ee0dabfd956a445fbe9f351e86a52767808558f20f4256e67fbbb3768e9da",
+"url": 
"https://dev-www.libreoffice.org/src/language-subtag-registry-2020-09-29.tar.bz2";,
+"sha256": 
"cbe9fca811a37056560aab73e9fc9d3522b46b6785cb02db165f521bf42c230f",
 "type": "file",
-"dest-filename": 
"external/tarballs/language-subtag-registry-2020-04-01.tar.bz2"
+"dest-filename": 
"external/tarballs/language-subtag-registry-2020-09-29.tar.bz2"
 },
 {
 "url": 
"https://dev-www.libreoffice.org/src/liblangtag-0.6.2.tar.bz2";,
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Michael Stahl (via logerrit)
 sw/source/core/doc/docbm.cxx |7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)

New commits:
commit 6a11f94ab366e1d736e97e31e908e86aa5d8b533
Author: Michael Stahl 
AuthorDate: Thu Nov 26 19:55:27 2020 +0100
Commit: Michael Stahl 
CommitDate: Fri Nov 27 09:46:29 2020 +0100

sw_fieldmarkhide: fix getFieldMarkFor() to return innermost field

... like it's supposed to; if it finds the outer field and then the next
field is inside but ends *before* the rPos then the loop is erroneously
terminated.

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

diff --git a/sw/source/core/doc/docbm.cxx b/sw/source/core/doc/docbm.cxx
index 406685c58691..c71178e1f005 100644
--- a/sw/source/core/doc/docbm.cxx
+++ b/sw/source/core/doc/docbm.cxx
@@ -1405,10 +1405,11 @@ namespace sw::mark
 return nullptr;
 auto pFieldmark(*itFieldmark);
 for ( ; itFieldmark != m_vFieldmarks.end()
-&& (**itFieldmark).IsCoveringPosition(rPos); ++itFieldmark)
+&& (**itFieldmark).GetMarkStart() <= rPos; ++itFieldmark)
 {   // find the innermost fieldmark
-if (pFieldmark->GetMarkStart() < (**itFieldmark).GetMarkStart()
-|| (**itFieldmark).GetMarkEnd() < pFieldmark->GetMarkEnd())
+if (rPos < (**itFieldmark).GetMarkEnd()
+&& (pFieldmark->GetMarkStart() < (**itFieldmark).GetMarkStart()
+|| (**itFieldmark).GetMarkEnd() < 
pFieldmark->GetMarkEnd()))
 {
 pFieldmark = *itFieldmark;
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


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

2020-11-27 Thread Caolán McNamara (via logerrit)
 vcl/unx/gtk3/gtk3gtkframe.cxx |4 
 1 file changed, 4 insertions(+)

New commits:
commit 283124fcf07fe8e38417579ce6d94ecf56715ac1
Author: Caolán McNamara 
AuthorDate: Thu Nov 26 17:13:28 2020 +
Commit: Caolán McNamara 
CommitDate: Fri Nov 27 09:45:53 2020 +0100

silence warnings about negative widths under RTL

I don't see any visual problems, but there are these
annoying warnings

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

diff --git a/vcl/unx/gtk3/gtk3gtkframe.cxx b/vcl/unx/gtk3/gtk3gtkframe.cxx
index 4680acf77a67..bb636e7874f8 100644
--- a/vcl/unx/gtk3/gtk3gtkframe.cxx
+++ b/vcl/unx/gtk3/gtk3gtkframe.cxx
@@ -2953,6 +2953,10 @@ void GtkSalFrame::damaged(sal_Int32 nExtentsX, sal_Int32 
nExtentsY,
 }
 #endif
 
+// quite a bit of noise in RTL mode with negative widths
+if (nExtentsWidth <= 0 || nExtentsHeight <= 0)
+return;
+
 gtk_widget_queue_draw_area(GTK_WIDGET(m_pFixedContainer),
nExtentsX, nExtentsY,
nExtentsWidth, nExtentsHeight);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: cui/inc extras/source vcl/Package_tipoftheday.mk

2020-11-27 Thread andreas kainz (via logerrit)
 cui/inc/tipoftheday.hrc  |8 
 extras/source/tipoftheday/formdocuments.png  |binary
 extras/source/tipoftheday/masterdocument.png |binary
 extras/source/tipoftheday/printnote.png  |binary
 extras/source/tipoftheday/statusbar.png  |binary
 vcl/Package_tipoftheday.mk   |4 
 6 files changed, 8 insertions(+), 4 deletions(-)

New commits:
commit c12bc5757c021f8ba3f802b26805964a11aaa71d
Author: andreas kainz 
AuthorDate: Fri Nov 27 08:54:43 2020 +0100
Commit: Andreas Kainz 
CommitDate: Fri Nov 27 09:35:38 2020 +0100

tdf#127294 add ToD Previews

Change-Id: Icd049b0ae313e4c3095975a98f8765372fa22abf
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106719
Tested-by: Jenkins
Reviewed-by: Andreas Kainz 

diff --git a/cui/inc/tipoftheday.hrc b/cui/inc/tipoftheday.hrc
index f5d06db9657e..95806eb9bff1 100644
--- a/cui/inc/tipoftheday.hrc
+++ b/cui/inc/tipoftheday.hrc
@@ -47,13 +47,13 @@ const std::tuple 
TIPOFTHEDAY_STRINGARRAY[] =
 {
  { NC_("RID_CUI_TIPOFTHEDAY", "%PRODUCTNAME offers a variety of user 
interface options to make you feel at home"), ".uno:ToolbarModeUI", 
"toolbarmode.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "Need to allow changes to parts of a 
read-only document in Writer? Insert frames or sections that can authorize 
changes."), 
"https://help.libreoffice.org/%PRODUCTVERSION/%LANGUAGENAME/text/swriter/guide/section_edit.html";,
 "tipoftheday_w.png"}, //local help missing
- { NC_("RID_CUI_TIPOFTHEDAY", "To print the notes of your slides go to 
File ▸ Print ▸ Impress tab and select Notes under Document ▸ Type."), "", 
"tipoftheday_i.png"},
+ { NC_("RID_CUI_TIPOFTHEDAY", "To print the notes of your slides go to 
File ▸ Print ▸ Impress tab and select Notes under Document ▸ Type."), "", 
"printnote.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "To start temporarily with a fresh user 
profile, or to restore a non-working %PRODUCTNAME, use Help ▸ Restart in Safe 
Mode."), "svx/ui/safemodedialog/SafeModeDialog", ""}, 
//https://help.libreoffice.org/%PRODUCTVERSION/%LANGUAGENAME/text/shared/01/profile_safe_mode.html
  { NC_("RID_CUI_TIPOFTHEDAY", "Writing a book? %PRODUCTNAME master 
document lets you manage large documents as a container for individual 
%PRODUCTNAME Writer files."), 
"https://help.libreoffice.org/%PRODUCTVERSION/%LANGUAGENAME/text/swriter/guide/globaldoc.html";,
 "tipoftheday_w.png"}, //local help missing
  { NC_("RID_CUI_TIPOFTHEDAY", "Create editable Hybrid PDFs with 
%PRODUCTNAME."), 
"https://wiki.documentfoundation.org/Documentation/HowTo/CreateAHybridPDF";, ""},
  { NC_("RID_CUI_TIPOFTHEDAY", "Explore the ten different functions in the 
status bar (at the bottom of the document window). Place the cursor over each 
field for an explanation. If not visible, use View ▸ Status Bar."), "", 
"tipoftheday_w.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "Want to sum a cell through several sheets? 
Refer to the range of sheets e.g. =SUM(Sheet1.A1:Sheet3.A1)."), "", 
"tipoftheday_c.png"},
- { NC_("RID_CUI_TIPOFTHEDAY", "Create fillable form documents (even PDFs) 
with %PRODUCTNAME."), 
"https://www.techrepublic.com/article/how-to-create-interactive-pdfs-with-libreoffice";,
 ""},
+ { NC_("RID_CUI_TIPOFTHEDAY", "Create fillable form documents (even PDFs) 
with %PRODUCTNAME."), 
"https://www.techrepublic.com/article/how-to-create-interactive-pdfs-with-libreoffice";,
 "formdocuments.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "Cannot see all the text in a cell? Expand 
the input line in the formula bar and you can scroll."), "", 
"tipoftheday_c.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "Optimize your table layout with Table ▸ 
Size ▸ Distribute Rows / Columns Evenly."), "", "tipoftheday_w.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "Find all expressions in brackets per Edit ▸ 
Find and Replace ▸ Find ▸ \\([^)]+\\) (check “Regular expressions”)"), "", 
"tipoftheday_w.png"},
@@ -66,7 +66,7 @@ const std::tuple 
TIPOFTHEDAY_STRINGARRAY[] =
  { NC_("RID_CUI_TIPOFTHEDAY", "You can restart the slide show after a 
pause specified at Slide Show ▸ Slide Show Settings ▸ Loop and repeat."), "", 
"tipoftheday_i.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "To distribute some text in multi-columns 
select the text and apply Format ▸ Columns."), "", "tipoftheday_w.png"},
  { NC_("RID_CUI_TIPOFTHEDAY", "Use View ▸ Value Highlighting to display 
cell contents in colors: Text/black, Formulas/green, Numbers/blue, Protected 
cells/grey background."), 
"https://help.libreoffice.org/%PRODUCTVERSION/%LANGUAGENAME/text/scalc/01/0308.html";,
 "tipoftheday_c.png"}, //local help missing
- { NC_("RID_CUI_TIPOFTHEDAY", "Create different master pages in a 
presentation template: View ▸ Master Slide and Slide ▸ New Master (or per 
toolbar or right click in slide pane)."), "", "tipoftheday_i.png"},
+ { NC_("RID_CUI_TIPOFTHEDAY", "Create different master pages in a 
presentation template: View ▸