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

RESOLVED FIXED in mozilla17

Status

()

defect
RESOLVED FIXED
7 years ago
3 months ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

Trunk
mozilla17
x86
macOS
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Assignee

Description

7 years ago
No description provided.
Assignee

Comment 1

7 years ago
Posted patch Patch (v1)Splinter Review
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

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

Comment 4

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

https://hg.mozilla.org/integration/mozilla-inbound/rev/f56f3d3f2c12
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.