Closed
Bug 2841
Opened 26 years ago
Closed 26 years ago
Enumerated types are generated each time a .idl file is included
Categories
(Core :: XPCOM, defect, P2)
Tracking
()
VERIFIED
FIXED
M4
People
(Reporter: scottputterman, Assigned: shaver)
Details
1. Create an .idl file that has an enumerated type.
2. Create another .idl file that includes the .idl file from step 1.
3. Run xpidl on both .idl files.
4. Notice that the generated header files both contain the enumerated type.
This causes a compiler error since the enumerated type is defined multiple
times.
Comment 2•26 years ago
|
||
I think this is a libIDL bug; Cc:ing andrewv.
Comment 3•26 years ago
|
||
I think this is a libIDL bug; Cc:ing andrewtv.
Comment 4•26 years ago
|
||
I think this is a libIDL bug, so I've asked andrewtv@usa.net to look into it.
Comment 5•26 years ago
|
||
It occurs to me that this means that you're putting your enums outside the
interface declaration. Have you tried putting them inside? Due to the way
XPConnect stores interface constants, top-level (global) enums may not be
visible to scripts, so that might be another reason.
(The aesthetics of nsInterface::valFoo vs nsInterface_valFoo are another plus,
IMHO.)
Reporter | ||
Comment 6•26 years ago
|
||
Currently the enums are outside of the interface declaration.
Updated•26 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Comment 7•26 years ago
|
||
This should be fixed by an upgrade to libIDL 0.5.9, but I will again plead for
you to move enums into interfaces, since XPConnect will not be able to handle
global enums. The IDL Author's Guide -- coming soon to
http://www.mozilla.org/scriptable/xpidl/idl-authors-guide/ -- will make a
similar plea, and I may in fact teach the xpidl tool to warn when it encounters
global enums or constants.
Updated•26 years ago
|
QA Contact: 3819
Comment 8•26 years ago
|
||
will mark verified assuming reporter agrees with fix
Comment 9•26 years ago
|
||
will mark verified assuming reporter agrees with fix
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Comment 10•26 years ago
|
||
didn't hear any complaining so marking verified
Comment 11•26 years ago
|
||
didn't hear any complaining so marking verified
Comment 12•26 years ago
|
||
bulk reassigning all bugs from shaver's @netscape account to his @mozill acount
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•