commit a65543d22a47112d9c4b815e8d1ef6b249d63d21
author Lei Zhang <thestig@chromium.org> Thu Apr 02 20:49:03 2020 +0000
committer Chromium commit bot <commit-bot@chromium.org> Thu Apr 02 20:49:03 2020 +0000
tree 07373185b448fa3bf57fc33c46d4a4616c9961f2
parent daabea6ba6c09f72e956e826b132f1d1b77acb9b

Use FX_AllocUninit() in outline_aa::allocate_block().

After allocate_block() gets called, add_cur_cell() immediately writes
into the newly allocated cell. So avoid zeroing out memory here.

This code is used frequently, so changing it makes a noticeable impact
on performance.

Bug: pdfium:1171
Change-Id: I5ebabf735f3eb84ec1a42b940da97cdbc2a50423
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/68250
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>

third_party/agg23/agg_rasterizer_scanline_aa.cpp

diff --git a/third_party/agg23/agg_rasterizer_scanline_aa.cpp b/third_party/agg23/agg_rasterizer_scanline_aa.cpp
index 9254d83..7850225 100644
--- a/third_party/agg23/agg_rasterizer_scanline_aa.cpp
+++ b/third_party/agg23/agg_rasterizer_scanline_aa.cpp
@@ -124,7 +124,7 @@
             m_cells = new_cells;
             m_max_blocks += cell_block_pool;
         }
-        m_cells[m_num_blocks++] = FX_Alloc(cell_aa, cell_block_size);
+        m_cells[m_num_blocks++] = FX_AllocUninit(cell_aa, cell_block_size);
     }
     m_cur_cell_ptr = m_cells[m_cur_block++];
 }