Dropped unused UniqueLock(Mutex*, ...) constructor that accepts a pointer to a mutex.
Also minor naming, comments and style improvements.
refactor: Drop unused `UniqueLock(Mutex*, ...)` constructor in sync.h #19913
pull hebasto wants to merge 2 commits into bitcoin:master from hebasto:200907-sync changing 1 files +6 −17-
hebasto commented at 10:00 PM on September 7, 2020: member
-
sync: Drop unused UniqueLock(Mutex*, ...) constructor 51269d198e
-
sync: Improve naming, comments, style bd77449604
- fanquake added the label Refactoring on Sep 7, 2020
- Maza1374 changes_requested
-
Maza1374 commented at 3:15 AM on September 8, 2020: none
.
- Maza1374 approved
-
MarcoFalke commented at 5:48 AM on September 8, 2020: member
This is useful when using a pointer as an optional alias to a mutex, no?
-
hebasto commented at 6:59 AM on September 8, 2020: member
This is useful when using a pointer as an optional alias to a mutex, no?
Correct. I'm using such aliases in my own experimental branches :facepalm:
Note to myself: add tests for this constructor.
- hebasto closed this on Sep 8, 2020
- DrahtBot locked this on Feb 15, 2022
Contributors
Labels