Closed
Bug 1696666
Opened 4 years ago
Closed 4 years ago
Remove WTF8Chars
Categories
(Core :: JavaScript Engine, task)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
88 Branch
| Tracking | Status | |
|---|---|---|
| firefox88 | --- | fixed |
People
(Reporter: anba, Assigned: anba)
Details
Attachments
(7 files)
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
Details | Review |
We don't use WTF8Chars anymore.
|
Assignee | |
Comment 1•4 years ago
|
||
Neither of two functions are called anymore anywhere.
|
|
Assignee | |
Comment 2•4 years ago
|
||
Depends on D107337
|
|
Assignee | |
Comment 3•4 years ago
|
||
Depends on D107338
|
|
Assignee | |
Comment 4•4 years ago
|
||
Depends on D107339
|
|
Assignee | |
Comment 5•4 years ago
|
||
Depends on D107340
|
|
Assignee | |
Comment 6•4 years ago
|
||
Remove template parameters from:
- GetUTF8AtomizationData
- MakeUTF8AtomHelper
- InflateUTF8CharsToBufferAndTerminate
- CopyAndInflateUTF8IntoBuffer
- InflateUTF8StringHelper
- InflateUTF8ToUTF16
- Utf8ToOneUcs4CharImpl
Depends on D107341
|
|
Assignee | |
Comment 7•4 years ago
|
||
Depends on D107342
Pushed by tcampbell@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/02b6452a7cf9
Part 1: Remove AtomizeWTF8Chars and JS::WTF8CharsToNewTwoByteCharsZ functions. r=tcampbell
https://hg.mozilla.org/integration/autoland/rev/32f8803201dd
Part 2: Inline AtomizeUTF8OrWTF8Chars. r=tcampbell
https://hg.mozilla.org/integration/autoland/rev/09f229a560ad
Part 3: Remove template parameter from AtomizeUTF8OrWTF8CharsWrapper. r=tcampbell
https://hg.mozilla.org/integration/autoland/rev/5dd96adb3b37
Part 4: Remove no longer used WTF8 AtomHasher::Lookup type. r=tcampbell
https://hg.mozilla.org/integration/autoland/rev/b36abc0f4d08
Part 5: Rename UTF8OrWTF8EqualsChars to UTF8EqualsChars. r=tcampbell
https://hg.mozilla.org/integration/autoland/rev/0231025f0c6f
Part 6: Remove template parameters from various functions. r=tcampbell
https://hg.mozilla.org/integration/autoland/rev/188912c5ffe8
Part 7: Remove WTF8Chars. r=tcampbell
|
||
Comment 9•4 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/02b6452a7cf9
https://hg.mozilla.org/mozilla-central/rev/32f8803201dd
https://hg.mozilla.org/mozilla-central/rev/09f229a560ad
https://hg.mozilla.org/mozilla-central/rev/5dd96adb3b37
https://hg.mozilla.org/mozilla-central/rev/b36abc0f4d08
https://hg.mozilla.org/mozilla-central/rev/0231025f0c6f
https://hg.mozilla.org/mozilla-central/rev/188912c5ffe8
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•