[Libreoffice-commits] core.git: compilerplugins/clang ucb/source

Stephan Bergmann (via logerrit) logerrit at kemper.freedesktop.org
Thu Oct 29 15:28:14 UTC 2020


 compilerplugins/clang/toolslong.cxx        |    3 ++-
 ucb/source/ucp/webdav-neon/NeonSession.cxx |    5 ++---
 2 files changed, 4 insertions(+), 4 deletions(-)

New commits:
commit 13c0326d237cb4860b121ceba8ecb04fe96ae479
Author:     Stephan Bergmann <sbergman at redhat.com>
AuthorDate: Thu Oct 29 10:30:32 2020 +0100
Commit:     Stephan Bergmann <sbergman at redhat.com>
CommitDate: Thu Oct 29 16:27:26 2020 +0100

    ne_lock::timeout is of type long, so keep using that here
    
    ...partially reverting 4fbd63860500b2db76df4d5aedbe5e3aa31fac69 "switching long
    to a 64-bit type on 64-bit windows"
    
    Change-Id: I858630e5de76942a001ba619085ee07183df6ff6
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/104994
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sbergman at redhat.com>

diff --git a/compilerplugins/clang/toolslong.cxx b/compilerplugins/clang/toolslong.cxx
index 32e94f8397e5..719b5b0caca5 100644
--- a/compilerplugins/clang/toolslong.cxx
+++ b/compilerplugins/clang/toolslong.cxx
@@ -125,7 +125,8 @@ void ToolsLong::run()
 
     StringRef fn(handler.getMainFileName());
     // sberg says this is fine
-    if (loplugin::isSamePathname(fn, SRCDIR "/pyuno/source/module/pyuno.cxx"))
+    if (loplugin::isSamePathname(fn, SRCDIR "/pyuno/source/module/pyuno.cxx")
+        || loplugin::isSamePathname(fn, SRCDIR "/ucb/source/ucp/webdav-neon/NeonSession.cxx"))
         return;
 
     TraverseDecl(compiler.getASTContext().getTranslationUnitDecl());
diff --git a/ucb/source/ucp/webdav-neon/NeonSession.cxx b/ucb/source/ucp/webdav-neon/NeonSession.cxx
index 4e9012a990dc..b914f41cd903 100644
--- a/ucb/source/ucp/webdav-neon/NeonSession.cxx
+++ b/ucb/source/ucp/webdav-neon/NeonSession.cxx
@@ -34,7 +34,6 @@
 #include <osl/diagnose.h>
 #include <osl/thread.h>
 #include <osl/time.h>
-#include <tools/long.hxx>
 #include <ne_socket.h>
 #include <ne_auth.h>
 #include <ne_redirect.h>
@@ -1569,7 +1568,7 @@ void NeonSession::LOCK( const OUString & inPath,
     }
 
     // Set the lock timeout
-    theLock->timeout = static_cast<tools::Long>(rLock.Timeout);
+    theLock->timeout = static_cast<long>(rLock.Timeout);
 
     // Set the lock owner
     OUString aValue;
@@ -1629,7 +1628,7 @@ bool NeonSession::LOCK( NeonLock * pLock,
     // save the current requested timeout, because ne_lock_refresh uses
     // pLock->timeout as an out parameter. This prevents a feedback-loop,
     // where we would request a shorter timeout on each refresh.
-    tools::Long timeout = pLock->timeout;
+    long timeout = pLock->timeout;
     const int theRetVal = ne_lock_refresh(m_pHttpSession, pLock);
     if (theRetVal == NE_OK)
     {


More information about the Libreoffice-commits mailing list