[Complex Relayers] Deduplicate l2r and r2l logic#1535
Conversation
Signed-off-by: Serban Iorga <serban@parity.io>
svyatonik
left a comment
svyatonik
left a comment
There was a problem hiding this comment.
Looks great, thank you! One thing that is confusing me is l2r and r2l :) When I've got PR notification I didn't even understand what this PR is about (relayer-to-?). IMO left_to_right and right_to_left gives more context when you look at the code, even though it is longer. But again - it may be my personal preference, so don't consider it as a requirement
Makes sense. I wanted to shorten the names a bit, but readability is more important. Done. |
* [Complex Relayers] Deduplicate l2r and r2l logic Signed-off-by: Serban Iorga <serban@parity.io> * Renamings
* [Complex Relayers] Deduplicate l2r and r2l logic Signed-off-by: Serban Iorga <serban@parity.io> * Renamings
2 participants
No description provided.