(trunk libT) #3933 "announcer.c peer parsing could be simpler" -- fixed.

Remove redundant code by using tr_peerMgrCompactToPex() and tr_peerMgrCompact6ToPex() to parse compact ipv4 and ipv6 peer lists. Simplify the old-style benc peer list parsing and fix a bug that returned too few bytes in the old-style peer array.

(trunk libT) #3906 "DHT ignores bind-address-ipv6" -- test fix.

Add code to honor the ipv6 bind address. Thanks to jch for saving me a little work by confirming the bug and pointing out where in the code the change needed to be made.

(trunk libT) #3927 "Use of libevent2 evbuffer_write on files..."

evbuffer_write() is intended for sockets. It works for files on Unix, but not on Windows, so we can't portably use it on files. Instead, use evbuffer_pullup() + write().

(trunk gtk) #3887: "CPU spikes to 100% on scrolling" -- workaround committed.

The patch avoids use of GtkCellRendererText's "sensitive" property, and instead renders paused torrents' text using the widget's style's color for insensitive text. This approach honors the theme's "insensitive" color setting and uses it to draw insensitive text with no etching or shadowing.

The better option is still for Murrine to fix its text shadow issue... but until then, here is a workaround.

(trunk libT) #3932  "Assertion failed: (tab != NULL), function tr_torrentAvailability"

Looks like tab can be NULL from the Mac client when magnet links are involved and the Mac client doesn't know how many pieces are available, so that assertion's not appropriate.

(trunk) #3901 "Confusing error message when libtool is missing" -- fixed.

