Closed Bug 458692 Opened 11 years ago Closed 11 years ago

Drop charset parameter from nsIMsgHeaderParser interfaces

Categories

(MailNews Core :: MIME, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.0b1

People

(Reporter: standard8, Assigned: standard8)

Details

(Keywords: memory-footprint)

Attachments

(1 file)

Attached patch The fixSplinter Review
Back in bug 414432 comment 7 we agreed to drop the charset parameter from the nsIMsgHeaderParser interfaces because it wasn't used and also not needed.

Attaching a patch which does just that and cleans up some of the documentation in nsIMsgHeaderParser.

I'm expecting that we can remove the noscript and have a string tidy up on the other functions, but that can come in a later bug/patch - simple bits first.
Attachment #341909 - Flags: superreview?(neil)
Attachment #341909 - Flags: review?(bienvenu)
Attachment #341909 - Flags: superreview?(neil) → superreview+
Attachment #341909 - Flags: review?(bienvenu) → review+
Checked in, changeset id  528:8115321452f9
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.0b1
You need to log in before you can comment on or make changes to this bug.