11195 - xpt_link cores when asked to link 1 file with no interfaces

Status: RESOLVED WONTFIX

(Core :: XPCOM, defect, P2)

Description

[John Bandhauer]

Do we want to make xpidl support idl files without interfaces? We need to either
make that work or suggest a better means for putterman to do what he wants to do
here:

[e-mail] (Scott Putterman) wrote:

Someone finally caught me on that :)
I always meant to try to figure it out.  Let me see if I can get this right.
The problem was that I wanted to
generate a .h file that had a couple of types defined that could also be used
from idl.  The problem was that
when I ran it, it complained unless I created a fake interface to go along with
it.  I promptly forgot about this
until you brought this up.

I really just cared about

typedef unsigned long nsMsgKey;
typedef unsigned long nsMsgViewIndex;

Scott


John Bandhauer wrote:

  Does this still need to be here? It is oddd to have 'hack' in the
  global interface namspace. What is the problem you were trying to
  solve?

  http://lxr.mozilla.org/seamonkey/source/mailnews/public/MailNewsTypes2.idl#22

  Thanks,

  John

Comment 1

[Mike McCabe]

Marking as assigned.

Comment 2

[Mike McCabe]

'xpidl -m header MailNewsTypes2.idl' and 'xpidl -m typelib MailNewsTypes2.xpt'
(should we generate an .xpt file for an interface-less idl file?) seem to work
just fine with the hacked interface commented out.  It looks like xpt_link fails
trying to link it, though.  Big ol' core dump.

Comment 3

[Mike McCabe]

Hm.  Scratch that; the failure seems to be limited to xpt_link coring whenever
asked to link exactly one file, which contains no interfaces.

Scott, was this the problem you were running into?

Comment 4

[Mike McCabe]

Hm.  Scratch that; the failure seems to be limited to xpt_link coring whenever
asked to link exactly one file, which contains no interfaces.

Scott, was this the problem you were running into?

Comment 5

[scottputterman]

Yes.  That was the problem. I'd forgotten about that.  If it isn't possible to
solve this, I'll try to look into moving these files into a different directory.
This is an old directory that hasn't really been used since it was first created
with the MailNewsTypes files(though those files are important).

Comment 6

[John Bandhauer]

I'm sure Mike will say when he wakes up... This is well worth fixing. The linker
shouldn't ought to crash. Hang on and we'll have it fixed soon.

Comment 7

[Mike McCabe]

Ahem.  This is well worth fixing. The linker shouldn't ought to crash. Hang on
and we'll have it fixed soon.

Comment 8

[Michael Ang]

Reassigning to [e-mail], since I'm already working on another
xpt_link crash.

Comment 9

[Michael Ang]

Mass reassign to mccabe since I'm outta here.

Comment 10

[Mike McCabe]

Mass accept as ASSIGNED of xpidl bugs

Comment 11

[Mike McCabe]

[SPAM] Marking milestone 'future' as part of nsbeta3 triage.

Comment 12

[Phil Schwartau]

Mass-reassigning mccabe's non-JS, non-Rhino bugs to jband (34 total). 

Would like to cc mccabe; but the mass-reassign page does not allow this. 
I'll leave it up to mccabe to decide if he wants to be cc'ed on these - 

Comment 13

[John Bandhauer]

mass reassign of xpidl bugs to [e-mail]