Handle OOM in intl_canonicalizeTimeZone
Categories
(Core :: JavaScript: Internationalization API, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox91 | --- | fixed |
People
(Reporter: anba, Assigned: anba)
Details
Attachments
(5 files)
Handle OOM in intl_canonicalizeTimeZone
and perform some style clean-ups.
Assignee | ||
Comment 1•3 years ago
|
||
FormatBuffer::toString()
returns nullptr
on OOM.
Assignee | ||
Comment 2•3 years ago
|
||
Delete some extra blank lines in error-handling code to align the code with the
rest of the code base.
Depends on D118019
Assignee | ||
Comment 3•3 years ago
|
||
It's not necessary to explicitly cast to void*
.
Depends on D118020
Assignee | ||
Comment 4•3 years ago
|
||
These occurences of const
in parameters were missed when const
was removed
in the previous "Unified Intl" patches.
Depends on D118021
Assignee | ||
Comment 5•3 years ago
|
||
Place JSContext*
as the last argument to indicate this function is infallible.
Most functions which have JSContext*
as their first argument are fallible, so
a SpiderMonkey convention is to move JSContext*
as the last parameter for
infallible functions.
Depends on D118022
Comment 7•3 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c30ff4af578c
https://hg.mozilla.org/mozilla-central/rev/0366857d15b3
https://hg.mozilla.org/mozilla-central/rev/c49f99d34cae
https://hg.mozilla.org/mozilla-central/rev/fa73c034ae5d
https://hg.mozilla.org/mozilla-central/rev/0bd5153b7662
Description
•