Re: [PATCH] Add authz support to the commit editor

This patch looks good, and I will likely commit it this evening with an
s/revv/rev/ in the log message.

Your testing methodology runs afoul of this part of the editor API:

 * At least one implementation of the editor interface is
 * asynchronous; an error from one operation may be detected some
 * number of operations later. As a result, an editor driver must not
 * assume that an error from an editing function resulted from the
 * particular operation being detected. Moreover, once an editing
 * function returns an error, the edit is dead; the only further
 * operation which may be called on the editor is abort_edit.

But since we're always testing over file:/// and this is part of the
test suite, I think that's okay. I may add a comment.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Aug 4 22:43:50 2005