Re: CVS update: MODIFIED: svntest ...

Greg Stein <gstein@lyra.org> writes:
> > +const char *
> > +svn_fs_db_lockfile (svn_fs_t *fs)
> > +{
> > + return apr_psprintf (fs->pool, "%s/%s",
> > + fs->lock_path,
> > + SVN_FS__REPOS_DB_LOCKFILE);
>
> The above is more efficient as:
>
> return apr_pstrcat (fs->pool,
> fs->lock_path,
> "/" SVN_FS__REPOS_DB_LOCKFILE,
> NULL);
>
> There are similar examples throughout...

Thanks, will change.

> (no need to psprintf the thing...)

Sheesh. Codin' on autopilot.

Thanks for watching, Greg.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:36:35 2006