You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by David Glasser <gl...@mit.edu> on 2006/08/11 06:18:17 UTC

[PATCH] modernize ra_local_test

I was playing around with adding a new RA API and noticed that the
ra_local C test file still uses the old ra_plugin API. The following
patch fixes this.

(There isn't a similar place to write tests for the other ra
implementations of features that aren't exposed by the command-line
clients, is there?)

[[[
Update ra_local C test to modern RA API.

* subversion/tests/libsvn_ra_local/ra-local-test.c
  (make_and_open_local_repos): Use svn_ra_session_t instead of
   svn_ra_plugin_t and a baton.
  (open_ra_session, get_youngest_rev): Adjust calls to above.
]]]

-- 
David Glasser | glasser@mit.edu | http://www.davidglasser.net/

Re: [PATCH] modernize ra_local_test

Posted by Garrett Rooney <ro...@electricjellyfish.net>.
On 8/11/06, David Glasser <gl...@mit.edu> wrote:
> On 8/11/06, David Glasser <gl...@mit.edu> wrote:
> > [[[
> > Update ra_local C test to modern RA API.
> >
> > * subversion/tests/libsvn_ra_local/ra-local-test.c
> >   (make_and_open_local_repos): Use svn_ra_session_t instead of
> >    svn_ra_plugin_t and a baton.
> >   (open_ra_session, get_youngest_rev): Adjust calls to above.
> > ]]]
>
> Oops. Somehow missed a compilation warning that I was using a wrong
> type.  Try this one.

Applied in r21048.

Thanks!

-garrett

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Re: [PATCH] modernize ra_local_test

Posted by David Glasser <gl...@mit.edu>.
On 8/11/06, David Glasser <gl...@mit.edu> wrote:
> [[[
> Update ra_local C test to modern RA API.
>
> * subversion/tests/libsvn_ra_local/ra-local-test.c
>   (make_and_open_local_repos): Use svn_ra_session_t instead of
>    svn_ra_plugin_t and a baton.
>   (open_ra_session, get_youngest_rev): Adjust calls to above.
> ]]]

Oops. Somehow missed a compilation warning that I was using a wrong
type.  Try this one.

--dave

-- 
David Glasser | glasser@mit.edu | http://www.davidglasser.net/