Closed Bug 495109 Opened 11 years ago Closed 11 years ago

Provide a reverse enumerator for messages in nsIMsgDBFolder

Categories

(MailNews Core :: Database, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.0b3

People

(Reporter: davida, Assigned: Bienvenu)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

so that one can efficiently and cheaply get the last N messages in a folder.
Flags: blocking-thunderbird3+
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.
Attachment #380288 - Flags: superreview?(bugzilla)
Attachment #380288 - Flags: review?(bugzilla)
Comment on attachment 380288 [details] [diff] [review]
very lightly tested

whoops, forgot to ask for review. Davida said this is working for him.
Davida needs this for his folder summary work, so putting in b3.
Status: NEW → ASSIGNED
Whiteboard: [needs review standard8]
Target Milestone: --- → Thunderbird 3.0b3
I'm doing some other cleanup of the db enumerators in bug 492963
Attachment #380288 - Flags: superreview?(bugzilla)
Attachment #380288 - Flags: superreview+
Attachment #380288 - Flags: review?(bugzilla)
Attachment #380288 - Flags: review+
Comment on attachment 380288 [details] [diff] [review]
very lightly tested

Looks good. r/sr=Standard8
fix checked in - now to unbitrot the patch in bug 492963
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [needs review standard8]
You need to log in before you can comment on or make changes to this bug.