There is no reason the make it synchronous. Also avoid holding a lock when calling the callback for good measure.
Details
Details
- Reviewers
PiRK - Group Reviewers
Restricted Project - Commits
- rABC7154ba1d5668: [avalanche] Make the BlockFinalized validation callback asynchronous
ninja all check-all
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| src/validation.cpp | ||
|---|---|---|
| 3706 ↗ | (On Diff #44307) | what do these brackets do? |
| src/validation.cpp | ||
|---|---|---|
| 3706 ↗ | (On Diff #44307) | It's C++'s way of defining a scope, basically before the LOCK would go all the way to the return true (and including the BlockFinalized call) and would then be released, but with the scope it only goes to the closing }. |