[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r13631 - in trunk/subversion: include libsvn_fs libsvn_fs_base libsvn_fs_fs libsvn_ra_local libsvn_repos mod_dav_svn svnserve tests/libsvn_fs tests/libsvn_repos

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: 2005-03-24 15:50:29 CET

"Peter N. Lundblad" <peter@famlundblad.se> writes:

> > * If @a path is non-existent, that's fine. The path is reserved, and
> > * a lock-token is returned.
>
> This got turned off later.

Ah, yes. Seems someone failed to change the docstring when making
that change...

> > @@ -1516,48 +1524,12 @@
> > svn_error_t *svn_fs_lock (svn_lock_t **lock,
> > svn_fs_t *fs,
> > const char *path,
> > + const char *token,
> > const char *comment,
> > - svn_boolean_t force,
> > long int timeout,
>
> That's not an apr_time_t as the log says. (I'd support that change,
> though.)

My fault. I started to make it an apr_time_t, then bailed (but forgot
to tweak the log). My reasoning was that an apr_time_t is supposed to
represent a moment in Time, not a span thereof. I dunno. I'd not -1
a change to apr_time_t if the community wants it.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Mar 24 15:54:45 2005

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.