Opened 7 years ago

Closed 5 years ago

Last modified 4 years ago

#2474 closed patch (Invalid)

Replace add_torrent with async_add_torrent

Reported by: Doadin Owned by:
Priority: minor Milestone: 2.x
Component: Core Version: develop
Keywords: Cc:

Description

http://dev.deluge-torrent.org/wiki/Plan/libtorrent

Other potential changes Replace add_torrent with the more efficient non-blocking async_add_torrent.

Attachments (1)

Replace add_torrent with async_add_torrent.patch (4.1 KB) - added by Doadin 7 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 7 years ago by Cas

I'm afraid that patch is incorrect plus it is more complicated than simply renaming the method call to libtorrent.

From lt man:

In order to add torrents more efficiently, consider using async_add_torrent() which returns immediately, without waiting for the torrent to add. Notification of the torrent being added is sent as add_torrent_alert.

Last edited 7 years ago by Cas (previous) (diff)

comment:2 Changed 7 years ago by Doadin

I was just starting to work on it. Mostly just opened this because there isnt a ticket for this item yet? So more just a spot to work on it and add patches and what not so its easy to collaborate on the issue.

comment:3 Changed 7 years ago by Cas

I appreciate the ticket being opened but the patch is no good, clearly untested and changes code that is irrelevant to the issue.

comment:4 Changed 5 years ago by Doadin

  • Resolution set to Invalid
  • Status changed from new to closed

comment:5 Changed 4 years ago by Cas

  • Milestone changed from 2.0.x to 2.x

Milestone renamed

Note: See TracTickets for help on using tickets.