I think my patch for buffering is not working very well with quarantine handling. Do you know what is the test for checking TB with quarantine turned on and off for pop3 download? Oh, I think I should be writing to Maildev mailing list about this. The patch looks innocent enough, but somewhere I have made some implicit assumptions how the file streams behave.
Bug 1733849 Comment 19 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
I think my patch for buffering is not working very well with quarantine handling. nsMsgBrkMBoxStore::GetNewMsgOutputStream( ......) calls InternalGetNewMsgOutputStream(aFolder, aNewMsgHdr, aResult) or nsQuarantinedOutputStream(mboxStream) depending on whether quarantine is on. Do you know what is the test for checking TB with quarantine turned on and off for pop3 download? I want to check the file handling in those tests. Oh, I think I should be writing to Maildev mailing list about this. The patch for buffered writing looks innocent enough, but somewhere I have made some implicit assumptions how the file streams behave and changing due to quarantine might have broken the assumptions, and the recent changes in the tree have made them clear.