Remove the fallible version of PL_DHashTableInit()

RESOLVED FIXED in Firefox 40

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: njn, Assigned: njn)

Tracking

unspecified
mozilla40
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox40 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

4 years ago
Redoing, more cautiously, some stuff from bug 1050035 that had to be backed out for intermittent failures.
Assignee

Comment 1

4 years ago
It's no longer needed now that entry storage isn't allocated there. (The other
possible causes of failures in that function are less interesting and simply
crashing is a reasonable thing to do for them.)

This also makes PL_DNewHashTable() infallible, so I removed some
now-unnecessary checks of its result.
Attachment #8599624 - Flags: review?(nfroyd)
Comment on attachment 8599624 [details] [diff] [review]
Remove the fallible version of PL_DHashTableInit()

Review of attachment 8599624 [details] [diff] [review]:
-----------------------------------------------------------------

Please post to dev-platform about this.
Attachment #8599624 - Flags: review?(nfroyd) → review+
https://hg.mozilla.org/mozilla-central/rev/efa98fd2a045
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in before you can comment on or make changes to this bug.