diff options
| author | Jan-Marek Glogowski <glogow@fbihome.de> | 2020-12-05 12:42:11 +0100 |
|---|---|---|
| committer | Jan-Marek Glogowski <glogow@fbihome.de> | 2020-12-07 14:27:14 +0100 |
| commit | 84af20ef3ea72190784e9e7be820684c2558ba8c (patch) | |
| tree | b8de2bfc31e10c2d70f689e633c583223874bebf /include/vcl/scheduler.hxx | |
| parent | e4b702718e83c1ad76f001e4d3641cc2dd17cd7b (diff) | |
Make SchedulerMutex non-recursive
While thinking about the "Unlock scheduler in deinit for
ProcessEventsToIdle" patch, I came to the conclusion, that this
mutex should actually be non-recursive. I had a look at the code
and did run "make check" for my symbol build on Linux, but for
the rest I'm reying on the LO CI.
Maybe this can be converted to a std::mutex later. I've updated
the vcl/README.scheduler and added a TODO.
Change-Id: Ib9cb086af74b51e48f99ebfa1201d14db12b140e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/107254
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
Diffstat (limited to 'include/vcl/scheduler.hxx')
| -rw-r--r-- | include/vcl/scheduler.hxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/vcl/scheduler.hxx b/include/vcl/scheduler.hxx index 9318b2109641..7219146cbf39 100644 --- a/include/vcl/scheduler.hxx +++ b/include/vcl/scheduler.hxx @@ -36,8 +36,8 @@ class VCL_DLLPUBLIC Scheduler final static void ImplStartTimer ( sal_uInt64 nMS, bool bForce, sal_uInt64 nTime ); - static void Lock( sal_uInt32 nLockCount = 1 ); - static sal_uInt32 Unlock( bool bUnlockAll = false ); + static void Lock(); + static void Unlock(); public: static constexpr sal_uInt64 ImmediateTimeoutMs = 0; |