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

[Stephan Bergmann (via logerrit)]

 compilerplugins/clang/toolslong.cxx |   15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

New commits:
commit b4534d1ea4c0d36f7e5220729c959d0ad148e352
Author:     Stephan Bergmann <sbergman at redhat.com>
AuthorDate: Fri Nov 27 14:34:10 2020 +0100
Commit:     Stephan Bergmann <sbergman at redhat.com>
CommitDate: Sun Nov 29 17:45:13 2020 +0100

    loplugin:toolslong: Exclude some Windows-specific files
    
    ...where I consider long and unsigned long to be appropriate choices, for one
    reason or another (e.g., in combination with strtol)
    
    Change-Id: I5bfd19ab8f295152ce1436d9c0027d31118cda50
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106776
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sbergman at redhat.com>

diff --git a/compilerplugins/clang/toolslong.cxx b/compilerplugins/clang/toolslong.cxx
index fd3f2e2bd932..fbf9ceb38773 100644
--- a/compilerplugins/clang/toolslong.cxx
+++ b/compilerplugins/clang/toolslong.cxx
@@ -142,7 +142,20 @@ 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 "/avmedia/source/win/framegrabber.cxx")
+        || loplugin::isSamePathname(fn, SRCDIR "/avmedia/source/win/manager.cxx")
+        || loplugin::isSamePathname(fn, SRCDIR "/avmedia/source/win/player.cxx")
+        || loplugin::isSamePathname(fn, SRCDIR "/avmedia/source/win/window.cxx")
+        || loplugin::isSamePathname(fn, SRCDIR "/connectivity/source/drivers/ado/AStatement.cxx")
+        || loplugin::isSamePathname(fn, SRCDIR "/connectivity/source/drivers/ado/Awrapado.cxx")
+        || loplugin::isSamePathname(fn, SRCDIR "/desktop/win32/source/loader.cxx")
+        || loplugin::isSamePathname(fn, SRCDIR "/extensions/source/activex/SOActiveX.cxx")
+        || loplugin::isSamePathname(fn, SRCDIR "/pyuno/source/module/pyuno.cxx")
+        || loplugin::isSamePathname(fn, SRCDIR
+                                    "/setup_native/source/win32/customactions/sellang/sellang.cxx")
+        || loplugin::isSamePathname(fn, SRCDIR
+                                    "/shell/source/win32/shlxthandler/ooofilt/stream_helper.cxx")
+        || loplugin::isSamePathname(fn, SRCDIR "/shell/source/win32/zipfile/zipfile.cxx")
         || loplugin::isSamePathname(fn, SRCDIR "/ucb/source/ucp/webdav-neon/NeonSession.cxx"))
         return;
     // these are places where the external API is actually "long"