Closed
Bug 1691886
Opened 4 years ago
Closed 3 years ago
Consider removing BufferSize
Categories
(Core :: JavaScript Engine, task, P3)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
89 Branch
Tracking | Status | |
---|---|---|
firefox89 | --- | fixed |
People
(Reporter: jandem, Assigned: lth)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
This wrapper around size_t
was helpful to mark the places that still needed changes, but deprecatedGetUint32
will be removed soon and then it probably makes sense to replace BufferSize
with size_t
.
Note that BufferSize
does not guarantee the wrapped size is <= the max ArrayBuffer
byte length, so I don't think it's that useful to keep around going forward.
This can wait a few weeks until we have had more fuzzing.
Assignee | ||
Comment 2•3 years ago
|
||
Remove the BufferSize abstraction, replace it with size_t everywhere.
Depends on D111100
Pushed by lhansen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/76f9bff99e53 Remove BufferSize. r=jandem
Comment 4•3 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox89:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•