Closed Bug 530618 Opened 15 years ago Closed 15 years ago

remove aol personal file cabinet aka pfc code

Categories

(Thunderbird :: General, defect)

Product:
Thunderbird
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Thunderbird 3.1a1

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Details

Attachments

(1 file)

proposed fix
19.77 KB, patch
neil
: review+
neil
: superreview+
Details | Diff | Splinter Review 
We have a bunch of code that has to do with the AOL personal file cabinet - it can be all removed.
Wonder how many of http://mxr.mozilla.org/l10n-mozilla1.9.1/search?string=5062 are actually the same phrase AOL used, assuming it was localized in those languages :)
OS: Windows 7 → All
Hardware: x86 → All
I'll do a patch for this.
Assignee: nobody → bienvenu
Status: NEW → ASSIGNED
Attached patch proposed fix — — Splinter Review 
removes unused code - I tried to get all the SM stuff as well.
Attachment #414136 - Flags: superreview?(neil)
Attachment #414136 - Flags: review?(neil)
Comment on attachment 414136 [details] [diff] [review]
proposed fix

>-  if (imapServer.storeReadMailInPFC)
So, who removed the storeReadMailInPFC attribute without fixing this???
that line is removed, according to the diff, and isn't in my tree anymore - but perhaps I horked the diff when trying to remove some other changes.
Oh, you're talking about past rev - I don't know.
Ah, so this sneaking past us in bug 476260...
Attachment #414136 - Flags: superreview?(neil)
Attachment #414136 - Flags: superreview+
Attachment #414136 - Flags: review?(neil)
Attachment #414136 - Flags: review+
fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.1a1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: