"Peter N. Lundblad" <peter@famlundblad.se> writes:
> > > Also, svn_client_ctx_t is extensible by design. It is possible
> > > to add the new notify function/baton as new
> > > fields. svn_client_create_context could init them to a
> > > function/baton that forwards to the old function. The other
> > > alternative is to revise svn_client_ctx_t, leading to revision
> > > of all client functions. Which do you prefer as the least ugly
> > > way? Or do I miss an obvious other alternative?
>
> I still want an opinion about the above paragraph.
So the idea is to put
svn_wc_notify_func_t2 notify_func2;
void *notify_baton2;
at the end of svn_client_ctx_t? But I'm unclear on exactly how
svn_client_create_context is supposed forward after that. That
function doesn't have a handle on the actual notify_func and baton the
caller is going to put in the ctx, so how can it set up the call
forwarding? All svn_client_create_context does is allocate the
structure; it doesn't currently initialize any fields.
I guess I'm not sure exactly what you're proposing. Could you
describe it in more detail, or in patch form?
Thanks,
-Karl
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Mar 2 17:02:38 2005