From 01e624fb699af3a8ee2f81620d59d366dac18f29 Mon Sep 17 00:00:00 2001 From: weili Date: Fri, 13 May 2016 16:59:50 -0700 Subject: Fix GCC warnings to be ready for GCC standalone build ChromeOS still compile pdfium with GCC. Fix the code to be ready to have workable GCC standalone build. The build file change will be in a follow-up CL. One warning is about maybe using uninitialized variable, the other is about enum and non-enum in a conditional statement. Review-Url: https://codereview.chromium.org/1981593002 --- fpdfsdk/fpdfxfa/fpdfxfa_doc.cpp | 2 +- third_party/agg23/0001-gcc-warning.patch | 17 +++++++++++++++++ third_party/agg23/README.pdfium | 2 ++ third_party/agg23/agg_path_storage.h | 6 +++--- 4 files changed, 23 insertions(+), 4 deletions(-) create mode 100644 third_party/agg23/0001-gcc-warning.patch diff --git a/fpdfsdk/fpdfxfa/fpdfxfa_doc.cpp b/fpdfsdk/fpdfxfa/fpdfxfa_doc.cpp index 72f67dd271..4b44669c0d 100644 --- a/fpdfsdk/fpdfxfa/fpdfxfa_doc.cpp +++ b/fpdfsdk/fpdfxfa/fpdfxfa_doc.cpp @@ -354,7 +354,7 @@ FX_BOOL CPDFXFA_Document::GetPopupPos(CXFA_FFWidget* hWidget, CPDFDoc_Environment* pEnv = m_pSDKDoc->GetEnv(); if (!pEnv) return FALSE; - FS_RECTF pageViewRect; + FS_RECTF pageViewRect = {0.0f, 0.0f, 0.0f, 0.0f}; pEnv->FFI_GetPageViewRect(pPage, pageViewRect); CFX_FloatRect rcAnchor; diff --git a/third_party/agg23/0001-gcc-warning.patch b/third_party/agg23/0001-gcc-warning.patch new file mode 100644 index 0000000000..759696ef03 --- /dev/null +++ b/third_party/agg23/0001-gcc-warning.patch @@ -0,0 +1,17 @@ +diff --git a/third_party/agg23/agg_path_storage.h b/third_party/agg23/agg_path_storage.h +index dc13851..7f21bac 100644 +--- a/third_party/agg23/agg_path_storage.h ++++ b/third_party/agg23/agg_path_storage.h +@@ -38,9 +38,9 @@ public: + } + unsigned vertex(FX_FLOAT* x, FX_FLOAT* y) + { +- return (m_vertex_idx < m_path->total_vertices()) ? +- m_path->vertex(m_vertex_idx++, x, y) : +- path_cmd_stop; ++ return (m_vertex_idx < m_path->total_vertices()) ++ ? m_path->vertex(m_vertex_idx++, x, y) ++ : static_cast(path_cmd_stop); + } + private: + const path_storage* m_path; diff --git a/third_party/agg23/README.pdfium b/third_party/agg23/README.pdfium index 54687934b6..3b73d4d4d3 100644 --- a/third_party/agg23/README.pdfium +++ b/third_party/agg23/README.pdfium @@ -12,3 +12,5 @@ Local Modifications: 0000-bug-466.patch: Fixes an infinite loop in stroke_calc_arc(). Various changes to use FX_ library functions. Possibly more? +0001-gcc-warning.patch: Fix a GCC warning about both enumeral and +non-enumeral type in conditional. diff --git a/third_party/agg23/agg_path_storage.h b/third_party/agg23/agg_path_storage.h index dc13851d09..7f21bacd09 100644 --- a/third_party/agg23/agg_path_storage.h +++ b/third_party/agg23/agg_path_storage.h @@ -38,9 +38,9 @@ public: } unsigned vertex(FX_FLOAT* x, FX_FLOAT* y) { - return (m_vertex_idx < m_path->total_vertices()) ? - m_path->vertex(m_vertex_idx++, x, y) : - path_cmd_stop; + return (m_vertex_idx < m_path->total_vertices()) + ? m_path->vertex(m_vertex_idx++, x, y) + : static_cast(path_cmd_stop); } private: const path_storage* m_path; -- cgit v1.2.3