Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Apr 2, 2018

Some fixups for #11742:

  • Add release notes for the new rpc
  • Fix a typo in the original pull
  • Make the mempool reference passed to CheckInputsFromMempoolAndCache const, since that function is called before we return from ATMP and we must not modify the mempool.

@jnewbery
Copy link
Contributor

jnewbery commented Apr 2, 2018

utACK fafcad3

@promag
Copy link
Contributor

promag commented Apr 6, 2018

utACK fafcad3.

@laanwj
Copy link
Member

laanwj commented Apr 7, 2018

utACK fafcad3

@laanwj laanwj merged commit fafcad3 into bitcoin:master Apr 7, 2018
laanwj added a commit that referenced this pull request Apr 7, 2018
fafcad3 doc: Add testmempoolaccept to release-notes (MarcoFalke)

Pull request description:

  Some fixups for #11742:

  * Add release notes for the new rpc
  * Fix a typo in the original pull
  * Make the mempool reference passed to `CheckInputsFromMempoolAndCache` const, since that function is called before we return from ATMP and we must not modify the mempool.

Tree-SHA512: 72c459ba69f7698a69c91d2592f10f7fb1864846c7d8c525050d48286f92ba5ec5fe554c54235b52fbd9a8f00226c526ad84584641ec39084e1a1310a261510d
@maflcko maflcko deleted the Mf1804-docReleaseNotes branch April 7, 2018 17:40
UdjinM6 pushed a commit to UdjinM6/dash that referenced this pull request May 22, 2021
…ing const

fafcad3 doc: Add testmempoolaccept to release-notes (MarcoFalke)

Pull request description:

  Some fixups for bitcoin#11742:

  * Add release notes for the new rpc
  * Fix a typo in the original pull
  * Make the mempool reference passed to `CheckInputsFromMempoolAndCache` const, since that function is called before we return from ATMP and we must not modify the mempool.

Tree-SHA512: 72c459ba69f7698a69c91d2592f10f7fb1864846c7d8c525050d48286f92ba5ec5fe554c54235b52fbd9a8f00226c526ad84584641ec39084e1a1310a261510d
@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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants