Comment on attachment 756486
Write updated popstate in tempfile first, then move over

Review of attachment 756486:
-----------------------------------------------------------------

Thanks so much for the patch; sorry it took so long for me to review it.

I'm OK with either landing this patch as is, or expanding it to refactor
out the other places in mailnews where we create safe, buffered output
streams.

::: mailnews/base/util/nsMsgUtils.cpp
@@ +1438,4 @@
>    return rv;
>  }
>  
> +nsresult MsgNewSafeBufferedFileOutputStream(nsIOutputStream **aResult,

There are a couple of other places in mailnews where we're creating safe
output streams; it would be great to update those to use this helper
function instead of repeating the code, as part of this patch.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/261595

Title:
  Thunderbird corrupts Mailbox when hd is full

To manage notifications about this bug go to:
https://bugs.launchpad.net/thunderbird/+bug/261595/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to