Closed
Bug 403683
Opened 18 years ago
Closed 18 years ago
Remove some unnecessary nsISupportsArray includes from toolkit.
Categories
(Toolkit Graveyard :: Build Config, defect)
Toolkit Graveyard
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: standard8, Assigned: standard8)
Details
Attachments
(1 file)
|
1.95 KB,
patch
|
Gavin
:
review+
damons
:
approval1.9+
|
Details | Diff | Splinter Review |
I've had these in my tree for a while now, and may as well clear them out.
The files that I'm removing these from no longer use nsISupportsArray and any bustage should be obvious. TB/SM/FF all build fine with this patch on my machine.
Attachment #288550 -
Flags: review?(ted.mielczarek)
|
||
Updated•18 years ago
|
Attachment #288550 -
Flags: review+
|
Assignee | |
Comment 1•18 years ago
|
||
Gavin, does the r+ mean I don't need to get it from Ted?
|
||
Comment 2•18 years ago
|
||
That was not really a build config patch, so you needed a toolkit peer anyway, so Gavin's review is what you wanted.
|
|
Assignee | |
Comment 3•18 years ago
|
||
(In reply to comment #2)
> That was not really a build config patch, so you needed a toolkit peer anyway,
> so Gavin's review is what you wanted.
Ok, apologies. It just seemed the best place to put it.
|
|
Assignee | |
Comment 4•18 years ago
|
||
Comment on attachment 288550 [details] [diff] [review]
Remove redundant includes.
Requesting approval 1.9.
Low risk patch that is just removing some unnecessary header includes.
Attachment #288550 -
Flags: review?(ted.mielczarek) → approval1.9?
|
||
Comment 5•18 years ago
|
||
Comment on attachment 288550 [details] [diff] [review]
Remove redundant includes.
a=release drivers.
Attachment #288550 -
Flags: approval1.9? → approval1.9+
|
|
Assignee | |
Comment 6•18 years ago
|
||
Checked in -> fixed.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
|
||
Updated•7 years ago
|
Product: Toolkit → Toolkit Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•