From b553bcb6a8e69de61f3b4abd257ee3161aa752c6 Mon Sep 17 00:00:00 2001 From: Jun Fang Date: Tue, 10 Nov 2015 18:49:04 +0800 Subject: Merge to XFA: Use standard true and false to replace 'TRUE' and 'FALSE' in pdfium_test BUG=446715 TBR=tsepez@chromium.org Review URL: https://codereview.chromium.org/1410073009 . Review URL: https://codereview.chromium.org/1413103007 . --- core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp | 2 +- core/src/fpdfapi/fpdf_parser/parser_int.h | 2 -- samples/pdfium_test.cc | 20 ++++++++++---------- 3 files changed, 11 insertions(+), 13 deletions(-) diff --git a/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp b/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp index 5ba7cbe87e..58420ffcfc 100644 --- a/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp +++ b/core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp @@ -4682,7 +4682,7 @@ FX_BOOL CPDF_HintTables::ReadPageHintTable(CFX_BitStream* hStream) { // where in the page's content stream the object is first referenced. FX_DWORD dwSharedNumeratorBits = hStream->GetBits(16); // Item 13: Skip Item 13 which has 16 bits. - FX_DWORD dwSharedDenominator = hStream->GetBits(16); + hStream->SkipBits(16); CPDF_Object* pPageNum = m_pLinearizedDict->GetElementValue(FX_BSTRC("N")); int nPages = pPageNum ? pPageNum->GetInteger() : 0; if (nPages < 1) diff --git a/core/src/fpdfapi/fpdf_parser/parser_int.h b/core/src/fpdfapi/fpdf_parser/parser_int.h index a9d5874f75..5e4dffa8b7 100644 --- a/core/src/fpdfapi/fpdf_parser/parser_int.h +++ b/core/src/fpdfapi/fpdf_parser/parser_int.h @@ -13,7 +13,6 @@ class CPDF_HintTables { public: CPDF_HintTables(CPDF_DataAvail* pDataAvail, CPDF_Dictionary* pLinearized) : m_pLinearizedDict(pLinearized), - m_pHintStream(nullptr), m_pDataAvail(pDataAvail), m_nFirstPageSharedObjs(0), m_szFirstPageObjOffset(0) {} @@ -35,7 +34,6 @@ class CPDF_HintTables { int ReadPrimaryHintStreamLength() const; CPDF_Dictionary* m_pLinearizedDict; - CPDF_Stream* m_pHintStream; CPDF_DataAvail* m_pDataAvail; FX_DWORD m_nFirstPageSharedObjs; FX_FILESIZE m_szFirstPageObjOffset; diff --git a/samples/pdfium_test.cc b/samples/pdfium_test.cc index 9ae9e2759f..3e71a60a13 100644 --- a/samples/pdfium_test.cc +++ b/samples/pdfium_test.cc @@ -332,14 +332,14 @@ FPDF_BOOL Is_Data_Avail(FX_FILEAVAIL* pThis, size_t offset, size_t size) { void Add_Segment(FX_DOWNLOADHINTS* pThis, size_t offset, size_t size) { } -FPDF_BOOL RenderPage(const std::string& name, - const FPDF_DOCUMENT& doc, - const FPDF_FORMHANDLE& form, - const int page_index, - const Options& options) { +bool RenderPage(const std::string& name, + const FPDF_DOCUMENT& doc, + const FPDF_FORMHANDLE& form, + const int page_index, + const Options& options) { FPDF_PAGE page = FPDF_LoadPage(doc, page_index); if (!page) { - return FALSE; + return false; } FPDF_TEXTPAGE text_page = FPDFText_LoadPage(page); FORM_OnAfterLoadPage(page, form); @@ -355,7 +355,7 @@ FPDF_BOOL RenderPage(const std::string& name, FPDF_BITMAP bitmap = FPDFBitmap_Create(width, height, 0); if (!bitmap) { fprintf(stderr, "Page was too large to be rendered.\n"); - return FALSE; + return false; } FPDFBitmap_FillRect(bitmap, 0, 0, width, height, 0xFFFFFFFF); @@ -393,7 +393,7 @@ FPDF_BOOL RenderPage(const std::string& name, FORM_OnBeforeClosePage(page, form); FPDFText_ClosePage(text_page); FPDF_ClosePage(page); - return TRUE; + return true; } void RenderPdf(const std::string& name, const char* pBuf, size_t len, @@ -430,7 +430,7 @@ void RenderPdf(const std::string& name, const char* pBuf, size_t len, FPDF_DOCUMENT doc; int nRet = PDF_DATA_NOTAVAIL; - FPDF_BOOL bIsLinearized = FALSE; + bool bIsLinearized = false; FPDF_AVAIL pdf_avail = FPDFAvail_Create(&file_avail, &file_access); if (FPDFAvail_IsLinearized(pdf_avail) == PDF_LINEARIZED) { fprintf(stderr, "Linearized path...\n"); @@ -450,7 +450,7 @@ void RenderPdf(const std::string& name, const char* pBuf, size_t len, nRet); return; } - bIsLinearized = TRUE; + bIsLinearized = true; } } else { fprintf(stderr, "Non-linearized path...\n"); -- cgit v1.2.3