The default bug view has changed. See this FAQ.

Enable the WebIDL parser to be used stand-alone for syntax checking

RESOLVED FIXED in mozilla17

Status

()

Core
DOM
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

Trunk
mozilla17
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

5 years ago
Created attachment 655080 [details] [diff] [review]
Patch (v1)
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #655080 - Flags: review?(khuey)
Comment on attachment 655080 [details] [diff] [review]
Patch (v1)

Review of attachment 655080 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/bindings/parser/WebIDL.py
@@ +3679,5 @@
> +            if options.verbose_errors:
> +                traceback.print_exc()
> +            else:
> +                print e
> +    parser.finish()

parser.finish() can throw too, say if you referenced an interface somewhere that was never defined.
Attachment #655080 - Flags: review?(khuey) → review+
(Assignee)

Comment 3

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/1bdc56a180bc
Target Milestone: --- → mozilla17
(Assignee)

Comment 4

5 years ago
Pushed a follow-up for python 2.5 compatibility:

https://hg.mozilla.org/integration/mozilla-inbound/rev/f56f3d3f2c12
https://hg.mozilla.org/mozilla-central/rev/1bdc56a180bc
https://hg.mozilla.org/mozilla-central/rev/f56f3d3f2c12
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.