Closed
Bug 1250949
Opened 8 years ago
Closed 8 years ago
Document how the WebIDL codegen converts dictionary members to C++ dictionary members
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla47
| Tracking | Status | |
|---|---|---|
| firefox47 | --- | fixed |
People
(Reporter: ehsan.akhgari, Assigned: bzbarsky)
Details
(Whiteboard: btpp-backlog)
Attachments
(1 file)
|
3.05 KB,
patch
|
ehsan.akhgari
:
review+
|
Details | Diff | Splinter Review |
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>.
|
Assignee | |
Updated•8 years ago
|
Flags: needinfo?(bzbarsky)
|
||
Updated•8 years ago
|
Whiteboard: btpp-backlog
|
|
Assignee | |
Comment 1•8 years ago
|
||
Attachment #8725436 -
Flags: review?(ehsan)
|
|
Assignee | |
Updated•8 years ago
|
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
|
|
Assignee | |
Updated•8 years ago
|
Flags: needinfo?(bzbarsky)
|
Reporter | |
Comment 2•8 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 4•8 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/5b8e45b227f3
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox47:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
|
||
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•