Closed
Bug 495109
Opened 15 years ago
Closed 15 years ago
Provide a reverse enumerator for messages in nsIMsgDBFolder
Categories
(MailNews Core :: Database, defect)
MailNews Core
Database
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.0b3
People
(Reporter: davida, Assigned: Bienvenu)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
|
(deleted),
patch
|
standard8
:
review+
standard8
:
superreview+
|
Details | Diff | Splinter Review |
so that one can efficiently and cheaply get the last N messages in a folder.
Flags: blocking-thunderbird3+
|
Assignee | |
Comment 1•15 years ago
|
||
This adds a ReverseEnumerateMessages() method to nsIMsgDatabase which should give you the newest message first - let me know if this works for you.
It would be good to have a test case for this - EnumerateMessages is already exercised by the existing test cases, but it wouldn't hurt to have a specific test that does both kinds of enumeration.
|
|
Assignee | |
Updated•15 years ago
|
Attachment #380288 -
Flags: superreview?(bugzilla)
Attachment #380288 -
Flags: review?(bugzilla)
|
|
Assignee | |
Comment 2•15 years ago
|
||
Comment on attachment 380288 [details] [diff] [review]
very lightly tested
whoops, forgot to ask for review. Davida said this is working for him.
|
|
Assignee | |
Comment 3•15 years ago
|
||
Davida needs this for his folder summary work, so putting in b3.
Status: NEW → ASSIGNED
Whiteboard: [needs review standard8]
Target Milestone: --- → Thunderbird 3.0b3
|
|
Assignee | |
Comment 4•15 years ago
|
||
I'm doing some other cleanup of the db enumerators in bug 492963
|
||
Updated•15 years ago
|
Attachment #380288 -
Flags: superreview?(bugzilla)
Attachment #380288 -
Flags: superreview+
Attachment #380288 -
Flags: review?(bugzilla)
Attachment #380288 -
Flags: review+
|
|
||
Comment 5•15 years ago
|
||
Comment on attachment 380288 [details] [diff] [review]
very lightly tested
Looks good. r/sr=Standard8
|
|
Assignee | |
Comment 6•15 years ago
|
||
fix checked in - now to unbitrot the patch in bug 492963
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Updated•15 years ago
|
Whiteboard: [needs review standard8]
You need to log in
before you can comment on or make changes to this bug.
Description
•