Closed Bug 312572 Opened 15 years ago Closed 15 years ago
Simplify syntax error handling/logging in IMAP parsing
The following simplifications can implemented easily and without changing any behavior of the IMAP implementation. * remove field |nsIMAPGenericParser::fSyntaxErrorLine| * remove method |nsIMAPGenericParser::::CreateSyntaxErrorLine()|
Comment on attachment 199684 [details] [diff] [review] Simplify syntax error handling/logging in IMAP parsing looks good - I'll try running with this, though I don't often have contact with servers that cause syntax errors :-)
Attachment #199684 - Flags: review?(bienvenu) → review+
fixed, thx, Hans. Sorry, I forgot to credit you in my checkin comment :-(
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Regarding the incomplete checkin comment: No Problem! (Anyway, it was a trivial patch, only leading to a small improvement of the code.) David, thank you for the timely processing of this bug!
I'm doing an LXR verification of this, because to be honest I'm assuming it works until someone files a bug on it ;-)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.