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

BOLT4: Correct hop count clarification #617

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@jtimon
Copy link
Contributor

commented May 30, 2019

The 20 hop count excludes the origin but includes the final node, right? thus it's 19 intermediate nodes but 20 channels.
This feels like a typo, please, ignore me if I'm wrong.

@t-bast

This comment has been minimized.

Copy link
Collaborator

commented Jun 4, 2019

You're right that the sentence is confusing, because hops refer to intermediate nodes:

The packet is routed through a number of intermediate nodes, called hops.

I think that it would be less confusing to say:

The longest route supported has 19 intermediate nodes and a maximum of 20 channels to be traversed.

@jtimon

This comment has been minimized.

Copy link
Contributor Author

commented Jun 6, 2019

I think that it would be less confusing to say:

The longest route supported has 19 intermediate nodes and a maximum of 20 channels to be traversed.

Yeah, I like your sentence better too, it's simpler.
Happy to change it to that if more people agree.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.