xpidl should generate Java interfaces in separate files

RESOLVED FIXED

Status

()

Core
XPCOM
P3
normal
RESOLVED FIXED
18 years ago
10 years ago

People

(Reporter: Braden, Assigned: Denis Sharypov)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

18 years ago
If an IDL file includes more than one interface definition, -mode java will
output all the interfaces in the same output file. javac doesn't like this. It
wants each interface in a separate file named "InterfaceName.java".

Comment 1

18 years ago
The Sun folks have been working on the xpidl backend.

Reassiging...
Assignee: mccabe → frankm

Comment 2

18 years ago
Frank Mitchell no longer works at Sun and his email is bouncing. Reassigning
his 21 bugs to george for reassignment.
Assignee: frankm → drapeau

Comment 3

18 years ago
Will work on this after PR 2.  Working on beginnings of this now.
Target Milestone: --- → M19

Comment 4

18 years ago
Re-assigning to lead engineer of BlackConnect, Igor Kushnirskiy.  Igor's group
is working on XPIDL modifications; perhaps this can be added to the list, or at
least evaluated here to see if/when this is possible.
Assignee: drapeau → idk

Comment 5

18 years ago
Reasigning to Denis Sahrypov.
Den will you take a look at this
Assignee: idk → sdv
(Assignee)

Comment 6

18 years ago
fixed. the sources to compile can be found 
at mozilla/java/xpcom/java/xpidl
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 7

18 years ago
Ack.

This means that you've forked the xpidl sources for the Java backend.

I would recommend against this.  Changes and fixes are still going into xpidl,
and many of them are also applicable to the Java backend.  There are IDL
verification functions in xpidl_util.c that the Java backend can use.

Is there a reason that the Java backend needs to be under
mozilla/java/xpcom/java/xpidl?  Why not use the backend already working under
mozilla/xpcom/xpidl?  We'd be happy to have you hacking on it.

Updated

10 years ago
Component: xpidl → XPCOM
QA Contact: mike+mozilla → xpcom
You need to log in before you can comment on or make changes to this bug.