Skip to content

Conversation

@rebroad
Copy link
Contributor

@rebroad rebroad commented Nov 11, 2016

Alternative to #9117 (and all the others that come after it!)...

@theuni
Copy link
Member

theuni commented Nov 11, 2016

It says right in the PR description for #9117 that it is meant to be an uncontroversial quick fix for a crash while the more complete fix is being worked on. That complete fix is included in #9128.

Dueling PR's really aren't helpful...

@maflcko
Copy link
Member

maflcko commented Nov 11, 2016

Agree with @theuni: We should focus on review of #9128. This seems to refactor the code in the wrong direction, closing for now.

@maflcko maflcko closed this Nov 11, 2016
@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.

3 participants