Closed
Bug 1743012
Opened 2 years ago
Closed 2 years ago
Add some missing `constexpr` to `CheckedInt` functions
Categories
(Core :: MFBT, defect)
Core
MFBT
Tracking
()
RESOLVED
FIXED
96 Branch
People
(Reporter: nika, Assigned: nika)
References
Details
Attachments
(1 file)
|
48 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-esr91+
|
Details | Review |
No description provided.
|
Assignee | |
Comment 1•2 years ago
|
||
Pushed by nlayzell@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9a2b5baa2694 Add missing `constexpr` qualifiers in `CheckedInt`, r=glandium
|
||
Comment 3•2 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox96:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch
|
|
Assignee | |
Comment 4•2 years ago
|
||
Comment on attachment 9252526 [details]
Bug 1743012 - Add missing constexpr qualifiers in CheckedInt, r=glandium
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: Dependency of bug 1741665
- User impact if declined: See bug 1741665
- Fix Landed on Version: 96
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Just adds a few
constexprqualifiers
Attachment #9252526 -
Flags: approval-mozilla-esr91?
|
||
Comment 5•2 years ago
|
||
Comment on attachment 9252526 [details]
Bug 1743012 - Add missing constexpr qualifiers in CheckedInt, r=glandium
Approved for 91.6esr.
Attachment #9252526 -
Flags: approval-mozilla-esr91? → approval-mozilla-esr91+
|
|
||
Comment 6•2 years ago
|
||
| bugherder uplift | ||
status-firefox-esr91:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•