On Tue, Feb 21, 2012 at 16:44, <hwri...@apache.org> wrote: > Author: hwright > Date: Tue Feb 21 21:44:13 2012 > New Revision: 1292047 > > URL: http://svn.apache.org/viewvc?rev=1292047&view=rev > Log: > Ev2 shims: Register the appropriate callbacks for use with the Ev2 shims every > time we fetch a commit editor from within the client.
I think you've leaked the "hidden shims" out too far with this change. Can't you put it back where you just registered the shims for every RA session. It certainly doesn't hurt to put them into the session, and let them be unused. On the baton, you're passing NULL for all of these, so it would seem that you could do the same when creating the RA session and installing shims. If you *do* need the baton sometimes, then pass the baton into the client-internal RA session creation. ... something. It just seems that we shouldn't be seeing this registration everywhere. Cheers, -g
