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

tests: Remove unused mininode functions {ser,deser}_int_vector(...). Remove unused imports. #11683

Merged

Conversation

practicalswift
Copy link
Contributor

@practicalswift practicalswift commented Nov 14, 2017

  • Remove unused mininode functions deser_int_vector(f) and ser_int_vector(l). Last use removed in 3858aab. Friendly ping @jnewbery :-)
  • Remove unused imports.

@practicalswift practicalswift changed the title tests: Remove unused mininode functions deser_int_vector(f) and ser_int_vector(l) tests: Remove unused mininode functions {ser,deser}_int_vector(...). Remove unused imports. Nov 14, 2017
@fanquake fanquake added the Tests label Nov 14, 2017
@promag
Copy link
Member

promag commented Nov 14, 2017

utACK 5122755.

@jnewbery
Copy link
Contributor

Thanks @practicalswift . Looks good but conflicts with #11648. Do you mind reviewing that first to avoid a rebase?

@practicalswift
Copy link
Contributor Author

@jnewbery Sure!

Should I cherry-pick f9cd9b1 into this PR, or should I await merge of #11648?

@jnewbery
Copy link
Contributor

Thanks! I don't think you need to do anything for now. If #11648 can get merged soon, then I'd just wait for that and rebase this on top. If #11648 takes a while, then it's fine for this to be merged and I'll rebase #11648 on top of this.

@practicalswift
Copy link
Contributor Author

@jnewbery Great! 👍

@practicalswift
Copy link
Contributor Author

Rebased! :-)

@jnewbery
Copy link
Contributor

Tested ACK f522fb7

Copy link
Member

@promag promag left a comment

Choose a reason for hiding this comment

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

Tested ACK.

@@ -18,7 +18,7 @@
import time

from test_framework.test_framework import BitcoinTestFramework
from test_framework.util import sync_blocks, connect_nodes, assert_equal
from test_framework.util import connect_nodes, assert_equal
Copy link
Member

Choose a reason for hiding this comment

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

Do we sort here?

@maflcko
Copy link
Member

maflcko commented Nov 17, 2017

utACK f522fb7

1 similar comment
@fanquake
Copy link
Member

utACK f522fb7

@maflcko maflcko merged commit f522fb7 into bitcoin:master Nov 26, 2017
maflcko pushed a commit that referenced this pull request Nov 26, 2017
…_vector(...). Remove unused imports.

f522fb7 tests: Remove unused mininode functions deser_int_vector(f) and ser_int_vector(l) (practicalswift)
0f3b752 Remove unused imports (practicalswift)

Pull request description:

  * Remove unused mininode functions `deser_int_vector(f)` and `ser_int_vector(l)`. Last use removed in 3858aab. Friendly ping @jnewbery :-)
  * Remove unused imports.

Tree-SHA512: 840c5623eae9f929561f6e86816883c5904ec1af82fc8d5e56dee1c0b1fe22e8600c10f7358ed8b556b3aec8c65c4910f6eee30e8015a573c4df8bef91124d3e
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Apr 4, 2020
…er}_int_vector(...). Remove unused imports.

f522fb7 tests: Remove unused mininode functions deser_int_vector(f) and ser_int_vector(l) (practicalswift)
0f3b752 Remove unused imports (practicalswift)

Pull request description:

  * Remove unused mininode functions `deser_int_vector(f)` and `ser_int_vector(l)`. Last use removed in 3858aab. Friendly ping @jnewbery :-)
  * Remove unused imports.

Tree-SHA512: 840c5623eae9f929561f6e86816883c5904ec1af82fc8d5e56dee1c0b1fe22e8600c10f7358ed8b556b3aec8c65c4910f6eee30e8015a573c4df8bef91124d3e
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Apr 5, 2020
…er}_int_vector(...). Remove unused imports.

f522fb7 tests: Remove unused mininode functions deser_int_vector(f) and ser_int_vector(l) (practicalswift)
0f3b752 Remove unused imports (practicalswift)

Pull request description:

  * Remove unused mininode functions `deser_int_vector(f)` and `ser_int_vector(l)`. Last use removed in 3858aab. Friendly ping @jnewbery :-)
  * Remove unused imports.

Tree-SHA512: 840c5623eae9f929561f6e86816883c5904ec1af82fc8d5e56dee1c0b1fe22e8600c10f7358ed8b556b3aec8c65c4910f6eee30e8015a573c4df8bef91124d3e
@practicalswift practicalswift deleted the remove-int-vector-serialization branch April 10, 2021 19:33
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Feb 21, 2022
…er}_int_vector(...). Remove unused imports.

f522fb7 tests: Remove unused mininode functions deser_int_vector(f) and ser_int_vector(l) (practicalswift)
0f3b752 Remove unused imports (practicalswift)

Pull request description:

  * Remove unused mininode functions `deser_int_vector(f)` and `ser_int_vector(l)`. Last use removed in 3858aab. Friendly ping @jnewbery :-)
  * Remove unused imports.

Tree-SHA512: 840c5623eae9f929561f6e86816883c5904ec1af82fc8d5e56dee1c0b1fe22e8600c10f7358ed8b556b3aec8c65c4910f6eee30e8015a573c4df8bef91124d3e
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants