writerfilter/source/dmapper/DomainMapper.cxx | 4 - writerfilter/source/dmapper/DomainMapperTableHandler.cxx | 15 ++--- writerfilter/source/dmapper/DomainMapper_Impl.cxx | 27 ++++------ writerfilter/source/dmapper/GraphicImport.cxx | 9 +-- writerfilter/source/dmapper/NumberingManager.cxx | 13 ++-- writerfilter/source/dmapper/OLEHandler.cxx | 9 +-- writerfilter/source/dmapper/PropertyMap.cxx | 4 - writerfilter/source/dmapper/StyleSheetTable.cxx | 4 - writerfilter/source/dmapper/TableManager.cxx | 5 + writerfilter/source/filter/RtfFilter.cxx | 5 + writerfilter/source/ooxml/OOXMLDocumentImpl.cxx | 36 ++++++------- writerfilter/source/rtftok/rtfdocumentimpl.cxx | 6 +- xmlhelp/Library_ucpchelp1.mk | 1 xmlhelp/source/cxxhelp/provider/resultsetforquery.cxx | 5 + xmloff/source/chart/SchXMLChartContext.cxx | 9 +-- xmloff/source/chart/SchXMLExport.cxx | 40 +++++++-------- xmloff/source/chart/SchXMLImport.cxx | 4 - xmloff/source/chart/SchXMLPlotAreaContext.cxx | 18 +++--- xmloff/source/chart/SchXMLSeries2Context.cxx | 16 +++--- xmloff/source/chart/SchXMLSeriesHelper.cxx | 8 +-- xmloff/source/core/DocumentSettingsContext.cxx | 7 +- xmloff/source/draw/animationexport.cxx | 5 + xmloff/source/draw/shapeexport.cxx | 4 - xmloff/source/style/XMLFontAutoStylePool.cxx | 5 + xmloff/source/text/XMLTextFrameContext.cxx | 11 ++-- xmloff/source/text/txtparai.cxx | 5 + xmlsecurity/source/helper/documentsignaturehelper.cxx | 5 + xmlsecurity/source/xmlsec/nss/nssinitializer.cxx | 5 + xmlsecurity/workben/pdfverify.cxx | 13 ++-- 29 files changed, 156 insertions(+), 142 deletions(-)
New commits: commit 2836b78b54650d117282a9a345677d1b4d9fc043 Author: Noel Grandin <noel.gran...@collabora.co.uk> AuthorDate: Tue Jun 18 15:38:17 2019 +0200 Commit: Noel Grandin <noel.gran...@collabora.co.uk> CommitDate: Tue Jun 18 20:03:42 2019 +0200 loplugin:logexceptionnicely in xmlhelp..xmlsecurity Change-Id: Idbf0de3f953b08f98144241346533ad30f49269e Reviewed-on: https://gerrit.libreoffice.org/74285 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk> diff --git a/xmlhelp/Library_ucpchelp1.mk b/xmlhelp/Library_ucpchelp1.mk index e12aabf96326..b76e0eb67c23 100644 --- a/xmlhelp/Library_ucpchelp1.mk +++ b/xmlhelp/Library_ucpchelp1.mk @@ -36,6 +36,7 @@ $(eval $(call gb_Library_use_libraries,ucpchelp1,\ cppuhelper \ helplinker \ sal \ + tl \ ucbhelper \ utl \ i18nlangtag \ diff --git a/xmlhelp/source/cxxhelp/provider/resultsetforquery.cxx b/xmlhelp/source/cxxhelp/provider/resultsetforquery.cxx index ac464c6d02c3..d599a217446d 100644 --- a/xmlhelp/source/cxxhelp/provider/resultsetforquery.cxx +++ b/xmlhelp/source/cxxhelp/provider/resultsetforquery.cxx @@ -26,6 +26,7 @@ #include <com/sun/star/lang/Locale.hpp> #include <helpcompiler/HelpSearch.hxx> +#include <tools/diagnose_ex.h> #if defined(__GNUC__) # pragma GCC visibility push (default) @@ -234,9 +235,9 @@ ResultSetForQuery::ResultSetForQuery( const uno::Reference< uno::XComponentConte aIndexFolderResultVectorVector.push_back( std::move(aIndexFolderResultVector) ); } - catch (const Exception &e) + catch (const Exception &) { - SAL_WARN("xmlhelp", e); + TOOLS_WARN_EXCEPTION("xmlhelp", ""); } if( bTemporary ) diff --git a/xmloff/source/chart/SchXMLChartContext.cxx b/xmloff/source/chart/SchXMLChartContext.cxx index 6cc3a29a7828..df143274ac32 100644 --- a/xmloff/source/chart/SchXMLChartContext.cxx +++ b/xmloff/source/chart/SchXMLChartContext.cxx @@ -27,6 +27,7 @@ #include "SchXMLTools.hxx" #include <osl/diagnose.h> #include <sal/log.hxx> +#include <tools/diagnose_ex.h> #include <unotools/mediadescriptor.hxx> #include <xmloff/xmlnmspe.hxx> #include <xmloff/xmlement.hxx> @@ -165,9 +166,9 @@ void lcl_removeEmptyChartTypeGroups( const uno::Reference< chart2::XChartDocumen } } } - catch(const uno::Exception& ex) + catch(const uno::Exception&) { - SAL_INFO("xmloff.chart", "Exception caught while removing empty chart types: " << ex); + TOOLS_INFO_EXCEPTION("xmloff.chart", "Exception caught while removing empty chart types"); } } @@ -301,9 +302,9 @@ static void lcl_setDataProvider(uno::Reference<chart2::XChartDocument> const & x xChartDoc->createInternalDataProvider(false); } } - catch (const uno::Exception & rEx) + catch (const uno::Exception &) { - SAL_INFO("xmloff.chart", "SchXMLChartContext::StartElement(): Exception caught: " << rEx); + TOOLS_INFO_EXCEPTION("xmloff.chart", "SchXMLChartContext::StartElement()"); } } diff --git a/xmloff/source/chart/SchXMLExport.cxx b/xmloff/source/chart/SchXMLExport.cxx index b1785d5330f4..46214498958b 100644 --- a/xmloff/source/chart/SchXMLExport.cxx +++ b/xmloff/source/chart/SchXMLExport.cxx @@ -853,9 +853,9 @@ lcl_TableData lcl_getDataForLocalTable( rTarget[i] <<= rSource[i]; } } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "something went wrong during table data collection: " << rEx); + TOOLS_INFO_EXCEPTION("xmloff.chart", "something went wrong during table data collection"); } return aResult; @@ -899,9 +899,9 @@ void lcl_exportNumberFormat( const OUString& rPropertyName, const Reference< bea } } } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "chart:exporting error bar ranges: " << rEx ); + TOOLS_INFO_EXCEPTION("xmloff.chart", "chart:exporting error bar ranges" ); } } @@ -1433,9 +1433,9 @@ void SchXMLExportHelper_Impl::parseDocument( Reference< chart::XChartDocument > Any aShapesAny = xDocPropSet->getPropertyValue("AdditionalShapes"); aShapesAny >>= mxAdditionalShapes; } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "AdditionalShapes not found: " << rEx ); + TOOLS_INFO_EXCEPTION("xmloff.chart", "AdditionalShapes not found" ); } if( mxAdditionalShapes.is()) @@ -1882,9 +1882,9 @@ void SchXMLExportHelper_Impl::exportPlotArea( rShapeExport->export3DSceneAttributes( xPropSet ); } } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "chart:exportPlotAreaException caught: " << rEx); + TOOLS_INFO_EXCEPTION("xmloff.chart", "chart:exportPlotAreaException caught"); } } @@ -2571,9 +2571,9 @@ void SchXMLExportHelper_Impl::exportSeries( xPropSet = SchXMLSeriesHelper::createOldAPISeriesPropertySet( aSeriesSeq[nSeriesIdx], mrExport.GetModel() ); } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "Series not found or no XPropertySet: " << rEx ); + TOOLS_INFO_EXCEPTION("xmloff.chart", "Series not found or no XPropertySet" ); continue; } if( xPropSet.is()) @@ -2587,9 +2587,9 @@ void SchXMLExportHelper_Impl::exportSeries( aAny = xPropSet->getPropertyValue( "MeanValue" ); aAny >>= bHasMeanValueLine; } - catch( const beans::UnknownPropertyException & rEx ) + catch( const beans::UnknownPropertyException & ) { - SAL_INFO("xmloff.chart", "Required property not found in DataRowProperties: " << rEx ); + TOOLS_INFO_EXCEPTION("xmloff.chart", "Required property not found in DataRowProperties" ); } const SvtSaveOptions::ODFDefaultVersion nCurrentODFVersion( SvtSaveOptions().GetODFDefaultVersion() ); @@ -2756,9 +2756,9 @@ void SchXMLExportHelper_Impl::exportSeries( Any aPropAny( xPropSet->getPropertyValue( "DataMeanValueProperties" )); aPropAny >>= xStatProp; } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "Exception caught during Export of series - optional DataMeanValueProperties not available: " << rEx ); + TOOLS_INFO_EXCEPTION("xmloff.chart", "Exception caught during Export of series - optional DataMeanValueProperties not available" ); } if( xStatProp.is() ) @@ -2994,9 +2994,9 @@ void SchXMLExportHelper_Impl::exportErrorBar( const Reference<beans::XPropertySe aAny >>= nErrorBarStyle; } } - catch( const beans::UnknownPropertyException & rEx ) + catch( const beans::UnknownPropertyException & ) { - SAL_INFO("xmloff.chart", "Required property not found in DataRowProperties: " << rEx ); + TOOLS_INFO_EXCEPTION("xmloff.chart", "Required property not found in DataRowProperties" ); } if( nErrorBarStyle != chart::ErrorBarStyle::NONE && (bNegative || bPositive)) @@ -3213,9 +3213,9 @@ void SchXMLExportHelper_Impl::exportDataPoints( xSeries, nElement, mrExport.GetModel() ); bExportNumFmt = true; } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "Exception caught during Export of data point: " << rEx ); + TOOLS_INFO_EXCEPTION("xmloff.chart", "Exception caught during Export of data point" ); } } else @@ -3281,9 +3281,9 @@ void SchXMLExportHelper_Impl::exportDataPoints( xPropSet = SchXMLSeriesHelper::createOldAPIDataPointPropertySet( xSeries, nCurrIndex, mrExport.GetModel() ); } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "Exception caught during Export of data point: " << rEx ); + TOOLS_INFO_EXCEPTION("xmloff.chart", "Exception caught during Export of data point" ); } if( xPropSet.is()) { diff --git a/xmloff/source/chart/SchXMLImport.cxx b/xmloff/source/chart/SchXMLImport.cxx index 5d9507cf0a92..a571ecc636a1 100644 --- a/xmloff/source/chart/SchXMLImport.cxx +++ b/xmloff/source/chart/SchXMLImport.cxx @@ -611,9 +611,9 @@ void SAL_CALL SchXMLImport::setTargetDocument(const uno::Reference<lang::XCompon } } } - catch (const uno::Exception & rEx) + catch (const uno::Exception &) { - SAL_INFO("xmloff.chart", "SchXMLChartContext::StartElement(): Exception caught: " << rEx); + TOOLS_INFO_EXCEPTION("xmloff.chart", "SchXMLChartContext::StartElement(): Exception caught"); } } diff --git a/xmloff/source/chart/SchXMLPlotAreaContext.cxx b/xmloff/source/chart/SchXMLPlotAreaContext.cxx index 46fee917baf1..1cf2785030a1 100644 --- a/xmloff/source/chart/SchXMLPlotAreaContext.cxx +++ b/xmloff/source/chart/SchXMLPlotAreaContext.cxx @@ -28,6 +28,7 @@ #include <comphelper/processfactory.hxx> #include <sal/log.hxx> +#include <tools/diagnose_ex.h> #include <xmloff/xmlnmspe.hxx> #include <xmloff/xmlement.hxx> #include <xmloff/nmspmap.hxx> @@ -116,9 +117,10 @@ void SchXML3DSceneAttributesHelper::getCameraDefaultFromDiagram( const uno::Refe maVUP.setZ( aCamGeo.vup.DirectionZ ); } } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "Exception caught for property NumberOfLines: " << rEx); + css::uno::Any ex( cppu::getCaughtException() ); + SAL_INFO("xmloff.chart", "Exception caught for property NumberOfLines: " << exceptionToString(ex)); } } @@ -382,9 +384,9 @@ void SchXMLPlotAreaContext::StartElement( const uno::Reference< xml::sax::XAttri xProp->getPropertyValue("NumberOfLines") >>= mnNumOfLinesProp; xProp->getPropertyValue("Volume") >>= mbStockHasVolume; } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "PlotAreaContext:EndElement(): Exception caught: " << rEx); + TOOLS_INFO_EXCEPTION("xmloff.chart", "PlotAreaContext:EndElement(): Exception caught"); } } // if @@ -552,9 +554,9 @@ void SchXMLPlotAreaContext::EndElement() xDiaProp->setPropertyValue("NumberOfLines", uno::makeAny( mnNumOfLinesProp )); } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "Exception caught for property NumberOfLines: " << rEx); + TOOLS_INFO_EXCEPTION("xmloff.chart", "Exception caught for property NumberOfLines"); } } @@ -567,9 +569,9 @@ void SchXMLPlotAreaContext::EndElement() xDiaProp->setPropertyValue("Volume", uno::makeAny( true )); } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "Exception caught for property Volume: " << rEx); + TOOLS_INFO_EXCEPTION("xmloff.chart", "Exception caught for property Volume"); } } } diff --git a/xmloff/source/chart/SchXMLSeries2Context.cxx b/xmloff/source/chart/SchXMLSeries2Context.cxx index c7a37c78ac00..857e47435c5c 100644 --- a/xmloff/source/chart/SchXMLSeries2Context.cxx +++ b/xmloff/source/chart/SchXMLSeries2Context.cxx @@ -860,9 +860,9 @@ void SchXMLSeries2Context::setStylesToSeries( SeriesDefaultsAndStyles& rSeriesDe } } } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "Exception caught during setting styles to series: " << rEx ); + TOOLS_INFO_EXCEPTION("xmloff.chart", "Exception caught during setting styles to series" ); } } } @@ -926,9 +926,9 @@ void SchXMLSeries2Context::setStylesToRegressionCurves( xRegCurveCont->addRegressionCurve( xRegCurve ); } } - catch( const uno::Exception& rEx ) + catch( const uno::Exception& ) { - SAL_INFO("xmloff.chart", "Exception caught during setting styles to series: " << rEx ); + TOOLS_INFO_EXCEPTION("xmloff.chart", "Exception caught during setting styles to series" ); } } @@ -995,9 +995,9 @@ void SchXMLSeries2Context::setStylesToStatisticsObjects( SeriesDefaultsAndStyles } } } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "Exception caught during setting styles to series: " << rEx ); + TOOLS_INFO_EXCEPTION("xmloff.chart", "Exception caught during setting styles to series" ); } } } @@ -1090,9 +1090,9 @@ void SchXMLSeries2Context::setStylesToDataPoints( SeriesDefaultsAndStyles& rSeri lcl_resetSymbolSizeForPointsIfNecessary( xPointProp, rImport, pPropStyleContext, pStylesCtxt ); } } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "Exception caught during setting styles to data points: " << rEx ); + TOOLS_INFO_EXCEPTION("xmloff.chart", "Exception caught during setting styles to data points" ); } } } // styles iterator diff --git a/xmloff/source/chart/SchXMLSeriesHelper.cxx b/xmloff/source/chart/SchXMLSeriesHelper.cxx index 18d1dc93a65e..b206d10df4d8 100644 --- a/xmloff/source/chart/SchXMLSeriesHelper.cxx +++ b/xmloff/source/chart/SchXMLSeriesHelper.cxx @@ -180,9 +180,9 @@ uno::Reference< beans::XPropertySet > SchXMLSeriesHelper::createOldAPISeriesProp } } } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "Exception caught SchXMLSeriesHelper::createOldAPISeriesPropertySet: " << rEx ); + TOOLS_INFO_EXCEPTION("xmloff.chart", "Exception caught SchXMLSeriesHelper::createOldAPISeriesPropertySet" ); } } @@ -215,9 +215,9 @@ uno::Reference< beans::XPropertySet > SchXMLSeriesHelper::createOldAPIDataPointP } } } - catch( const uno::Exception & rEx ) + catch( const uno::Exception & ) { - SAL_INFO("xmloff.chart", "Exception caught SchXMLSeriesHelper::createOldAPIDataPointPropertySet: " << rEx ); + TOOLS_INFO_EXCEPTION("xmloff.chart", "Exception caught SchXMLSeriesHelper::createOldAPIDataPointPropertySet" ); } } diff --git a/xmloff/source/core/DocumentSettingsContext.cxx b/xmloff/source/core/DocumentSettingsContext.cxx index ec3252251e3d..305d1d94853b 100644 --- a/xmloff/source/core/DocumentSettingsContext.cxx +++ b/xmloff/source/core/DocumentSettingsContext.cxx @@ -46,6 +46,7 @@ #include <rtl/ustrbuf.hxx> #include <sal/log.hxx> #include <osl/diagnose.h> +#include <tools/diagnose_ex.h> #include <unotools/configmgr.hxx> #include "xmlenums.hxx" @@ -746,10 +747,10 @@ void XMLConfigItemMapIndexedContext::EndElement() { xForbChars->setForbiddenCharacters( aLocale, aForbid ); } - catch (uno::Exception const& e) + catch (uno::Exception const&) { - SAL_WARN("xmloff.core", - "Exception while importing forbidden characters: " << e); + TOOLS_WARN_EXCEPTION("xmloff.core", + "Exception while importing forbidden characters"); } } } diff --git a/xmloff/source/draw/animationexport.cxx b/xmloff/source/draw/animationexport.cxx index 8055e01515fe..a55f1e991685 100644 --- a/xmloff/source/draw/animationexport.cxx +++ b/xmloff/source/draw/animationexport.cxx @@ -56,6 +56,7 @@ #include <sax/tools/converter.hxx> #include <sal/log.hxx> #include <osl/diagnose.h> +#include <tools/diagnose_ex.h> #include <xmloff/unointerfacetouniqueidentifiermapper.hxx> #include "sdxmlexp_impl.hxx" @@ -537,9 +538,9 @@ static OUString lcl_StoreMediaAndGetURL(SvXMLExport & rExport, OUString const& r return urlPath; } - catch (uno::Exception const& e) + catch (uno::Exception const&) { - SAL_INFO("xmloff", "exception while storing embedded media: '" << e << "'"); + TOOLS_INFO_EXCEPTION("xmloff", "exception while storing embedded media"); } return OUString(); } diff --git a/xmloff/source/draw/shapeexport.cxx b/xmloff/source/draw/shapeexport.cxx index d798a8a382a7..89a40effd932 100644 --- a/xmloff/source/draw/shapeexport.cxx +++ b/xmloff/source/draw/shapeexport.cxx @@ -3227,9 +3227,9 @@ lcl_StoreMediaAndGetURL(SvXMLExport & rExport, return urlPath; } - catch (uno::Exception const& e) + catch (uno::Exception const&) { - SAL_INFO("xmloff", "exception while storing embedded media: " << e); + TOOLS_INFO_EXCEPTION("xmloff", "exception while storing embedded media"); } return OUString(); } diff --git a/xmloff/source/style/XMLFontAutoStylePool.cxx b/xmloff/source/style/XMLFontAutoStylePool.cxx index 407f1ae8ecc6..d16997977f10 100644 --- a/xmloff/source/style/XMLFontAutoStylePool.cxx +++ b/xmloff/source/style/XMLFontAutoStylePool.cxx @@ -28,6 +28,7 @@ #include <vcl/embeddedfontshelper.hxx> #include <osl/file.hxx> #include <sal/log.hxx> +#include <tools/diagnose_ex.h> #include <com/sun/star/beans/XPropertySet.hpp> #include <com/sun/star/embed/ElementModes.hpp> @@ -649,9 +650,9 @@ OUString XMLFontAutoStylePool::embedFontFile(OUString const & fileUrl, OUString return sInternalName; } } - } catch( const Exception& e ) + } catch( const Exception& ) { - SAL_WARN( "xmloff", "Exception when embedding a font file:" << e ); + TOOLS_WARN_EXCEPTION( "xmloff", "Exception when embedding a font file" ); } return OUString(); } diff --git a/xmloff/source/text/XMLTextFrameContext.cxx b/xmloff/source/text/XMLTextFrameContext.cxx index d317ee2808b5..4a7360abe482 100644 --- a/xmloff/source/text/XMLTextFrameContext.cxx +++ b/xmloff/source/text/XMLTextFrameContext.cxx @@ -19,6 +19,7 @@ #include <osl/diagnose.h> #include <sal/log.hxx> +#include <tools/diagnose_ex.h> #include <comphelper/base64.hxx> #include <com/sun/star/lang/XMultiServiceFactory.hpp> #include <com/sun/star/text/TextContentAnchorType.hpp> @@ -699,9 +700,9 @@ void XMLTextFrameContext_Impl::Create() { xTextImportHelper->InsertTextContent(xTxtCntnt); } - catch (lang::IllegalArgumentException const& e) + catch (lang::IllegalArgumentException const&) { - SAL_WARN("xmloff.text", "Cannot import part of the text - probably an image in the text frame? " << e); + TOOLS_WARN_EXCEPTION("xmloff.text", "Cannot import part of the text - probably an image in the text frame?"); return; } } @@ -1294,10 +1295,10 @@ void XMLTextFrameContext_Impl::SetName() { xNamed->setName(m_sOrigName); } - catch (uno::Exception const& e) + catch (uno::Exception const&) { // fdo#71698 document contains 2 frames with same draw:name - SAL_INFO("xmloff.text", "SetName(): exception setting \"" - << m_sOrigName << "\": " << e); + TOOLS_INFO_EXCEPTION("xmloff.text", "SetName(): exception setting \"" + << m_sOrigName << "\""); } } } diff --git a/xmloff/source/text/txtparai.cxx b/xmloff/source/text/txtparai.cxx index 03ed99fecb5c..8676390dabe6 100644 --- a/xmloff/source/text/txtparai.cxx +++ b/xmloff/source/text/txtparai.cxx @@ -25,6 +25,7 @@ #include <rtl/ustring.hxx> #include <rtl/ustrbuf.hxx> #include <sal/log.hxx> +#include <tools/diagnose_ex.h> #include <com/sun/star/lang/XMultiServiceFactory.hpp> #include <com/sun/star/text/XTextFrame.hpp> @@ -2077,9 +2078,9 @@ void XMLParaContext::EndElement() xTxtImport->GetText()->insertTextContent( xAttrCursor, xContent, true ); } - catch (uno::RuntimeException const& e) + catch (uno::RuntimeException const&) { - SAL_INFO("xmloff.text", "could not insert index mark, presumably in editengine text " << e); + TOOLS_INFO_EXCEPTION("xmloff.text", "could not insert index mark, presumably in editengine text"); } } break; diff --git a/xmlsecurity/source/helper/documentsignaturehelper.cxx b/xmlsecurity/source/helper/documentsignaturehelper.cxx index 66a058c1d3dd..37faa606c32f 100644 --- a/xmlsecurity/source/helper/documentsignaturehelper.cxx +++ b/xmlsecurity/source/helper/documentsignaturehelper.cxx @@ -39,6 +39,7 @@ #include <rtl/uri.hxx> #include <sal/log.hxx> #include <svx/xoutbmp.hxx> +#include <tools/diagnose_ex.h> #include <xmloff/attrlist.hxx> #include <xsecctl.hxx> @@ -379,9 +380,9 @@ SignatureStreamHelper DocumentSignatureHelper::OpenSignatureStream( aHelper.xSignatureStorage = rxStore->openStorageElement("_xmlsignatures", nSubStorageOpenMode); aHelper.nStorageFormat = embed::StorageFormats::OFOPXML; } - catch (const io::IOException& rException) + catch (const io::IOException&) { - SAL_WARN_IF(nOpenMode != css::embed::ElementModes::READ, "xmlsecurity.helper", "DocumentSignatureHelper::OpenSignatureStream: " << rException); + TOOLS_WARN_EXCEPTION_IF(nOpenMode != css::embed::ElementModes::READ, "xmlsecurity.helper", "DocumentSignatureHelper::OpenSignatureStream:"); } } diff --git a/xmlsecurity/source/xmlsec/nss/nssinitializer.cxx b/xmlsecurity/source/xmlsec/nss/nssinitializer.cxx index d77e2d4a4fcb..4813d9cac10c 100644 --- a/xmlsecurity/source/xmlsec/nss/nssinitializer.cxx +++ b/xmlsecurity/source/xmlsec/nss/nssinitializer.cxx @@ -31,6 +31,7 @@ #include <osl/file.hxx> #include <osl/thread.h> #include <sal/log.hxx> +#include <tools/diagnose_ex.h> #include <unotools/tempfile.hxx> #include <salhelper/singletonref.hxx> @@ -260,9 +261,9 @@ OString getMozillaCurrentProfile( const css::uno::Reference< css::uno::XComponen return OUStringToOString(sUserSetCertPath, osl_getThreadTextEncoding()); } } - catch (const uno::Exception &e) + catch (const uno::Exception &) { - SAL_WARN("xmlsecurity.xmlsec", "getMozillaCurrentProfile: caught " << e); + TOOLS_WARN_EXCEPTION("xmlsecurity.xmlsec", "getMozillaCurrentProfile:"); } // third, dig around to see if there's one available diff --git a/xmlsecurity/workben/pdfverify.cxx b/xmlsecurity/workben/pdfverify.cxx index efe0461b9f14..191ca8c087eb 100644 --- a/xmlsecurity/workben/pdfverify.cxx +++ b/xmlsecurity/workben/pdfverify.cxx @@ -18,6 +18,7 @@ #include <osl/file.hxx> #include <sal/log.hxx> #include <sal/main.h> +#include <tools/diagnose_ex.h> #include <vcl/pngwrite.hxx> #include <vcl/svapp.hxx> #include <vcl/graphicfilter.hxx> @@ -67,10 +68,10 @@ int pdfVerify(int nArgc, char** pArgv) { xComponentContext = cppu::defaultBootstrap_InitialComponentContext(); } - catch (const uno::RuntimeException& rException) + catch (const uno::RuntimeException&) { - SAL_WARN("xmlsecurity.pdfio", - "cppu::defaultBootstrap_InitialComponentContext() failed: " << rException); + TOOLS_WARN_EXCEPTION("xmlsecurity.pdfio", + "cppu::defaultBootstrap_InitialComponentContext() failed:"); return 1; } uno::Reference<lang::XMultiComponentFactory> xMultiComponentFactory @@ -92,10 +93,10 @@ int pdfVerify(int nArgc, char** pArgv) { xSEInitializer = xml::crypto::SEInitializer::create(xComponentContext); } - catch (const uno::DeploymentException& rException) + catch (const uno::DeploymentException&) { - SAL_WARN("xmlsecurity.pdfio", - "DeploymentException while creating SEInitializer: " << rException); + TOOLS_WARN_EXCEPTION("xmlsecurity.pdfio", + "DeploymentException while creating SEInitializer:"); return 1; } uno::Reference<xml::crypto::XXMLSecurityContext> xSecurityContext commit 697f8edbb0154ed72a42e4e8d7c4942cfe7dd59d Author: Noel Grandin <noel.gran...@collabora.co.uk> AuthorDate: Tue Jun 18 15:37:40 2019 +0200 Commit: Noel Grandin <noel.gran...@collabora.co.uk> CommitDate: Tue Jun 18 20:03:33 2019 +0200 loplugin:logexceptionnicely in writerfilter Change-Id: Ie01d0e6af6af0cfeb46cdde38f52ab068b64db6b Reviewed-on: https://gerrit.libreoffice.org/74284 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk> diff --git a/writerfilter/source/dmapper/DomainMapper.cxx b/writerfilter/source/dmapper/DomainMapper.cxx index 195e360a671d..0eef649f96d3 100644 --- a/writerfilter/source/dmapper/DomainMapper.cxx +++ b/writerfilter/source/dmapper/DomainMapper.cxx @@ -3162,9 +3162,9 @@ void DomainMapper::lcl_text(const sal_uInt8 * data_, size_t len) m_pImpl->appendTextPortion( sText, pContext ); } } - catch( const uno::RuntimeException& e ) + catch( const uno::RuntimeException& ) { - SAL_WARN("writerfilter", "failed. Message :" << e); + TOOLS_WARN_EXCEPTION("writerfilter", ""); } } diff --git a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx index 27a710f5a0b6..f1ca99fc77bf 100644 --- a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx +++ b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx @@ -35,6 +35,7 @@ #include "util.hxx" #include <osl/diagnose.h> #include <sal/log.hxx> +#include <tools/diagnose_ex.h> #include <comphelper/sequence.hxx> #ifdef DBG_UTIL @@ -915,9 +916,9 @@ static bool lcl_emptyRow(std::vector<RowSequence_t>& rTableRanges, sal_Int32 nRo if (bRangesAreNotEqual) return false; } - catch (const lang::IllegalArgumentException& e) + catch (const lang::IllegalArgumentException&) { - SAL_WARN( "writerfilter.dmapper", "compareRegionStarts() failed: " << e); + TOOLS_WARN_EXCEPTION( "writerfilter.dmapper", "compareRegionStarts() failed"); return false; } return true; @@ -1102,18 +1103,16 @@ void DomainMapperTableHandler::endTable(unsigned int nestedTableLevel, bool bTab } } } - catch ( const lang::IllegalArgumentException &e ) + catch ( const lang::IllegalArgumentException & ) { - SAL_INFO("writerfilter.dmapper", - "Conversion to table error: " << e); + TOOLS_INFO_EXCEPTION("writerfilter.dmapper", "Conversion to table error"); #ifdef DBG_UTIL TagLogger::getInstance().chars(std::string("failed to import table!")); #endif } - catch ( const uno::Exception &e ) + catch ( const uno::Exception & ) { - SAL_INFO("writerfilter.dmapper", - "Exception during table creation: " << e); + TOOLS_INFO_EXCEPTION("writerfilter.dmapper", "Exception during table creation"); } // If we have a table with a start and an end position, we should make it a floating one. diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.cxx b/writerfilter/source/dmapper/DomainMapper_Impl.cxx index d70ebb443691..1e6a5a1e3f76 100644 --- a/writerfilter/source/dmapper/DomainMapper_Impl.cxx +++ b/writerfilter/source/dmapper/DomainMapper_Impl.cxx @@ -1564,9 +1564,9 @@ void DomainMapper_Impl::finishParagraph( const PropertyMapPtr& pPropertyMap, con { OSL_FAIL( "IllegalArgumentException in DomainMapper_Impl::finishParagraph" ); } - catch(const uno::Exception& e) + catch(const uno::Exception&) { - SAL_WARN( "writerfilter.dmapper", "finishParagraph() " << e ); + TOOLS_WARN_EXCEPTION( "writerfilter.dmapper", "finishParagraph()" ); } } @@ -2242,10 +2242,9 @@ void DomainMapper_Impl::PopAnnotation() } m_aAnnotationPositions.erase( m_nAnnotationId ); } - catch (uno::Exception const& e) + catch (uno::Exception const&) { - SAL_WARN("writerfilter.dmapper", - "Cannot insert annotation field: " << e); + TOOLS_WARN_EXCEPTION("writerfilter.dmapper", "Cannot insert annotation field"); } m_xAnnotationField.clear(); @@ -2436,9 +2435,9 @@ void DomainMapper_Impl::PushShapeContext( const uno::Reference< drawing::XShape m_bParaChanged = true; getTableManager().setIsInShape(true); } - catch ( const uno::Exception& e ) + catch ( const uno::Exception& ) { - SAL_WARN("writerfilter.dmapper", "Exception when adding shape: " << e); + TOOLS_WARN_EXCEPTION("writerfilter.dmapper", "Exception when adding shape"); } } /* @@ -4902,9 +4901,9 @@ void DomainMapper_Impl::CloseFieldCommand() //set the text field if there is any pContext->SetTextField( uno::Reference< text::XTextField >( xFieldInterface, uno::UNO_QUERY ) ); } - catch( const uno::Exception& e ) + catch( const uno::Exception& ) { - SAL_WARN( "writerfilter.dmapper", "Exception in CloseFieldCommand(): " << e ); + TOOLS_WARN_EXCEPTION( "writerfilter.dmapper", "Exception in CloseFieldCommand()" ); } pContext->SetCommandCompleted(); } @@ -5073,10 +5072,9 @@ void DomainMapper_Impl::SetFieldResult(OUString const& rResult) } } } - catch (const uno::Exception& e) + catch (const uno::Exception&) { - SAL_WARN("writerfilter.dmapper", - "DomainMapper_Impl::SetFieldResult: " << e); + TOOLS_WARN_EXCEPTION("writerfilter.dmapper", "DomainMapper_Impl::SetFieldResult"); } } } @@ -5911,10 +5909,9 @@ uno::Reference<container::XIndexAccess> DomainMapper_Impl::GetCurrentNumberingRu uno::Reference<beans::XPropertySet> xStyle(xNumberingStyles->getByName(aListName), uno::UNO_QUERY); xRet.set(xStyle->getPropertyValue("NumberingRules"), uno::UNO_QUERY); } - catch (const uno::Exception& e) + catch (const uno::Exception&) { - SAL_WARN("writerfilter.dmapper", - "GetCurrentNumberingRules: exception caught: " << e); + TOOLS_WARN_EXCEPTION("writerfilter.dmapper", "GetCurrentNumberingRules: exception caught"); } return xRet; } diff --git a/writerfilter/source/dmapper/GraphicImport.cxx b/writerfilter/source/dmapper/GraphicImport.cxx index d5d09a90dcbb..bf80052e33dd 100644 --- a/writerfilter/source/dmapper/GraphicImport.cxx +++ b/writerfilter/source/dmapper/GraphicImport.cxx @@ -49,6 +49,7 @@ #include <rtl/ustrbuf.hxx> #include <sal/log.hxx> #include <rtl/math.hxx> +#include <tools/diagnose_ex.h> #include <comphelper/string.hxx> #include <comphelper/sequenceashashmap.hxx> #include <comphelper/sequence.hxx> @@ -377,9 +378,9 @@ public: xGraphicObjectProperties->setPropertyValue(getPropertyName( PROP_TITLE ), uno::makeAny( title )); } - catch( const uno::Exception& e ) + catch( const uno::Exception& ) { - SAL_WARN("writerfilter", "failed. Message :" << e); + TOOLS_WARN_EXCEPTION("writerfilter", "failed"); } } @@ -1387,9 +1388,9 @@ uno::Reference<text::XTextContent> GraphicImport::createGraphicObject(uno::Refer } } } - catch( const uno::Exception& e ) + catch( const uno::Exception& ) { - SAL_WARN("writerfilter", "failed. Message :" << e); + TOOLS_WARN_EXCEPTION("writerfilter", ""); } return xGraphicObject; } diff --git a/writerfilter/source/dmapper/NumberingManager.cxx b/writerfilter/source/dmapper/NumberingManager.cxx index 666ee674cddc..e5bf229d174e 100644 --- a/writerfilter/source/dmapper/NumberingManager.cxx +++ b/writerfilter/source/dmapper/NumberingManager.cxx @@ -39,6 +39,7 @@ #include <osl/diagnose.h> #include <rtl/ustring.hxx> #include <sal/log.hxx> +#include <tools/diagnose_ex.h> #include <comphelper/sequence.hxx> #include <comphelper/propertyvalue.hxx> @@ -624,19 +625,19 @@ void ListDef::CreateNumberingRules( DomainMapper& rDMapper, OUString sNumRulesName = getPropertyName( PROP_NUMBERING_RULES ); xStyle->setPropertyValue( sNumRulesName, uno::makeAny( m_xNumRules ) ); } - catch( const lang::IllegalArgumentException& e ) + catch( const lang::IllegalArgumentException& ) { - SAL_WARN( "writerfilter", e ); + TOOLS_WARN_EXCEPTION( "writerfilter", "" ); assert( !"Incorrect argument to UNO call" ); } - catch( const uno::RuntimeException& e ) + catch( const uno::RuntimeException& ) { - SAL_WARN( "writerfilter", e ); + TOOLS_WARN_EXCEPTION( "writerfilter", "" ); assert( !"Incorrect argument to UNO call" ); } - catch( const uno::Exception& e ) + catch( const uno::Exception& ) { - SAL_WARN( "writerfilter", e ); + TOOLS_WARN_EXCEPTION( "writerfilter", "" ); } } diff --git a/writerfilter/source/dmapper/OLEHandler.cxx b/writerfilter/source/dmapper/OLEHandler.cxx index 8eb6fddeb303..c59e041b28a4 100644 --- a/writerfilter/source/dmapper/OLEHandler.cxx +++ b/writerfilter/source/dmapper/OLEHandler.cxx @@ -27,6 +27,7 @@ #include <rtl/ustring.hxx> #include <sal/log.hxx> #include <osl/diagnose.h> +#include <tools/diagnose_ex.h> #include <unotools/mediadescriptor.hxx> #include <officecfg/Office/Common.hxx> #include <com/sun/star/beans/PropertyValue.hpp> @@ -124,9 +125,9 @@ void OLEHandler::lcl_attribute(Id rName, Value & rVal) xShapeProps->getPropertyValue( getPropertyName( PROP_BITMAP ) ) >>= m_xReplacement; } - catch( const uno::Exception& e ) + catch( const uno::Exception& ) { - SAL_WARN("writerfilter", "Exception in OLE Handler: " << e); + TOOLS_WARN_EXCEPTION("writerfilter", "Exception in OLE Handler"); } // No need to set the wrapping here as it's either set in oox or will be set later } @@ -175,9 +176,9 @@ void OLEHandler::lcl_sprm(Sprm & rSprm) if( m_rDomainMapper.IsInHeaderFooter() ) xShapeProps->setPropertyValue("Opaque", uno::makeAny(m_nWrapMode != text::WrapTextMode_THROUGH)); } - catch( const uno::Exception& e ) + catch( const uno::Exception& ) { - SAL_WARN("writerfilter", "Exception in OLE Handler: " << e); + TOOLS_WARN_EXCEPTION("writerfilter", "Exception in OLE Handler"); } } } diff --git a/writerfilter/source/dmapper/PropertyMap.cxx b/writerfilter/source/dmapper/PropertyMap.cxx index 68f1f7119564..a85b80735b1a 100644 --- a/writerfilter/source/dmapper/PropertyMap.cxx +++ b/writerfilter/source/dmapper/PropertyMap.cxx @@ -841,9 +841,9 @@ void SectionPropertyMap::CopyHeaderFooterTextProperty( const uno::Reference< bea xTxt->copyText( xPrevTxt ); } - catch ( const uno::Exception& e ) + catch ( const uno::Exception& ) { - SAL_INFO( "writerfilter", "An exception occurred in SectionPropertyMap::CopyHeaderFooterTextProperty( ) - " << e ); + TOOLS_INFO_EXCEPTION( "writerfilter", "An exception occurred in SectionPropertyMap::CopyHeaderFooterTextProperty( )" ); } } diff --git a/writerfilter/source/dmapper/StyleSheetTable.cxx b/writerfilter/source/dmapper/StyleSheetTable.cxx index 3e41aa93404e..bfd535949bf4 100644 --- a/writerfilter/source/dmapper/StyleSheetTable.cxx +++ b/writerfilter/source/dmapper/StyleSheetTable.cxx @@ -373,9 +373,9 @@ void StyleSheetTable_Impl::SetPropertiesToDefault(const uno::Reference<style::XS { xPropertyState->setPropertyToDefault(aPropertyNames[i]); } - catch(const uno::Exception& rException) + catch(const uno::Exception&) { - SAL_INFO("writerfilter", "setPropertyToDefault(" << aPropertyNames[i] << ") failed: " << rException); + TOOLS_INFO_EXCEPTION("writerfilter", "setPropertyToDefault(" << aPropertyNames[i] << ") failed"); } } } diff --git a/writerfilter/source/dmapper/TableManager.cxx b/writerfilter/source/dmapper/TableManager.cxx index 8377d356668e..f0194ca499ce 100644 --- a/writerfilter/source/dmapper/TableManager.cxx +++ b/writerfilter/source/dmapper/TableManager.cxx @@ -23,6 +23,7 @@ #include "util.hxx" #include <sal/log.hxx> +#include <tools/diagnose_ex.h> namespace writerfilter { @@ -304,9 +305,9 @@ void TableManager::resolveCurrentTable() mpTableDataHandler->endTable(mTableDataStack.size() - 1, m_bTableStartsAtCellStart); } - catch (css::uno::Exception const& e) + catch (css::uno::Exception const&) { - SAL_WARN("writerfilter", "resolving of current table failed with: " << e); + TOOLS_WARN_EXCEPTION("writerfilter", "resolving of current table failed"); } } mState.resetTableProps(); diff --git a/writerfilter/source/filter/RtfFilter.cxx b/writerfilter/source/filter/RtfFilter.cxx index a80c82c43e2f..3c1e975d8ad7 100644 --- a/writerfilter/source/filter/RtfFilter.cxx +++ b/writerfilter/source/filter/RtfFilter.cxx @@ -31,6 +31,7 @@ #include <cppuhelper/supportsservice.hxx> #include <osl/file.hxx> #include <sal/log.hxx> +#include <tools/diagnose_ex.h> #include <unotools/mediadescriptor.hxx> #include <unotools/streamwrap.hxx> #include <unotools/ucbstreamhelper.hxx> @@ -153,9 +154,9 @@ sal_Bool RtfFilter::filter(const uno::Sequence< beans::PropertyValue >& rDescrip throw lang::WrappedTargetRuntimeException("", static_cast<OWeakObject*>(this), anyEx); } - catch (const uno::Exception& e) + catch (const uno::Exception&) { - SAL_INFO("writerfilter", "Exception caught: " << e); + TOOLS_INFO_EXCEPTION("writerfilter", "Exception caught"); } if (xStatusIndicator.is()) diff --git a/writerfilter/source/ooxml/OOXMLDocumentImpl.cxx b/writerfilter/source/ooxml/OOXMLDocumentImpl.cxx index 9601a5311923..e8386c5898ef 100644 --- a/writerfilter/source/ooxml/OOXMLDocumentImpl.cxx +++ b/writerfilter/source/ooxml/OOXMLDocumentImpl.cxx @@ -83,10 +83,10 @@ void OOXMLDocumentImpl::resolveFastSubStream(Stream & rStreamHandler, { pStream = OOXMLDocumentFactory::createStream(mpStream, nType); } - catch (uno::Exception const& e) + catch (uno::Exception const&) { - SAL_INFO("writerfilter.ooxml", "resolveFastSubStream: exception while " - "resolving stream " << nType << " : " << e); + TOOLS_INFO_EXCEPTION("writerfilter.ooxml", "resolveFastSubStream: exception while " + "resolving stream " << nType); return; } OOXMLStream::Pointer_t savedStream = mpStream; @@ -137,10 +137,10 @@ uno::Reference<xml::dom::XDocument> OOXMLDocumentImpl::importSubStream(OOXMLStre { pStream = OOXMLDocumentFactory::createStream(mpStream, nType); } - catch (uno::Exception const& e) + catch (uno::Exception const&) { - SAL_INFO("writerfilter.ooxml", "importSubStream: exception while " - "importing stream " << nType << " : " << e); + TOOLS_INFO_EXCEPTION("writerfilter.ooxml", "importSubStream: exception while " + "importing stream " << nType); return xRet; } @@ -153,10 +153,10 @@ uno::Reference<xml::dom::XDocument> OOXMLDocumentImpl::importSubStream(OOXMLStre uno::Reference<xml::dom::XDocumentBuilder> xDomBuilder(xml::dom::DocumentBuilder::create(xContext)); xRet = xDomBuilder->parse(xInputStream); } - catch (uno::Exception const& e) + catch (uno::Exception const&) { - SAL_INFO("writerfilter.ooxml", "importSubStream: exception while " - "parsing stream " << nType << " : " << e); + TOOLS_INFO_EXCEPTION("writerfilter.ooxml", "importSubStream: exception while " + "parsing stream " << nType); return xRet; } } @@ -614,10 +614,10 @@ void OOXMLDocumentImpl::resolveGlossaryStream(Stream & /*rStream*/) { pStream = OOXMLDocumentFactory::createStream(mpStream, OOXMLStream::GLOSSARY); } - catch (uno::Exception const& e) + catch (uno::Exception const&) { - SAL_INFO("writerfilter.ooxml", "resolveGlossaryStream: exception while " - "createStream for glossary" << OOXMLStream::GLOSSARY << " : " << e); + TOOLS_INFO_EXCEPTION("writerfilter.ooxml", "resolveGlossaryStream: exception while " + "createStream for glossary" << OOXMLStream::GLOSSARY); return; } uno::Reference<embed::XRelationshipAccess> xRelationshipAccess; @@ -685,10 +685,10 @@ void OOXMLDocumentImpl::resolveGlossaryStream(Stream & /*rStream*/) uno::Reference<xml::dom::XDocumentBuilder> xDomBuilder(xml::dom::DocumentBuilder::create(xContext)); xDom = xDomBuilder->parse(xInputStream); } - catch (uno::Exception const& e) + catch (uno::Exception const&) { - SAL_INFO("writerfilter.ooxml", "importSubStream: exception while " - "parsing stream of Type" << nType << " : " << e); + TOOLS_INFO_EXCEPTION("writerfilter.ooxml", "importSubStream: exception while " + "parsing stream of Type" << nType); return; } @@ -767,10 +767,10 @@ void OOXMLDocumentImpl::resolveEmbeddingsStream(const OOXMLStream::Pointer_t& pS if (Stream) resolveEmbeddingsStream(Stream); } - catch (uno::Exception const& e) + catch (uno::Exception const&) { - SAL_INFO("writerfilter.ooxml", "resolveEmbeddingsStream: can't find header/footer whilst " - "resolving stream " << streamType << " : " << e); + TOOLS_INFO_EXCEPTION("writerfilter.ooxml", "resolveEmbeddingsStream: can't find header/footer whilst " + "resolving stream " << streamType); return; } } diff --git a/writerfilter/source/rtftok/rtfdocumentimpl.cxx b/writerfilter/source/rtftok/rtfdocumentimpl.cxx index 43463dabc152..b829c8aa9fd2 100644 --- a/writerfilter/source/rtftok/rtfdocumentimpl.cxx +++ b/writerfilter/source/rtftok/rtfdocumentimpl.cxx @@ -25,6 +25,7 @@ #include <filter/msfilter/util.hxx> #include <filter/msfilter/rtfutil.hxx> #include <comphelper/string.hxx> +#include <tools/diagnose_ex.h> #include <tools/globname.hxx> #include <tools/datetimeutils.hxx> #include <comphelper/classids.hxx> @@ -3161,10 +3162,9 @@ RTFError RTFDocumentImpl::popState() xDocumentPropertyContainer->addProperty( rKey, beans::PropertyAttribute::REMOVABLE, aValue); } - catch (const uno::Exception& rException) + catch (const uno::Exception&) { - SAL_WARN("writerfilter.rtf", - "failed to set property " << rKey << ": " << rException); + TOOLS_WARN_EXCEPTION("writerfilter.rtf", "failed to set property " << rKey); } } } _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits