Blackconnect doesn't handle typedefs

RESOLVED FIXED

Status

--
major
RESOLVED FIXED
17 years ago
4 years ago

People

(Reporter: edburns, Assigned: blackconnect)

Tracking

Trunk
All
Other

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

17 years ago
Hello,

When trying to use blackconnect to expose mozilla's cache implementation to 
webclient users, I found it necessary to access the nsICache interface.  This 
interface has two typedefs, which blackconnect appears to ignore:

 28 typedef long nsCacheStoragePolicy;
 29 typedef long nsCacheAccessMode;
 
When I run xpidl like this:

..\..\..\dist\WIN32_D.OBJ\bin\xpidl -m java -w -I..\..\..\dist\idl\ nsICache.idl

I get:

nsICache.idl:108: Error: const declaration 'ACCESS_NONE' must be of type short 
or long
(Reporter)

Comment 1

17 years ago
Created attachment 44975 [details]
modified xpidl_java.c.  Search for the string "edburns" for my thoughts on how to fix this.
(Reporter)

Updated

17 years ago
Blocks: 94192
(Assignee)

Comment 2

17 years ago
Created attachment 45501 [details] [diff] [review]
fix for the bug
(Assignee)

Comment 3

17 years ago
We need to substitute typedefed types before any processing happen.
See http://bugzilla.mozilla.org/showattachment.cgi?attach_id=45501
for details of the fix.

Also xpidl.c xpidl_idl.c were removed from the mozilla/java/xpcom/java/xpidl/
directory.
Now we are using default files from the mozilla/xpcom/typelib/xpidl directory.

The fix is checked into the mozilla trunk.
closing as fixed
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.