Skip to content

Coinjoin cs adjustments#4463

Merged
UdjinM6 merged 8 commits into
dashpay:developfrom
PastaPastaPasta:coinjoin-cs-adjustments
Sep 30, 2021
Merged

Coinjoin cs adjustments#4463
UdjinM6 merged 8 commits into
dashpay:developfrom
PastaPastaPasta:coinjoin-cs-adjustments

Conversation

@PastaPastaPasta
Copy link
Copy Markdown
Member

No description provided.

@PastaPastaPasta PastaPastaPasta added this to the 18 milestone Sep 26, 2021
Comment thread src/coinjoin/coinjoin-client.cpp
Comment thread src/coinjoin/coinjoin-client.cpp
Copy link
Copy Markdown

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@UdjinM6 UdjinM6 merged commit 7508790 into dashpay:develop Sep 30, 2021
@PastaPastaPasta PastaPastaPasta deleted the coinjoin-cs-adjustments branch September 30, 2021 19:06
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Jun 19, 2022
* Adjust coinjoin-client-options.* by making values atomic, removing CCriticalSection, and adjusting constructor to be default

* coinjoin-client.* adjust cs_deqsessions usage

* coinjoin add GUARDED_BY notations

* coinjoin make some values atomic

* coinjoin: move lock to where needed

* Ensure values are properly protected

* LOCK(cs_vecqueue) as needed

* Ensure vecOutputs is properly protected

Signed-off-by: pasta <pasta@dashboost.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants