A notification framework for libsvn_repos

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.
Hmmm....)

Thoughts?

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