5416 - should restrict (or warn) about structs passed by value

Status: RESOLVED WONTFIX

(Core :: XPCOM, defect, P4)

Description

[John Bandhauer]

The 'native' declaration syntax allows for declaring type aliases for structs
without tagging them as ptr or ref.

native foo(JSContext);
  rather than
[ptr] native foo(JSContext);

Such types used in params will result in generate a C++ header in which the
struct is passed by value. *Usually* this would be bad. However, there are small
structs (like longlongs on some platforms and point structs) that fit in a word
and *perhaps* ought to be allowed in interfaces. Still, more often than not such
a declaration is a mistake that ought to be flagged.

Comment 1

[Mike McCabe]

Marking these as ASSIGNED.  Pesky bugzilla-daemon.

Comment 2

[Mike McCabe]

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

Comment 3

[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 4

[John Bandhauer]

mass reassign of xpidl bugs to [e-mail]