Created an attachment (id=319890)
an attempt to fix this problem
This patch fixes handling of addresses of the form 'Spät, Karl <email address hidden>' for me. Please give it a try. The Patch is against HEAD (May 07, 2008).
Rationale:
In many parts of the MailNews sources, MIME headers containing RFC 2822 address lists are treated by decoding MIME first, then expanding address lists. This is wrong if the MIME-encoded data contains a comma (MIME encoded data should be treated as an RFC 2822 atom).
This patch changes the behavior such that MIME decoding is done last, immediately before displaying the headers, and in particular, after splitting up the address list.
Individual changes:
- nsMsgCompFields::SplitRecipients(): run postponed late MIME conversions
- nsMsgDBView::FetchAuthor(): fixes message list display
- nsMsgCompose::CreateMessage(): postpone MIME conversion for some fields (for "Reply")
- QuotingOutputStreamListener():: postpone more MIME conversions (for "Reply")
- QuotingOutputStreamListener::OnStopRequest(): dito
- CreateCompositionFields(): dito, for "Edit as New"
- MimeHeaders_write_all_headers(): dito, for message display window
Created an attachment (id=319890)
an attempt to fix this problem
This patch fixes handling of addresses of the form 'Spät, Karl <email address hidden>' for me. Please give it a try. The Patch is against HEAD (May 07, 2008).
Rationale:
In many parts of the MailNews sources, MIME headers containing RFC 2822 address lists are treated by decoding MIME first, then expanding address lists. This is wrong if the MIME-encoded data contains a comma (MIME encoded data should be treated as an RFC 2822 atom).
This patch changes the behavior such that MIME decoding is done last, immediately before displaying the headers, and in particular, after splitting up the address list.
Individual changes:
- nsMsgCompFields ::SplitRecipien ts(): run postponed late MIME conversions :FetchAuthor( ): fixes message list display :CreateMessage( ): postpone MIME conversion for some fields (for "Reply") reamListener( ):: postpone more MIME conversions (for "Reply") reamListener: :OnStopRequest( ): dito onFields( ): dito, for "Edit as New" write_all_ headers( ): dito, for message display window
- nsMsgDBView:
- nsMsgCompose:
- QuotingOutputSt
- QuotingOutputSt
- CreateCompositi
- MimeHeaders_