From 53c1ec56cf91f765931cd1f63c1fdc9f8e058530 Mon Sep 17 00:00:00 2001
From: "putterman%netscape.com"
 <putterman%netscape.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Fri, 8 Sep 2000 05:46:35 +0000
Subject: [PATCH] some js warning fixes. r=bienvenu.

git-svn-id: svn://10.0.0.236/trunk@78493 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../base/resources/content/commandglue.js       |  5 +++--
 .../content/mail3PaneWindowCommands.js          |  1 +
 .../resources/content/msgMail3PaneWindow.js     | 17 +++--------------
 3 files changed, 7 insertions(+), 16 deletions(-)

diff --git a/mozilla/mailnews/base/resources/content/commandglue.js b/mozilla/mailnews/base/resources/content/commandglue.js
index 71930474e93..f9297a83f5b 100644
--- a/mozilla/mailnews/base/resources/content/commandglue.js
+++ b/mozilla/mailnews/base/resources/content/commandglue.js
@@ -541,6 +541,7 @@ function SortFolderPane(column, sortKey)
 	//Remove the sortActive attribute because we don't want this pane to have any
 	//sort styles.
 	node.setAttribute("sortActive", "false");
+	return true;
 }
 
 function SortColumn(node, sortKey, secondarySortKey, direction)
@@ -814,8 +815,8 @@ function GetNextMessageAfterDelete(messages)
 	if(!nextMessage)
 	{
 
-		var curMessage = messages[0];
-		var nextMessage = null;
+		curMessage = messages[0];
+		nextMessage = null;
 		//search forward
 		while(curMessage)
 		{
diff --git a/mozilla/mailnews/base/resources/content/mail3PaneWindowCommands.js b/mozilla/mailnews/base/resources/content/mail3PaneWindowCommands.js
index 16a528c8819..2cb1faf6e1f 100644
--- a/mozilla/mailnews/base/resources/content/mail3PaneWindowCommands.js
+++ b/mozilla/mailnews/base/resources/content/mail3PaneWindowCommands.js
@@ -315,6 +315,7 @@ var DefaultController =
 			default:
 				return false;
 		}
+		return false;
 	},
 
 	doCommand: function(command)
diff --git a/mozilla/mailnews/base/resources/content/msgMail3PaneWindow.js b/mozilla/mailnews/base/resources/content/msgMail3PaneWindow.js
index 689a14ca48a..78235c38b50 100644
--- a/mozilla/mailnews/base/resources/content/msgMail3PaneWindow.js
+++ b/mozilla/mailnews/base/resources/content/msgMail3PaneWindow.js
@@ -352,7 +352,7 @@ function loadStartFolder(startFolderUri)
 			if(!inboxFolder) return;
 
 			var resource = inboxFolder.QueryInterface(Components.interfaces.nsIRDFResource);
-			var startFolderUri = resource.Value;
+			startFolderUri = resource.Value;
 
 			//first, let's see if it's already in the dom.  This will make life easier.
 			//We need to make sure content is built by this time
@@ -478,6 +478,8 @@ function FindInSidebar(currentWindow, id)
 		if(frameItem)
 			return frameItem;
 	}
+
+	return null;
 }
 
 function GetThreadAndMessagePaneSplitter()
@@ -569,19 +571,6 @@ function GetSelectedFolder()
 
 }
 
-function GetSelectedMessage()
-{
-	var tree = GetThreadTree();
-	var selection = tree.selectedItems;
-	if(selection.length > 0)
-		return selection[0];
-	else
-		return null;
-
-}
-
-
-
 function GetServer(uri)
 {
     if (!uri) return null;