Closed
Bug 998779
Opened 10 years ago
Closed 10 years ago
Remove unused includes of nsHashtable.h
Categories
(Core :: XPCOM, defect)
Core
XPCOM
Tracking
()
RESOLVED
FIXED
mozilla31
People
(Reporter: mccr8, Assigned: mccr8)
References
Details
Attachments
(2 files)
6.25 KB,
patch
|
froydnj
:
review+
|
Details | Diff | Splinter Review |
754 bytes,
patch
|
froydnj
:
review+
|
Details | Diff | Splinter Review |
Quite a few of these have crept in.
Assignee | ||
Comment 1•10 years ago
|
||
This could have just been part of the final removal patch, but the includes were annoying me.
Assignee | ||
Comment 2•10 years ago
|
||
I threw in a few bonus removals that look unused. https://tbpl.mozilla.org/?tree=Try&rev=9eefdca8ba82
Attachment #8409457 -
Flags: review?(nfroyd)
Updated•10 years ago
|
Attachment #8409457 -
Flags: review?(nfroyd) → review+
Assignee | ||
Comment 3•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/956f6d2dfc81
Comment 4•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/956f6d2dfc81
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
Assignee | ||
Comment 5•10 years ago
|
||
Attachment #8410345 -
Flags: review?(nfroyd)
Assignee | ||
Comment 6•10 years ago
|
||
Oops, I noticed on MXR this morning that I missed one.
Updated•10 years ago
|
Attachment #8410345 -
Flags: review?(nfroyd) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 7•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/98f12f724867
Comment 8•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/98f12f724867
You need to log in
before you can comment on or make changes to this bug.
Description
•