diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2022-11-04 09:47:05 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2022-11-04 13:14:47 +0100 |
commit | 493c88ab4e597c55e920dbec2262058ecc6b6922 (patch) | |
tree | d56f426ca7836ed51d4cfb7343847f138072eb7d /dbaccess/source/ui/uno | |
parent | b1671feb791178f9c498b0d00a5aad486bccfa03 (diff) |
calling getCaughtException twice in one catch block is not allowed
It seems to work, but it's not guaranteed to do so, and violates the
documented contract.
Regression from
commit 85a5233275ab83fe0d6b943b298695387d97accd
Author: Noel Grandin <noel.grandin@collabora.co.uk>
Date: Mon Oct 31 12:08:50 2022 +0200
error when copying table and the copy table wizard creates the
primary key
Change-Id: Ib8422017e627e825865c96dd9fbacbe0e518e96b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/142264
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'dbaccess/source/ui/uno')
-rw-r--r-- | dbaccess/source/ui/uno/copytablewizard.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/dbaccess/source/ui/uno/copytablewizard.cxx b/dbaccess/source/ui/uno/copytablewizard.cxx index 5fd10ee37e75..fab5e3966cbd 100644 --- a/dbaccess/source/ui/uno/copytablewizard.cxx +++ b/dbaccess/source/ui/uno/copytablewizard.cxx @@ -1416,8 +1416,8 @@ void CopyTableWizard::impl_doCopy_nothrow() } catch( const Exception& ) { - TOOLS_WARN_EXCEPTION("dbaccess", ""); aError = ::cppu::getCaughtException(); + SAL_WARN("dbaccess", exceptionToString(aError)); // silence the error of the user cancelling the parameter's dialog SQLException aSQLError; |