Skip to content

Conversation

@yyforyongyu
Copy link
Contributor

7539 is obsoleted by 8439, as mentioned in 7539.

@ysangkok
Copy link
Contributor

ysangkok commented Apr 6, 2020

This fixes #646

Copy link
Collaborator

@cdecker cdecker left a comment

Choose a reason for hiding this comment

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

ACK

Copy link
Collaborator

@t-bast t-bast left a comment

Choose a reason for hiding this comment

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

I've looked briefly at RFC 8439 and the errata filed against 7539, if I didn't miss anything there is no functional change, only clean-up and performance benchmarks added so this looks good!

Copy link
Collaborator

@rustyrussell rustyrussell left a comment

Choose a reason for hiding this comment

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

Ack 3a6039e

Copy link
Collaborator

@cfromknecht cfromknecht left a comment

Choose a reason for hiding this comment

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

LGTM

@t-bast t-bast merged commit 50b7391 into lightning:master Aug 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants