diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2017-09-28 13:59:13 +0200 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2017-09-30 09:30:03 +0200 |
commit | 0d43f5176d34145b0965fdb2536cd6f454e985c5 (patch) | |
tree | cdce3fff899e08e9aa4aa405213b23fd5a4da7f6 /compilerplugins | |
parent | 736a4e659cff2ef5f900301d7d288494f9b98178 (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.cxx | 2 |
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" |