The default bug view has changed. See this FAQ.

Remove some redundant nsVoidArray includes and member variable

RESOLVED FIXED in Thunderbird 23.0

Status

MailNews Core
Backend
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

Trunk
Thunderbird 23.0
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
Created attachment 747343 [details] [diff] [review]
The fix

I found a member variable that wasn't being used, plus some unnecessary includes that we no longer need - attaching patch (passed on try on windows).
Attachment #747343 - Flags: review?(neil)
(Assignee)

Updated

4 years ago
Summary: Remove some redundant nsVoidArray headers and member variable → Remove some redundant nsVoidArray includes and member variable

Updated

4 years ago
Attachment #747343 - Flags: review?(neil) → review+

Updated

4 years ago
Status: NEW → ASSIGNED
Keywords: checkin-needed
Looks like Mark already landed this.

https://hg.mozilla.org/comm-central/rev/fa82bc15f789
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 23.0
(Assignee)

Comment 2

4 years ago
Yep. aceman: thanks for setting checkin-needed, but I usually land my own patches (as I have access, it saves someone else the work) and not setting it avoids potential checkin conflicts - however, if after a week or so I've obviously forgotten, please do poke me.
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.