fix various unused and uninitialize variables, newline at end of file, etc.

mailnews/local/src/nsPop3Protocol.cpp

@@ -421,8 +421,6 @@ NS_IMETHODIMP nsPop3Protocol::OnStopBinding(nsIURL* aURL, nsresult aStatus,
 
 nsPop3Protocol::nsPop3Protocol(nsIURL* aURL) : nsMsgLineBuffer(NULL, FALSE)
 {
-    nsresult rv = 0;
-
 	NS_INIT_REFCNT();
 	Initialize(aURL);
 }
@@ -462,7 +460,6 @@ void nsPop3Protocol::Initialize(nsIURL * aURL)
 		if (NS_SUCCEEDED(rv) && m_nsIPop3URL)
 		{
 			// extract the file name and create a file transport...
-			const char * fileName = nsnull;
             nsINetService* pNetService;
             rv = nsServiceManager::GetService(kNetServiceCID,
                                               nsINetService::GetIID(),
@@ -2536,8 +2533,6 @@ nsPop3Protocol::ProcessPop3State (nsIURL* aURL, nsIInputStream* aInputStream,
                having to worry about the server timing out on us while
                we wait for user input. */
 			  {
-            char *fmt1 = 0, *fmt2 = 0;
-            
             /* If we're just checking for new mail (biff) then don't
                prompt the user for a password; just tell him we don't
                know whether he has new mail. */

mailnews/local/src/nsPop3Service.h

@@ -54,4 +54,4 @@ protected:
 
 };
 
-#endif /* nsPop3Service_h___ */
+#endif /* nsPop3Service_h___ */

mailnews/local/src/nsPop3URL.cpp

@@ -346,7 +346,7 @@ nsresult nsPop3URL::ParseURL(const nsString& aSpec, const nsIURL* aURL)
     // protocol spec. A protocol spec is an alphanumeric string of 1 or
     // more characters that is terminated with a colon.
     PRBool isAbsolute = PR_FALSE;
-    char* cp;
+    char* cp = nsnull;
     char* ap = cSpec;
     char ch;
     while (0 != (ch = *ap)) {