Closed Bug 1370593 Opened 2 years ago Closed 2 years ago

avoid redundant lookups in nsRefPtrHashtable::Remove

Categories

(Core :: XPCOM, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: froydnj, Assigned: froydnj)

Details

Attachments

(1 file)

We already have a handle to the entry in the hashtable, we can remove
that directly, rather than implicitly looking up the entry to remove in
Remove().
Comment on attachment 8874904 [details] [diff] [review]
avoid redundant lookups in nsRefPtrHashtable::Remove

Nice!
Attachment #8874904 - Flags: review?(mats) → review+
Pushed by nfroyd@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f7be035185f6
avoid redundant lookups in nsRefPtrHashtable::Remove; r=mats
https://hg.mozilla.org/mozilla-central/rev/f7be035185f6
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.