Garrett Rooney <rooneg@electricjellyfish.net> writes:
> On Tuesday, May 20, 2003, at 09:50 PM, Greg Stein wrote:
>
> > On Tue, May 20, 2003 at 01:26:57PM -0500, cmpilato@tigris.org wrote:
> >> ...
> >> (apply_textdelta): Use the file baton's pool to store the tmpfile
> >> and its path.
> >
> > How about: keep the file in the handler pool, and rely on that to
> > auto close
> > the file when the handler is done. Then strdup the path into the
> > file baton.
> > You'd eliminate the need to store the pool in the baton (as that pool
> > sitting there could lead to Badness).
So predictable... Ben and I were joking about how you'd say this. In
fact, I was actually going to do it this way. Ben stopped me, though,
saying that that was how things had been before, and they were
purposely changed ... he just couldn't remember why.
> The original reason the file was moved to the baton was so we could
> explicitly close it (flushing data to disk) before doing a
> copy-and-translate. If you move it to any pool, be sure that the
> auto-close happens before that occurs.
And there's the "why".
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed May 21 19:46:24 2003