Re: [Issue 806] Changed - De-kluge libsvn_ra_dav's fix for issue #797, use clean fix

Karl Fogel <kfogel@newton.ch.collab.net> writes:

> So, one solution might be to have a field in the access baton that
> stores deferred log items. When one invokes
>
> svn_wc__run_log (adm_access, pool);
>
> it would first take everything pent up in adm_access->final_log_items
> (or whatever) and append it to the log, atomically of course, *then*
> run the log.
>
> Any holes in that?

Sounds OK. It sounds like you are storing them in memory, with the
access baton pointing to them. An alternative is to hold an open
apr_file_t in the access baton, and write to that. Then
svn_wc__run_log uses the apr_file_t, either as part of the log, or as
the whole log. In fact, the entire log file could be accumulated this
way.

This is a bit like another idea which is to have the access baton be
aware of when a log file is running. Certain operations are allowed
when running a log file that are prohibited at other times.

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org