From d147d93d2aa4836914187f1bc48cbd9da7ef6911 Mon Sep 17 00:00:00 2001
From: Charles Kerr <charles@transmissionbt.com>
Date: Sun, 24 May 2009 16:05:28 +0000
Subject: [PATCH] (trunk web) fix striping in the file list, which I broke last
 night

---
 web/javascript/torrent.js      |  2 +-
 web/javascript/transmission.js | 24 ++++++++++--------------
 2 files changed, 11 insertions(+), 15 deletions(-)

diff --git a/web/javascript/torrent.js b/web/javascript/torrent.js
index 4e58d6b30..f6616b2ba 100644
--- a/web/javascript/torrent.js
+++ b/web/javascript/torrent.js
@@ -655,7 +655,7 @@ TorrentFile.prototype = {
 		this.readAttributes(file_data);
 
 		var li = document.createElement('li');
-		li.classNameConst = 'inspector_torrent_file_list_entry ' + ((this._index%2)?'odd':'event');
+		li.classNameConst = 'inspector_torrent_file_list_entry ' + ((this._index%2)?'odd':'even');
 		li.className = li.classNameConst;
 
 		var wanted_div = document.createElement('div');
diff --git a/web/javascript/transmission.js b/web/javascript/transmission.js
index a8b4df022..696a5432b 100644
--- a/web/javascript/transmission.js
+++ b/web/javascript/transmission.js
@@ -132,9 +132,8 @@ Transmission.prototype =
 		}
 	},
 	loadImages: function() {
-		for(var i = 0; i<arguments.length; i++) {
-			jQuery("<img>").attr("src", arguments[i]);
-		}
+		for( var i=0, row; row=arguments[i]; ++i )
+			jQuery("<img>").attr("src", row);
 	},
     
 	/*
@@ -610,13 +609,13 @@ Transmission.prototype =
 		var tab_ids = $(tab).parent('#inspector_tabs').find('.inspector_tab').map(
 			function() { return $(this).attr('id'); } 
 		);
-		for( var i=0; i<tab_ids.length; ++i ) {
-			if (tab.id == tab_ids[i]) {
-				$('#' + tab_ids[i]).addClass('selected');
-				$('#' + tab_ids[i] + '_container').show();
+		for( var i=0, row; row=tab_ids[i]; ++i ) {
+			if (tab.id == row) {
+				$('#'+row).addClass('selected');
+				$('#'+row+'_container').show();
 			} else {
-				$('#' + tab_ids[i]).removeClass('selected');
-				$('#' + tab_ids[i] + '_container').hide();
+				$('#'+row).removeClass('selected');
+				$('#'+row+'_container').hide();
 			}
 		}
 		this.hideiPhoneAddressbar();
@@ -1394,11 +1393,9 @@ Transmission.prototype =
 		// decide which torrents to keep showing
 		var allTorrents = this.getAllTorrents( );
 		var keep = [ ];
-		for( var i=0, len=allTorrents.length; i<len; ++i ) {
-			var t = allTorrents[i];
+		for( var i=0, t; t=allTorrents[i]; ++i )
 			if( t.test( this[Prefs._FilterMode], this._current_search ) )
 				keep.push( t );
-		}
 
 		// sort the keepers
 		Torrent.sortTorrents( keep, this[Prefs._SortMethod],
@@ -1409,8 +1406,7 @@ Transmission.prototype =
 		this.deselectAll( );
 
 		// hide the ones we're not keeping
-		for( var i=keep.length; i<this._rows.length; ++i ) {
-			var e = this._rows[i];
+		for( var i=keep.length, e; e=this._rows[i]; ++i ) {
 			delete e._torrent;
 			e[0].style.display = 'none';
 		}