(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

use a float instead of a double for file unit size

#3878 don't show the units of a partial file size when it's the same as the full file size's units

#3877 Match Finder's file size display: start at KB, each increase in units increases the number of decimals shown, and truncate trailing zeros

Improve error messages in tr-dht.c.

Avoid a descriptor leak when binding the IPv6 DHT socket fails.

(trunk libT) minor code cleanup in fdlimit.c

(trunk gtk) #3876 "Give filesize its own column in the Files tree" -- added.

(trunk libT) very minor code cleanup -- use an enum instead of magic numbers, etc.

replace the "Reverse Sort Order" menu item with "Ascending" and "Descending" menu items

capitalize "By" in "Sort Transfers By"