diff options
author | fanquake <fanquake@gmail.com> | 2023-12-13 09:50:27 +0000 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2023-12-13 10:06:16 +0000 |
commit | f48a789385add6df3582b2a957db6156f9f36f2b (patch) | |
tree | 06ed1b2d0849ffe58b85f68c45a8f3e22827b55c /share | |
parent | d646ca35d991e4f694096fdbd2d2ebd8cebf244e (diff) | |
parent | fa3da629a1aebcb4500803d7417feed8e34285b0 (diff) |
Merge bitcoin/bitcoin#28075: util: Remove DirIsWritable, GetUniquePath
fa3da629a1aebcb4500803d7417feed8e34285b0 Remove DirIsWritable, GetUniquePath (MarcoFalke)
fad3a9793b71df5bb0b17cc3758cf3466d08c015 Return LockResult::ErrorWrite in LockDirectory (MarcoFalke)
fa0afe740843c308f6287b923f1f4d758cf2a3f6 refactor: Return enum in LockDirectory (MarcoFalke)
Pull request description:
`GetUniquePath` is only used in tests and in `DirIsWritable`. The check by `DirIsWritable` is redundant with the check done in `LockDirectory`.
Fix the redundancy by removing everything, except `LockDirectory`.
ACKs for top commit:
TheCharlatan:
Re-ACK fa3da629a1aebcb4500803d7417feed8e34285b0
hebasto:
ACK fa3da629a1aebcb4500803d7417feed8e34285b0, I have reviewed the code and it looks OK.
Tree-SHA512: e95f18cd586de7582e9c08ac7ddb860bfcfcbc8963804f45c5784c5e4c0598dc59ae7e45dd4daf30a5020dbf8433f5db2ad06e46a8676371982003790043c6c9
Diffstat (limited to 'share')
0 files changed, 0 insertions, 0 deletions