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

[Noel Grandin (via logerrit)]

 dbaccess/source/ui/browser/formadapter.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 371a91aaae0500f60f73968ff90a3d61e21b6d1d
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Fri Nov 5 15:54:08 2021 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Fri Nov 5 17:49:29 2021 +0100

    Werror=nonnull in SbaXFormAdapter::cancel
    
    ever since
        commit 38d78d3c30183b11f5b643c147667b1d30325784
        Author: Ocke Janssen <o...@openoffice.org>
        Date:   Thu Oct 26 13:46:14 2000 +0000
        new datasource browser
    
    spotted by new gcc (Ubuntu 11.2.0-7ubuntu2) 11.2.0
    
    Change-Id: If054b949f4b4f93e785d315ddeb3423b52d1f55b
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/124759
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/dbaccess/source/ui/browser/formadapter.cxx 
b/dbaccess/source/ui/browser/formadapter.cxx
index 4b98babb26c9..a1e9b1f31ecb 100644
--- a/dbaccess/source/ui/browser/formadapter.cxx
+++ b/dbaccess/source/ui/browser/formadapter.cxx
@@ -1305,7 +1305,7 @@ 
IMPLEMENT_PROPERTY_LISTENER_ADMINISTRATION(SbaXFormAdapter, VetoableChangeListen
 void SAL_CALL SbaXFormAdapter::cancel()
 {
     Reference< css::util::XCancellable >  xCancel(m_xMainForm, UNO_QUERY);
-    if (xCancel.is())
+    if (!xCancel.is())
         return;
     xCancel->cancel();
 }