Closed
Bug 616816
Opened 15 years ago
Closed 15 years ago
CRs in nsMsgXFViewThread.cpp nsIMailboxUrl.idl certFetchingStatus.js mimecms.cpp
Categories
(MailNews Core :: Backend, defect)
MailNews Core
Backend
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.3a2
People
(Reporter: neil, Assigned: neil)
Details
Attachments
(1 file)
|
13.18 KB,
patch
|
Bienvenu
:
review+
|
Details | Diff | Splinter Review |
nsMsgXFViewThread.cpp: 514 lines, 12 CRs
nsIMailboxUrl.idl: 94 lines, 94 CRs
certFetchingStatus.js: 322 lines, 3 CRs
mimecms.cpp: 739 lines, 12 CRs
|
Assignee | |
Comment 1•15 years ago
|
||
|
||
Comment 2•15 years ago
|
||
Comment on attachment 495366 [details] [diff] [review]
Proposed patch
can you fix the indention in nsIMailboxUrl.idl while you're here, e.g.,:
+ const long ActionParseMailbox = 0;
+ const long ActionFetchMessage = 1;
+ const long ActionCopyMessage = 2;
+ const long ActionMoveMessage = 3;
and some of the comment lines above...
Attachment #495366 -
Flags: review?(bienvenu) → review+
|
|
Assignee | |
Comment 3•15 years ago
|
||
Pushed changeset 07d07bebd7f0 to comm-central.
I reindented everything with 2 spaces.
|
|
Assignee | |
Updated•15 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
|
||
Updated•15 years ago
|
Target Milestone: --- → Thunderbird 3.3a2
You need to log in
before you can comment on or make changes to this bug.
Description
•