Fix a few more -Wreorder warnings.
authorNico Weber <thakis@chromium.org>
Thu, 31 Jul 2014 17:07:04 +0000 (10:07 -0700)
committerNico Weber <thakis@chromium.org>
Thu, 31 Jul 2014 17:07:04 +0000 (10:07 -0700)
No intended behavior change.

BUG=pdfium:29
R=bo_xu@foxitsoftware.com

Review URL: https://codereview.chromium.org/436483002

core/src/fpdfdoc/doc_vt.cpp
fpdfsdk/include/javascript/Document.h
fpdfsdk/src/fsdk_mgr.cpp
fpdfsdk/src/fxedit/fxet_list.cpp
fpdfsdk/src/javascript/Document.cpp
fpdfsdk/src/javascript/JS_Runtime.cpp
fpdfsdk/src/javascript/JS_Value.cpp
fpdfsdk/src/pdfwindow/PWL_Note.cpp

index f3b9bbb..fd3f5ba 100644 (file)
@@ -317,7 +317,7 @@ void CSection::ClearWord(const CPVT_WordPlace & place)
     delete m_WordArray.GetAt(place.nWordIndex);
     m_WordArray.RemoveAt(place.nWordIndex);
 }
-CTypeset::CTypeset(CSection * pSection) : m_pSection(pSection), m_pVT(pSection->m_pVT), m_rcRet(0.0f, 0.0f, 0.0f, 0.0f)
+CTypeset::CTypeset(CSection * pSection) : m_rcRet(0.0f, 0.0f, 0.0f, 0.0f), m_pVT(pSection->m_pVT), m_pSection(pSection)
 {
 }
 CTypeset::~CTypeset()
@@ -804,21 +804,21 @@ void CTypeset::OutputLines()
     m_rcRet = CPVT_FloatRect(fMinX, fMinY, fMaxX, fMaxY);
 }
 CPDF_VariableText::CPDF_VariableText() :
-    m_pVTProvider(NULL),
-    m_pVTIterator(NULL),
-    m_bInitial(FALSE),
-    m_bRichText(FALSE),
+    m_nLimitChar(0),
+    m_nCharArray(0),
     m_bMultiLine(FALSE),
     m_bLimitWidth(FALSE),
     m_bAutoFontSize(FALSE),
-    m_nLimitChar(0),
-    m_nCharArray(0),
     m_nAlignment(0),
+    m_fLineLeading(0.0f),
     m_fCharSpace(0.0f),
-    m_fFontSize(0.0f),
     m_nHorzScale(100),
     m_wSubWord(0),
-    m_fLineLeading(0.0f)
+    m_fFontSize(0.0f),
+    m_bInitial(FALSE),
+    m_bRichText(FALSE),
+    m_pVTProvider(NULL),
+    m_pVTIterator(NULL)
 {
 }
 CPDF_VariableText::~CPDF_VariableText()
@@ -1687,8 +1687,8 @@ IPDF_VariableText_Provider*       CPDF_VariableText::SetProvider(IPDF_VariableText_Pro
     return pOld;
 }
 CPDF_VariableText_Iterator::CPDF_VariableText_Iterator(CPDF_VariableText * pVT):
-    m_pVT(pVT),
-    m_CurPos(-1, -1, -1)
+    m_CurPos(-1, -1, -1),
+    m_pVT(pVT)
 {
 }
 CPDF_VariableText_Iterator::~CPDF_VariableText_Iterator()
index ca8b51b..764bbbc 100644 (file)
@@ -41,7 +41,7 @@ class Field;
 
 struct IconElement
 {
-       IconElement() : IconName(L""), IconStream(NULL), NextIcon(NULL){}
+       IconElement() : IconName(L""), NextIcon(NULL), IconStream(NULL) {}
        virtual ~IconElement()
        {
        }
index 9052c37..52173b4 100644 (file)
@@ -218,9 +218,14 @@ CJS_RuntimeFactory* GetJSRuntimeFactory()
        return &s_JSRuntimeFactory;
 }
 
-CPDFDoc_Environment::CPDFDoc_Environment(CPDF_Document * pDoc):m_pInfo(NULL),m_pIFormFiller(NULL),
-                                                               m_pAnnotHandlerMgr(NULL),m_pActionHandler(NULL),m_pJSRuntime(NULL),
-                                                               m_pSDKDoc(NULL), m_pPDFDoc(pDoc)
+CPDFDoc_Environment::CPDFDoc_Environment(CPDF_Document* pDoc) :
+       m_pAnnotHandlerMgr(NULL),
+       m_pActionHandler(NULL),
+       m_pJSRuntime(NULL),
+       m_pInfo(NULL),
+       m_pSDKDoc(NULL),
+       m_pPDFDoc(pDoc),
+       m_pIFormFiller(NULL)
 {
 
        m_pSysHandler = NULL;
@@ -595,7 +600,7 @@ CFX_WideString      CPDFSDK_Document::GetPath()
 }
 
 
-CPDFSDK_PageView::CPDFSDK_PageView(CPDFSDK_Document* pSDKDoc,CPDF_Page* page):m_pSDKDoc(pSDKDoc),m_page(page)
+CPDFSDK_PageView::CPDFSDK_PageView(CPDFSDK_Document* pSDKDoc,CPDF_Page* page):m_page(page),m_pSDKDoc(pSDKDoc)
 {
        CPDFSDK_InterForm* pInterForm = pSDKDoc->GetInterForm();
        if(pInterForm)
index 63aee1b..64ed694 100644 (file)
@@ -119,7 +119,7 @@ CFX_WideString CFX_ListItem::GetText() const
 
 /* ------------------------------------ CFX_List --------------------------------- */
 
-CFX_List::CFX_List() : m_pFontMap(NULL), m_fFontSize(0.0f), m_bMultiple(FALSE) 
+CFX_List::CFX_List() : m_fFontSize(0.0f), m_pFontMap(NULL), m_bMultiple(FALSE)
 {
 }
 
@@ -507,12 +507,12 @@ void CPLST_Select::Done()
 /* ------------------------------------ CFX_ListCtrl --------------------------------- */
 
 CFX_ListCtrl::CFX_ListCtrl() : m_pNotify(NULL),
+       m_bNotifyFlag(FALSE),
        m_ptScrollPos(0.0f,0.0f),
        m_nSelItem(-1),
        m_nFootIndex(-1),
-       m_bCtrlSel(FALSE),      
-       m_nCaretIndex(-1),
-       m_bNotifyFlag(FALSE)
+       m_bCtrlSel(FALSE),
+       m_nCaretIndex(-1)
 {
 }
 
index 0c4d4f4..229d977 100644 (file)
@@ -158,11 +158,11 @@ FX_BOOL   CJS_Document::InitInstance(IFXJS_Context* cc)
 /* --------------------------------- Document --------------------------------- */
 
 Document::Document(CJS_Object* pJSObject) : CJS_EmbedObj(pJSObject),
-       m_cwBaseURL(L""),
+       m_isolate(NULL),
        m_pIconTree(NULL),
        m_pDocument(NULL),
-       m_bDelay(FALSE),
-       m_isolate(NULL)
+       m_cwBaseURL(L""),
+       m_bDelay(FALSE)
 {
 }
 
index 543540b..be11b04 100644 (file)
@@ -104,8 +104,8 @@ void CJS_RuntimeFactory::ReleaseGlobalData()
 CJS_Runtime::CJS_Runtime(CPDFDoc_Environment * pApp) : 
        m_pApp(pApp),
        m_pDocument(NULL),
-       m_pFieldEventPath(NULL),
        m_bBlocking(FALSE),
+       m_pFieldEventPath(NULL),
        m_bRegistered(FALSE)
 {
        m_isolate = v8::Isolate::New();
index 0217ea1..90b324a 100644 (file)
 
 /* ---------------------------- CJS_Value ---------------------------- */
 
-CJS_Value::CJS_Value(v8::Isolate* isolate) : m_isolate(isolate),m_eType(VT_unknown)
+CJS_Value::CJS_Value(v8::Isolate* isolate) : m_eType(VT_unknown),m_isolate(isolate)
 {
 }
-CJS_Value::CJS_Value(v8::Isolate* isolate, v8::Handle<v8::Value> pValue,FXJSVALUETYPE t) :m_isolate(isolate), m_pValue(pValue) , m_eType(t)
+CJS_Value::CJS_Value(v8::Isolate* isolate, v8::Handle<v8::Value> pValue,FXJSVALUETYPE t) :
+       m_pValue(pValue), m_eType(t), m_isolate(isolate)
 {
 }
 
index b04b0f5..b0028b6 100644 (file)
@@ -681,10 +681,10 @@ void CPWL_Note_Contents::EnableRead(FX_BOOL bEnabled)
 /* ---------------------------------- CPWL_NoteItem ---------------------------------- */
 
 CPWL_NoteItem::CPWL_NoteItem() :
-       m_pPrivateData(NULL),
        m_pSubject(NULL),
        m_pDateTime(NULL),
        m_pContents(NULL),
+       m_pPrivateData(NULL),
        m_sAuthor(L""),
        m_fOldItemHeight(0.0f),
        m_bSizeChanged(FALSE),
@@ -1205,13 +1205,13 @@ CPWL_Note::CPWL_Note(IPopup_Note* pPopupNote, IPWL_NoteNotify* pNoteNotify, IPWL
        m_pAuthor(NULL),
        m_pIcon(NULL),
        m_pCloseBox(NULL),
-       m_pContentsBar(NULL),
        m_pLBBox(NULL),
        m_pRBBox(NULL),
+       m_pContentsBar(NULL),
        m_pOptions(NULL),
+       m_pNoteNotify(pNoteNotify),
        m_bResizing(FALSE),
        m_rcCaption(0,0,0,0),
-       m_pNoteNotify(pNoteNotify),
        m_bEnalbleNotify(TRUE),
        m_pPopupNote(pPopupNote)
 {