Bug 271118
| Summary: | Drop Vector::append(T*, size_t) and use Vector::append(std::span<const T>) instead | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Chris Dumez <cdumez> |
| Component: | Web Template Framework | Assignee: | Diego Pino <dpino> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | dpino, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Chris Dumez
Drop Vector::append(T*, size_t) and use Vector::append(std::span<const T>) instead. This is part of an effort to use std::span more in the codebase, for better security.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Chris Dumez
Pull request: https://github.com/WebKit/WebKit/pull/26014
EWS
Committed 276319@main (c904754b70e1): <https://commits.webkit.org/276319@main>
Reviewed commits have been landed. Closing PR #26014 and removing active labels.
Radar WebKit Bug Importer
<rdar://problem/124982635>
Diego Pino
Build error in GTK-Linux-64-bit-Release-Debian-11-Build:
https://build.webkit.org/#/builders/992/builds/750
```
../../../Source/WebCore/Modules/indexeddb/server/IDBSerialization.cpp:356:36: error: ‘auto’ parameter not permitted in this context
356 | Vector<uint8_t> dataVector(std::span { data, size });
|
```
Diego Pino
Re-opening for pull request https://github.com/webkit/webkit/pull/26107
EWS
Committed 276336@main (20ecbb6bc4e7): <https://commits.webkit.org/276336@main>
Reviewed commits have been landed. Closing PR #26107 and removing active labels.