[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: rev 4923 - in trunk/subversion: include libsvn_wc

From: <cmpilato_at_collab.net>
Date: 2003-02-19 15:32:00 CET

Greg Stein <gstein@lyra.org> writes:

> On Tue, Feb 18, 2003 at 12:20:25PM -0600, cmpilato@tigris.org wrote:
> >...
> > +/* Public API */
> > +void
> > +svn_wc_get_username_provider (const svn_auth_provider_t **provider,
> > + void **provider_baton,
> > + apr_pool_t *pool)
> > +{
> > + provider_baton_t *pb = apr_pcalloc (pool, sizeof (*pb));
> > svn_auth_provider_t *prov = apr_palloc (pool, sizeof (*prov));
> >
> > prov->cred_kind = SVN_AUTH_CRED_SIMPLE;
> > - prov->first_credentials = simple_wc_first_creds;
> > + prov->first_credentials = username_first_creds;
> > prov->next_credentials = NULL; /* no retry. */
> > - prov->save_credentials = simple_wc_save_creds;
> > + prov->save_credentials = username_save_creds;
> >
> > *provider = prov;
> > *provider_baton = pb;
>
> The svn_auth_provider_t structure should be declared as a static const
> structure. There is no reason to place it onto the heap -- the API was
> designed to avoid the heap.

Ah yes. We switched to a heap allocation back when we were
dynamically (and conditionally) setting up the vtable functions. Will
switch it (them, really) back.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Feb 19 15:34:12 2003

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.