You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
So just rough thinking, the shouldAdd isn't needed here, you can simply continue the outer loop if id == node.Server this would practically split loc in half. Also, this can be moved into a new function for clarity.
The text was updated successfully, but these errors were encountered:
We already have a function which handles that, so I will use that instead. Sweet. Although, that function still uses the shouldAdd variable since whether or not to continue is decided in another loop.
https://github.com/Whiteblock/genesis/blob/dev/status/network.go#L34-L43
So just rough thinking, the
shouldAdd
isn't needed here, you can simplycontinue
the outer loop ifid == node.Server
this would practically split loc in half. Also, this can be moved into a new function for clarity.The text was updated successfully, but these errors were encountered: