-
Notifications
You must be signed in to change notification settings - Fork 578
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
Moved Multiqueue, invQueue and addrQueue to network module #2256
Conversation
anand-skss
commented
Jun 23, 2024
•
edited
Loading
edited
- Moved Multiqueue, invQueue and addrQueue to network module
cda064e
to
36ecc83
Compare
@@ -7,7 +7,7 @@ | |||
# magic imports! | |||
import connectionpool | |||
from protocol import assembleAddrMessage | |||
from queues import addrQueue # FIXME: init with queue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please check Fixme comment, i'm not sure whether i remove this comment or not.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure either, let's leave it as it is.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok, just next time I'd put the helper_random -> random change into a separate PR.
@@ -7,7 +7,7 @@ | |||
# magic imports! | |||
import connectionpool | |||
from protocol import assembleAddrMessage | |||
from queues import addrQueue # FIXME: init with queue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure either, let's leave it as it is.