diff options
author | Tom Sepez <tsepez@chromium.org> | 2018-04-18 20:14:40 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-04-18 20:14:40 +0000 |
commit | 4e636b74000b0c97be28ccc52fa1f2a5bb8c80ca (patch) | |
tree | a7d1e266ab10cde42f07dd727f1c0f8dd1a9045a /core/fpdfapi/parser/cpdf_stream.h | |
parent | 24582d2c000f11ed4bfcb2579b79d10afc809062 (diff) | |
download | pdfium-4e636b74000b0c97be28ccc52fa1f2a5bb8c80ca.tar.xz |
Revert "Return pdfium::span<wchar_t> from WideString::GetBuffer()."
This reverts commit 154e18f9a862975abecebe77b8f5fb418418d14c.
Reason for revert: Generate CL to merge to beta branch
Original change's description:
> Return pdfium::span<wchar_t> from WideString::GetBuffer().
>
> Adds bounds checking "for free", but beware of span outliving
> a ReleaseBuffer() call. Scoping as such avoids the possibility
> of using an invalid span (and it is flagged as a lifetime issue).
>
> Change-Id: Ica63f4b1429823d0254ec6951aeaeb08160cb93c
> Reviewed-on: https://pdfium-review.googlesource.com/30310
> Reviewed-by: dsinclair <dsinclair@chromium.org>
> Commit-Queue: Tom Sepez <tsepez@chromium.org>
TBR=tsepez@chromium.org,dsinclair@chromium.org
# Not skipping CQ checks because original CL landed > 1 day ago.
Change-Id: Ie1ec9434215584a024538ca8edeb59dea555af48
Reviewed-on: https://pdfium-review.googlesource.com/30830
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
(cherry picked from commit 0d32b8fda53e02c1036d39f7290d4f59f2b58ca4)
Reviewed-on: https://pdfium-review.googlesource.com/30951
Diffstat (limited to 'core/fpdfapi/parser/cpdf_stream.h')
0 files changed, 0 insertions, 0 deletions