From 4684c83a4e9114a63c64744064dd190e4d459c6f Mon Sep 17 00:00:00 2001 From: Nico Weber Date: Thu, 6 Aug 2015 14:38:03 -0700 Subject: clang-format all pdfium code, again. Also add a presubmit that checks for this so I don't have to keep doing it. No behavior change. Generated by: find . -name '*.cpp' -o -name '*.h' | \ grep -E -v 'third_party|thirdparties|lpng_v163' | \ xargs ../../buildtools/mac/clang-format -i BUG=none R=thestig@chromium.org Review URL: https://codereview.chromium.org/1272653005 . --- fpdfsdk/include/fsdk_baseform.h | 22 ++++----- fpdfsdk/include/fxedit/fxet_list.h | 8 ++-- fpdfsdk/include/javascript/JS_Define.h | 6 +-- fpdfsdk/include/javascript/app.h | 86 ---------------------------------- fpdfsdk/include/pdfwindow/PWL_Wnd.h | 42 ++++++++--------- fpdfsdk/src/fpdfview.cpp | 1 - fpdfsdk/src/fsdk_baseform.cpp | 17 +------ fpdfsdk/src/fxedit/fxet_ap.cpp | 5 -- fpdfsdk/src/javascript/app.cpp | 4 +- fpdfsdk/src/pdfwindow/PWL_ListBox.cpp | 2 +- 10 files changed, 44 insertions(+), 149 deletions(-) (limited to 'fpdfsdk') diff --git a/fpdfsdk/include/fsdk_baseform.h b/fpdfsdk/include/fsdk_baseform.h index b685ae6a8e..7b69e5d7fa 100644 --- a/fpdfsdk/include/fsdk_baseform.h +++ b/fpdfsdk/include/fsdk_baseform.h @@ -43,18 +43,18 @@ typedef struct _PDFSDK_FieldAction { bRC = TRUE; } - FX_BOOL bModifier; // in - FX_BOOL bShift; // in - int nCommitKey; // in - CFX_WideString sChange; // in[out] + FX_BOOL bModifier; // in + FX_BOOL bShift; // in + int nCommitKey; // in + CFX_WideString sChange; // in[out] CFX_WideString sChangeEx; // in - FX_BOOL bKeyDown; // in - int nSelEnd; // in[out] - int nSelStart; // in[out] - CFX_WideString sValue; // in[out] - FX_BOOL bWillCommit; // in - FX_BOOL bFieldFull; // in - FX_BOOL bRC; // in[out] + FX_BOOL bKeyDown; // in + int nSelEnd; // in[out] + int nSelStart; // in[out] + CFX_WideString sValue; // in[out] + FX_BOOL bWillCommit; // in + FX_BOOL bFieldFull; // in + FX_BOOL bRC; // in[out] } PDFSDK_FieldAction; class CPDFSDK_Widget : public CPDFSDK_Annot { public: diff --git a/fpdfsdk/include/fxedit/fxet_list.h b/fpdfsdk/include/fxedit/fxet_list.h index 4aa35684cf..8c5584a118 100644 --- a/fpdfsdk/include/fxedit/fxet_list.h +++ b/fpdfsdk/include/fxedit/fxet_list.h @@ -339,10 +339,10 @@ class CFX_ListCtrl : public CFX_List { FX_BOOL m_bNotifyFlag; CPDF_Point m_ptScrollPos; CPLST_Select m_aSelItems; // for multiple - int32_t m_nSelItem; // for single - int32_t m_nFootIndex; // for multiple - FX_BOOL m_bCtrlSel; // for multiple - int32_t m_nCaretIndex; // for multiple + int32_t m_nSelItem; // for single + int32_t m_nFootIndex; // for multiple + FX_BOOL m_bCtrlSel; // for multiple + int32_t m_nCaretIndex; // for multiple }; #endif // FPDFSDK_INCLUDE_FXEDIT_FXET_LIST_H_ diff --git a/fpdfsdk/include/javascript/JS_Define.h b/fpdfsdk/include/javascript/JS_Define.h index 7c3afd7359..db4b1c360d 100644 --- a/fpdfsdk/include/javascript/JS_Define.h +++ b/fpdfsdk/include/javascript/JS_Define.h @@ -37,10 +37,10 @@ struct JSMethodSpec { #define BEGIN_JS_STATIC_CONST(js_class_name) \ JSConstSpec js_class_name::JS_Class_Consts[] = { #define JS_STATIC_CONST_ENTRY_NUMBER(const_name, pValue) \ - { const_name, pValue, L"", 0 } \ + { const_name, pValue, L"", 0 } \ , -#define JS_STATIC_CONST_ENTRY_STRING(const_name, pValue) \ - { const_name, 0, pValue, 1 } \ +#define JS_STATIC_CONST_ENTRY_STRING(const_name, pValue) \ + { const_name, 0, pValue, 1 } \ , #define END_JS_STATIC_CONST() \ { 0, 0, 0, 0 } \ diff --git a/fpdfsdk/include/javascript/app.h b/fpdfsdk/include/javascript/app.h index 221a7c1d89..b13121323b 100644 --- a/fpdfsdk/include/javascript/app.h +++ b/fpdfsdk/include/javascript/app.h @@ -36,71 +36,6 @@ class CJS_TimerObj : public CJS_Object { DECLARE_JS_CLASS(CJS_TimerObj); }; -// struct APP_MENUITEM_ARRAY; -// -// struct APP_MENUITEM -// { -// APP_MENUITEM() : oSubMenu(NULL), cName(L""), cReturn(L""), -// bMarked(false), bEnabled(true) -// { -// } -// CFX_WideString cName; -// CFX_WideString cReturn; -// APP_MENUITEM_ARRAY* oSubMenu; -// bool bMarked; -// bool bEnabled; -// }; - -// struct APP_MENUITEM_ARRAY -// { -// APP_MENUITEM_ARRAY() : m_hMenu(NULL), pContents(NULL), nSize(0) -// { -// -// } -// APP_MENUITEM * pContents; -// HMENU m_hMenu; -// int nSize; -// }; - -// struct APP_MENU; -// struct APP_MENU_ARRAY -// { -// APP_MENU_ARRAY(): -// pContent(NULL) -// { -// } -// -// APP_MENU* pContent; -// }; - -// struct APP_MENU -// { -// APP_MENU():bSubMenu(false), -// SubMenuItems(NULL), -// cwMenuItemName(L""), -// hMenu(NULL), -// iSize(0) -// { -// -// } -// -// APP_MENU(CFX_WideString &cwName): -// cwMenuItemName(cwName), -// bSubMenu(false), -// SubMenuItems(NULL), -// hMenu(NULL), -// iSize(0) -// { -// -// } -// -// CFX_WideString cwMenuItemName; -// bool bSubMenu; -// APP_MENU_ARRAY* SubMenuItems; -// int iSize; -// HMENU hMenu; -// }; - class app : public CJS_EmbedObj { public: app(CJS_Object* pJSObject); @@ -226,25 +161,8 @@ class app : public CJS_EmbedObj { CFX_WideString& sError); private: - // FX_DWORD AppGetTickCount(); void TimerProc(CJS_Timer* pTimer); void RunJsScript(CJS_Runtime* pRuntime, const CFX_WideString& wsScript); - // void ParsePopupMenuObj(APP_MENUITEM * - //ppMenuItem,JSObject * pObj); - // void DeleteMenuItems(APP_MENUITEM_ARRAY * - //pMenuItems); - // void AddMenuItem(APP_MENUITEM_ARRAY * pMenuItems, HMENU - // hMenu, MENUITEMINFO MenuItemInfo); - // void InitMenuItemInfo(MENUITEMINFO& - // MenuItemInfo); - // void DestroyPopUpMenu(); - - // void ParserMenuItem(APP_MENU* pHead, const - // CJS_Parameters¶ms); - // void AddItemToMenu(APP_MENU* pHead, HMENU hMenu, - // MENUITEMINFO MenuItemInfo); - // void DestroyMenuItems(APP_MENU* - // pHead); public: static CFX_WideString SysPathToPDFPath(const CFX_WideString& sOldPath); @@ -254,10 +172,6 @@ class app : public CJS_EmbedObj { bool m_bRuntimeHighLight; CFX_ArrayTemplate m_aTimer; - // APP_MENU* m_pMenuHead; - - public: - // static CReader_App* s_App; }; class CJS_App : public CJS_Object { diff --git a/fpdfsdk/include/pdfwindow/PWL_Wnd.h b/fpdfsdk/include/pdfwindow/PWL_Wnd.h index 63fd28b6d1..0845bb17d1 100644 --- a/fpdfsdk/include/pdfwindow/PWL_Wnd.h +++ b/fpdfsdk/include/pdfwindow/PWL_Wnd.h @@ -226,28 +226,28 @@ struct PWL_CREATEPARAM { eCursorType(FXCT_ARROW), mtChild(1, 0, 0, 1, 0, 0) {} - CPDF_Rect rcRectWnd; // required + CPDF_Rect rcRectWnd; // required IFX_SystemHandler* pSystemHandler; // required - IFX_Edit_FontMap* pFontMap; // required for text window - IPWL_Provider* pProvider; // required for self coordinate - IPWL_FocusHandler* pFocusHandler; // optional - FX_DWORD dwFlags; // optional - CPWL_Color sBackgroundColor; // optional - FX_HWND hAttachedWnd; // required for no-reader framework - IPWL_SpellCheck* pSpellCheck; // required for spellchecking - int32_t nBorderStyle; // optional - int32_t dwBorderWidth; // optional - CPWL_Color sBorderColor; // optional - CPWL_Color sTextColor; // optional - CPWL_Color sTextStrokeColor; // optional - int32_t nTransparency; // optional - FX_FLOAT fFontSize; // optional - CPWL_Dash sDash; // optional - void* pAttachedData; // optional - CPWL_Wnd* pParentWnd; // ignore - CPWL_MsgControl* pMsgControl; // ignore - int32_t eCursorType; // ignore - CPDF_Matrix mtChild; // ignore + IFX_Edit_FontMap* pFontMap; // required for text window + IPWL_Provider* pProvider; // required for self coordinate + IPWL_FocusHandler* pFocusHandler; // optional + FX_DWORD dwFlags; // optional + CPWL_Color sBackgroundColor; // optional + FX_HWND hAttachedWnd; // required for no-reader framework + IPWL_SpellCheck* pSpellCheck; // required for spellchecking + int32_t nBorderStyle; // optional + int32_t dwBorderWidth; // optional + CPWL_Color sBorderColor; // optional + CPWL_Color sTextColor; // optional + CPWL_Color sTextStrokeColor; // optional + int32_t nTransparency; // optional + FX_FLOAT fFontSize; // optional + CPWL_Dash sDash; // optional + void* pAttachedData; // optional + CPWL_Wnd* pParentWnd; // ignore + CPWL_MsgControl* pMsgControl; // ignore + int32_t eCursorType; // ignore + CPDF_Matrix mtChild; // ignore }; class CPWL_Timer { diff --git a/fpdfsdk/src/fpdfview.cpp b/fpdfsdk/src/fpdfview.cpp index 8895dce076..9ad3384803 100644 --- a/fpdfsdk/src/fpdfview.cpp +++ b/fpdfsdk/src/fpdfview.cpp @@ -155,7 +155,6 @@ DLLEXPORT void STDCALL FPDF_SetSandBoxPolicy(FPDF_DWORD policy, return FSDK_SetSandBoxPolicy(policy, enable); } - DLLEXPORT FPDF_DOCUMENT STDCALL FPDF_LoadDocument(FPDF_STRING file_path, FPDF_BYTESTRING password) { // NOTE: the creation of the file needs to be by the embedder on the diff --git a/fpdfsdk/src/fsdk_baseform.cpp b/fpdfsdk/src/fsdk_baseform.cpp index bc041fd05f..ca0c29efa4 100644 --- a/fpdfsdk/src/fsdk_baseform.cpp +++ b/fpdfsdk/src/fsdk_baseform.cpp @@ -396,8 +396,6 @@ void CPDFSDK_Widget::DrawShadow(CFX_RenderDevice* pDevice, int nFieldType = GetFieldType(); if (m_pInterForm->IsNeedHighLight(nFieldType)) { - // if (nFieldType != FIELDTYPE_PUSHBUTTON) - // { CPDF_Rect rc = GetRect(); FX_COLORREF color = m_pInterForm->GetHighlightColor(nFieldType); uint8_t alpha = m_pInterForm->GetHighlightAlpha(); @@ -411,10 +409,6 @@ void CPDFSDK_Widget::DrawShadow(CFX_RenderDevice* pDevice, pPageView->GetCurrentMatrix(page2device); page2device.Transform(((FX_FLOAT)rc.left), ((FX_FLOAT)rc.bottom), rcDevice.left, rcDevice.bottom); - // pEnv->FFI_PageToDevice(m_pPageView->GetPDFPage(), rc.left, - // rc.bottom, &rcDevice.left, &rcDevice.bottom); - // pEnv->FFI_PageToDevice(m_pPageView->GetPDFPage(), rc.right, - // rc.top, &rcDevice.right, &rcDevice.top); page2device.Transform(((FX_FLOAT)rc.right), ((FX_FLOAT)rc.top), rcDevice.right, rcDevice.top); @@ -424,7 +418,6 @@ void CPDFSDK_Widget::DrawShadow(CFX_RenderDevice* pDevice, FX_RECT rcDev((int)rcDevice.left, (int)rcDevice.top, (int)rcDevice.right, (int)rcDevice.bottom); pDevice->FillRect(&rcDev, argb); - /* }*/ } } @@ -573,10 +566,7 @@ void CPDFSDK_Widget::ResetAppearance_PushButton() { ASSERT(pDoc != NULL); CPDFDoc_Environment* pEnv = pDoc->GetEnv(); - CBA_FontMap FontMap( - this, - pEnv->GetSysHandler()); //, - //ISystemHandle::GetSystemHandler(m_pBaseForm->GetEnv())); + CBA_FontMap FontMap(this, pEnv->GetSysHandler()); FontMap.Initial(); FontMap.SetAPType("N"); @@ -1136,10 +1126,7 @@ void CPDFSDK_Widget::ResetAppearance_TextField(const FX_WCHAR* sValue) { ASSERT(pDoc != NULL); CPDFDoc_Environment* pEnv = pDoc->GetEnv(); - CBA_FontMap FontMap( - this, - pEnv->GetSysHandler()); //, - //ISystemHandle::GetSystemHandler(m_pBaseForm->GetEnv())); + CBA_FontMap FontMap(this, pEnv->GetSysHandler()); FontMap.Initial(); pEdit->SetFontMap(&FontMap); diff --git a/fpdfsdk/src/fxedit/fxet_ap.cpp b/fpdfsdk/src/fxedit/fxet_ap.cpp index b9900d899b..4e92099417 100644 --- a/fpdfsdk/src/fxedit/fxet_ap.cpp +++ b/fpdfsdk/src/fxedit/fxet_ap.cpp @@ -196,11 +196,6 @@ CFX_ByteString IFX_Edit::GetSelectAppearanceStream( CPVT_Word word; CPVT_Line line; if (pIterator->GetWord(word) && pIterator->GetLine(line)) { - // CPDF_Rect rcWordSel = CPDF_Rect(word.ptWord.x,line.ptLine.y + - // line.fLineDescent, - // word.ptWord.x+word.fWidth,line.ptLine.y + - //line.fLineAscent); - sRet << word.ptWord.x + ptOffset.x << " " << line.ptLine.y + line.fLineDescent << " " << word.fWidth << " " << line.fLineAscent - line.fLineDescent << " re\nf\n"; diff --git a/fpdfsdk/src/javascript/app.cpp b/fpdfsdk/src/javascript/app.cpp index 29007a4a2a..24f00ca262 100644 --- a/fpdfsdk/src/javascript/app.cpp +++ b/fpdfsdk/src/javascript/app.cpp @@ -302,10 +302,10 @@ FX_BOOL app::alert(IFXJS_Context* cc, CJS_Value* pValue = new CJS_Value(isolate); // if (iLenth == 1) // pValue = new - //CJS_Value(isolate); + // CJS_Value(isolate); // else if (iLenth > 1) // pValue = new - //CJS_Value[iLenth]; + // CJS_Value[iLenth]; for (int i = 0; i < iLenth; i++) { carray.GetElement(i, *pValue); diff --git a/fpdfsdk/src/pdfwindow/PWL_ListBox.cpp b/fpdfsdk/src/pdfwindow/PWL_ListBox.cpp index ba75e55fdf..b252bffb8e 100644 --- a/fpdfsdk/src/pdfwindow/PWL_ListBox.cpp +++ b/fpdfsdk/src/pdfwindow/PWL_ListBox.cpp @@ -183,7 +183,7 @@ void CPWL_ListBox::DrawThisAppearance(CFX_RenderDevice* pDevice, if (m_pList->IsItemSelected(i)) { // CPWL_Utils::DrawFillRect(pDevice, pUser2Device, rcItem, - //ArgbEncode(255,0,51,113)); + // ArgbEncode(255,0,51,113)); IFX_SystemHandler* pSysHandler = GetSystemHandler(); if (pSysHandler && pSysHandler->IsSelectionImplemented()) { IFX_Edit::DrawEdit( -- cgit v1.2.3