Comment 285 for bug 52667

Revision history for this message
In , Blake Winton (bwinton) wrote :

Created an attachment (id=380341)
Dare I say, the final version of the patch?

(In reply to comment #227)
> Are you using these?
> + const nsMsgViewCommandTypeValue replyToAll = 33;
> + const nsMsgViewCommandTypeValue replyToList = 34;

Nope. I've removed them.

> Unless I'm missing a later use of msgURI, I think this:
> + let msgURI = GetLoadedMessage();
> + let msgHdr = messenger.msgHdrFromURI(msgURI);
> can just be
> + let msgHdr = messenger.msgHdrFromURI(GetLoadedMessage());

Fixed.

Would one of you or Magnus please commit this when you get a chance?

In the meantime, I'll start working on the enabling/disabling patch.

Thank you,
Blake.