Closed
Bug 967929
Opened 11 years ago
Closed 11 years ago
Remove nsSupportsHashtable
Categories
(Core :: XPCOM, defect)
Core
XPCOM
Tracking
()
RESOLVED
FIXED
mozilla30
People
(Reporter: mccr8, Assigned: mccr8)
References
Details
(Keywords: dev-doc-complete)
Attachments
(1 file, 1 obsolete file)
5.05 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → continuation
Assignee | ||
Comment 2•11 years ago
|
||
Attachment #8375849 -
Attachment is obsolete: true
Attachment #8379775 -
Flags: superreview?(benjamin)
Attachment #8379775 -
Flags: review?(nfroyd)
Assignee | ||
Comment 3•11 years ago
|
||
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)
Updated•11 years ago
|
Attachment #8379775 -
Flags: review?(benjamin) → review+
Assignee | ||
Comment 4•11 years ago
|
||
Assignee | ||
Comment 5•11 years ago
|
||
Comment 6•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
Assignee | ||
Comment 7•11 years ago
|
||
In case anybody is curious, here's the commit from 1999 that added nsSupportsHashtable (and nsObjectHashtable):
https://github.com/mozilla/gecko-dev/commit/d3f90ec7553f4943b927a4931b12c54ffe77eaba
Assignee | ||
Comment 8•11 years ago
|
||
I removed the brief reference to nsSupportsHashtable from the Detailed XPCOM hashtable guide.
Keywords: dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•