-
Notifications
You must be signed in to change notification settings - Fork 447
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
RestAPI start download endpoint #2464
Conversation
Now, only the starting of the DHT is executed on the thread pool. The check itself is executed on the reactor thread.
40496b3
to
03d27fc
Compare
b848e9f
to
5fdcbb1
Compare
@lfdversluis Would you review this please? |
@@ -304,7 +302,11 @@ def network_create_engine_wrapper(self, pstate, initialdlstatus=None, checkpoint | |||
|
|||
self.handle = self.ltmgr.add_torrent(self, atp) | |||
|
|||
if self.handle: | |||
# DEBUG: |
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.
Don't forget to remove this debug statement if it's stable ;)
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.
Yes, I would like to invite @ardhipoetra to use this in his PR #2399.
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.
noted! :)
@brussee good work 👍 |
@devos50 Thanks! You too! |
@@ -185,20 +200,6 @@ def test_get_share_mode(self): | |||
self.libtorrent_download_impl.handle.status().share_mode = True | |||
self.assertTrue(self.libtorrent_download_impl.get_share_mode()) | |||
|
|||
def test_set_share_mode(self): |
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.
are you deleting this test as well as the wrapper function?
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 will put it back, but use at own risk!
@whirm Ready. |
This problematic wrapper function is now unused: (a flag is used instead by @devos50)