harmony-dev / beacon-chain-java

Java implementation of Ethereum 2.0 Beacon Chain
Apache License 2.0
34 stars 9 forks source link

Refactor method name to distinguish by parameter name #210

Closed shahankhatch closed 4 years ago

shahankhatch commented 4 years ago

This refactor is to make it clearer when magic is applied to the first parameter in WhoAreYouPacket. For both create methods, I thought the Bytes and Bytes32 types of the first parameter were too closely related, making it hard for me to distinguish when generalizing types (integration of this discv5 into Artemis uses Tuweni Bytes).

zilm13 commented 4 years ago

Cool, I like it