Skip to content
This repository was archived by the owner on Jun 17, 2022. It is now read-only.

Remove get_int/get_int64 in favor of getInt<> #35

Merged
merged 1 commit into from
May 17, 2022
Merged

Remove get_int/get_int64 in favor of getInt<> #35

merged 1 commit into from
May 17, 2022

Conversation

maflcko
Copy link

@maflcko maflcko commented May 17, 2022

No need to have several methods which do the same thing.

Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

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

utACK 9372038

@maflcko
Copy link
Author

maflcko commented May 17, 2022

Thanks, fixed typo in the test

Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

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

ACK 462c503

@fanquake fanquake merged commit 0626532 into bitcoin-core:bitcoin-fork May 17, 2022
@maflcko maflcko deleted the 2205-int-strip-🙂 branch May 17, 2022 16:04
maflcko pushed a commit to maflcko/bitcoin-core that referenced this pull request May 30, 2022
de4f73ddca Merge bitcoin-core/univalue-subtree#36: Drop overloaded members
076c051488 Drop overloaded members
06265321de Merge bitcoin-core/univalue-subtree#35: Remove get_int/get_int64 in favor of getInt<>
462c503aa4 Remove get_int/get_int64 in favor of getInt<>
68c8f5532d Merge bitcoin-core/univalue-subtree#34: doc: remove TODO
297c53a5ee doc: remove TODO

git-subtree-dir: src/univalue
git-subtree-split: de4f73ddca40487179e9ed08c6f6aa745d6cbba3
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants