313038 - Tracking bug: clean-up parsing of IMAP server response

Status: RESOLVED INVALID

(MailNews Core :: Networking: IMAP, enhancement)

Description

[Hans-Andreas Engel]

Traditionally, the IMAP parser code contains many internal inconsistencies,
kludges which work around these inconsistencies, and inefficiencies.  On the
other hand, the code works very well (up to performance issues) for many IMAP
servers.

The goal is to eliminate the problems in the code, but at the same time the
behavior for a given server response should be maintained in order to keep the
code working well.

To ensure that this latter criterion is satisfied, I decided to achieve this
goal by splitting up the required changes in small steps; these steps are
tracked in this bug.

Comment 1

[Hans-Andreas Engel]

As of 2004-07-30, the code involved in IMAP parsing was in
http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/mailnews/imap/src/nsIMAPGenericParser.cpp&rev=1.38&root=/cvsroot
http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/mailnews/imap/src/nsImapServerResponseParser.cpp&rev=1.112&root=/cvsroot
http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/mailnews/imap/src/nsIMAPBodyShell.cpp&rev=1.43&root=/cvsroot

nsIMAPGenericParser.cpp            766 lines
nsImapServerResponseParser.cpp    2950 lines
nsIMAPBodyShell.cpp               1770 lines

Comment 2

[Wayne Mery (:wsmwk)]

all bugs cited are fixed.
closing INVALID