Closed Bug 736890 Opened 9 years ago Closed 9 years ago
Speed up building of layout/svg/base/src by cleaning up #include directives in its header files
This is the equivalent of bug 721131, but for layout/svg/base/src header files. This patch cleans up the header files to make them include what they use, and nothing more. For me, I get a 13% speed up for a warm, full rebuild in that directory.
Turns out there were a few other places outside of the SVG code that were depending on SVG headers to include headers that they need.
Comment on attachment 607014 [details] [diff] [review] patch Looks good to me. I didn't bother to verify that all the added #includes and 'class'/'struct' declarations are actually necessary -- I'm trusting you and the include-what-you-use tool on that. (Plus, even if this did add an extraneous #include or declaration, it shouldn't really cause any problems, and this would still be a net win.)
Attachment #607014 - Flags: review?(dholbert) → review+
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla14
You need to log in before you can comment on or make changes to this bug.