Skip to content

Remove no longer used mempool.exists(outpoint) - #10684

Merged
laanwj merged 1 commit into
bitcoin:masterfrom
morcos:lessHaveCoin
Jun 28, 2017
Merged

Remove no longer used mempool.exists(outpoint)#10684
laanwj merged 1 commit into
bitcoin:masterfrom
morcos:lessHaveCoin

Conversation

@morcos

@morcos morcos commented Jun 27, 2017

Copy link
Copy Markdown
Contributor

@sipa feel free to close if you prefer to leave this in for future work, but this function's use was removed by #10581

@sipa

sipa commented Jun 27, 2017

Copy link
Copy Markdown
Member

ACK, can always be added back later if needed.

@gmaxwell gmaxwell left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

utACK

@sdaftuar

Copy link
Copy Markdown
Member

utACK

@fanquake

Copy link
Copy Markdown
Member

utACK 22378ad

@TheBlueMatt

TheBlueMatt commented Jun 28, 2017

Copy link
Copy Markdown
Contributor

All aboard the obvious-ack train before it gets merged!

@laanwj

laanwj commented Jun 28, 2017

Copy link
Copy Markdown
Member

obvious utACK 22378ad

@laanwj
laanwj merged commit 22378ad into bitcoin:master Jun 28, 2017
laanwj added a commit that referenced this pull request Jun 28, 2017
22378ad Remove no longer used mempool.exists(outpoint) (Alex Morcos)

Tree-SHA512: e98ffdf9039a1bdd84a388dc2da43221b8850bab413dffbd18c20fb06c791ff364f824d1b3b7c0a696987ae126e52b4ee200bb63c5c46e9782c1973c6d888d32
@morcos
morcos deleted the lessHaveCoin branch June 28, 2017 09:32
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 17, 2019
22378ad Remove no longer used mempool.exists(outpoint) (Alex Morcos)

Tree-SHA512: e98ffdf9039a1bdd84a388dc2da43221b8850bab413dffbd18c20fb06c791ff364f824d1b3b7c0a696987ae126e52b4ee200bb63c5c46e9782c1973c6d888d32
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants