protocol/state: replace state.Output with bc.OutputID #535

Merged
merged 3 commits into from Feb 16, 2017

Projects

None yet

3 participants

@bobg
Contributor
bobg commented Feb 15, 2017

Turns out we no longer need to carry around the TxOutput.

The indexer was using the TxOutput in a state.Output, but it can get that same info elsewhere, and now does.

@bobg
Contributor
bobg commented Feb 15, 2017

PTAL

@bobg bobg added the PTAL label Feb 15, 2017
@bobg
Contributor
bobg commented Feb 16, 2017

PTAL

@kr
Member
kr commented Feb 16, 2017

👀

@kr
kr approved these changes Feb 16, 2017 View changes
@kr
Member
kr commented Feb 16, 2017

LGTM

bobg added some commits Feb 15, 2017
@bobg @chainbot bobg Looks like we don't need a TxOutput in a state.Output. 68befc5
@bobg @chainbot bobg Propagate changes to some tests. 6465ec3
@bobg @chainbot bobg Remove state.Output altogether.
a7fda5a
@chainbot chainbot merged commit bfe08f2 into main Feb 16, 2017

3 checks passed

licence/cla Contributor License Agreement is signed.
Details
wercker/cored Wercker pipeline passed
Details
wercker/java Wercker pipeline passed
Details
@chainbot chainbot deleted the simpler-state branch Feb 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment