Bug 1415154 - Follow-up: minor tweaks (order of includes, etc.). r=aceman DONTBUILD
authorJorg K <jorgk@jorgk.com>
Tue, 07 Nov 2017 16:22:56 +0100
changeset 29330 8fe22762188be80640d56ebd6c0ea50b4a59611c
parent 29329 f840d467eaa12f82528102916acad001cb24fefa
child 29331 88393c26b4210a43142076cc1e95e41e6dee5d14
push id2068
push userclokep@gmail.com
push dateMon, 13 Nov 2017 19:02:14 +0000
treeherdercomm-beta@9c7e7ce8672b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersaceman
bugs1415154
Bug 1415154 - Follow-up: minor tweaks (order of includes, etc.). r=aceman DONTBUILD
mailnews/local/src/nsMailboxProtocol.cpp
mailnews/news/src/nsNNTPProtocol.cpp
--- a/mailnews/local/src/nsMailboxProtocol.cpp
+++ b/mailnews/local/src/nsMailboxProtocol.cpp
@@ -17,35 +17,34 @@
 #include "nsICopyMsgStreamListener.h"
 #include "nsMsgMessageFlags.h"
 #include "prtime.h"
 #include "mozilla/Logging.h"
 #include "mozilla/SlicedInputStream.h"
 #include "prerror.h"
 #include "prprf.h"
 #include "nspr.h"
-
-static mozilla::LazyLogModule MAILBOX("MAILBOX");
-
 #include "nsIFileStreams.h"
 #include "nsIStreamTransportService.h"
 #include "nsIStreamConverterService.h"
 #include "nsIIOService.h"
 #include "nsNetUtil.h"
 #include "nsMsgUtils.h"
 #include "nsIMsgWindow.h"
 #include "nsIMimeHeaders.h"
 #include "nsIMsgPluggableStore.h"
 #include "nsISeekableStream.h"
 #include "nsStreamUtils.h"
 
 #include "nsIMsgMdnGenerator.h"
 
 using namespace mozilla;
 
+static LazyLogModule MAILBOX("MAILBOX");
+
 /* the output_buffer_size must be larger than the largest possible line
  * 2000 seems good for news
  *
  * jwz: I increased this to 4k since it must be big enough to hold the
  * entire button-bar HTML, and with the new "mailto" format, that can
  * contain arbitrarily long header fields like "references".
  *
  * fortezza: proxy auth is huge, buffer increased to 8k (sigh).
@@ -398,17 +397,17 @@ NS_IMETHODIMP nsMailboxProtocol::OnStopR
   // protocol connection....
   m_nextState = MAILBOX_DONE;
 
   // the following is for smoke test purposes. QA is looking at this "Mailbox Done" string which
   // is printed out to the console and determining if the mail app loaded up correctly...obviously
   // this solution is not very good so we should look at something better, but don't remove this
   // line before talking to me (mscott) and mailnews QA....
 
-  MOZ_LOG(MAILBOX, mozilla::LogLevel::Info, ("Mailbox Done\n"));
+  MOZ_LOG(MAILBOX, LogLevel::Info, ("Mailbox Done\n"));
 
   // when on stop binding is called, we as the protocol are done...let's close down the connection
   // releasing all of our interfaces. It's important to remember that this on stop binding call
   // is coming from netlib so they are never going to ping us again with on data available. This means
   // we'll never be going through the Process loop...
 
   if (m_multipleMsgMoveCopyStream)
   {
--- a/mailnews/news/src/nsNNTPProtocol.cpp
+++ b/mailnews/news/src/nsNNTPProtocol.cpp
@@ -18,20 +18,20 @@
 #include "nsMsgI18N.h"
 #include "nsINNTPNewsgroupPost.h"
 #include "nsMsgBaseCID.h"
 #include "nsMsgNewsCID.h"
 
 #include "nsINntpUrl.h"
 #include "prmem.h"
 #include "prtime.h"
-#include "mozilla/Logging.h"
 #include "prerror.h"
 #include "nsString.h"
 #include "mozilla/Attributes.h"
+#include "mozilla/Logging.h"
 #include "mozilla/Services.h"
 #include "mozilla/SlicedInputStream.h"
 #include "mozilla/mailnews/MimeHeaderParser.h"
 #include "nsContentUtils.h"
 
 #include "prprf.h"
 #include <algorithm>