Remove debugging messages.

Implement outgoing uTP connections.

Export uTP send_to callback.

Fix peerIoReconnect for uTP sockets.

Implement pacing of reads.

This should cause uTP sockets to respect read bandwidth limits.  I'm not so
sure about the values we return for the read buffer size -- perhaps we
should allow some slack for network latency?

Maintain information about enabled/disabled I/O.

Set the read buffer of UTP sockets to a well-defined value.

Compute accurate overhead for uTP packets.

Push testig for UTP enabled further down.

There's no need to test for DHT/uTP being enabled in tr-udp.  The DHT
will silently discard any packets directed at the wrong session (see the
beginning of dhtCallback).  As to uTP, we need to grok uTP packets
to close any remaining connections after we disabled uTP, so it's better
to participate in uTP, just reject any incoming connections.

Fix typo.

make a note that we need to implement UTP_GetGlobalUTPBytesSent in libtransmission

(trunk T) add preliminary enable/disable toggles in daemon, gtk, and qt clients. 'preliminary' because this may be widened to a tri-state of inabled, disabled, and incoming only

(trunk libT) add C and RPC API for getting/setting uTP enabled flag

(trunk libT) silence compiler warnings

automake/autoconf; #includes in libtransmission

import libutp into third-party/ and plug it in to autoconf, automake

Propagate uTP and ut_holepunch information over PEX.

Use dummy callbacks in peerIoReconnect.

Tweak dummy callbacks.

Don't abort, just log a message and continue.

Don't call UTP_Close multiple times.

Greg says it's evil.

(gtk) Add tooltip for uTP peers.

Shutdown UTP socket on short write.

Schedule a uTP write in peerIoWriteBuf.

Complete the uTP callbacks.

Add dummy UTP callbacks.

Libutp will sometimes call our callbacks after we called UTP_Close,
notably to notify us of the UTP_STATE_DESTROYING state change, but
also, for some reason, to ask us about our read buffer.  The simplest
way to avoid issues with that is to switch to a second set of callbacks.

Add assertions to libevent callbacks.

They should never be called with a UTP socket.

First pass at uTP callbacks.

Update tr-utp.c to be compatible with libevent2.

Fix io_dtor for uTP sockets.

Check for UTP timeouts every 50ms or so.

Greg says that works for them.

Accept UTP connexions.

Modify peer-io to work with UTP sockets.

This is not supposed to work yet -- it just adds new fields to the data
structures to get the UTP code to compile.

Fix incorrect use of tr_timerAdd.

Participate in UTP.

This adds code to participate in the UTP protocol, but without doing anything
useful yet -- we just shut down immediately any incoming connexion request.

(trunk libT) #3767 "rarest first policy" -- disable the assertReplicationCountIsExact() assertion

(trunk libT) #3767 "rarest first policy" -- possible fix for assertion error reported by rolcol

Resolve the problem described @ https://trac.transmissionbt.com/ticket/3767#comment:38

(trunk libT) #3767 "rarest first policy" -- remove some debugging code.

My initial thinking was to let nightly users test this out, but it's not necessary.

(trunk libT) #3767 "rarest first policy" -- fixed.

This commit, started by a patch from athy, implements a rarest first policy when deciding which pieces to request from peers. It keeps a count of how many peers have each piece, and updates the count when getting bitfields, have, have all, and have none messages, as well as decrementing the counts when peers disconnect.

This running total is generated only for downloading torrents. Seeds don't have this overhead.

(trunk) #4032 "Better error detection / reporting in http announces" -- added to trunk.

This patch adds two new flags to the callback function -- did_connect and did_timeout -- that are calculated inside of web.c using information from libcurl. This allows the announcer to detect timeouts more accurately and also to distinguish between unresponsive peers (which get the preexisting "Tracker did not respond" error message) and unconnectable peers (which get a new error message, "Could not connect to tracker").

revert r11894

(trunk dht) apply patch from jch to increase the number of token_bucket_tokens.

Make the initial DHT bootstrap more persistent.

We now try to contact the bootstrap nodes up to six times.
A better solution might be to reattempt bootstrap every half hour
or so.  This might be beneficial to people whose connectivity
changes while Transmission is running.

(trunk libT) #3656 "endgame could be faster" -- fixed. Patch by harrydb.

(trunk gtk) #3866 "popup menu for file list manipulation" -- remove unused code in file-list.c

This code can be pulled back out of svn and readded if it's needed in the future.

(trunk) make tr_torrentName() a public function.

This has been a private function in libtransmission for awhile now but it makes more sense as a public function.

adjust the spacing between the "Add" and "Cancel" buttons in the Add windows

(trunk libT) #4016 "blocklists don't handle overlapping/unsorted rules" -- fixed.

(trunk) #4022 "please update .gitignore" -- done.

rename "Clear Completed" to "Cleanup Completed"

