Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove unused dummy_tx variable from FillPSBT #13962

Merged
merged 1 commit into from Aug 14, 2018

Conversation

@dongcarl
Copy link
Contributor

@dongcarl dongcarl commented Aug 14, 2018

No description provided.

@achow101
Copy link
Member

@achow101 achow101 commented Aug 14, 2018

ACK 16bcc1b

grep'd for dummy_tx and it was only used here.

@domob1812
Copy link
Contributor

@domob1812 domob1812 commented Aug 14, 2018

utACK 16bcc1b - nice find!

@Empact
Copy link
Member

@Empact Empact commented Aug 14, 2018

utACK 16bcc1b introduced in a4b06fb and never used

ken2812221 pushed a commit to ken2812221/bitcoin that referenced this pull request Aug 14, 2018
16bcc1b Remove unused dummy_tx variable from FillPSBT (Carl Dong)

Pull request description:

Tree-SHA512: e7652126bca2e87d445d4190aa0b4192d9575bc0c280d063302ca420be51e7a04fcbc24e0e7f5ec1f18938f0a596901e1285e9afc9b33ca3da78177938791163
@laanwj laanwj merged commit 16bcc1b into bitcoin:master Aug 14, 2018
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@laanwj
Copy link
Member

@laanwj laanwj commented Aug 14, 2018

utACK 16bcc1b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
You can’t perform that action at this time.