diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2020-06-08 07:03:36 -0400 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2020-06-08 07:04:27 -0400 |
| commit | 374fd6fc8b4c6539173d5fa0008266aafdb6c668 (patch) | |
| tree | 961e9b27e183d3b2d84bf1f9c79486bc657a170f /src/sync.cpp | |
| parent | 8496dbeba6878d4a123995170a0ad8d125c17695 (diff) | |
| parent | cc5c0d2299b09c58cd9962ca5075ffa53f2633c0 (diff) | |
Merge #19189: refactor: Replace RecursiveMutex with Mutex in timedata.cpp
cc5c0d2299b09c58cd9962ca5075ffa53f2633c0 refactor: Fix formatting of timedata.cpp (Hennadii Stepanov)
c2410ceb844a443caf6dd8c6df976b9e24724d06 refactor: Replace RecursiveMutex with Mutex in timedata.cpp (Hennadii Stepanov)
Pull request description:
Only `GetTimeOffset()` and `AddTimeData()` functions lock this mutex. They do not call itself recursively, and do not call each other either directly or indirectly. Therefore, the `g_timeoffset_mutex` could be a non-recursive mutex.
Related to #19180.
ACKs for top commit:
MarcoFalke:
ACK cc5c0d2299b09c58cd9962ca5075ffa53f2633c0 , checked the second commit with --word-diff-regex=. --ignore-all-space -U0 🦉
vasild:
ACK cc5c0d22 verified that recursion is not happening
Tree-SHA512: 38f6df689374d4a1a0e9aedb3ed5e885d8285c4da6b75f9bc84ae036936a159ef8276462db33b4f4dd5c71c6312fa9b45380f7a5726959665bc71dc39031be88
Diffstat (limited to 'src/sync.cpp')
0 files changed, 0 insertions, 0 deletions