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

rpc: Use enum instead of string for filtertype_name #25485

Merged
merged 1 commit into from
Jun 28, 2022

Conversation

w0xlt
Copy link
Contributor

@w0xlt w0xlt commented Jun 27, 2022

This PR changes the getblockfilter RPC to use BlockFilterType enum instead of a repeated string for filtertype_name.

Copy link
Member

@furszy furszy left a comment

Choose a reason for hiding this comment

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

ACK baf4efe

Copy link
Contributor

@brunoerg brunoerg left a comment

Choose a reason for hiding this comment

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

ACK baf4efe

@maflcko maflcko merged commit ee3ba5a into bitcoin:master Jun 28, 2022
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Jun 28, 2022
…name`

baf4efe rpc: use enum instead of string for filter type (w0xlt)

Pull request description:

  This PR changes the `getblockfilter` RPC to use `BlockFilterType` enum instead of a repeated string for `filtertype_name`.

ACKs for top commit:
  furszy:
    ACK baf4efe
  brunoerg:
    ACK baf4efe

Tree-SHA512: 31c79c0a5f0b17fd69b399bb026f523003b656733d6b7d5ffe665921a8cc0f1e0334d2e465145cd89fbd85e196059cf56f4f11563bbc92948b0606080ca76524
@w0xlt w0xlt deleted the filtertype_enum branch July 8, 2022 12:42
@bitcoin bitcoin locked and limited conversation to collaborators Jul 8, 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.

5 participants