The default bug view has changed. See this FAQ.

Speed up building of layout/svg/base/src by cleaning up #include directives in its source files

RESOLVED FIXED in mozilla14

Status

()

Core
SVG
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: jwatt, Assigned: jwatt)

Tracking

(Blocks: 2 bugs)

Trunk
mozilla14
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [include-what-you-use])

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 609008 [details] [diff] [review]
patch

Following on from bug 721131, this patch cleans up the headers in the source files in layout/svg/base/src. The build time is only cut by 2% this time.
Attachment #609008 - Flags: review?(dholbert)
(Assignee)

Updated

5 years ago
Blocks: 738969
Comment on attachment 609008 [details] [diff] [review]
patch

>+// Keep in (case insensitive) order:

Extreme nit, which I'm only mentioning because this comment may end up appearing across a significant fraction of the codebase if these bugs take off, and it'd be nice to be consistent:

I believe "case-insensitive" (with a hyphen) is more commonly-accepted (/ correct?) than "case insensitive" (no hyphen). For what it's worth, https://en.wikipedia.org/wiki/Case_sensitivity has it hyphenated, and there's a section on the "Talk" page discussing this very question: https://en.wikipedia.org/wiki/Talk:Case_sensitivity#hyphen-sensitive

It doesn't really matter, but if you agree & feel like doing a search-and-replace across the patch, I wouldn't complain. :)
Attachment #609008 - Flags: review?(dholbert) → review+
(Assignee)

Comment 2

5 years ago
Sure thing, I made that change and pushed:

https://hg.mozilla.org/integration/mozilla-inbound/rev/78e3ec2adbdf
Target Milestone: --- → mozilla14
https://hg.mozilla.org/mozilla-central/rev/78e3ec2adbdf
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

4 years ago
Blocks: 903843
You need to log in before you can comment on or make changes to this bug.