remove selected transfers when removing completed transfers, removing through RPC, etc. (whenever removing transfers that aren't the selected transfers)

Add option to remove completed transfers

(trunk gtk) #4012 "Cannot set individual seeding time in GTK+ client" -- fixed.

The code in gtk/details.c was still using the pre-release RPC key "seedInactiveLimit" when it should have been using "seedIdleLimit" instead.

(trunk libT) rename "tr_ip_range" as "tr_ipv4_range" in blocklist.c

(trunk) #4019 "Minor patch, kill useless casts in transmission source." -- fixed. patch by jlouis

jlouis used Coccinelle to scan transmission's source for redundant code that casts a type to its own type.

pixel adjustment to the tracker add/remove button

change the add/remove tracker buttons in the inspector to the rounded rect style, and add tooltips

adjust the pop up button style in the the inspector - options tab

increase the width of the bandwidth limit fields in the inspector

center the "No Torrents Selected" in the inspector

inspector: update the displayed global seeding settings only when necessary; update only the displayed global seeding settings instead of all the options

#4017 display the global seeding settings in the inspector

fix a warning

fix a crash with the URL window, and memory leak with the URL window, and update the "Open" button of the URL window when shown

make the buttons in the URL window localizable

remember the URL sheet window size

enable the URL sheet's add button when shown if text is already entered

remember the text in the URL sheet

move the "open torrent URL" sheet into its own xib file

#3989 Menubar not translated in Dutch localisation

(trunk) improvement to r11864

Since the contents of getcwd() are undefined on error, explicitly terminate the buffer string if getcwd() fails.

(trunk libt) silence a couple of identical compiler warnings: "ignoring return value of ‘getcwd’, declared with attribute warn_unused_result".

(trunk libT) silence compiler warning in libtransmission/fdlimit.c: "ignoring return value of ‘ftruncate’, declared with attribute warn_unused_result"

(trunk libT) fix gcc compiler warning in libtransmission/makemeta.c: "ignoring return value of ‘read’, declared with attribute warn_unused_result"

(trunk gtk) msgwin.c: silence compiler warning "warning: ‘pinned_to_new’ may be used uninitialized in this function"

#3696 Publish Web Interface via Wide-Area Bonjour (in addition to Link-Local)

format the number of times opened

#4002 Update to JQuery 1.5

(trunk libT) fix compiler warning: "peer-io.h:282: warning: inlining failed in call to ‘evbuffer_add_uint8’: optimizing for size and code size would grow"

(trunk libT) #3991 "connection encryption stopped working" -- silence minor compiler warning. patch by ijuxda.

bump to version 2.21+

(trunk) bump to 2.21

(trunk libT) #3993 "Version (-V) option no longer outputs svn revision no." -- fixed.

update-version-h.sh tries to use {{{svnversion}}} when possible. But when it's not, it looks through the "$Id:" lines in the source file comments and uses the largest version number it finds. The new files tr-dht.[ch] didn't have the line of its $Id: comment formatted in the way update-version-h.sh expected. tr-dht.[ch]'s $Id: line has been homogenized to be like everyone else's...

add the application type to Info.plist

#3987 Display issue in the main window in the German localisation

set the visibility priority for high and low priority menu items

have miniupnp use the project's flags

#3994 don't compile assert statements in Release mode

(libT) #3991 "Connection encryption stopped working" -- fixed.

This is a pretty straightfoward bug: the call to evbuffer_peek() should not have been wrapped in assert().

(trunk gtk) refresh the translations again for 2.21

(trunk libT) silence minor "unused function argument" compiler warning

(trunk qt) refresh the strings for transmission-qt 2.20; updated es translation from rb07 :)

bump version to 2.20+

bump to 2.20

set 2.20 release date in NEWS

(trunk libT) #3975 "tr_bencToFile() contains unnecessary calls to stat() and unlink()" -- fixed for win32.

remove() doesn't have the same behavior on Windows. On that platform, we should use MoveFileEx( oldpath, newpath, MOVEFILE_REPLACE_EXISTING )." Thanks to rb07 for testing & confirming the fix.

(trunk gtk) #3866 "Popup menu for file list manipulation" -- removing from the 2.20 milestone

Looking over the 2.20 tickets, this one keeps sticking out. A popup menu may be useful if the "delete files" ticket is accepted in 2.30, but for this release (2.20) it doesn't really add anything new. One lesson from the 2.1x cycle was that features are a harder to remove than to add. So I'm going to hold off on adding this feature until it has a clear raison d'être.

update French localization for 2.2

#3675 handle trashing data over RPD in the Mac client

(trunk) #3675 "Not all .part files are removed" -- handle trashing files via RPC.

When libtransmission gets a "remove torrent" request from RPC, it tries to delegate the work. This is because the GTK+ and Mac clients don't want torrents disappearing in a different thread and causing possible thread issues. So the GTK+ and Mac clients get notification about this via libtransmission's RPC callback and remove the torrents themselves. Unfortunately, that notification doesn't include information about whether or not to delete local data.

This commit adds that information to the RPC callback so that the Mac and GTK+ clients will know whether or not to trash the local files when a third-party RPC client requests that at torrent and its files be deleted.

(trunk libT) #3931 "'Announce is Queued' but doesn't get announced" -- remove the 'unresponsive tracker' penalty from torrents whose announce time has been reached.

The 'bad tracker' penalty was introduced in 2009 after a top tier trackers went down. Announces to it would hang, tying up an announce slot in libcurl for minutes at a time. If a user had enough torrents from that tracker, it could bottleneck all announce slots. The workaround was to deprioritize failing trackers so that they wouldn't obstruct other trackers.

Its implementation could be better, however. There are two parts:

  1. Deciding how frequently to retry unresponsive trackers

  2. Once an unresponsive tracker announce was ready to go, it would be bumped down the queue if other announces were ready too.

Part 2 probably contributes to #3931. If there are enough torrents loaded, there will always be good tracker announces that get pushed ahead of a bad one in the queue. Modifying 2's heuristics would be one option, but it seems simpler to remove it altogether now that getRetryInterval() grades more hashly for consecutive failures. Altering the retry interval also gives better visual feedback to users than Part 2 did.

This commit removes "Part 2" as described above.