Closed Bug 1486782 Opened Last year Closed Last year

Remove some unnecessary #includes of gc/Zone.h

Categories

(Core :: JavaScript: GC, enhancement)

61 Branch
enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: jonco, Assigned: jonco)

Details

Attachments

(1 file)

Zone.h is included in a bunch of places where it doesn't need to be.
I factored out the definition of regexp related enums into a new header so we don't need to include the whole of RegExpShared.h in the frontend.
Attachment #9004561 - Flags: review?(sphink)
Comment on attachment 9004561 [details] [diff] [review]
bug1486782-remove-zone-includes

Review of attachment 9004561 [details] [diff] [review]:
-----------------------------------------------------------------

That's an impressive set of removals.
Attachment #9004561 - Flags: review?(sphink) → review+
Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/fcae304349d8
Remove unnecessary includes of gc/Zone.h r=sfink
https://hg.mozilla.org/mozilla-central/rev/fcae304349d8
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.