(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

(trunk gtk) removed the unused MC_NAME column from the torrent GtkListStore.

(trunk libT) when processing errors in peer-io.c's event_read_cb() and tr_peerIoTryRead() functions, don't call tr_net_strerror() unless logging is turned on s.t. the string will be used.

(trunk libT) when processing an error in tr_peerIoTryWrite(), don't call tr_net_strerror() unless logging is turned on s.t. the string will be used.

(trunk gtk) use GQuarks instead of strings for keys in tr-icon and tr-window

(trunk gtk) use GQuarks instead of strings for keys in the filterbar's key/value pairs

(trunk gtk) more heap pruning: in the main window's torrent list, only update the case-insensitive "collated name" of a torrent when the torrent's metadata changes (such as when a magnet link's metadata finishes downloading).

(trunk libT) more heap pruning: use a hash, rather than an allocated string in tr-core when watching for a torrent's tracker list to change

(trunk libT) oops, r12313 committed the wrong version of list.c

(trunk libT) keep a pool of reusable tr_list nodes

(trunk libT) Use tr_strndup() instead of tr_strdup() in tr_urlParse() to avoid a couple of strlen() calls

(trunk libT) use alloca() for allocating the block array in updateBlockRequests()

(trunk libT) #4165 "crash on startup introduced in r12262" -- experimental commit

(trunk libT) remove redundant assert()s

(trunk libT) remove unnecessary memmove()s from rechokeDownloads()

remove unneeded include

(trunk gtk) more heap pruning: avoid unnecessary malloc() + free() calls in gtr_get_host_from_url()

(trunk libT) more heap pruning: avoid an unnecessary malloc() + free() when encrypting outbound messages

(trunk libT) avoid an unnecessary malloc() + free() call when we finish downloading a block.

Pre-libevent2, this free() was useful in helping keep the peer's incoming piece data buffer from growing too large because that could be a significant amount of wasted space given enough peers. However now that we're using the libevent2 code, that piece data buffer gets handed off to the block cache, so most of the time we're freeing an evbuffer that doesn't have any inernal chains allocated anyway.

(trunk libT) when reading piece data in from a socket, avoid two unnecessary calls to memcpy()

small cleanup when determining if two announce URLs differ only by scheme

(trunk libT) fix UMR bug reported by wereHamster

Reinstate including tcp.h in net.c.

Removed in r12225, this broke netSetCongestionControl.

(trunk libT) don't bother calling rechokeUploads() and rechokeDownloads() on torrents which have no peers.

(trunk libT) #4138 "use stdbool.h instead of tr_bool" -- fix watch.c error reported by zebulon501 in trac. :)

(trunk libT) better multiscrape

(trunk libT) tr_torrentGetFileMTime() used to require two stat() calls.. now it only requires one.

(trunk libT) tr_torrentFindFile2(): avoid an unnecessary malloc() + free() if the file we're looking for is complete and doesn't have a ".part" suffix

(trunk libT) simplify tr_torrent's hasAnyLocalData() to be more readable, and also to avoid yet another unnecessary malloc() and free()

(trunk libT) more malloc pruning. *yawn*

tr_metainfoGetBasename(): avoid an unnecessary malloc() + free()
getannounce(): avoid an unnecessary malloc() + free() per each tracker

(trunk libT) avoid an unnecessary malloc/free in tr_urlIsValidTracker() and tr_urlIsValid()

(trunk libT) copyediting in resume.c

(trunk libT) streamline parsing the metainfo's file list

(trunk libT) since the JSON parser gives us the string's length, we can call tr_strndup() instead of tr_strdup()

(trunk gtk) remove tr-torrent.c from POTFILES.in

update to 2.30b1

update jQuery to 1.5.2

update German localization for 2.3

(trunk libT) announcer.c: fold the tiers and trackers into fewer mallocs/frees

(trunk libT) copyediting: peer-io's "EncryptionMode" type had a name too similar to tr_encryption_mode... rename it to reduce potential confusion

(trunk libT) save 48 bytes per tr_torrent object. whoo!

(trunk libT) use aggregation for the tr_bandwidth objects owned by tr_session and tr_torrent

update Russian localization to 2.30

(trunk libT) fix never-executed code branch detected by clang static analyzer

(trunk gtk) remove "dead nested assignment" wart detected by clang static analyzer

(trunk libT) remove dead logic branch detected by clang static analyzer

(trunk libT) fix "dangerous pointer arithmetic" warning detected by clang static analyzer

(trunk libT) fix minor dead store detected by clang static analyzer

(trunk libT) fix ABR detected by clang static analyzer in tr_torrentStat.

When copying a value to s->errorString, it used memcpy(a,b,sizeof(a)) where sizeof(a) was larger than sizeof(b). Fixed by replacing the memcpy() call with tr_strlcpy().

(trunk libT) fix potential UMR detected by clang static analyzer: maxPeers could be uninitialized in some pathological logic branches

(trunk libT) fix potential null pointer dereference in announcer.c

(trunk libT) fix file descriptor leak in tr-dht.c

(trunk libT) #4153 "Protect against non-monotonic time in libutp" -- try again.

(trunk libT) #4153 "Protect against non-monotonic time in libutp" -- another attempt at getting it to compile on win32

(trunk libT) #4153 "Protect against non-monotonic time in libutp" -- move UTP_GetMilliseconds() and UTP_GetMicroseconds() outside of the if-not-windows ifdef section. Based on feedback from rb07 in ticket comment 5

(trunk daemon) fix minor text formatting bug when the ETA is being shown in seconds

(trunk libT) #4157 "'transmission-remote -si' doesn't show alternate download speeds correctly" -- fixed.

(trunk daemon) #4156 "request: transmission-remote --session-info report whether µTP is enabled" -- added.

(trunk libT) merge tr_bitfieldTestFast() and tr_bitfieldTest()

(trunk libT) handle situations where we don't know the bitfield's upper bound in advance. This comes up sometimes with magnet links.

Don't explicitly protect against non-monotonic gettimeofday.

Since we're now protecting against non-monotonic time unconditionally,
there's no need for additional protection in the gettimeofday case.

Protect against monotonic time being non-monotonic, take 2.

This is a better fix, suggested by ghazel.

Protect against monotonic time being non-monotonic.

Within utp.cpp, there's a bunch of assertions that will cause us
to crash if monotonic time isn't.  While I have no idea about Windows,
under Unix we use real time instead of monotonic time if POSIX clocks
are not available, and real time can be stepped backwards.  Since the
protection is cheap, we perform it on all platforms.

(trunk libT) fix endgame detection bug introduced in r12248

(trunk libT) fix a crasher introduced yesterday in r12248

(trunk libT) safeguard against a minor memory leak if io->sokcet couldn't be opened.

(trunk libT) fix bitfield.c assertion failure reported by Rolcol

(trunk libT) don't cache unininteresting peer addresses between sessions

For example, if we're both seeds, or if the peer is not connectible, don't bother caching it for the next session. If it's still alive, we'll find it up through DHT or tracker announces next time around. As with r12253, this commit's intention is to reduce the number of unproductive peer_atoms that we waste time trying to connect to.

(trunk libT) better scrape management on private trackers.

If a private tracker scrape says that there are no downloaders in the swarm, mark all the peers in the private swarm as seeds. This can greatly reduce unnecessary overhead on large seedboxes. We don't do this same trick on public torrents, since a public tracker won't know of all the peers.

(trunk libT) fix bitfield assertion failure due to invalid assumption in tr_bitfieldIsValid()

(trunk libT) more completion and bitfield cleanup: (1) fix regression in tr_cpSizeWhenDone() reported by Waldorf, (2) make simple one-liner functions inlined

(trunk mac) delete bitset.[ch] from the xcode file. No, I don't know what I'm doing, but what's the worst that could happen, it could break the build... :)

(trunk libT) whoops, remove 4 debugging fprintf()'s from the previous commit

(trunk libT) break the mac build and introduce new crashes.

This is partially to address #4145 "Downloads stuck at 100%" by refactoring the bitset, bitfield, and tr_completion; however, the ripple effect is larger than usual so things may get worse in the short term before getting better.

livings124: to fix the mac build, remove bitset.[ch] from xcode

(trunk libT) strip trailing whitespace

(trunk libT) fix memory leak in tr_utf8clean()'s utility function to_utf8()

drop two NSLogs

update Spanish localization for 2.3

Attempt to fix #4148 "Mac client gives inappropriate error if data volume is not mounted"

add µTP to the help documentation

update NEWS

update Credits to match AUTHORS

update Italian localization for 2.3

(trunk libT) remove the tr_session argument from the announce/scrape response callbacks

(trunk libT) minor code cleanup in announcer-udp: whitespace & function names

draft of NEWS for 2.30

add Andreas Nilsson to the credits

(trunk) fix a handful of small memory leaks that valgrind found.