summaryrefslogtreecommitdiff
path: root/compilerplugins
diff options
context:
space:
mode:
authorStephan Bergmann <sbergman@redhat.com>2017-09-28 13:59:13 +0200
committerStephan Bergmann <sbergman@redhat.com>2017-09-30 09:30:03 +0200
commit0d43f5176d34145b0965fdb2536cd6f454e985c5 (patch)
treecdce3fff899e08e9aa4aa405213b23fd5a4da7f6 /compilerplugins
parent736a4e659cff2ef5f900301d7d288494f9b98178 (diff)
Access to ProcessEventsToSignal's bSignal is racy
...so spell out its single use, locking the appropriate mutex around the access Change-Id: I8e8f47de1979f5a80cf1ad65e5ec24d25145c463 Reviewed-on: https://gerrit.libreoffice.org/42908 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'compilerplugins')
-rw-r--r--compilerplugins/clang/constparams.cxx2
1 files changed, 0 insertions, 2 deletions
diff --git a/compilerplugins/clang/constparams.cxx b/compilerplugins/clang/constparams.cxx
index 024c4d8e3430..62300f4d5eb2 100644
--- a/compilerplugins/clang/constparams.cxx
+++ b/compilerplugins/clang/constparams.cxx
@@ -181,8 +181,6 @@ bool ConstParams::VisitFunctionDecl(const FunctionDecl * functionDecl)
|| name == "convert_slashes"
// UNO component entry points
|| name.endswith("component_getFactory")
- // in Scheduler::, wants to loop until a reference to a bool becomes true
- || name == "ProcessEventsToSignal"
// external API
|| name == "Java_com_sun_star_sdbcx_comp_hsqldb_StorageNativeOutputStream_flush"
|| name == "egiGraphicExport"