Skip to content

Conversation

cfromknecht
Copy link
Contributor

In light of #2570, this PR ensures that the local names of wtwire feature bits are uniform between even/odd features. This ensures that Has properly returns true if the feature vector supports optional or required for a given feature bit

@cfromknecht cfromknecht changed the title watchtower/wtwire/features: make wtwire feature uniform watchtower/wtwire/features: make wtwire features uniform Feb 7, 2019
@cfromknecht cfromknecht added this to the 0.6 milestone Feb 7, 2019
Copy link
Member

@Roasbeef Roasbeef left a comment

Choose a reason for hiding this comment

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

LGTM 💥

Super subtle thing in the API derived from the lnwire package. We may want to consider adding an init() method that runs a sanity check over the specified values.

@Roasbeef Roasbeef merged commit 16d69e8 into lightningnetwork:master Feb 9, 2019
@cfromknecht cfromknecht deleted the wtwire-uniform-feature-names branch February 9, 2019 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants