Skip to content
Commit 85bdd640 authored by François Degros's avatar François Degros Committed by Copybara-Service
Browse files

[zip] Change one of FileWriterDelegate's constructor

No functional change. Optimization only.

Changed the constructor of FileWriterDelegate that takes ownership of
the File. It now moves the passed File by value rather than by
unique_ptr. This avoid one extra level of indirection, and dynamic
memory allocation.

BUG=None
TEST=autoninja -C out/Default zlib_unittests && out/Default/zlib_unittests
TEST=autoninja -C out/Default components_unittests && out/Default/components_unittests --gtest_filter='UnzipTest.*'

Change-Id: Idca9b1836189d20fca10681fcb4d00c6d392eae4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3511609


Reviewed-by: default avatarNoel Gordon <noel@chromium.org>
Commit-Queue: François Degros <fdegros@chromium.org>
Cr-Commit-Position: refs/heads/main@{#979053}
NOKEYCHECK=True
GitOrigin-RevId: d915951cf63c82c7bf5ed6250c3799660cb81280
parent aa6909a5
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment