diff --git a/mailnews/local/src/nsParseMailbox.cpp b/mailnews/local/src/nsParseMailbox.cpp
index 7b277fbb012..130a4999bd3 100644
--- a/mailnews/local/src/nsParseMailbox.cpp
+++ b/mailnews/local/src/nsParseMailbox.cpp
@@ -779,7 +779,7 @@ NS_IMETHODIMP nsParseMailMessageState::GetHeaders(char ** pHeaders)
   NS_ENSURE_ARG_POINTER(pHeaders);
   nsCString crlfHeaders;
   char *curHeader = m_headers.GetBuffer();
-  for (PRInt32 headerPos = 0; headerPos < m_headers.GetBufferPos();)
+  for (PRUint32 headerPos = 0; headerPos < m_headers.GetBufferPos();)
   {
     crlfHeaders.Append(curHeader);
     crlfHeaders.Append(CRLF);
diff --git a/mailnews/local/src/nsPop3Sink.cpp b/mailnews/local/src/nsPop3Sink.cpp
index c402fe1ac71..092db2a102e 100644
--- a/mailnews/local/src/nsPop3Sink.cpp
+++ b/mailnews/local/src/nsPop3Sink.cpp
@@ -476,7 +476,7 @@ nsPop3Sink::IncorporateComplete(nsIMsgWindow *msgWindow)
 {
   if (m_buildMessageUri && m_baseMessageUri)
   {
-      PRUint32 msgKey = -1;
+      PRUint32 msgKey;
       m_newMailParser->GetEnvelopePos(&msgKey);
       m_messageUri.SetLength(0);
       nsBuildLocalMessageURI(m_baseMessageUri, msgKey, m_messageUri);
diff --git a/mailnews/mime/emitters/src/nsMimeHtmlEmitter.cpp b/mailnews/mime/emitters/src/nsMimeHtmlEmitter.cpp
index 924074bbf12..8d54eeb15b4 100644
--- a/mailnews/mime/emitters/src/nsMimeHtmlEmitter.cpp
+++ b/mailnews/mime/emitters/src/nsMimeHtmlEmitter.cpp
@@ -207,7 +207,7 @@ NS_IMETHODIMP nsMimeHtmlDisplayEmitter::WriteHTMLHeaders()
       headerNames[numHeadersAdded] = headerInfo->name;
 
       if (nsCRT::strcasecmp("Date", headerInfo->name) == 0)
-        nsresult rv = GenerateDateString(headerInfo->value, &headerValues[numHeadersAdded]);
+        rv = GenerateDateString(headerInfo->value, &headerValues[numHeadersAdded]);
       else
       {
         // optimization: if we aren't in view all header view mode, we only show a small set of the total # of headers.