[Libreoffice-commits] core.git: vcl/qt5 vcl/source vcl/unx vcl/workben
Noel Grandin (via logerrit)
logerrit at kemper.freedesktop.org
Tue Jun 18 08:47:46 UTC 2019
vcl/qt5/Qt5Widget.cxx | 5 +++--
vcl/source/app/session.cxx | 5 +++--
vcl/source/app/svdata.cxx | 5 +++--
vcl/source/app/svmain.cxx | 5 +++--
vcl/source/gdi/gdimtf.cxx | 5 +++--
vcl/source/gdi/vectorgraphicdata.cxx | 5 +++--
vcl/source/image/ImplImageTree.cxx | 17 +++++++++--------
vcl/source/window/window.cxx | 9 +++++----
vcl/unx/generic/fontmanager/fontconfig.cxx | 5 +++--
vcl/unx/gtk/gtksalframe.cxx | 5 +++--
vcl/unx/gtk3/gtk3gtkframe.cxx | 3 ++-
vcl/workben/mtfdemo.cxx | 5 +++--
vcl/workben/outdevgrind.cxx | 5 +++--
vcl/workben/svdem.cxx | 5 +++--
vcl/workben/svpclient.cxx | 3 ++-
vcl/workben/svptest.cxx | 5 +++--
vcl/workben/vcldemo.cxx | 5 +++--
17 files changed, 57 insertions(+), 40 deletions(-)
New commits:
commit a2e4b76e29e491bac4f9e6dfd9929dfd49a4b05e
Author: Noel Grandin <noel.grandin at collabora.co.uk>
AuthorDate: Tue Jun 18 09:33:28 2019 +0200
Commit: Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Tue Jun 18 10:46:40 2019 +0200
loplugin:logexceptionnicely in vcl
Change-Id: I0ed575a11c84c2e8aabfa1b4204ba6ae27393d5f
Reviewed-on: https://gerrit.libreoffice.org/74245
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>
diff --git a/vcl/qt5/Qt5Widget.cxx b/vcl/qt5/Qt5Widget.cxx
index b46b48b35366..657c5e8e73f7 100644
--- a/vcl/qt5/Qt5Widget.cxx
+++ b/vcl/qt5/Qt5Widget.cxx
@@ -47,6 +47,7 @@
#include <vcl/commandevent.hxx>
#include <vcl/event.hxx>
#include <window.h>
+#include <tools/diagnose_ex.h>
#include <com/sun/star/accessibility/XAccessibleContext.hpp>
#include <com/sun/star/accessibility/XAccessibleEditableText.hpp>
@@ -613,9 +614,9 @@ static bool lcl_retrieveSurrounding(sal_Int32& rPosition, sal_Int32& rAnchor, QS
if (xAccessible.is())
xText = FindFocusedEditableText(xAccessible->getAccessibleContext());
}
- catch (const uno::Exception& e)
+ catch (const uno::Exception&)
{
- SAL_WARN("vcl.qt5", "Exception in getting input method surrounding text: " << e);
+ TOOLS_WARN_EXCEPTION("vcl.qt5", "Exception in getting input method surrounding text");
}
bool result = false;
diff --git a/vcl/source/app/session.cxx b/vcl/source/app/session.cxx
index 5679c7a9fb63..c96d179cf90e 100644
--- a/vcl/source/app/session.cxx
+++ b/vcl/source/app/session.cxx
@@ -24,6 +24,7 @@
#include <cppuhelper/basemutex.hxx>
#include <cppuhelper/compbase.hxx>
+#include <tools/diagnose_ex.h>
#include <vcl/svapp.hxx>
#include <factory.hxx>
@@ -376,8 +377,8 @@ void VCLSession::disposing() {
try {
listener.m_xListener->disposing(src);
SAL_INFO("vcl.se.debug", " call Listener disposing");
- } catch (css::uno::RuntimeException & e) {
- SAL_WARN("vcl.se", "ignoring " << e);
+ } catch (css::uno::RuntimeException &) {
+ TOOLS_WARN_EXCEPTION("vcl.se", "ignoring");
}
}
}
diff --git a/vcl/source/app/svdata.cxx b/vcl/source/app/svdata.cxx
index 93335d48624b..2a3fd1162528 100644
--- a/vcl/source/app/svdata.cxx
+++ b/vcl/source/app/svdata.cxx
@@ -22,6 +22,7 @@
#include <comphelper/lok.hxx>
#include <comphelper/processfactory.hxx>
#include <rtl/process.h>
+#include <tools/diagnose_ex.h>
#include <tools/gen.hxx>
#include <unotools/resmgr.hxx>
#include <uno/current_context.hxx>
@@ -244,9 +245,9 @@ vcl::Window *ImplGetDefaultContextWindow()
pContext->acquire();
#endif
}
- catch (const css::uno::Exception& e)
+ catch (const css::uno::Exception&)
{
- SAL_WARN("vcl", "unable to create Default Window: " << e);
+ TOOLS_WARN_EXCEPTION("vcl", "unable to create Default Window");
}
}
}
diff --git a/vcl/source/app/svmain.cxx b/vcl/source/app/svmain.cxx
index 51e47674b604..64395164e8ba 100644
--- a/vcl/source/app/svmain.cxx
+++ b/vcl/source/app/svmain.cxx
@@ -102,6 +102,7 @@
#include <opengl/watchdog.hxx>
#include <basegfx/utils/systemdependentdata.hxx>
+#include <tools/diagnose_ex.h>
#if OSL_DEBUG_LEVEL > 0
#include <typeinfo>
@@ -344,9 +345,9 @@ bool InitVCL()
osl_setEnvironment(envVar.pData, aLocaleString.pData);
}
}
- catch (const uno::Exception &e)
+ catch (const uno::Exception &)
{
- SAL_INFO("vcl.app", "Unable to get ui language: '" << e);
+ TOOLS_INFO_EXCEPTION("vcl.app", "Unable to get ui language:");
}
pSVData->mpDefInst->AfterAppInit();
diff --git a/vcl/source/gdi/gdimtf.cxx b/vcl/source/gdi/gdimtf.cxx
index eb730fe578a7..8c01f8302535 100644
--- a/vcl/source/gdi/gdimtf.cxx
+++ b/vcl/source/gdi/gdimtf.cxx
@@ -22,6 +22,7 @@
#include <memory>
#include <sal/log.hxx>
#include <osl/diagnose.h>
+#include <tools/diagnose_ex.h>
#include <tools/helpers.hxx>
#include <tools/stream.hxx>
#include <tools/vcompat.hxx>
@@ -426,10 +427,10 @@ bool GDIMetaFile::ImplPlayWithRenderer( OutputDevice* pOut, const Point& rPos, S
{
throw; // runtime errors are fatal
}
- catch (const uno::Exception& e)
+ catch (const uno::Exception&)
{
// ignore errors, no way of reporting them here
- SAL_WARN("vcl.gdi", "GDIMetaFile::ImplPlayWithRenderer: " << e);
+ TOOLS_WARN_EXCEPTION("vcl.gdi", "GDIMetaFile::ImplPlayWithRenderer");
}
return false;
diff --git a/vcl/source/gdi/vectorgraphicdata.cxx b/vcl/source/gdi/vectorgraphicdata.cxx
index 8c936f00cfd7..3d32af0ee0e7 100644
--- a/vcl/source/gdi/vectorgraphicdata.cxx
+++ b/vcl/source/gdi/vectorgraphicdata.cxx
@@ -17,6 +17,7 @@
* the License at http://www.apache.org/licenses/LICENSE-2.0 .
*/
+#include <tools/diagnose_ex.h>
#include <tools/stream.hxx>
#include <sal/log.hxx>
#include <vcl/vectorgraphicdata.hxx>
@@ -78,9 +79,9 @@ BitmapEx convertPrimitive2DSequenceToBitmapEx(
aRetval = vcl::unotools::bitmapExFromXBitmap(xIntBmp);
}
}
- catch (const uno::Exception& e)
+ catch (const uno::Exception&)
{
- SAL_WARN("vcl", "Got no graphic::XPrimitive2DRenderer! : " << e);
+ TOOLS_WARN_EXCEPTION("vcl", "Got no graphic::XPrimitive2DRenderer!");
}
catch (const std::exception& e)
{
diff --git a/vcl/source/image/ImplImageTree.cxx b/vcl/source/image/ImplImageTree.cxx
index d879759b5486..e1bf651e5b02 100644
--- a/vcl/source/image/ImplImageTree.cxx
+++ b/vcl/source/image/ImplImageTree.cxx
@@ -41,6 +41,7 @@
#include <rtl/bootstrap.hxx>
#include <rtl/uri.hxx>
+#include <tools/diagnose_ex.h>
#include <tools/stream.hxx>
#include <tools/urlobj.hxx>
#include <implimagetree.hxx>
@@ -256,9 +257,9 @@ OUString ImplImageTree::getImageUrl(OUString const & rName, OUString const & rSt
}
}
}
- catch (const uno::Exception & e)
+ catch (const uno::Exception &)
{
- SAL_INFO("vcl", e);
+ TOOLS_INFO_EXCEPTION("vcl", "");
}
aStyle = fallbackStyle(aStyle);
@@ -296,9 +297,9 @@ std::shared_ptr<SvMemoryStream> ImplImageTree::getImageStream(OUString const & r
}
}
}
- catch (const uno::Exception & e)
+ catch (const uno::Exception &)
{
- SAL_INFO("vcl", e);
+ TOOLS_INFO_EXCEPTION("vcl", "");
}
aStyle = fallbackStyle(aStyle);
@@ -413,9 +414,9 @@ bool ImplImageTree::doLoadImage(ImageRequestParameters& rParameters)
{
throw;
}
- catch (const uno::Exception& e)
+ catch (const uno::Exception&)
{
- SAL_INFO("vcl", "ImplImageTree::doLoadImage exception: " << e);
+ TOOLS_INFO_EXCEPTION("vcl", "ImplImageTree::doLoadImage");
}
if (bFound)
@@ -681,9 +682,9 @@ bool ImplImageTree::checkPathAccess()
{
throw;
}
- catch (const uno::Exception & e)
+ catch (const uno::Exception &)
{
- SAL_INFO("vcl", "ImplImageTree::zip file location " << e << " for " << rIconSet.maURL);
+ TOOLS_INFO_EXCEPTION("vcl", "ImplImageTree::zip file location " << rIconSet.maURL);
return false;
}
return rNameAccess.is();
diff --git a/vcl/source/window/window.cxx b/vcl/source/window/window.cxx
index 27f027eb3224..5a0b505e2aef 100644
--- a/vcl/source/window/window.cxx
+++ b/vcl/source/window/window.cxx
@@ -70,6 +70,7 @@
#include <comphelper/lok.hxx>
#include <comphelper/processfactory.hxx>
#include <unotools/configmgr.hxx>
+#include <tools/diagnose_ex.h>
#include <tools/debug.hxx>
#include <cassert>
@@ -3288,9 +3289,9 @@ Reference< XClipboard > Window::GetClipboard()
= css::datatransfer::clipboard::SystemClipboard::create(
comphelper::getProcessComponentContext());
}
- catch (DeploymentException & e)
+ catch (DeploymentException const &)
{
- SAL_WARN("vcl.window", "ignoring " << e);
+ TOOLS_WARN_EXCEPTION("vcl.window", "ignoring");
}
}
@@ -3329,9 +3330,9 @@ Reference< XClipboard > Window::GetPrimarySelection()
mpWindowImpl->mpFrameData->mxSelection = s_xSelection;
#endif
}
- catch (RuntimeException & e)
+ catch (RuntimeException const &)
{
- SAL_WARN("vcl.window", "ignoring " << e);
+ TOOLS_WARN_EXCEPTION("vcl.window", "ignoring");
}
}
diff --git a/vcl/unx/generic/fontmanager/fontconfig.cxx b/vcl/unx/generic/fontmanager/fontconfig.cxx
index b518493a967d..3f0a2aa4418b 100644
--- a/vcl/unx/generic/fontmanager/fontconfig.cxx
+++ b/vcl/unx/generic/fontmanager/fontconfig.cxx
@@ -32,6 +32,7 @@
#include <i18nutil/unicode.hxx>
#include <rtl/strbuf.hxx>
#include <sal/log.hxx>
+#include <tools/diagnose_ex.h>
#include <unicode/uchar.h>
#include <unicode/uscript.h>
#include <officecfg/Office/Common.hxx>
@@ -863,9 +864,9 @@ IMPL_LINK_NOARG(PrintFontManager, autoInstallFontLangSupport, Timer *, void)
css::uno::Reference<XSyncDbusSessionHelper> xSyncDbusSessionHelper(SyncDbusSessionHelper::create(comphelper::getProcessComponentContext()));
xSyncDbusSessionHelper->InstallFontconfigResources(comphelper::containerToSequence(m_aCurrentRequests), "hide-finished");
}
- catch (const css::uno::Exception& e)
+ catch (const css::uno::Exception&)
{
- SAL_INFO("vcl.fonts", "InstallFontconfigResources problem, caught " << e);
+ TOOLS_INFO_EXCEPTION("vcl.fonts", "InstallFontconfigResources problem");
// Disable this method from now on. It's simply not available on some systems
// and leads to an error dialog being shown each time this is called tdf#104883
std::shared_ptr<comphelper::ConfigurationChanges> batch( comphelper::ConfigurationChanges::create() );
diff --git a/vcl/unx/gtk/gtksalframe.cxx b/vcl/unx/gtk/gtksalframe.cxx
index 75526e55a59b..425ad0562d82 100644
--- a/vcl/unx/gtk/gtksalframe.cxx
+++ b/vcl/unx/gtk/gtksalframe.cxx
@@ -34,6 +34,7 @@
#include <rtl/bootstrap.hxx>
#include <rtl/process.h>
#include <sal/log.hxx>
+#include <tools/diagnose_ex.h>
#include <vcl/floatwin.hxx>
#include <vcl/svapp.hxx>
#include <vcl/window.hxx>
@@ -3509,9 +3510,9 @@ static uno::Reference<accessibility::XAccessibleEditableText> lcl_GetxText(vcl::
if (xAccessible.is())
xText = FindFocusedEditableText(xAccessible->getAccessibleContext());
}
- catch(const uno::Exception& e)
+ catch(const uno::Exception&)
{
- SAL_WARN( "vcl.gtk", "Exception in getting input method surrounding text: " << e);
+ TOOLS_WARN_EXCEPTION( "vcl.gtk", "Exception in getting input method surrounding text");
}
return xText;
}
diff --git a/vcl/unx/gtk3/gtk3gtkframe.cxx b/vcl/unx/gtk3/gtk3gtkframe.cxx
index 888a7fb9aeb4..1061345344bc 100644
--- a/vcl/unx/gtk3/gtk3gtkframe.cxx
+++ b/vcl/unx/gtk3/gtk3gtkframe.cxx
@@ -36,6 +36,7 @@
#include <rtl/bootstrap.hxx>
#include <rtl/process.h>
#include <sal/log.hxx>
+#include <tools/diagnose_ex.h>
#include <vcl/floatwin.hxx>
#include <vcl/svapp.hxx>
#include <vcl/weld.hxx>
@@ -4083,7 +4084,7 @@ static uno::Reference<accessibility::XAccessibleEditableText> lcl_GetxText(vcl::
}
catch(const uno::Exception& e)
{
- SAL_WARN( "vcl.gtk3", "Exception in getting input method surrounding text: " << e);
+ TOOLS_WARN_EXCEPTION( "vcl.gtk3", "Exception in getting input method surrounding text");
}
return xText;
}
diff --git a/vcl/workben/mtfdemo.cxx b/vcl/workben/mtfdemo.cxx
index 7addd2bc54a3..0ee726e051f8 100644
--- a/vcl/workben/mtfdemo.cxx
+++ b/vcl/workben/mtfdemo.cxx
@@ -19,6 +19,7 @@
#include <vcl/gdimtf.hxx>
#include <vcl/wmf.hxx>
+#include <tools/diagnose_ex.h>
#include <tools/urlobj.hxx>
#include <tools/stream.hxx>
#include <tools/vcompat.hxx>
@@ -91,9 +92,9 @@ public:
Application::Execute();
}
- catch (const css::uno::Exception& e)
+ catch (const css::uno::Exception&)
{
- SAL_WARN("vcl.app", "Fatal: " << e);
+ TOOLS_WARN_EXCEPTION("vcl.app", "Fatal");
return 1;
}
catch (const std::exception& e)
diff --git a/vcl/workben/outdevgrind.cxx b/vcl/workben/outdevgrind.cxx
index ac6f12aaa373..8283a5e59da2 100644
--- a/vcl/workben/outdevgrind.cxx
+++ b/vcl/workben/outdevgrind.cxx
@@ -19,6 +19,7 @@
#include <sal/main.h>
#include <sal/log.hxx>
+#include <tools/diagnose_ex.h>
#include <tools/extendapplicationenvironment.hxx>
#include <cppuhelper/bootstrap.hxx>
@@ -731,9 +732,9 @@ SAL_IMPLEMENT_MAIN()
DeInitVCL();
}
- catch (const css::uno::Exception& e)
+ catch (const css::uno::Exception&)
{
- SAL_WARN("vcl.app", "Fatal: " << e);
+ TOOLS_WARN_EXCEPTION("vcl.app", "Fatal");
return EXIT_FAILURE;
}
catch (const std::exception& e)
diff --git a/vcl/workben/svdem.cxx b/vcl/workben/svdem.cxx
index 579141839cc4..dd9027a81be5 100644
--- a/vcl/workben/svdem.cxx
+++ b/vcl/workben/svdem.cxx
@@ -19,6 +19,7 @@
#include <sal/main.h>
#include <sal/log.hxx>
+#include <tools/diagnose_ex.h>
#include <tools/extendapplicationenvironment.hxx>
#include <cppuhelper/bootstrap.hxx>
@@ -57,9 +58,9 @@ SAL_IMPLEMENT_MAIN()
::Main();
DeInitVCL();
}
- catch (const Exception& e)
+ catch (const Exception&)
{
- SAL_WARN("vcl.app", "Fatal: " << e);
+ TOOLS_WARN_EXCEPTION("vcl.app", "Fatal");
return 1;
}
catch (const std::exception &e)
diff --git a/vcl/workben/svpclient.cxx b/vcl/workben/svpclient.cxx
index 1b78996642e7..9ffdbcb38f2c 100644
--- a/vcl/workben/svpclient.cxx
+++ b/vcl/workben/svpclient.cxx
@@ -36,6 +36,7 @@
#include <vcl/bitmapex.hxx>
#include <vcl/graphicfilter.hxx>
#include <vcl/graph.hxx>
+#include <tools/diagnose_ex.h>
#include <tools/extendapplicationenvironment.hxx>
#include <tools/stream.hxx>
@@ -79,7 +80,7 @@ SAL_IMPLEMENT_MAIN()
}
catch (const Exception& e)
{
- SAL_WARN("vcl", "Fatal: " << e);
+ TOOLS_WARN_EXCEPTION("vcl", "Fatal");
return 1;
}
catch (const std::exception& e)
diff --git a/vcl/workben/svptest.cxx b/vcl/workben/svptest.cxx
index e2425807a8e9..a79cda4df507 100644
--- a/vcl/workben/svptest.cxx
+++ b/vcl/workben/svptest.cxx
@@ -19,6 +19,7 @@
#include <sal/main.h>
#include <sal/log.hxx>
+#include <tools/diagnose_ex.h>
#include <tools/extendapplicationenvironment.hxx>
#include <cppuhelper/bootstrap.hxx>
@@ -67,9 +68,9 @@ SAL_IMPLEMENT_MAIN()
::Main();
DeInitVCL();
}
- catch (const Exception& e)
+ catch (const Exception&)
{
- SAL_WARN("vcl.app", "Fatal: " << e);
+ TOOLS_WARN_EXCEPTION("vcl.app", "Fatal");
return 1;
}
catch (const std::exception &e)
diff --git a/vcl/workben/vcldemo.cxx b/vcl/workben/vcldemo.cxx
index befde4053660..3685e905a1ec 100644
--- a/vcl/workben/vcldemo.cxx
+++ b/vcl/workben/vcldemo.cxx
@@ -31,6 +31,7 @@
#include <vcl/ptrstyle.hxx>
#include <salhelper/thread.hxx>
+#include <tools/diagnose_ex.h>
#include <tools/urlobj.hxx>
#include <tools/stream.hxx>
#include <vcl/svapp.hxx>
@@ -2376,9 +2377,9 @@ public:
xWidgets.disposeAndClear();
xPopup.disposeAndClear();
}
- catch (const css::uno::Exception& e)
+ catch (const css::uno::Exception&)
{
- SAL_WARN("vcl.app", "Fatal: " << e);
+ TOOLS_WARN_EXCEPTION("vcl.app", "Fatal");
return 1;
}
catch (const std::exception& e)
More information about the Libreoffice-commits
mailing list