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

A notification framework for libsvn_repos

From: Hyrum K. Wright <hyrum_wright_at_mail.utexas.edu>
Date: Fri, 28 May 2010 10:22:04 -0500

In poking around libsvn_repos lately, I've found a somewhat neglected corner
of the library: the notification system. Several functions return
notifications back to the caller (typically 'svnadmin'), but they all use
different callback types. In the case of svn_repos_load_fs2(), the output
is generated directly inside of libsvn_repos(!), and just fed into a stream
the caller provides.

I think we can unify the notifications in libsvn_repos without too much
difficulty by emulating the svn_wc_notify_t framework. It would give
callers much more fine-grained control over knowing what is going on in the
process, rather than having to parse a stream or implement 3 different
callbacks. With the prevalence of remove-administration tools, I think this
is a good benefit.

My one concern is that this will be too much overhead for something like
load, which creates lots of notifications. (Actually, load could probably
just reuse the same baton, resetting values differently each time.


Received on 2010-05-28 17:22:44 CEST

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