You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@subversion.apache.org by Daniel Shahaf <d....@daniel.shahaf.name> on 2011/06/09 15:13:57 UTC

Re: API review - svn_cache_config.h

Julian Foad wrote on Thu, Jun 09, 2011 at 11:22:23 +0100:
> svn_cache_config.h
>   # Why is the cache config header separate from the cache header?

Because one is public and one is private?

>   svn_cache_config_t
>   svn_get_cache_config()
>   svn_set_cache_config()
>     # Rename these two to svn_cache_config_*().
> 

+1

Re: API review - svn_cache_config.h

Posted by Julian Foad <ju...@wandisco.com>.

On Thu, 2011-06-09 at 16:13 +0300, Daniel Shahaf wrote:
> Julian Foad wrote on Thu, Jun 09, 2011 at 11:22:23 +0100:
> > svn_cache_config.h
> >   # Why is the cache config header separate from the cache header?
> 
> Because one is public and one is private?

OK, that makes sense.

> >   svn_cache_config_t
> >   svn_get_cache_config()
> >   svn_set_cache_config()
> >     # Rename these two to svn_cache_config_*().
> > 
> 
> +1

r1133780.

Thanks.

- Julian