diff options
| author | weili <weili@chromium.org> | 2016-05-13 16:59:50 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-05-13 16:59:50 -0700 |
| commit | 01e624fb699af3a8ee2f81620d59d366dac18f29 (patch) | |
| tree | 6e5e4313d5fd11ec7c4d249a08d61e83e56a2a74 /third_party/agg23 | |
| parent | 8adc153ea6fe778d19dc33cd2c5b87aed312b29b (diff) | |
| download | pdfium-01e624fb699af3a8ee2f81620d59d366dac18f29.tar.xz | |
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
Diffstat (limited to 'third_party/agg23')
| -rw-r--r-- | third_party/agg23/0001-gcc-warning.patch | 17 | ||||
| -rw-r--r-- | third_party/agg23/README.pdfium | 2 | ||||
| -rw-r--r-- | third_party/agg23/agg_path_storage.h | 6 |
3 files changed, 22 insertions, 3 deletions
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<unsigned>(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<unsigned>(path_cmd_stop); } private: const path_storage* m_path; |