Closed Bug 967929 Opened 11 years ago Closed 11 years ago

Remove nsSupportsHashtable

Categories

(Core :: XPCOM, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla30

People

(Reporter: mccr8, Assigned: mccr8)

References

Details

(Keywords: dev-doc-complete)

Attachments

(1 file, 1 obsolete file)

This is deprecated, and there are only 3 uses of it (the various XBL forward declarations are bogus), so it shouldn't be too hard to get rid of it.
Depends on: 967931
Depends on: 967941
Depends on: 967999
Depends on: 968012
Blocks: 971264
Attached patch Remove nsSupportsHashtable. (obsolete) — Splinter Review
Assignee: nobody → continuation
Attachment #8375849 - Attachment is obsolete: true
Attachment #8379775 - Flags: superreview?(benjamin)
Attachment #8379775 - Flags: review?(nfroyd)
Comment on attachment 8379775 [details] [diff] [review] Remove nsSupportsHashtable. I guess it is overkill for this simple patch to have a reviewer and a super reviewer...
Attachment #8379775 - Flags: superreview?(benjamin)
Attachment #8379775 - Flags: review?(nfroyd)
Attachment #8379775 - Flags: review?(benjamin)
Attachment #8379775 - Flags: review?(benjamin) → review+
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
In case anybody is curious, here's the commit from 1999 that added nsSupportsHashtable (and nsObjectHashtable): https://github.com/mozilla/gecko-dev/commit/d3f90ec7553f4943b927a4931b12c54ffe77eaba
I removed the brief reference to nsSupportsHashtable from the Detailed XPCOM hashtable guide.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: