Page MenuHomePhabricator

Merge #11599: scripted-diff: Small locking rename
ClosedPublic

Authored by jasonbcox on May 8 2019, 19:11.

Details

Summary

190bf62be1214b072513c7fd7e01cc191723967c scripted-diff: Small locking rename (Russell Yanofsky)

Pull request description:

Call sync.h primitives "locks" and "mutexes" instead of "blocks" and "waitable critical sections" to match current coding conventions and c++11 standard names.

This PR does not rename the "CCriticalSection" class (though this could be done as a followup) because it's used everywhere and would swamp the other changes in this PR. Plain mutexes should mostly be preferred instead of recursive mutexes in new code anyway.

Tree-SHA512: 39b5b2be8f7a98227be8ab0648bdbb1b620944659bdc1eb9a15b0fcc0c930457fa0c03170cfedaeee0007ea716c526b31a8d84a86dd2333ce9d8bfabd773fe45

Backport of Core PR11599
https://github.com/bitcoin/bitcoin/pull/11599/files

Test Plan

make check
test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

jasonbcox created this revision.May 8 2019, 19:11
Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 8 2019, 19:11
jasonbcox edited the summary of this revision. (Show Details)May 9 2019, 00:16
markblundeberg accepted this revision.May 9 2019, 01:02
markblundeberg added a subscriber: markblundeberg.

For fun, I tried running the script and picked up some changes:

  • new { and } in sync.h -- I guess you added those manually.
  • the comment removed from sync.h leaves a residue of /** and */ (due to ABC/core commenting style changes).
This revision is now accepted and ready to land.May 9 2019, 01:02
Fabien accepted this revision.May 9 2019, 06:20
This revision was automatically updated to reflect the committed changes.