Closed Bug 488907 Opened 12 years ago Closed 12 years ago

Tidy up some includes in nsIMsgFolder.idl/nsIMsgWindow.idl

Categories

(MailNews Core :: Backend, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.0b3

People

(Reporter: standard8, Assigned: standard8)

Details

Attachments

(1 file)

Attached patch The fixSplinter Review
I'm working on a patch that will hopefully improve how we do our nsIMsgStatusFeedback/nsIMsgProgress hooks. For that I need to redo some includes in a couple of the idl files that will affect quite a few places in mailnews, so I'm separating out to a separate patch.

I've basically gone for declaring the interface in the idl, and including the relevant .h files only where necessary.

Tested on Linux, Mac & Windows.
Attachment #373382 - Flags: superreview?(bienvenu)
Attachment #373382 - Flags: review?(bienvenu)
Comment on attachment 373382 [details] [diff] [review]
The fix

sorry for the delay, builds on windows.
Attachment #373382 - Flags: superreview?(bienvenu)
Attachment #373382 - Flags: superreview+
Attachment #373382 - Flags: review?(bienvenu)
Attachment #373382 - Flags: review+
Checked in: http://hg.mozilla.org/comm-central/rev/f02f2adc5f62
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.