(trunk libT) CPU improvement in torrent.c's torrentInit()

Add the new torrent to the session's torrent list by prepending it instead of appending it. That way we don't have to walk the list in order to add it. tr_session.torrentList is an unordered list, so there's no real difference between prepending and appending.

(trunk libT) CPU optimization in peer-mgr.c's rechokeDownloads()

Instead of recalculating interesting pieces for each peer we loop through, calculate them just once into a bitfield and then reuse that bitfield inside the loop.

(trunk libT) small CPU optimization in bitfield.c's countRange()

(trunk libT) CPU improvement in peer-mgr.c's getPeerCandidates()

getPeerCandiates() used to read through all the torrents to determine the global peer connections, and then again to determine the global peer candidates. Now this is done in one loop instead of two.

(trunk libT) CPU optimization in tr_bandwidthClamp().

Don't call tr_time_msec() if it's not necessary. This was one of the top CPU sinks in profiling, so removing it is a nice improvement in cases when it's not necessary, such as when speed limiting is disabled.

(trunk libT) torrent-get CPU optimization.

Move the tr_torrentStat() call outside of the addField() loop so that it's only called once instead of N times.

#4232 Extend tooltip support to disabled files in Inspector->Files

(trunk libT) experimental fix for the unterminated announce URLs reported by blacklion in the forums

(trunk libT) #4227 "invalid URLs aren't filtered out of .torrents' webseed lists"

If we can't parse a URL provided in the .torrent files' webseed list, that URL should be discarded.

(trunk libT) #2338 "Add uTP support" -- increase the block bandwidth to 3000 bytes in phase 1 as suggested by jch in comment:120.

The rationale is that by using 3000 bytes we'll send one full-size frame straight away, and leave enough buffered data for the next frame to go out in a timely manner.

#4002 Update to jQuery 1.6

update the mac libevent config files

(trunk gtk) use ngettext() instead of gtr_dngettext() to address https://bugs.launchpad.net/ubuntu/+source/transmission/+bug/760761

(trunk libT) the functions tr_peerMsgsSetChoke() and tr_peerMsgsSetInterested() have bool arguments whose types never got switched from "int" to "bool" when "bool" was adopted.

(trunk, gtk) #4218 "open url should set focus on the open button by default" -- fix r12409 oops

(trunk gtk) #4218 "open url should set focus on the open button by default" -- done.

Include dht-0.19.

This fixes a bug with parsing of announce messages, noticed by cjdelisle.

(trunk gtk) #4203 "gnome-network-properties doesn't exist in GNOME 3" -- fixed with patch from lesnoland

(trunk) bump user-agent prefix to 2.30 beta 4

(trunk libT) very minor copyediting: remove a little unwanted whitespace

Don't allow the Add Magnet window to be resized vertically

(trunk libT) simplify the code in peer-mgr.c's rechokeDownloads().

The new code has fewer calls to malloc/free and, more importantly, has less copy-and-pasted code to handle the different rechoke states.

(trunk libT) #4215 "bug in unchoking algorithm" -- fixed.

(trunk gtk) in the details dialog, use the GQuark versions of g_object_{get,set}_data()

(trunk gtk) tweak to r12398: because most of the strings in category_filter_model_update() will be duplicates of each other, use g_string_chunk_insert_const() instead of g_string_insert().

(trunk gtk) in filter.c's category_filter_model_update(), use a string chunk for all those temporary strings that are allocated in a looop

(trunk libT) tr_torrentGetFileMTime(): if the file being looked at is aleady open in fdlimit's file cache, use that cached handle instead of deriving our own.

(trunk gtk) gtr_get_host_from_url(): avoid a couple of malloc() + free() calls.

(trunk libT) in peer-mgr.c's refillUpkeep(), allocate the temporary array outside of the loop instead of inside...

(trunk libT) #4212 "excessive UDP logfile entries in debug mode" -- fixed.

(trunk libT) heap pruning: use tr_bencToBuf() instead of tr_bencToStr() when building LTEP messages. This saves us from a handful of unnecessary malloc+memcpy+free calls in each instance where the change is made.

(trunk libT) heap pruning: using the same mechanism as in r12388, avoid an unnecessary malloc+memcpy+free in tr_bencToFile()

(trunk libT) tweak the ABI of tr_bencToBuf() to match the way it's being used

(trunk libT) heap pruning: using the same mechanism as in r12388, avoid an unnecessary malloc+memcpy+free when building the URL for a webseed download request

