Closed
Bug 224199
Opened 22 years ago
Closed 18 years ago
Code cleanup: nsMsgSortOrder and nsMsgSortType
Categories
(SeaMonkey :: MailNews: Message Display, defect)
SeaMonkey
MailNews: Message Display
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bugzillaPost120030in, Assigned: bugzilla)
Details
Attachments
(1 file)
|
1.78 KB,
patch
|
mnyromyr
:
review+
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
http://lxr.mozilla.org/seamonkey/search?string=nsMsgSortOrder shows
the three enums defined by
enum nsMsgSortOrder
are all unused.
|
Reporter | |
Comment 1•22 years ago
|
||
Ditto : http://lxr.mozilla.org/seamonkey/search?string=nsMsgSortType .
Am I missing something? Since TBird is becoming the flagship, should I be
looking there instead? (I happened upon these whil looking to change the default
sort from ascending to descending.)
|
||
Comment 2•21 years ago
|
||
You should change both Seamonkey and TBird, if you have the time.
|
|
||
Comment 3•21 years ago
|
||
Matthew, are you planning on working on this? If not, can you/I punt it back to
nobody@mozilla.org?
|
|
Reporter | |
Comment 4•21 years ago
|
||
Belatedly punting. My compile platform is out of commission, no time to fix.
Assignee: bugzillaPost120030in → nobody
|
||
Updated•21 years ago
|
Product: Browser → Seamonkey
|
||
Updated•18 years ago
|
Assignee: nobody → mail
Severity: normal → trivial
QA Contact: esther
|
Assignee | |
Comment 5•18 years ago
|
||
According to MXR neither nsMsgSortOrder nor nsMsgSortType are used anywhere in the code.
Assignee: mail → aqualon
Status: NEW → ASSIGNED
Attachment #285770 -
Flags: superreview?(bienvenu)
Attachment #285770 -
Flags: review?(mnyromyr)
|
|
Assignee | |
Updated•18 years ago
|
Summary: Code cleanup: nsMsgSortOrder → Code cleanup: nsMsgSortOrder and nsMsgSortType
|
||
Comment 6•18 years ago
|
||
Comment on attachment 285770 [details] [diff] [review]
Remove nsMsgSortOrder and nsMsgSortType
thx for the patch
Attachment #285770 -
Flags: superreview?(bienvenu) → superreview+
|
||
Comment 7•18 years ago
|
||
Comment on attachment 285770 [details] [diff] [review]
Remove nsMsgSortOrder and nsMsgSortType
JFTR: we're using nsMsgViewSortOrder and nsMsgViewSortType these days.
Landed on trunk.
Attachment #285770 -
Flags: review?(mnyromyr) → review+
|
|
||
Updated•18 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
OS: Windows XP → All
Hardware: PC → All
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•