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.
Created an attachment (id=380341)
Dare I say, the final version of the patch?
(In reply to comment #227) dTypeValue replyToAll = 33; dTypeValue replyToList = 34;
> Are you using these?
> + const nsMsgViewComman
> + const nsMsgViewComman
Nope. I've removed them.
> Unless I'm missing a later use of msgURI, I think this: msgHdrFromURI( msgURI) ; msgHdrFromURI( GetLoadedMessag e());
> + let msgURI = GetLoadedMessage();
> + let msgHdr = messenger.
> can just be
> + let msgHdr = messenger.
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.