Closed
Bug 71722
Opened 24 years ago
Closed 24 years ago
nsIMenuItem IID and nsMenuItem CID are the same
Categories
(Core :: XUL, defect)
Tracking
()
RESOLVED
FIXED
mozilla0.8.1
People
(Reporter: sfraser_bugs, Assigned: mikepinkerton)
Details
Attachments
(1 file)
1.95 KB,
patch
|
Details | Diff | Splinter Review |
nsIMenuItem's IID:
http://lxr.mozilla.org/seamonkey/source/widget/public/nsIMenuItem.h#34
conflicts with nsMenuItems's CID:
http://lxr.mozilla.org/seamonkey/source/widget/public/nsWidgetsCID.h#137
How many more conflicts do we have?
Reporter | ||
Comment 1•24 years ago
|
||
These three also conflict:
nsIMenuBar's IID:
http://lxr.mozilla.org/seamonkey/source/widget/public/nsIMenuBar.h#33
nsIMenuListeners's IID:
http://lxr.mozilla.org/seamonkey/source/widget/public/nsIMenuListener.h#30
nsMenuBar's CID:
http://lxr.mozilla.org/seamonkey/source/widget/public/nsWidgetsCID.h#127
There are gonna be some real bugs here. Saari, what were you thinking?
Reporter | ||
Comment 2•24 years ago
|
||
The IID comment and definition in nsIMenuListener.h don't match, so this doesn't
actually conflict. But please fix the comment; it screws up lxr searches.
Assignee | ||
Comment 4•24 years ago
|
||
Reporter | ||
Comment 5•24 years ago
|
||
sr=sfraser
Assignee | ||
Comment 6•24 years ago
|
||
fixed. r=saari
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•