(trunk libT) heap pruning: using the same mechanism as in r12388, avoid an unnecessary malloc+memcpy+free when TR_CURL_VERBOSE is logging HTTP messages

(trunk libT) heap pruning: avoid unnecessary malloc+memcpy+frees in announcer.

1. when creating announce URLs in announcer-http.c
2. when creating scrape URLs in announcer-http.c
3. when deep-logging what announces are in a torrent's queue in announcer.c

(trunk libT) heap pruning: lazy-allocate the array in peer-mgr.c's getPeersToClose().

(trunk libT) more heap pruning: avoid an unnecessary evbuffer_new() + evbuffer_free() in tr_peerIoWriteBytes()

(trunk libT) constify the tr_scrape_request argument to scrape_request_delegate().

(trunk libT) more heap pruning: build the "cookie_filename" field once when creating the web thread, instead of each time a new tr_web_task is added.

(trunk libT) use tr_malloc() instead of malloc() in tr_loadFile() and in the fallback code for tr_valloc()

(trunk libT) #4209 "Shortcut UDP tracker test" -- prioritize the UDP handler functions based on frequency (uTP, DHT, UTP tracker) as outlined by jch in comment:5. Also, don't call the uTP or DHT handlers when uTP or DHT is disabled in the system preferences.

To avoid the function call overhead described by jch, instead of calling tr_sessionIsUTPEnabled() and tr_sessionIsDHTEnabled(), we test for WITH_UTP to be defined and test the tr_session.isUTPEnabled and tr_session.isDHTEnabled flags directly.

(trunk libT) #4209 "Shortcut UDP tracker test" -- the goal of #4209 is to minimize the cost of the UDP event callback function, so apply the heap pruning eye to that by removing an unnecessary malloc/free call there.

Previously we allocated a 4096 character buffer each time; now we allocate it on the stack. It seems all the distros and OS flavors that Transmission runs on have multi-MB default stack sizes, so a hardwired 4K array should be safe.

#4206 "RPC Documentation missing defualt value for 'torrent-set-location' argument 'move'" -- fixed.

(trunk libT) #4204 "Availability is the same as the amount downloaded when there are no seeders" -- possible fix

It looks like we had the wrong test on the replication count array size.

