Document how the WebIDL codegen converts dictionary members to C++ dictionary members

RESOLVED FIXED in Firefox 47

Status

()

Core
DOM
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Away for a while, Assigned: bz)

Tracking

unspecified
mozilla47
Points:
---

Firefox Tracking Flags

(firefox47 fixed)

Details

(Whiteboard: btpp-backlog)

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
Boris and I had a conversation about this on IRC today where he explained to me how this stuff fits together.  It would be nice to put some of this as documentation inside getMemberConversion().

The conversation begins at <http://logs.glob.uno/?c=content#c357795> and ends at <http://logs.glob.uno/?c=content#c357849>.
Flags: needinfo?(bzbarsky)
Whiteboard: btpp-backlog
Created attachment 8725436 [details] [diff] [review]
Add some better documentation to the dictionary init codegen
Attachment #8725436 - Flags: review?(ehsan)
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Flags: needinfo?(bzbarsky)
(Reporter)

Comment 2

2 years ago
Comment on attachment 8725436 [details] [diff] [review]
Add some better documentation to the dictionary init codegen

Thanks!
Attachment #8725436 - Flags: review?(ehsan) → review+

Comment 3

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/5b8e45b227f3

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/5b8e45b227f3
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox47: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
You need to log in before you can comment on or make changes to this bug.