[Debtorrent-commits] r32 - in /debtorrent/trunk: DebTorrent/BT1/Rerequester.py DebTorrent/BT1/StorageWrapper.py DebTorrent/download_bt1.py TODO

[camrdale-guest at users.alioth.debian.org]

Author: camrdale-guest
Date: Thu May  3 03:21:30 2007
New Revision: 32

URL: http://svn.debian.org/wsvn/debtorrent/?sc=1&rev=32
Log:
Cleaned up TODO stuff

Modified:
    debtorrent/trunk/DebTorrent/BT1/Rerequester.py
    debtorrent/trunk/DebTorrent/BT1/StorageWrapper.py
    debtorrent/trunk/DebTorrent/download_bt1.py
    debtorrent/trunk/TODO

Modified: debtorrent/trunk/DebTorrent/BT1/Rerequester.py
URL: http://svn.debian.org/wsvn/debtorrent/debtorrent/trunk/DebTorrent/BT1/Rerequester.py?rev=32&op=diff
==============================================================================
--- debtorrent/trunk/DebTorrent/BT1/Rerequester.py (original)
+++ debtorrent/trunk/DebTorrent/BT1/Rerequester.py Thu May  3 03:21:30 2007
@@ -163,7 +163,6 @@
             self.announce(event)
 
     def announce(self, event = 3, callback = lambda: None, specialurl = None):
-        # TODO: think about re-enabling compact peer lists once testing is complete
         if specialurl is not None:
             s = self.url+'&uploaded=0&downloaded=0&left=1'   # don't add to statistics
             if self.howmany() >= self.maxpeers:

Modified: debtorrent/trunk/DebTorrent/BT1/StorageWrapper.py
URL: http://svn.debian.org/wsvn/debtorrent/debtorrent/trunk/DebTorrent/BT1/StorageWrapper.py?rev=32&op=diff
==============================================================================
--- debtorrent/trunk/DebTorrent/BT1/StorageWrapper.py (original)
+++ debtorrent/trunk/DebTorrent/BT1/StorageWrapper.py Thu May  3 03:21:30 2007
@@ -281,7 +281,6 @@
             s = sh.digest()
             if s == self.hashes[i]:
                 self._markgot(i, i)
-# TODO: Determine if this moving of pieces (for 'normal' allocation) is needed
 #            elif ( self.check_targets.get(s)
 #                   and self._piecelen(i) == self._piecelen(self.check_targets[s][-1]) ):
 #                self._markgot(self.check_targets[s].pop(), i)
@@ -354,7 +353,6 @@
         if not self.holes:
             return False
         self.numholes = float(len(self.holes))
-        # TODO: determine if this is a huge waste of memory (160MB)
         self.alloc_buf = chr(0xFF) * max(self.piece_sizes)
         if self.alloc_type == 'pre-allocate':
             self.bgalloc_enabled = True

Modified: debtorrent/trunk/DebTorrent/download_bt1.py
URL: http://svn.debian.org/wsvn/debtorrent/debtorrent/trunk/DebTorrent/download_bt1.py?rev=32&op=diff
==============================================================================
--- debtorrent/trunk/DebTorrent/download_bt1.py (original)
+++ debtorrent/trunk/DebTorrent/download_bt1.py Thu May  3 03:21:30 2007
@@ -317,7 +317,6 @@
 
     return response
 
-# TODO: make this function work
 def get_packages(file, url, errorfunc):
     try:
         if file:

Modified: debtorrent/trunk/TODO
URL: http://svn.debian.org/wsvn/debtorrent/debtorrent/trunk/TODO?rev=32&op=diff
==============================================================================
--- debtorrent/trunk/TODO (original)
+++ debtorrent/trunk/TODO Thu May  3 03:21:30 2007
@@ -12,7 +12,8 @@
 between them like it was previously. This may not be an issue after a maximum 
 piece size is introduced, though pre-allocation may still be necessary to serve 
 downloaded files while other downloads continue. Pre-allocation with priorities 
-enabled does not pre-allocate the entire archive.
+enabled does not pre-allocate the entire archive. Later, when multiple pieces per 
+file are used, an allocation method could be used within a file.
 
 
 Comments
@@ -87,3 +88,12 @@
 Packages files every week/day/hour. Might need to parse the 
 /var/cache/apt/archive directory when run the first time to hardlink in files 
 that have already been downloaded.
+
+
+Re-enable compact peer lists
+
+Compact peer lists have been disbaled, both in the tracker and client, to 
+enable DNS names to work. This is currently needed because the host that the 
+tracker and seed are being tested on is behind a Firewall/NAT, and so the DNS 
+names are needed. This will not be aproble when a more permanent home for the 
+tracker is found.