On Mon, Jan 30, 2006 at 10:21:22AM -0800, Zac Medico wrote: > Zac Medico wrote: > > Okay, I've created a file-like class called atomic_ostream and it is now > > used for both write_atomic() and writedict(). > > I've been using this patch locally with no problems. Do we have > any more feedback or are people satisfied with it? IMO we need > something like this, if not for (unsupported) parallel merges, at > least to prevent loss of an important file when it is being > overwritten and an IO error occurs (see bug 114133).
Meh.... you're not supposed to call me on being a slacker for not commenting. :) No complaints with this going into svn for upcoming 2.1_pre*, but I'd like to see the class rewritten actually- the file object lacks a lot of capabilities that a normal file object has. I'd suggest deriving straight from the file class, and just doing changes in close and open. Should be a much smaller class also ;) ~harring
pgpiD5Co9VAKj.pgp
Description: PGP signature
