status-im
/
nim-eth
mirror of https://github.com/status-im/nim-eth.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Small clarification to protocol name length doc 

As per conversation in PR
This commit is contained in:
Bruno Škvorc 2019-08-20 16:29:40 +02:00 committed by zah
parent 843f9c3916
commit f3aad8f925
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
eth/p2p/rlpx.nim
View File

@ -583,7 +583,8 @@ proc p2pProtocolBackendImpl*(protocol: P2PProtocol): Backend =
isSubprotocol = protocol.version > 0
if protocol.rlpxName.len == 0: protocol.rlpxName = protocol.name
# By convention, all Ethereum protocol names must be abbreviated to 3 letters
# By convention, all Ethereum protocol names were abbreviated to 3 letters,
# but this informal spec has since been relaxed (e.g. `hive`).
doAssert protocol.rlpxName.len > 2
new result