Closed Bug 1159972 Opened 6 years ago Closed 6 years ago

Remove the fallible version of PL_DHashTableInit()

Categories

(Core :: XPCOM, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: njn, Assigned: njn)

References

Details

Attachments

(1 file)

Redoing, more cautiously, some stuff from bug 1050035 that had to be backed out for intermittent failures.
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
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in before you can comment on or make changes to this bug.