Closed Bug 777205 Opened 7 years ago Closed 7 years ago

Should remove unused wrap length value in nsMsgAttachmentHandler::UrlExit

Categories

(MailNews Core :: Composition, defect, trivial)

defect
Not set
trivial

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 17.0

People

(Reporter: hiro, Assigned: hiro)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Attached patch FixSplinter Review
No description provided.
Attachment #645618 - Flags: review?(mbanner)
Good catch. The width is determined but the variable never used. The real text conversion is done in ConvertBufToPlainText() and that one fetches the pref itself (and caps the width differently).

This will help bug 155219 to reduce confusion about the various manglings of the pref value.
Assignee: nobody → hiikezoe
Blocks: 155219
Status: NEW → ASSIGNED
aceman, thank you for the info. There are lots of wrap length issues. We need WADA there too. ;-p
Attachment #645618 - Flags: review?(mbanner) → review+
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/2368e7da5f4b
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 17.0
You need to log in before you can comment on or make changes to this bug.