Optimise UDP dispatch (fixes #4209).

Make quick inline tests for the DHT and UDP tracker protocols, which
avoids calling tau_handle_message for each µTP packet.

(trunk daemon) #2338 "Add uTP support" -- added er13's patch from comment:115 to add uTP enable/disable commands to transmission-remote. Thanks er13!

(trunk utils) add '-o' description to transmission-create's manpage.

(trunk gtk) very minor tweak in deciding which rows to update in the torrent model.

Since we only display speed to two decimal places, we don't need to compare previous and current torrent speed with more precision than that.

(trunk) revert r12372 based on feedback from er13: "Many embedded systems use uClibc++ (and not the gcc' libstdc++) as an implementation of Standard C++ Library. r12372 breaks the compilation of transmission for such systems."

(trunk libt) #4164 "__FD_SETSIZE impact on open-file-limit and peer-limit-global" -- commit patch by romanr to take FD_SETSIZE into account before calling setrlimit()

(trunk) fix "__gxx_personality_v0" when compiling Transmission on some versions of gcc/g++

The problem is coming from gcc getting confused by having a C program (Transmission) link against a C++ library (libutp). In gcc, C++ code has an implicit dependency on libstdc++ for the __gxx_personality_v0 function.

More info @ http://stackoverflow.com/questions/329059/what-is-gxx-personality-v0-for

bump to 2.30 beta 3

tweak to the reveal button in the inspector in the French localization

update Dutch localization for 2.30

(trunk libT) more heap pruning: don't allocate peer-msgs' incoming block buffer until it's needed. If we're seeding, we won't need it.

(trunk libT) heap and event pruning: don't create evtimers for periodic pex messages if the torrent doesn't allow pex (such as, if it's on a private tracker).

Previously, we unconditionally created the evtimer, and then checked each time to see if pex was allowed.

(trunk libT) more heap pruning: use composition rather than aggregation for the tr_crypto object owned by tr_peerIo.

(trunk libT) use FD_SETSIZE instead of __FD_SETSIZE

(trunk libT) #4165 "__FD_SETSIZE impact on open-file-limit and peer-limit-global" -- add safety guard in the file cache to prevent too many open files.

#4104 Drop Mac PPC support

(trunk libT) changes to the bandwidth allocator's phaseOne step as suggested by Vincent in #2338 comment:108

(trunk qt) sync Qt client with r12355

Ignore configuration files from Xcode 4.

(trunk gtk) more heap pruning, in tr_core

Tweak how we loop through the torrents in the model, again to avoid unnecessary GtkTreePath temporaries.

(trunk gtk) Avoid unnecessary GtkComboBox queries.

Instead of calling gtk_combo_box_get_active_iter() on the filterbar's two comboboxes in every periodic update, keep the state information in a local struct and update it when the selection changes. That way the filter code doesn't even need to know about the GtkComboBox or the GtkTreeModel.

(trunk gtk) more heap pruning: minor GtkTreeModel changes

Use gtk_tree_model_iter_nth_child() instead of gtk_tree_model_get_iter_first() to avoid an unnecessary gtk_tree_path object being created and destroyed. This is a very minor change and I'm not sure how useful it really is, but it doesn't hurt.

(trunk gtk) more heap pruning:

querying gconf2 each time the curl callack function is called is expensive, so query it once -- then again later, if the proxy settings change -- and remember the values in a local struct.

(trunk gtk) transmission-gtk requires GTK+ 2.8 or higher, so remove the `#if GTK_CHECK_VERSION(2,8,0)' checks because we already know the answer.

(trunk gtk) more heap pruning: give DetailsImpl an internal GString buffer so that it doesn't have to create and free so many temporary strings

(trunk gtk) more heap pruning: give torrent-cell-renderer an internal GString buffer so that we don't have to create & free so many short-term strings

update French localization for 2.30

(trunk libT) fix 2.30b1 memory corruption bug when editing trackers.

The problem was that the new number of trackers was not being kept and the old count was retained. So if the count changed, tr_torrentTrackers() could return dangling pointers to the caller.

(trunk gtk) minor janitorial: const correctness in tr-core's compare_by_name()

#4182 Remember filter search text between runs

#4175 fix typo

(trunk libT) #4175 "very bizarre and unwieldly logfile in 2.30b2" -- fixed.

6789ABCDE

(trunk libT) remove unneeded assert()ion

(trunk libT) use a better data struct for the tr_datatype list in peer-io

(trunk web) #3820 "web ui flicking to selection" -- apply test patch suggested by fx

(trunk libT) #4175 "very bizarre and unwieldy logfile in 2.30b2" -- another test commit for gunzip :)

(trunk libT) #4175 "bizarre and unwieldly logfile in 2.30b2" -- possible fix for gunzip to test

bump version number to 2.30b2

(trunk libT) #4173 "crashing with r12315 in Debian" -- apply patch from comment:8 for testing in 2.30b2

catch invalid addresses when loading tracker favicons

(trunk libT) in peer-mgr's closeBadPeers() function, don't bother doing any of the setup/teardown calculations if the peer array is empty.

(trunk libT) use get_next_scrape_time() inside the scrape error handler.

get_next_scrape_time() was introduced in r12297. The rationale is that by rounding all scrape times to their nearest 10th second, they will tend to occur in batches and improve multiscrape.

(trunk libT) #3931 "'Announce is Queued' but torrent doesn't announce itself to trackers" -- add more debugging information for the next announce interval when an announce error is encountered, as requested by Sardok in comment:70 of #3931

(trunk libT) #4173 "Crashing with r12315 in Debian" -- possible fix?

(trunk libT) trivial commit fixing typo in the comments from r12297

(trunk gtk) more heap pruning: use a GStringChunk for the collated torrent names in tr-core.c

add an extra check when generating status strings in a TorrentCell

(trunk libT) more heap pruning: avoid four unnecessary malloc() + free() calls per tr_peer.

This commit also changes tr_recentHistory from being a general-purpose tool to being a little more hardcoded for the only purpose it's used, in tr_peerMgr. If its files (history.[ch]) don't find any other "customers" in libtransmission, eventually it should be demoted to being a private helper class inside of peer-mgr.c and have the history.[ch] files removed from the build.

update European Portuguese localization for 2.3

(trunk libT) remove an assertion from bitfield that doesn't always need to be true

when removing trackers, handle hidden duplicates

(trunk libT) fix ABR error when parsing a URL with no path and no trailing slash after the host. Error reported by livings124

(trunk libT) more heap pruning: avoid an unnecessary malloc() + free() when reading PadD during an outbound encrypted handshake

(trunk libT) more heap pruning: avoid an unnecessary malloc() + strcmp() + free() when parsing the initial handshake string from a peer