Closed
Bug 416712
Opened 18 years ago
Closed 18 years ago
Drop redundant nsISupportsArray include from libpref.
Categories
(Core :: Preferences: Backend, defect)
Core
Preferences: Backend
Tracking
()
RESOLVED
FIXED
People
(Reporter: standard8, Assigned: standard8)
Details
Attachments
(1 file)
|
613 bytes,
patch
|
benjamin
:
review+
benjamin
:
superreview+
mtschrep
:
approval1.9+
|
Details | Diff | Splinter Review |
I've been building with this patch in for a while now, drops a redundant nsISupportsArray.h include from libpref: http://mxr.mozilla.org/seamonkey/search?find=%2Fmodules%2Flibpref%2F&string=nsISupportsArray - nothing else uses it in libpref.
Attachment #302446 -
Flags: superreview?(benjamin)
Attachment #302446 -
Flags: review?(benjamin)
|
||
Updated•18 years ago
|
Attachment #302446 -
Flags: superreview?(benjamin)
Attachment #302446 -
Flags: superreview+
Attachment #302446 -
Flags: review?(benjamin)
Attachment #302446 -
Flags: review+
|
Assignee | |
Comment 1•18 years ago
|
||
Comment on attachment 302446 [details] [diff] [review]
The fix
Requesting 1.9 approval for simple one-line patch that removes a redundant include. Any problems will be shown up at compilation, though there shouldn't be any.
Attachment #302446 -
Flags: approval1.9?
|
||
Updated•18 years ago
|
Attachment #302446 -
Flags: approval1.9? → approval1.9+
|
|
Assignee | |
Comment 2•18 years ago
|
||
Patch checked in -> fixed.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•