Fix member initializer re-ordered warnings by re-ordering member initializers. r=jag rs=waterson b=105974

content/base/src/nsDOMAttribute.cpp

@@ -49,7 +49,7 @@
 
 nsDOMAttribute::nsDOMAttribute(nsIContent* aContent, nsINodeInfo *aNodeInfo,
                                const nsAReadableString& aValue)
-  : mNodeInfo(aNodeInfo), mValue(aValue), mContent(aContent), mChild(nsnull),
+  : mContent(aContent), mNodeInfo(aNodeInfo), mValue(aValue), mChild(nsnull),
     mChildList(nsnull)
 {
   NS_ABORT_IF_FALSE(mNodeInfo, "We must get a nodeinfo here!");

content/html/content/src/nsHTMLFormElement.cpp

@@ -657,8 +657,9 @@ nsHTMLFormElement::IndexOfControl(nsIFormControl* aControl, PRInt32* aIndex)
 // a lightweight collection implementation somewhere
 
 nsFormControlList::nsFormControlList(nsIDOMHTMLFormElement* aForm)
-  : mForm(aForm), mNameLookupTable(NS_FORM_CONTROL_LIST_HASHTABLE_SIZE),
+  : mForm(aForm),
-    mNoNameLookupTable(nsnull)
+    mNoNameLookupTable(nsnull),
+    mNameLookupTable(NS_FORM_CONTROL_LIST_HASHTABLE_SIZE)
 {
   NS_INIT_REFCNT();
 }

content/html/style/src/nsCSSDeclaration.cpp

@@ -227,11 +227,11 @@ nsCSSText::nsCSSText(void)
 nsCSSText::nsCSSText(const nsCSSText& aCopy)
   : mWordSpacing(aCopy.mWordSpacing),
     mLetterSpacing(aCopy.mLetterSpacing),
-    mDecoration(aCopy.mDecoration),
     mVerticalAlign(aCopy.mVerticalAlign),
     mTextTransform(aCopy.mTextTransform),
     mTextAlign(aCopy.mTextAlign),
     mTextIndent(aCopy.mTextIndent),
+    mDecoration(aCopy.mDecoration),
     mTextShadow(nsnull),
     mUnicodeBidi(aCopy.mUnicodeBidi),
     mLineHeight(aCopy.mLineHeight),

content/html/style/src/nsCSSStruct.cpp

@@ -227,11 +227,11 @@ nsCSSText::nsCSSText(void)
 nsCSSText::nsCSSText(const nsCSSText& aCopy)
   : mWordSpacing(aCopy.mWordSpacing),
     mLetterSpacing(aCopy.mLetterSpacing),
-    mDecoration(aCopy.mDecoration),
     mVerticalAlign(aCopy.mVerticalAlign),
     mTextTransform(aCopy.mTextTransform),
     mTextAlign(aCopy.mTextAlign),
     mTextIndent(aCopy.mTextIndent),
+    mDecoration(aCopy.mDecoration),
     mTextShadow(nsnull),
     mUnicodeBidi(aCopy.mUnicodeBidi),
     mLineHeight(aCopy.mLineHeight),

content/xul/document/src/nsXULCommandDispatcher.cpp

@@ -75,7 +75,7 @@ static PRLogModuleInfo* gLog;
 ////////////////////////////////////////////////////////////////////////
 
 nsXULCommandDispatcher::nsXULCommandDispatcher(nsIDocument* aDocument)
-    : mDocument(aDocument), mFocusController(nsnull), mUpdaters(nsnull)
+    : mFocusController(nsnull), mDocument(aDocument), mUpdaters(nsnull)
 {
 	NS_INIT_REFCNT();
 

db/mork/src/morkEnv.cpp

@@ -109,8 +109,8 @@ morkEnv::morkEnv(const morkUsage& inUsage, nsIMdbHeap* ioHeap,
 , mEnv_DoTrace( morkBool_kFalse )
 , mEnv_AutoClear( morkAble_kDisabled )
 , mEnv_ShouldAbort( morkBool_kFalse )
-, mEnv_OwnsHeap ( morkBool_kFalse )
 , mEnv_BeVerbose( morkEnv_kBeVerbose )
+, mEnv_OwnsHeap ( morkBool_kFalse )
 {
   MORK_ASSERT(ioSlotHeap && ioFactory );
   if ( ioSlotHeap )
@@ -150,8 +150,8 @@ morkEnv::morkEnv(morkEnv* ev, /*i*/
 , mEnv_DoTrace( morkBool_kFalse )
 , mEnv_AutoClear( morkAble_kDisabled )
 , mEnv_ShouldAbort( morkBool_kFalse )
-, mEnv_OwnsHeap ( morkBool_kFalse )
 , mEnv_BeVerbose( morkEnv_kBeVerbose )
+, mEnv_OwnsHeap ( morkBool_kFalse )
 {
   // $$$ do we need to refcount the inSelfAsMdbEnv nsIMdbEnv??
   

embedding/browser/webBrowser/nsDocShellTreeOwner.cpp

@@ -904,7 +904,8 @@ NS_INTERFACE_MAP_END
 ChromeTooltipListener :: ChromeTooltipListener ( nsWebBrowser* inBrowser, nsIWebBrowserChrome* inChrome ) 
   : mWebBrowser(inBrowser), mWebBrowserChrome(inChrome),
      mTooltipListenerInstalled(PR_FALSE),
-     mShowingTooltip(PR_FALSE), mMouseClientX(0), mMouseClientY(0)
+     mMouseClientX(0), mMouseClientY(0),
+     mShowingTooltip(PR_FALSE)
 {
   NS_INIT_REFCNT();
 
@@ -1336,7 +1337,9 @@ NS_INTERFACE_MAP_END
 // ChromeTooltipListener ctor
 //
 ChromeContextMenuListener :: ChromeContextMenuListener ( nsWebBrowser* inBrowser, nsIWebBrowserChrome* inChrome ) 
-  : mWebBrowser(inBrowser), mWebBrowserChrome(inChrome), mContextMenuListenerInstalled(PR_FALSE)
+  : mContextMenuListenerInstalled(PR_FALSE),
+    mWebBrowser(inBrowser),
+    mWebBrowserChrome(inChrome)
 {
   NS_INIT_REFCNT();
 } // ctor

embedding/browser/webBrowser/nsWebBrowser.cpp

@@ -77,13 +77,17 @@ static NS_DEFINE_CID(kLookAndFeelCID, NS_LOOKANDFEEL_CID);
 //*****************************************************************************
 
 nsWebBrowser::nsWebBrowser() : mDocShellTreeOwner(nsnull), 
-   mInitInfo(nsnull), mContentType(typeContentWrapper),
+   mInitInfo(nsnull),
-   mParentNativeWindow(nsnull), mParentWidget(nsnull), mParent(nsnull),
+   mContentType(typeContentWrapper),
-   mProgressListener(nsnull), mListenerArray(nsnull),
+   mParentNativeWindow(nsnull),
+   mProgressListener(nsnull),
    mBackgroundColor(0),
    mPersistCurrentState(nsIWebBrowserPersist::PERSIST_STATE_READY),
+   mPersistResult(NS_OK),
    mPersistFlags(nsIWebBrowserPersist::PERSIST_FLAGS_NONE),
-   mPersistResult(NS_OK)
+   mParentWidget(nsnull),
+   mParent(nsnull),
+   mListenerArray(nsnull)
 {
     NS_INIT_REFCNT();
     mInitInfo = new nsWebBrowserInitInfo();

embedding/browser/webBrowser/nsWebBrowserPersist.cpp

@@ -117,8 +117,8 @@ nsWebBrowserPersist::nsWebBrowserPersist() :
     mCancel(PR_FALSE),
     mJustStartedLoading(PR_TRUE),
     mCompleted(PR_FALSE),
-    mPersistResult(NS_OK),
+    mPersistFlags(PERSIST_FLAGS_NONE),
-    mPersistFlags(PERSIST_FLAGS_NONE)
+    mPersistResult(NS_OK)
 {
     NS_INIT_REFCNT();
 }

gfx/src/nsPrintOptionsImpl.cpp

@@ -91,20 +91,20 @@ nsFont* nsPrintOptions::sDefaultFont = nsnull;
  *	@update 6/21/00 dwc
  */
 nsPrintOptions::nsPrintOptions() :
+  mPrintOptions(0L),
   mPrintRange(kRangeAllPages),
   mStartPageNum(1),
   mEndPageNum(1),
-  mPrintOptions(0L),
+  mPrintFrameType(kFramesAsIs),
+  mHowToEnableFrameUI(kFrameEnableNone),
+  mIsCancelled(PR_FALSE),
+  mPrintSilent(PR_FALSE),
+  mPrintPageDelay(500),
   mPrintReversed(PR_FALSE),
   mPrintInColor(PR_TRUE),
   mPaperSize(kLetterPaperSize),
   mOrientation(kPortraitOrientation),
-  mPrintToFile(PR_FALSE),
+  mPrintToFile(PR_FALSE)
-  mPrintFrameType(kFramesAsIs),
-  mHowToEnableFrameUI(kFrameEnableNone),
-  mIsCancelled(PR_FALSE),
-  mPrintPageDelay(500),
-  mPrintSilent(PR_FALSE)
 {
   NS_INIT_ISUPPORTS();
 

layout/base/nsBidiPresUtils.cpp

@@ -69,12 +69,12 @@ NS_NewContinuingTextFrame(nsIPresShell* aPresShell, nsIFrame** aResult);
 extern nsresult
 NS_NewDirectionalFrame(nsIFrame** aNewFrame, PRUnichar aChar);
 
-nsBidiPresUtils::nsBidiPresUtils() : mSuccess(NS_ERROR_FAILURE),
+nsBidiPresUtils::nsBidiPresUtils() : mArraySize(8),
-                                     mBidiEngine(nsnull),
-                                     mUnicodeUtils(nsnull),
-                                     mArraySize(8),
                                      mIndexMap(nsnull),
-                                     mLevels(nsnull)
+                                     mLevels(nsnull),
+                                     mSuccess(NS_ERROR_FAILURE),
+                                     mBidiEngine(nsnull),
+                                     mUnicodeUtils(nsnull)
 {
   mBidiEngine = do_GetService("@mozilla.org/intl/bidi;1");
   if (mBidiEngine) {

layout/base/nsPresShell.cpp

@@ -252,7 +252,13 @@ protected:
 // Counting Class
 class IndiReflowCounter {
 public:
-  IndiReflowCounter(ReflowCountMgr * aMgr = nsnull):mMgr(aMgr),mCounter(aMgr),mFrame(nsnull), mCount(0), mHasBeenOutput(PR_FALSE) {}
+  IndiReflowCounter(ReflowCountMgr * aMgr = nsnull)
+    : mFrame(nsnull),
+      mCount(0),
+      mMgr(aMgr),
+      mCounter(aMgr),
+      mHasBeenOutput(PR_FALSE)
+    {}
   virtual ~IndiReflowCounter() {}
 
   nsAutoString mName;

layout/base/src/nsBidiPresUtils.cpp

@@ -69,12 +69,12 @@ NS_NewContinuingTextFrame(nsIPresShell* aPresShell, nsIFrame** aResult);
 extern nsresult
 NS_NewDirectionalFrame(nsIFrame** aNewFrame, PRUnichar aChar);
 
-nsBidiPresUtils::nsBidiPresUtils() : mSuccess(NS_ERROR_FAILURE),
+nsBidiPresUtils::nsBidiPresUtils() : mArraySize(8),
-                                     mBidiEngine(nsnull),
-                                     mUnicodeUtils(nsnull),
-                                     mArraySize(8),
                                      mIndexMap(nsnull),
-                                     mLevels(nsnull)
+                                     mLevels(nsnull),
+                                     mSuccess(NS_ERROR_FAILURE),
+                                     mBidiEngine(nsnull),
+                                     mUnicodeUtils(nsnull)
 {
   mBidiEngine = do_GetService("@mozilla.org/intl/bidi;1");
   if (mBidiEngine) {

layout/generic/nsTextTransformer.cpp

@@ -162,8 +162,8 @@ nsTextTransformer::nsTextTransformer(nsILineBreaker* aLineBreaker,
     mLineBreaker(aLineBreaker),
     mWordBreaker(aWordBreaker),
     mBufferPos(0),
-    mFlags(0),
+    mTextTransform(NS_STYLE_TEXT_TRANSFORM_NONE),
-    mTextTransform(NS_STYLE_TEXT_TRANSFORM_NONE)
+    mFlags(0)
 {
   MOZ_COUNT_CTOR(nsTextTransformer);
 

layout/html/base/src/nsPresShell.cpp

@@ -252,7 +252,13 @@ protected:
 // Counting Class
 class IndiReflowCounter {
 public:
-  IndiReflowCounter(ReflowCountMgr * aMgr = nsnull):mMgr(aMgr),mCounter(aMgr),mFrame(nsnull), mCount(0), mHasBeenOutput(PR_FALSE) {}
+  IndiReflowCounter(ReflowCountMgr * aMgr = nsnull)
+    : mFrame(nsnull),
+      mCount(0),
+      mMgr(aMgr),
+      mCounter(aMgr),
+      mHasBeenOutput(PR_FALSE)
+    {}
   virtual ~IndiReflowCounter() {}
 
   nsAutoString mName;

layout/html/base/src/nsTextTransformer.cpp

@@ -162,8 +162,8 @@ nsTextTransformer::nsTextTransformer(nsILineBreaker* aLineBreaker,
     mLineBreaker(aLineBreaker),
     mWordBreaker(aWordBreaker),
     mBufferPos(0),
-    mFlags(0),
+    mTextTransform(NS_STYLE_TEXT_TRANSFORM_NONE),
-    mTextTransform(NS_STYLE_TEXT_TRANSFORM_NONE)
+    mFlags(0)
 {
   MOZ_COUNT_CTOR(nsTextTransformer);
 

layout/style/nsCSSDeclaration.cpp

@@ -227,11 +227,11 @@ nsCSSText::nsCSSText(void)
 nsCSSText::nsCSSText(const nsCSSText& aCopy)
   : mWordSpacing(aCopy.mWordSpacing),
     mLetterSpacing(aCopy.mLetterSpacing),
-    mDecoration(aCopy.mDecoration),
     mVerticalAlign(aCopy.mVerticalAlign),
     mTextTransform(aCopy.mTextTransform),
     mTextAlign(aCopy.mTextAlign),
     mTextIndent(aCopy.mTextIndent),
+    mDecoration(aCopy.mDecoration),
     mTextShadow(nsnull),
     mUnicodeBidi(aCopy.mUnicodeBidi),
     mLineHeight(aCopy.mLineHeight),

layout/style/nsCSSStruct.cpp

@@ -227,11 +227,11 @@ nsCSSText::nsCSSText(void)
 nsCSSText::nsCSSText(const nsCSSText& aCopy)
   : mWordSpacing(aCopy.mWordSpacing),
     mLetterSpacing(aCopy.mLetterSpacing),
-    mDecoration(aCopy.mDecoration),
     mVerticalAlign(aCopy.mVerticalAlign),
     mTextTransform(aCopy.mTextTransform),
     mTextAlign(aCopy.mTextAlign),
     mTextIndent(aCopy.mTextIndent),
+    mDecoration(aCopy.mDecoration),
     mTextShadow(nsnull),
     mUnicodeBidi(aCopy.mUnicodeBidi),
     mLineHeight(aCopy.mLineHeight),

layout/svg/base/src/nsSVGPathFrame.cpp

@@ -81,9 +81,9 @@ NS_NewSVGPathFrame(nsIPresShell* aPresShell, nsIFrame** aNewFrame)
 // nsSVGPathFrame cntr
 //
 nsSVGPathFrame::nsSVGPathFrame() :
-  mPath(nsnull),
   mX(0),
-  mY(0)
+  mY(0),
+  mPath(nsnull)
 {
 
 }

layout/xul/base/src/nsPopupSetFrame.cpp

@@ -73,9 +73,9 @@
 
 nsPopupFrameList::nsPopupFrameList(nsIContent* aPopupContent, nsPopupFrameList* aNext)
 :mNextPopup(aNext), 
- mElementContent(nsnull), 
- mPopupContent(aPopupContent),
  mPopupFrame(nsnull),
+ mPopupContent(aPopupContent),
+ mElementContent(nsnull), 
  mCreateHandlerSucceeded(PR_FALSE),
  mLastPref(-1,-1)
 {

layout/xul/base/src/nsXULTreeGroupFrame.cpp

@@ -104,10 +104,18 @@ NS_INTERFACE_MAP_END_INHERITING(nsBoxFrame)
 
 // Constructor
 nsXULTreeGroupFrame::nsXULTreeGroupFrame(nsIPresShell* aPresShell, PRBool aIsRoot, nsIBoxLayout* aLayoutManager)
-:nsBoxFrame(aPresShell, aIsRoot, aLayoutManager), mFrameConstructor(nsnull), mPresContext(nsnull),
+  : nsBoxFrame(aPresShell, aIsRoot, aLayoutManager),
- mOuterFrame(nsnull), mAvailableHeight(10000), mTopFrame(nsnull), mBottomFrame(nsnull), mLinkupFrame(nsnull),
+    mFrameConstructor(nsnull),
- mContentChain(nsnull), mYDropLoc(nsTreeItemDragCapturer::kNoDropLoc), mDropOnContainer(PR_FALSE),
+    mPresContext(nsnull),
- mOnScreenRowCount(-1)
+    mOuterFrame(nsnull),
+    mAvailableHeight(10000),
+    mTopFrame(nsnull),
+    mBottomFrame(nsnull),
+    mLinkupFrame(nsnull),
+    mContentChain(nsnull),
+    mOnScreenRowCount(-1),
+    mYDropLoc(nsTreeItemDragCapturer::kNoDropLoc),
+    mDropOnContainer(PR_FALSE)
 {}
 
 // Destructor

layout/xul/base/src/nsXULTreeOuterGroupFrame.cpp

@@ -259,11 +259,25 @@ NS_NewXULTreeOuterGroupFrame(nsIPresShell* aPresShell, nsIFrame** aNewFrame, PRB
 // Constructor
 nsXULTreeOuterGroupFrame::nsXULTreeOuterGroupFrame(nsIPresShell* aPresShell, PRBool aIsRoot, nsIBoxLayout* aLayoutManager)
   : nsXULTreeGroupFrame(aPresShell, aIsRoot, aLayoutManager),
- mBatchCount(0), mRowGroupInfo(nsnull), mRowHeight(0), mCurrentIndex(0), mOldIndex(0),
+    mBatchCount(0),
- mTreeIsSorted(PR_FALSE), mDragOverListener(nsnull), mCanDropBetweenRows(PR_TRUE),
+    mRowGroupInfo(nsnull),
- mRowHeightWasSet(PR_FALSE), mReflowCallbackPosted(PR_FALSE), mYPosition(0), mScrolling(PR_FALSE),
+    mRowHeight(0),
- mScrollSmoother(nsnull), mTimePerRow(TIME_PER_ROW_INITAL), mAdjustScroll(PR_FALSE), mTreeItemTag(nsXULAtoms::treeitem),
+    mCurrentIndex(0),
- mTreeRowTag(nsXULAtoms::treerow), mTreeChildrenTag(nsXULAtoms::treechildren), mStringWidth(-1)
+    mOldIndex(0),
+    mTreeIsSorted(PR_FALSE),
+    mCanDropBetweenRows(PR_TRUE),
+    mDragOverListener(nsnull),
+    mRowHeightWasSet(PR_FALSE),
+    mReflowCallbackPosted(PR_FALSE),
+    mScrolling(PR_FALSE),
+    mAdjustScroll(PR_FALSE),
+    mYPosition(0),
+    mScrollSmoother(nsnull),
+    mTimePerRow(TIME_PER_ROW_INITAL),
+    mTreeItemTag(nsXULAtoms::treeitem),
+    mTreeRowTag(nsXULAtoms::treerow),
+    mTreeChildrenTag(nsXULAtoms::treechildren),
+    mStringWidth(-1)
 {
 }
 

mailnews/addrbook/src/nsAbMDBDirectory.cpp

@@ -71,8 +71,8 @@ nsAbMDBDirectory::nsAbMDBDirectory(void)
   :  nsAbMDBRDFResource(),
      mInitialized(PR_FALSE),
 	 mIsMailingList(-1),
-     mPerformingQuery(PR_FALSE),
+     mIsQueryURI(PR_FALSE),
-     mIsQueryURI(PR_FALSE)
+     mPerformingQuery(PR_FALSE)
 {
 	NS_NewISupportsArray(getter_AddRefs(mSubDirectories));
 }

mailnews/imap/src/nsImapMailFolder.cpp

@@ -5415,9 +5415,10 @@ nsImapMailFolder::CopyStreamMessage(nsIMsgDBHdr* message,
 }
 
 nsImapMailCopyState::nsImapMailCopyState() : m_msgService(nsnull),
-    m_isMove(PR_FALSE), m_selectedState(PR_FALSE), m_curIndex(0),
+    m_isMove(PR_FALSE), m_selectedState(PR_FALSE),
+    m_isCrossServerOp(PR_FALSE), m_curIndex(0),
     m_totalCount(0), m_streamCopy(PR_FALSE), m_dataBuffer(nsnull),
-    m_leftOver(0), m_isCrossServerOp(PR_FALSE), m_allowUndo(PR_FALSE)
+    m_leftOver(0), m_allowUndo(PR_FALSE)
 {
     NS_INIT_REFCNT();
 }

modules/oji/src/nsCSecurityContext.cpp

@@ -233,8 +233,8 @@ MOZ_DECL_CTOR_COUNTER(nsCSecurityContext)
 nsCSecurityContext::nsCSecurityContext(JSContext* cx)
                    : m_pJStoJavaFrame(NULL), m_pJSCX(cx),
                      m_pPrincipal(NULL),
-                     m_HasUniversalBrowserReadCapability(PR_FALSE),
+                     m_HasUniversalJavaCapability(PR_FALSE),
-                     m_HasUniversalJavaCapability(PR_FALSE)
+                     m_HasUniversalBrowserReadCapability(PR_FALSE)
 {
     MOZ_COUNT_CTOR(nsCSecurityContext);
     NS_INIT_REFCNT();
@@ -277,8 +277,8 @@ nsCSecurityContext::nsCSecurityContext(JSContext* cx)
 nsCSecurityContext::nsCSecurityContext(nsIPrincipal *principal)
                    : m_pJStoJavaFrame(NULL), m_pJSCX(NULL),
                      m_pPrincipal(principal),
-                     m_HasUniversalBrowserReadCapability(PR_FALSE),
+                     m_HasUniversalJavaCapability(PR_FALSE),
-                     m_HasUniversalJavaCapability(PR_FALSE)
+                     m_HasUniversalBrowserReadCapability(PR_FALSE)
 {
     MOZ_COUNT_CTOR(nsCSecurityContext);
     NS_INIT_REFCNT();

netwerk/streamconv/converters/nsBinHexDecoder.cpp

@@ -63,9 +63,10 @@ static NS_DEFINE_CID(kStreamConverterServiceCID, NS_STREAMCONVERTERSERVICE_CID);
 #define CR  '\015'
 #define LF '\012'
 
-nsBinHexDecoder::nsBinHexDecoder() : mCRC(0), mFileCRC(0), mOctetin(26), 
+nsBinHexDecoder::nsBinHexDecoder() :
+  mState(0), mCRC(0), mFileCRC(0), mOctetin(26), 
   mDonePos(3), mInCRC(0), mCount(0), mMarker(0), mPosInbuff(0), 
-  mPosOutputBuff(0), mState(0)
+  mPosOutputBuff(0)
 {
   NS_INIT_ISUPPORTS();
   mDataBuffer = nsnull;

widget/src/xpwidgets/nsTransferable.cpp

@@ -77,7 +77,7 @@ NS_IMPL_ISUPPORTS1(nsTransferable, nsITransferable)
 struct DataStruct
 {
   DataStruct ( const char* aFlavor )
-    : mFlavor(aFlavor), mDataLen(0), mCacheFileName(nsnull) { }
+    : mDataLen(0), mFlavor(aFlavor), mCacheFileName(nsnull) { }
   ~DataStruct();
   
   const nsCString& GetFlavor() const { return mFlavor; }

xpfe/appshell/src/nsXULWindow.cpp

@@ -98,8 +98,9 @@ static NS_DEFINE_CID(kWindowMediatorCID, NS_WINDOWMEDIATOR_CID);
 nsXULWindow::nsXULWindow() : mChromeTreeOwner(nsnull), 
    mContentTreeOwner(nsnull), mPrimaryContentTreeOwner(nsnull),
    mContinueModalLoop(PR_FALSE), mModalStatus(NS_OK), mChromeLoaded(PR_FALSE), 
-   mShowAfterLoad(PR_FALSE), mIntrinsicallySized(PR_FALSE), mCenterAfterLoad(PR_FALSE),
+   mShowAfterLoad(PR_FALSE), mIntrinsicallySized(PR_FALSE),
-   mHadChildWindow(PR_FALSE), mZlevel(nsIXULWindow::normalZ), mIsHiddenWindow(PR_FALSE)
+   mCenterAfterLoad(PR_FALSE), mIsHiddenWindow(PR_FALSE),
+   mHadChildWindow(PR_FALSE), mZlevel(nsIXULWindow::normalZ)
 {
   NS_INIT_REFCNT();
 }

xpfe/components/history/src/nsGlobalHistory.h

@@ -398,8 +398,8 @@ protected:
       mHistory(aHistory),
       mURLColumn(aURLColumn),
       mCommentColumn(aCommentColumn),
-      mSelectValue(aSelectValue),
+      mExclude(aExclude),
-      mExclude(aExclude) {}
+      mSelectValue(aSelectValue) {}
 
   protected:
     virtual PRBool   IsResult(nsIMdbRow* aRow);

xpinstall/wizard/libxpnet/src/nsSocket.cpp

@@ -77,8 +77,8 @@ static int sbWinSockInited = FALSE;
 #endif
 
 nsSocket::nsSocket(char *aHost, int aPort, int (*aEventPumpCB)(void)) :
-    mHost(aHost),
     mEventPumpCB( aEventPumpCB ),
+    mHost(aHost),
     mPort(aPort),
     mFd(-1),
     mListenFd(-1)
@@ -86,8 +86,8 @@ nsSocket::nsSocket(char *aHost, int aPort, int (*aEventPumpCB)(void)) :
 }
 
 nsSocket::nsSocket(char *aHost, int aPort) :
-    mHost(aHost),
     mEventPumpCB( NULL ),
+    mHost(aHost),
     mPort(aPort),
     mFd(-1),
     mListenFd(-1)