|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
58 bytes, text/x-review-board-request
|Details | Review|
With bug 1307357 landed, now we have five Servo arc types, and with the progress of CSSOM support, more such types will be added. Currently adding such a type needs adding code in five different places among three different files. And forgetting to do so sometimes leads to build failures which is only shown in non-stylo builds (which means you may not be aware until pushing to the server). It would be easier to manage if we have a file listing all arc types, and use the list in those places.
After having this, we can make the regen script (and in the future, the build script) parse this list file for arc types directly.
Attachment #8813461 - Flags: review?(bobbyholley) → review?(manishearth)
Seems fine, but let's have Manish look it over since he's done more work on the FFI layer recently than I have.
Comment on attachment 8813461 [details] Bug 1319614 - Move Servo Arc types to a list file. https://reviewboard.mozilla.org/r/94866/#review95726
Attachment #8813461 - Flags: review?(manishearth) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/a74bd474c1a6 Move Servo Arc types to a list file. r=manishearth
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox53: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.