New issue
Advanced search Search tips

Issue 770501 link

Starred by 2 users
Status: Untriaged
Owner: ----
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task



Sign in to add a comment

Replace der::Input with base::Span<const uint8_t>

Project Member Reported by davidben@chromium.org, Sep 30 2017 
Thoughts? They're really the exact same type. This won't interfere with moving to BoringSSL because we have an analogous bssl::Span<const uint8_t> and, hopefully, everything will settle on absl::Span<const uint8_t> and, later, std::span<const uint8_t> in the future.

Comment 1 by eroman@chromium.org, Oct 1 2017 

sgtm

Comment 2 by rsleevi@chromium.org, Oct 2 2017 

SGTM. I was looking over the delta of changes from the spec, but I think it's a net-win overall :)

Comment 3 by davidben@chromium.org, Oct 2 2017

Summary: Replace der::Input with base::Span<const uint8_t> (was: Replace der::Input with base::Span<const uint8_t>?)

Comment 4 by davidben@chromium.org, Oct 2 2017

Status: Available (was: Unconfirmed)
Project Member

Comment 5 by sheriffbot@chromium.org, Oct 2

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
►

Sign in to add a comment