Closed Bug 40152 Opened 20 years ago Closed 19 years ago

xpidl uses location of typedef when reporting errors involving types

Categories

(Core :: XPCOM, defect, P3, minor)

defect

Tracking

()

RESOLVED FIXED
Future

People

(Reporter: mike+mozilla, Assigned: mike+mozilla)

Details

Attachments

(1 file)

This showed up in the typelib backend.

Old behavior

../../dist/bin/xpidl -m typelib -w -I ../../dist/idl -I. -o _xpidlgen/nsISample
nsISample.idl
nsrootidl.idl:44: Error: can't find matching argument for [size_is(4)]

In this case, the param was of type PRInt32, and nsrootidl.idl:44 was the
location of the PRInt32 typedef - not of the actual problem.

Fix in hand.  With fix:

../../dist/bin/xpidl -m typelib -w -I ../../dist/idl -I. -o _xpidlgen/nsISample
nsISample.idl
nsISample.idl:49: Error: can't find matching argument for [size_is(4)]

... which is what we want.
[SPAM] Marking milestone 'future' as part of nsbeta3 triage.
Status: NEW → ASSIGNED
Target Milestone: --- → Future
sr=jband
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Component: xpidl → XPCOM
QA Contact: mike+mozilla → xpcom
You need to log in before you can comment on or make changes to this bug.