Open
Bug 1412441
Opened 7 years ago
Updated 2 years ago
stylo: Merge structs.rs into bindings.rs
Categories
(Core :: CSS Parsing and Computation, enhancement, P3)
Core
CSS Parsing and Computation
Tracking
()
NEW
People
(Reporter: xidorn, Unassigned)
References
(Blocks 1 open bug)
Details
IIUC, the point to have separate structs.rs and bindings.rs was because we have two structs files in the past, for debug and release build respectively.
In bug 1400084 we stopped generating two structs.rs, but I forgot that we can also merge structs.rs into bindings.rs.
We should probably do that here.
It should help the build time of style crate since we would no longer need to scan all the headers twice.
Reporter | ||
Comment 1•7 years ago
|
||
I expect bug 1412486 to change some code in build_gecko.rs, so it's probably better holding on until that gets landed (since that should happen sooner).
Depends on: 1412486
Comment 2•7 years ago
|
||
status-firefox57=wontfix unless someone thinks this bug should block 57
status-firefox57:
--- → wontfix
Comment 3•7 years ago
|
||
P3 because this bug doesn't block us from shipping stylo-chrome or Android.
Priority: -- → P3
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•