From 4bed2d42879537cf0fa7bbc0caefaed480df8c56 Mon Sep 17 00:00:00 2001 From: "dbaron@fas.harvard.edu" Date: Thu, 25 Oct 2001 03:21:53 +0000 Subject: [PATCH] Fix member initializer re-ordered warnings by re-ordering member initializers. r=jag rs=waterson b=105974 --- content/base/src/nsDOMAttribute.cpp | 2 +- .../html/content/src/nsHTMLFormElement.cpp | 5 ++-- content/html/style/src/nsCSSDeclaration.cpp | 2 +- content/html/style/src/nsCSSStruct.cpp | 2 +- .../document/src/nsXULCommandDispatcher.cpp | 2 +- db/mork/src/morkEnv.cpp | 4 +-- .../webBrowser/nsDocShellTreeOwner.cpp | 7 +++-- embedding/browser/webBrowser/nsWebBrowser.cpp | 12 ++++++--- .../webBrowser/nsWebBrowserPersist.cpp | 4 +-- gfx/src/nsPrintOptionsImpl.cpp | 14 +++++----- layout/base/nsBidiPresUtils.cpp | 10 +++---- layout/base/nsPresShell.cpp | 8 +++++- layout/base/src/nsBidiPresUtils.cpp | 10 +++---- layout/generic/nsTextTransformer.cpp | 4 +-- layout/html/base/src/nsPresShell.cpp | 8 +++++- layout/html/base/src/nsTextTransformer.cpp | 4 +-- layout/style/nsCSSDeclaration.cpp | 2 +- layout/style/nsCSSStruct.cpp | 2 +- layout/svg/base/src/nsSVGPathFrame.cpp | 4 +-- layout/xul/base/src/nsPopupSetFrame.cpp | 4 +-- layout/xul/base/src/nsXULTreeGroupFrame.cpp | 16 +++++++++--- .../xul/base/src/nsXULTreeOuterGroupFrame.cpp | 26 ++++++++++++++----- modules/oji/src/nsCSecurityContext.cpp | 8 +++--- .../streamconv/converters/nsBinHexDecoder.cpp | 5 ++-- widget/src/xpwidgets/nsTransferable.cpp | 2 +- xpfe/appshell/src/nsXULWindow.cpp | 5 ++-- xpfe/components/history/src/nsGlobalHistory.h | 4 +-- xpinstall/wizard/libxpnet/src/nsSocket.cpp | 4 +-- 28 files changed, 112 insertions(+), 68 deletions(-) diff --git a/content/base/src/nsDOMAttribute.cpp b/content/base/src/nsDOMAttribute.cpp index 81714a5f8268..1bdf5cd97db3 100644 --- a/content/base/src/nsDOMAttribute.cpp +++ b/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!"); diff --git a/content/html/content/src/nsHTMLFormElement.cpp b/content/html/content/src/nsHTMLFormElement.cpp index 089aea6e8a6b..e3af81dc4437 100644 --- a/content/html/content/src/nsHTMLFormElement.cpp +++ b/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), - mNoNameLookupTable(nsnull) + : mForm(aForm), + mNoNameLookupTable(nsnull), + mNameLookupTable(NS_FORM_CONTROL_LIST_HASHTABLE_SIZE) { NS_INIT_REFCNT(); } diff --git a/content/html/style/src/nsCSSDeclaration.cpp b/content/html/style/src/nsCSSDeclaration.cpp index 84e72f57e176..2cc40c360910 100644 --- a/content/html/style/src/nsCSSDeclaration.cpp +++ b/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), diff --git a/content/html/style/src/nsCSSStruct.cpp b/content/html/style/src/nsCSSStruct.cpp index 84e72f57e176..2cc40c360910 100644 --- a/content/html/style/src/nsCSSStruct.cpp +++ b/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), diff --git a/content/xul/document/src/nsXULCommandDispatcher.cpp b/content/xul/document/src/nsXULCommandDispatcher.cpp index f34561b602c2..fee5cef62ab5 100644 --- a/content/xul/document/src/nsXULCommandDispatcher.cpp +++ b/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(); diff --git a/db/mork/src/morkEnv.cpp b/db/mork/src/morkEnv.cpp index 1d88dac0fa30..e7bc39a580d7 100644 --- a/db/mork/src/morkEnv.cpp +++ b/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?? diff --git a/embedding/browser/webBrowser/nsDocShellTreeOwner.cpp b/embedding/browser/webBrowser/nsDocShellTreeOwner.cpp index 951fa239bdfc..2f56421978e1 100644 --- a/embedding/browser/webBrowser/nsDocShellTreeOwner.cpp +++ b/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 diff --git a/embedding/browser/webBrowser/nsWebBrowser.cpp b/embedding/browser/webBrowser/nsWebBrowser.cpp index 5b66d001121c..44ae91ca484e 100644 --- a/embedding/browser/webBrowser/nsWebBrowser.cpp +++ b/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), - mParentNativeWindow(nsnull), mParentWidget(nsnull), mParent(nsnull), - mProgressListener(nsnull), mListenerArray(nsnull), + mInitInfo(nsnull), + mContentType(typeContentWrapper), + 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(); diff --git a/embedding/browser/webBrowser/nsWebBrowserPersist.cpp b/embedding/browser/webBrowser/nsWebBrowserPersist.cpp index 2368467a0aeb..4b45c7c72fc1 100644 --- a/embedding/browser/webBrowser/nsWebBrowserPersist.cpp +++ b/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(); } diff --git a/gfx/src/nsPrintOptionsImpl.cpp b/gfx/src/nsPrintOptionsImpl.cpp index ac17aa9ffbc8..107b119f0f9a 100644 --- a/gfx/src/nsPrintOptionsImpl.cpp +++ b/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), - mPrintFrameType(kFramesAsIs), - mHowToEnableFrameUI(kFrameEnableNone), - mIsCancelled(PR_FALSE), - mPrintPageDelay(500), - mPrintSilent(PR_FALSE) + mPrintToFile(PR_FALSE) { NS_INIT_ISUPPORTS(); diff --git a/layout/base/nsBidiPresUtils.cpp b/layout/base/nsBidiPresUtils.cpp index 72dbb9b35902..5815c6de071a 100644 --- a/layout/base/nsBidiPresUtils.cpp +++ b/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), - mBidiEngine(nsnull), - mUnicodeUtils(nsnull), - mArraySize(8), +nsBidiPresUtils::nsBidiPresUtils() : 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) { diff --git a/layout/base/nsPresShell.cpp b/layout/base/nsPresShell.cpp index 141fc8729c23..14faf6c1e2a6 100644 --- a/layout/base/nsPresShell.cpp +++ b/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; diff --git a/layout/base/src/nsBidiPresUtils.cpp b/layout/base/src/nsBidiPresUtils.cpp index 72dbb9b35902..5815c6de071a 100644 --- a/layout/base/src/nsBidiPresUtils.cpp +++ b/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), - mBidiEngine(nsnull), - mUnicodeUtils(nsnull), - mArraySize(8), +nsBidiPresUtils::nsBidiPresUtils() : 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) { diff --git a/layout/generic/nsTextTransformer.cpp b/layout/generic/nsTextTransformer.cpp index 8fb2bdc0914a..559c25ab7aa9 100644 --- a/layout/generic/nsTextTransformer.cpp +++ b/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); diff --git a/layout/html/base/src/nsPresShell.cpp b/layout/html/base/src/nsPresShell.cpp index 141fc8729c23..14faf6c1e2a6 100644 --- a/layout/html/base/src/nsPresShell.cpp +++ b/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; diff --git a/layout/html/base/src/nsTextTransformer.cpp b/layout/html/base/src/nsTextTransformer.cpp index 8fb2bdc0914a..559c25ab7aa9 100644 --- a/layout/html/base/src/nsTextTransformer.cpp +++ b/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); diff --git a/layout/style/nsCSSDeclaration.cpp b/layout/style/nsCSSDeclaration.cpp index 84e72f57e176..2cc40c360910 100644 --- a/layout/style/nsCSSDeclaration.cpp +++ b/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), diff --git a/layout/style/nsCSSStruct.cpp b/layout/style/nsCSSStruct.cpp index 84e72f57e176..2cc40c360910 100644 --- a/layout/style/nsCSSStruct.cpp +++ b/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), diff --git a/layout/svg/base/src/nsSVGPathFrame.cpp b/layout/svg/base/src/nsSVGPathFrame.cpp index a9cc86d97f2b..218b0b516fa8 100644 --- a/layout/svg/base/src/nsSVGPathFrame.cpp +++ b/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) { } diff --git a/layout/xul/base/src/nsPopupSetFrame.cpp b/layout/xul/base/src/nsPopupSetFrame.cpp index 6c4d03d0ae18..049ee37b340e 100644 --- a/layout/xul/base/src/nsPopupSetFrame.cpp +++ b/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) { diff --git a/layout/xul/base/src/nsXULTreeGroupFrame.cpp b/layout/xul/base/src/nsXULTreeGroupFrame.cpp index 9ac09ddc2781..9ba6b5111509 100644 --- a/layout/xul/base/src/nsXULTreeGroupFrame.cpp +++ b/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), - mOuterFrame(nsnull), mAvailableHeight(10000), mTopFrame(nsnull), mBottomFrame(nsnull), mLinkupFrame(nsnull), - mContentChain(nsnull), mYDropLoc(nsTreeItemDragCapturer::kNoDropLoc), mDropOnContainer(PR_FALSE), - mOnScreenRowCount(-1) + : nsBoxFrame(aPresShell, aIsRoot, aLayoutManager), + mFrameConstructor(nsnull), + mPresContext(nsnull), + mOuterFrame(nsnull), + mAvailableHeight(10000), + mTopFrame(nsnull), + mBottomFrame(nsnull), + mLinkupFrame(nsnull), + mContentChain(nsnull), + mOnScreenRowCount(-1), + mYDropLoc(nsTreeItemDragCapturer::kNoDropLoc), + mDropOnContainer(PR_FALSE) {} // Destructor diff --git a/layout/xul/base/src/nsXULTreeOuterGroupFrame.cpp b/layout/xul/base/src/nsXULTreeOuterGroupFrame.cpp index 7f05e1104131..12b5a30f58e5 100644 --- a/layout/xul/base/src/nsXULTreeOuterGroupFrame.cpp +++ b/layout/xul/base/src/nsXULTreeOuterGroupFrame.cpp @@ -258,12 +258,26 @@ 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), - mTreeIsSorted(PR_FALSE), mDragOverListener(nsnull), mCanDropBetweenRows(PR_TRUE), - mRowHeightWasSet(PR_FALSE), mReflowCallbackPosted(PR_FALSE), mYPosition(0), mScrolling(PR_FALSE), - mScrollSmoother(nsnull), mTimePerRow(TIME_PER_ROW_INITAL), mAdjustScroll(PR_FALSE), mTreeItemTag(nsXULAtoms::treeitem), - mTreeRowTag(nsXULAtoms::treerow), mTreeChildrenTag(nsXULAtoms::treechildren), mStringWidth(-1) + : nsXULTreeGroupFrame(aPresShell, aIsRoot, aLayoutManager), + mBatchCount(0), + mRowGroupInfo(nsnull), + mRowHeight(0), + mCurrentIndex(0), + 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) { } diff --git a/modules/oji/src/nsCSecurityContext.cpp b/modules/oji/src/nsCSecurityContext.cpp index 12c3d65ab5d0..82c7684e2f9a 100644 --- a/modules/oji/src/nsCSecurityContext.cpp +++ b/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(); diff --git a/netwerk/streamconv/converters/nsBinHexDecoder.cpp b/netwerk/streamconv/converters/nsBinHexDecoder.cpp index 5e497ed4e528..8f8a737f14ad 100644 --- a/netwerk/streamconv/converters/nsBinHexDecoder.cpp +++ b/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; diff --git a/widget/src/xpwidgets/nsTransferable.cpp b/widget/src/xpwidgets/nsTransferable.cpp index 05a7c840ddc0..392dd7abd783 100644 --- a/widget/src/xpwidgets/nsTransferable.cpp +++ b/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; } diff --git a/xpfe/appshell/src/nsXULWindow.cpp b/xpfe/appshell/src/nsXULWindow.cpp index 2f6e19692e7a..d6952b8eff86 100644 --- a/xpfe/appshell/src/nsXULWindow.cpp +++ b/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), - mHadChildWindow(PR_FALSE), mZlevel(nsIXULWindow::normalZ), mIsHiddenWindow(PR_FALSE) + mShowAfterLoad(PR_FALSE), mIntrinsicallySized(PR_FALSE), + mCenterAfterLoad(PR_FALSE), mIsHiddenWindow(PR_FALSE), + mHadChildWindow(PR_FALSE), mZlevel(nsIXULWindow::normalZ) { NS_INIT_REFCNT(); } diff --git a/xpfe/components/history/src/nsGlobalHistory.h b/xpfe/components/history/src/nsGlobalHistory.h index 31e3d9b56fdf..ddc077bb0faf 100644 --- a/xpfe/components/history/src/nsGlobalHistory.h +++ b/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); diff --git a/xpinstall/wizard/libxpnet/src/nsSocket.cpp b/xpinstall/wizard/libxpnet/src/nsSocket.cpp index 894ea50ebefa..5265b0b6b92e 100644 --- a/xpinstall/wizard/libxpnet/src/nsSocket.cpp +++ b/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)