Closed Bug 1486528 Opened 2 years ago Closed 2 years ago

Remove unused nsIMemory includes

Categories

(Core :: XPCOM, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: marco, Assigned: marco)

References

(Blocks 2 open bugs)

Details

Attachments

(1 file)

nsMemoryImpl is not tested at all (at least on Linux and Windows), I thought it might be unused.
By inspecting the code, it looks like it might be used.

I've found several #includes that seem unnecessary though.
Attached patch PatchSplinter Review
Assignee: nobody → mcastelluccio
Status: NEW → ASSIGNED
Attachment #9004282 - Flags: review?(n.nethercote)
Attachment #9004282 - Flags: review?(n.nethercote) → review+
Pushed by mcastelluccio@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/481d502afc69
Remove some unnecessary nsIMemory imports. r=njn
https://hg.mozilla.org/mozilla-central/rev/481d502afc69
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.