Following up on r1583977, tweak the dead transaction checks in our tests.

* subversion/tests/cmdline/commit_tests.py
  (failed_commit, start_commit_hook_test): Use run_and_verify_svnadmin()
    helper instead of directly comparing OUTPUT and ERRPUT.  This makes
    the check stricter, because we now also test the exit code and removes
    unnecessary boilerplate.


git-svn-id: https://svn.apache.org/repos/asf/subversion/trunk@1585549 13f79535-47bb-0310-9956-ffa450edef68
* subversion/libsvn_fs_base/lock.c
  (svn_fs_base__unlock): Don't leak error.

Found by: julianfoad


git-svn-id: https://svn.apache.org/repos/asf/subversion/trunk@1585564 13f79535-47bb-0310-9956-ffa450edef68
Remove an unnecessary callback wrapper.

* subversion/libsvn_fs/fs-loader.c
  (struct lock_many_baton_t, lock_many_cb): Remove.
  (svn_fs_lock_many): Pass callers callback.

Found by: julianfoad


git-svn-id: https://svn.apache.org/repos/asf/subversion/trunk@1585566 13f79535-47bb-0310-9956-ffa450edef68
* subversion/svnserve/serve.c
  (lock_many, unlock_many): Tweak comments.

* subversion/include/svn_fs.h
  (svn_fs_lock_target_t,
   svn_fs_lock_target_create,
   svn_fs_lock_target_set_token): Tweak comments.


git-svn-id: https://svn.apache.org/repos/asf/subversion/trunk@1585568 13f79535-47bb-0310-9956-ffa450edef68