Closed Bug 654317 Opened 9 years ago Closed 9 years ago

unneeded saving of filter files during folder move/copies slows the process

Categories

(MailNews Core :: Filters, defect)

x86
Windows 7
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 5.0b1

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Details

(Keywords: perf)

Attachments

(1 file, 1 obsolete file)

Attached patch proposed fix (obsolete) — Splinter Review
I was deleting a local folder hierarchy, and it was taking forever. Using my top-secret performance investigation technique, I discovered that we were spending most of the time saving the filter list unnecessarily. Fix upcoming.

Asking for review from rkent on the slightly dubious basis that this nominally involves filters.
Attachment #529597 - Flags: review?(kent)
Attachment #529597 - Attachment is obsolete: true
Attachment #529597 - Flags: review?(kent)
Attachment #529607 - Flags: review?(kent)
Keywords: perf
Summary: unneeded saving of filter files during folder move/copies → unneeded saving of filter files during folder move/copies slows the process
Comment on attachment 529607 [details] [diff] [review]
make sure we set found

Review of attachment 529607 [details] [diff] [review]:

The essence of this is "check found value not address", not "make sure we set found" as the found are already all set upstream AFAICT, but no matter, the patch is good.
Attachment #529607 - Flags: review?(kent) → review+
fixed on trunk - http://hg.mozilla.org/comm-central/rev/7e6e9dc6e84d

the name of the second patch was trying to describe the difference between the first and second patch, which is that we need to set *found to PR_FALSE when we don't find a matching filter, instead of only setting it to PR_TRUE when we do find a filter.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.3a4
You need to log in before you can comment on or make changes to this bug.