summaryrefslogtreecommitdiff
path: root/compilerplugins/clang/test/unnecessarylocking.cxx
diff options
context:
space:
mode:
Diffstat (limited to 'compilerplugins/clang/test/unnecessarylocking.cxx')
-rw-r--r--compilerplugins/clang/test/unnecessarylocking.cxx114
1 files changed, 114 insertions, 0 deletions
diff --git a/compilerplugins/clang/test/unnecessarylocking.cxx b/compilerplugins/clang/test/unnecessarylocking.cxx
new file mode 100644
index 000000000000..a2319623140a
--- /dev/null
+++ b/compilerplugins/clang/test/unnecessarylocking.cxx
@@ -0,0 +1,114 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4; fill-column: 100 -*- */
+/*
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ */
+
+#include <mutex>
+#include <osl/mutex.hxx>
+
+static std::mutex gSolarMutex;
+
+class SolarMutexGuard
+{
+ std::unique_lock<std::mutex> lock;
+
+public:
+ SolarMutexGuard()
+ : lock(gSolarMutex)
+ {
+ }
+};
+
+namespace test1
+{
+struct Foo
+{
+ int m_foo;
+ int bar1()
+ // expected-error@+1 {{unnecessary locking [loplugin:unnecessarylocking]}}
+ {
+ SolarMutexGuard guard;
+ return 1;
+ }
+ // no warning expected
+ int bar2()
+ {
+ SolarMutexGuard guard;
+ return m_foo;
+ }
+};
+}
+
+namespace test2
+{
+int free_function() { return 1; }
+
+struct Foo
+{
+ std::mutex m_aMutex;
+ osl::Mutex m_aOslMutex;
+ int m_foo;
+
+ int bar1()
+ // expected-error@+1 {{unnecessary locking [loplugin:unnecessarylocking]}}
+ {
+ std::unique_lock guard(m_aMutex);
+ return 1;
+ }
+ int bar2()
+ // expected-error@+1 {{unnecessary locking [loplugin:unnecessarylocking]}}
+ {
+ std::scoped_lock guard(m_aMutex);
+ return 1;
+ }
+ // no warning expected
+ int bar3()
+ {
+ std::scoped_lock guard(m_aMutex);
+ return m_foo;
+ }
+ int bar4()
+ // expected-error@+1 {{unnecessary locking [loplugin:unnecessarylocking]}}
+ {
+ ::osl::Guard<::osl::Mutex> aGuard(m_aOslMutex);
+ return 1;
+ }
+ int bar5()
+ {
+ // expected-error@+1 {{unnecessary locking [loplugin:unnecessarylocking]}}
+ {
+ std::unique_lock guard(m_aMutex);
+ return free_function();
+ }
+ }
+ osl::Mutex& getOslMutex() { return m_aOslMutex; }
+ int bar6()
+ // expected-error@+1 {{unnecessary locking [loplugin:unnecessarylocking]}}
+ {
+ ::osl::Guard<::osl::Mutex> aGuard(getOslMutex());
+ return 1;
+ }
+};
+}
+
+// Calling anything on VCLUnoHelper means we need the SolarMutex
+class VCLUnoHelper
+{
+public:
+ static int CreateToolkit();
+};
+namespace test4
+{
+// no warning expected
+void bar1()
+{
+ SolarMutexGuard guard;
+ VCLUnoHelper::CreateToolkit();
+}
+}
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab cinoptions=b1,g0,N-s cinkeys+=0=break: */