Skip to content

Squash two logic branches in SubmitDenominate into one#2270

Merged
UdjinM6 merged 1 commit into
dashpay:developfrom
UdjinM6:squashlogic
Sep 11, 2018
Merged

Squash two logic branches in SubmitDenominate into one#2270
UdjinM6 merged 1 commit into
dashpay:developfrom
UdjinM6:squashlogic

Conversation

@UdjinM6
Copy link
Copy Markdown

@UdjinM6 UdjinM6 commented Sep 7, 2018

Should not change the behaviour.

@UdjinM6
Copy link
Copy Markdown
Author

UdjinM6 commented Sep 9, 2018

Includes #2277 for now, will rebase after merge

@UdjinM6 UdjinM6 changed the title Squash two logic branches in SubmitDenominate into one [WIP] Squash two logic branches in SubmitDenominate into one Sep 9, 2018
@UdjinM6 UdjinM6 changed the title [WIP] Squash two logic branches in SubmitDenominate into one Squash two logic branches in SubmitDenominate into one Sep 11, 2018
@UdjinM6
Copy link
Copy Markdown
Author

UdjinM6 commented Sep 11, 2018

rebased, pls review

Copy link
Copy Markdown

@nmarley nmarley 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 28e0476 into dashpay:develop Sep 11, 2018
bool CPrivateSendClientSession::SubmitDenominate(CConnman& connman)
{
// This is just a local helper
auto GetStartRound = [](bool fMixLowest, bool fScanFromTheMiddle) -> int
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

FYI, the -> int is luckely not required in most cases when using lambdas. It will guess the type based on the return statements found inside the lambda.

CryptoCentric pushed a commit to absolute-community/absolute that referenced this pull request May 20, 2019
@UdjinM6 UdjinM6 deleted the squashlogic branch November 26, 2020 11:38
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.

3 participants