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

Re: API to get revision size on disk

From: Evgeny Kotkov <evgeny.kotkov_at_visualsvn.com>
Date: Sat, 13 Apr 2019 00:43:50 +0300

Julian Foad <julianfoad_at_apache.org> writes:

> I have reviewed and tested it. Can you commit?

Fixed the typo and committed in https://svn.apache.org/r1857435, thanks!

> * We could keep the existing private API declarations in
> include/private/svn_fs_fs_private.h as well as the new ioctl versions of
> them, rather than moving the existing declarations to a local header. It
> would reduce the patch size, and some people might like the ability to
> link to libsvn_fs_fs directly and and call those functions directly. But
> I don't feel strongly about this; either way is fine for me.

For now I chose to move the declarations to libsvn_fs_fs/fs_fs.h, to avoid
having two parallel interfaces and to only export the version that works
through the FS loader.

Evgeny Kotkov
Received on 2019-04-12 23:58:47 CEST

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.