autogen.sh invokes autoreconf without checking to see if it fails (as it does if libtool can't be found). In such cases it also invokes a broken doomed version of the configure script, which generates a lot of red herring messages that distract from the real problem encountered by autoreconf.

The fix is to exit autogen.sh if its call to autoreconf fails.

(trunk qt) #3930 - "Add URL" - auto-copy links from the clipboard

When opening the "Add URL" dialog, check the clipboard selection & clipboard proper to see if either contains a URL, magnet link, or info hash code. If it has any of those things, put that in the dialog's entry field as the default action. Patch by cantabile.

(trunk gtk) #3930: Add drag-and-drop support for info hashes.

The "Add URL" dialog accepts URLs, magnet links, and infohashes, but only copyies urls and magnet links from the clipboard. Patch by cantabile.

(trunk gtk) include the torrent hashcode in the model's collated name.

This simplifies sorting by name by merging the primary and secondary keys (case-insensitive name, and hash string) into a single key.

(trunk gtk) set GtkLabel's "single-line-mode" flag to TRUE in the toolbar, in the stats dialog, and in the details dialog's info tab.

According to the GTK+ documentation, this "can be an advantage in situations where resizing the label because of text changes would be distracting, e.g. in a statusbar." It doesn't seem to prevent relayout in GTK+ 2.20.1, but maybe other versions of GTK+ will make better use of the flag.

(trunk gtk) companion commit to r11738 to reduce unnecessary re-rendering in the main window

The main window called gtk_tree_model_filter_refilter() once per second to refresh the torrent list's filtering. This is not an efficient approach: gtk_tree_model_filter_refilter() emits a "row changed" event for every row, causing unnecessary re-rendering.

I've removed the call to gtk_tree_model_filter_refilter() and expanded the model to includes all the fields necessary for filtering. That way we only fire "row changed" events for rows that actually change.

By reducing the number of renders in steady state, this might ameliorate https://bugs.launchpad.net/ubuntu/+source/transmission/+bug/655024

However it will *not* help the related "CPU spikes to 100% on scrolling" ticket at https://trac.transmissionbt.com/ticket/3887 because rendering paused torrents is still exceptionally expensive in the murrine theme.

(trunk gtk) #3916 "Reorder the list when metadata is downloaded" -- fix r11737 typo.

(trunk gtk) make "gtr_label_set_text" public and use it everywhere instead of gtk_label_set_text()

Some of the refresh events to the main window's torrent list are caused by main window relayout caused by the toolbar's GtkLabels recalculating their size after being updated once per second. To prevent relayout in some trivial cases, I'm replacing the gtk_label_set_text() calls with gtr_label_set_text() because the latter doesn't update the label widget if the old and new text strings are the same.

There are other changes that can handle more important cases -- I'll test those out next.

(trunk gtk) #3916 "Reorder the list when metadata is downloaded" -- fixed.

When tr_info.name changes, update the collated name in the main window's TreeStore.

#3915 "RPC Documentation should reflect in the X-Transmission-Session-Id headers" -- fixed.

Add documentation for handling 409 error messages from the Transmission RPC server.

Remove the documentation for PUT notation from the rpc spec

The PUT hack was useful back when Transmission's RPC was first written, but it doesn't scale well to any of the commands that make its RPC useful. We should deprecate its use and not advertise this feature anymore.

tweak Credits.rtf

mark the peer-id and user-agent for 2.20 beta 1

mention #3870 in the NEWS

(trunk libT) #3870 "transmission doesn't recheck after getting initial 404" -- fixed.

The exponentially-growing interval between retries had a bug that caused intervals to be too long if no successful announce had ever been made. This commit fixes the code that calculates the interval.

(trunk libT) #3923 "remove redundant announcer.c debug message" -- done.

Mention dht-0.18 in NEWS.

Import dht-0.18.

(trunk) #3926 "use 'Open Torrent' instead of 'Add Torrent' in GTK+ and Qt clients" -- fix accelerator ambiguity.

Using Ctrl-O for "Open File" causes a conflict with the existing Ctrl-O for "Open Folder." The HIG-compliant shortcut wins the conflict, so "Open Folder" gets a new accelerator.

(trunk qt) #3926 "use 'Open Torrent' instead of 'Add Torrent' in GTK+ and Qt clients" -- sync the accelerator with the GTK+ client

(trunk qt) #3925 "qtr.pro cleanup" -- patch from rb07 to handle libevent2 more gracefully via pkg-config

(trunk qt) #3926 "use "Open Torrent" instead of "Add Torrent" in GTK+ and Qt clients" -- one more instance

(trunk gtk) #3909 -- don't use Ctrl-M as the shortcut for "Copy Magnet Link to Clipboard."

(trunk) #3926: use "Open Torrent" instead of "Add Torrent" in GTK+ and Qt clients -- change the filename from add-dialog.[ch] to open-dialog.[ch]. whoooo

(trunk) #3926: use "Open Torrent" instead of "Add Torrent" in GTK+ and Qt clients -- text changed.

(trunk libT) #3921 "no overhead for reads" -- fixed. thanks to jch for reporting this.

(trunk libT) minor code tweak to crypto.c's tr_sha1() function

tr_sha1(): remove unnecessary casts from va_arg

(trunk qt) #3917 "Text on QPushButton is not updated after selecting 'Torrent Done' script" -- fixed. patch by cantabile.

(trunk) #3914 "strlsize() passes variable of incompatible type to the tr_formatter_size_B()" -- fixed.

remove mention of µTP from 2.20 NEWS. let's not get ahead of ourselves -- we shouldn't imply future µTP support when that decision hasn't been made yet.

Update NEWS.

(trunk gtk) source code formatting tweak

(trunk gtk) #3913 "Area to display a torrent's comment is small" -- fixed.

Update the copyright year in the source code comments.

The Berne Convention says that the copyright year is moot, so instead of adding another year to each file as in previous years, I've removed the year altogether from the source code comments in libtransmission, gtk, qt, utils, daemon, and cli.

Juliusz's copyright notice in tr-dht and Johannes' copyright notice in tr-lpd have been left alone; it didn't seem appropriate to modify them.

(trunk libT) #3912 "tr_torrentGetMetadataPercent() may return NaN" -- fixed.

#3519 "webseeds don't work" -- the `active' filter in the web client didn't test for webseed activity.

cleanup of who calls the libtransmission thread locking functions.

In some cases we were calling them from deep inside libtransmission, when they should have been called directly from the public-visible API functions: tr_torrentWebSpeeds_KBps(), tr_torrentPeers(), tr_torrentTrackers().

(trunk gtk) #3519 "webseeds don't work" -- the `active' filter in the GTK+ client didn't test for webseed activity.

code cleanup for tr_peerMgrTorrentAvailability().

1. move responsibility for getting a thread lock back to the public API fucntion, tr_torrentAvailability().
2. if the torrent doesn't have metadata, stop after zeroing out the table
3. minor code formatting cleanup

#3911: tr_peerMgrTorrentAvailability() doesn't sample across the entire torrent' -- fixed.

tweak COPYING

the text in the last change was weak.

tweak gtk+ section... lots of little fixes there

grammar fix

update strings file

update NEWS

(trunk libT) demote the remaining lazy-verify debug messages from the debug window to deep logging

trivial minor changes to indentation, text strings, etc.

first draft of 2.20 news

(trunk gtk) #3907 "in the details dialog's peers tab, 'status' is a misnomer" -- changed to 'flags'.

(trunk gtk) #3908 "details dialog's information tab shows (verified + unverified) + unverified" -- fixed.

Make ensureAtomExists add to the flags of an existing atom.

Flags are cumulative -- if we learn that a given peer groks encryption
from one source, and that it accepts incoming connections from another,
then the relevant atom should have both flags.

Consolidate both versions of tr_isPex into a single function.

There used to be two versions of tr_isPex; one correct in peer-mgr.c,
and one buggy in resume.c.  The buggy version caused us to reject all
peers with non-trivial flags when resuming.

first cut at a .gitignore file

#3686 "add rpc command to cleanly shutdown daemon" - added for Mac client

Fix transmission-remote regression introduced in the last commit. Thanks to Rolcol for spotting this. Mumble, grumble.

#3686 "add rpc command to cleanly shutdown daemon" -- added for libT, tr-daemon, tr-gtk, tr-remote, and the rpc spec

update MainMenu.xib to recognize the change in r11684

change the filter bar outlet to FilterBarView

use a string instead of an int when setting peer-socket-tos

(trunk daemon) "--logfile is unreliable on Solaris 10" -- fixed.

Make the peer-socket-tos preference more user-friendly.

Possible values are now the strings default, lowcost, throughput etc.
Numeric values are used for non-standard values.

(trunk libT) #2191 "NAT-PMP and UPnP should also map UDP port" -- committed patch by starix.

#3881 Right click torrent context menu does not work with Firefox / Firegestures / Ubuntu

tweak MainMenu.xib's tags

push COPYING date to 2011

Make the peer-congestion-algorithm preference visible.

When I first implemented this code, I made the preference hidden --
unless you set it yourself, it will not appear in the sttings file.
Now that it's been documented and advertised, make it visible.

(trunk gtk) #3903 "Scroll to new messages when viewing new messages in the message window" -- added.

(trunk libT) fix a minor error in the log message for prefetch errors. thanks to Chinstrap for spotting this.

(trunk libT) #3898 "Add 'Add' and 'Remove' buttons to the tracker list" -- done.

(trunk qt) #3899 "typo in qt/details.cc" -- fixed. patch by cantabile

(trunk gtk) #3897 "filterbar should use tr_torrentStatCached() instead of tr_torrentStat()" -- fixed.

(trunk gtk) #3896 "Redundant call to tr_torrentStat() in tr_core_update()" -- fixed.

(trunk libT) #3894 "tr_peerIo.hasFinishedConnecting should be removed" -- committed.

(trunk libT) fix formatting error in a couple of old calls to dbgmsg()

fix typo in the 'blocklist contains N rules' text message

Fix incorrect overflow handling in the DHT's parser.

Don't complain about unexpected DHT packets.

 #3634 Use dht-0.17

Import dht-0.17.

Fix incorrect usage of event_new in UDP code.

Apparently you need to pass session->event_base as the first argument.

Fix UDP error handling.

fix Mac build

Update tr-{dht,udp} to libevent2.

Fix tr-udp for the no IPv6 case.

Move handling of UDP I/O to tr-udp.c.

Use the dht-0.16 API in tr-dht.c.
Recvfrom is now done outside the dht code, which makes it possible to share
the socket with UTP.

Import dht-0.16.

Move creation of the UDP sockets to tr-udp.c.

Implement tr_dhtInit and Uninit, use it in session.c.

Update copyright notices.

Add files tr-udp.[ch] to the build.

if file size is less than 10 KB, show one decimal place

silence a warning