Skip to content

Conversation

fanquake
Copy link
Member

I've cherry-picked some of the commits out of #22924, and made minor changes (like fixing named args).

@DrahtBot
Copy link
Contributor

DrahtBot commented Mar 25, 2022

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #24661 (refactor: Use clang-tidy syntax for C++ named arguments by fanquake)
  • #23599 (Tidy up RPCTxSerializationFlags by MarcoFalke)
  • #23319 (rpc: Return fee and prevout (utxos) to getrawtransaction by dougEfresh)
  • #21283 (Implement BIP 370 PSBTv2 by achow101)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

Copy link
Member

@maflcko maflcko left a comment

Choose a reason for hiding this comment

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

review ACK 9aad284 😼

Show signature

Signature:

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

review ACK 9aad2844aa7047121e72cb0bd416cc14101e2b7c 😼
-----BEGIN PGP SIGNATURE-----

iQGzBAEBCgAdFiEE+rVPoUahrI9sLGYTzit1aX5ppUgFAlwqrYAACgkQzit1aX5p
pUigxAwAuyqNsS2+EPh36vJ7QjbJzWcX1/55TU4yjULBwisAqfcoU06nfp5nmlWB
lJ7y4iJJ/ZgcS/XP5Lf3yCKz6KmnViT1T6CGrEVsIFnZQ70cyzMm+Ikt2NkeDZE5
f8QqOW6wucqWC4kt8Cu9557cuwYQy7lkk7fZdNcOQhN8PC0J2MKp6VOf3Q63cQYF
HJmM5cXeo0e5+O+k+aAKABxOEjV5dJH8dKXZfPVgb8jeZuHlm1PTvRsIulWYWPRg
WbZXkRcN/+v0EEetVa8IhYV4tWpoiWzHXrGLnTo4MzDttsL7PFVKba3Pv01iXflx
eY+wbE6W4wwYfTn9S+UwHFA5MolNeuwMnC2N1MWuI6HBMqQ2SilP1zbb5bX2QZby
VLiJ4UB8pMDWqY2mBsDEfGKscgdOR6qK38zH8iFmfWIwKphFT5sRzTHR2f3rALmN
u3NnyMcmn+YvWyDgyTWC/e2g2TQeIAHeux9wQHrcyAXqvOw9qtTab5EE1FexLk/T
JdbYgGFz
=/ATi
-----END PGP SIGNATURE-----

@maflcko
Copy link
Member

maflcko commented Mar 31, 2022

re-ACK 9563a64 🕓

Show signature

Signature:

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

re-ACK 9563a645c22a455da3d2d305ed0eef4266b1d322 🕓
-----BEGIN PGP SIGNATURE-----

iQGzBAEBCgAdFiEE+rVPoUahrI9sLGYTzit1aX5ppUgFAlwqrYAACgkQzit1aX5p
pUjyqQv/WChwsfIm2X3iwc5IJ6TYwiH81nmMk6ELbpIwzjD3uqWS3ordwelmW9jH
MbCxAv44LJ4IBG6XzFXKBKwuHCmuky7/5EfmBVlX/H6ztnamQMXDbYswpYpQiIq+
HXu8f2pDe/+U025h36rALlmc9OuCBQWlKmBzktnOhCaSg4LD+72qqQQ+B8Avr6DE
1tv/s0u1/5lC8DvsjENGJi+ah6fO87qp2n5+deFFEBpQt40EmHbVJcHM+suDvP3d
qIBomRIYP5OxGB1oQ4xcq5+7/qpgdxh/elN9Q1diaDWuCmYQ46Omnft8yvWu4YSS
0fLoFkXztAMYgAd4yrgs8OUQAN4DdSc3xBuYwtaxmhHvFl+wnWUhScFE7/bTtjRb
RlrEljbFIYjJASQzETp9WHhgfMRq6ARaJc+ulS0+g1UAMNg3PX8taSkX7YmJaN2l
Q4JO2N/Rw0feV94pOKNyOByhkwieAz7izabBeahq/Yr+2iJzGKSrxzOUar5z9G0K
KjIqtwNr
=O5zM
-----END PGP SIGNATURE-----

@maflcko maflcko merged commit a2e1590 into bitcoin:master Mar 31, 2022
@fanquake fanquake deleted the 22924_rebased branch March 31, 2022 07:03
@mjdietzx
Copy link
Contributor

Post merge review ACK 9563a64

sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Apr 3, 2022
@bitcoin bitcoin locked and limited conversation to collaborators Mar 31, 2023
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.

4 participants