pyxpt: Report IIDs when they differ

VERIFIED FIXED in mozilla12

Status

()

Core
Build Config
--
enhancement
VERIFIED FIXED
6 years ago
5 years ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

Trunk
mozilla12
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Code is
{
1210                         else:
1211                             # Same name, different IIDs, raise an exception
1212                             raise DataError, \
1213                                   "Typelibs contain definitions of interface %s"\
1214                                   " with different IIDs!" % i.name
1215                 elif i.iid == j.iid and i.iid != Interface.UNRESOLVED_IID:
1216                     # Same IID, different names, raise an exception
1217                     raise DataError, \
1218                           "Typelibs contain definitions of interface %s"\
1219                           " with different names (%s vs. %s)!" %  \
1220                           (i.iid, i.name, j.name)
}

1) Line 1213:
IIDs should be reported, as names are at line 1218.

2) Both:
Could it be possible to report the involved xpts too?
With their timestamp?
(Assignee)

Comment 1

6 years ago
Created attachment 591389 [details] [diff] [review]
(Av1) xpt.py: Report IIDs when they differ
[Checked in: Comment 3]

This should fix part 1.
(Untested.)
Attachment #591389 - Flags: review?
(Assignee)

Updated

6 years ago
Attachment #591389 - Flags: review? → review?(mh+mozilla)
Comment on attachment 591389 [details] [diff] [review]
(Av1) xpt.py: Report IIDs when they differ
[Checked in: Comment 3]

Ted knowns the code better than I do.
Attachment #591389 - Flags: review?(mh+mozilla) → review?(ted.mielczarek)
Attachment #591389 - Flags: review?(ted.mielczarek) → review+
(Assignee)

Comment 3

6 years ago
Comment on attachment 591389 [details] [diff] [review]
(Av1) xpt.py: Report IIDs when they differ
[Checked in: Comment 3]

https://hg.mozilla.org/mozilla-central/rev/03ae304e45af
Attachment #591389 - Attachment description: (Av1) xpt.py: Report IIDs when they differ → (Av1) xpt.py: Report IIDs when they differ [Checked in: Comment 3]
(Assignee)

Updated

6 years ago
Assignee: nobody → sgautherie.bz
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Summary: pyxpt: Give more details about interfaces with different IID/names → pyxpt: Report IIDs when they differ
Target Milestone: --- → mozilla12
(Assignee)

Comment 4

6 years ago
(In reply to Serge Gautherie (:sgautherie) from comment #0)
> 2) Both:
> Could it be possible to report the involved xpts too?
> With their timestamp?

Is it worth that I filed another bug about part 2, or would that be wontfix_able/ed?
Seems totally reasonable to have the xpt filenames in the error messages.
(Assignee)

Comment 6

6 years ago
V.Fixed, per bug 720952 comment 6.
Status: RESOLVED → VERIFIED
(Assignee)

Comment 7

6 years ago
(In reply to Ted Mielczarek [:ted, :luser] from comment #5)
> Seems totally reasonable to have the xpt filenames in the error messages.

Bug 721065
(Assignee)

Updated

5 years ago
Blocks: 721065
You need to log in before you can comment on or make changes to this bug.