Re: svn commit: r981091 - in /subversion/branches/performance/subversion: include/private/svn_file_cache.h libsvn_subr/svn_file_cache.c
Blair Zajac wrote:
> On 7/31/10 1:47 PM, stefan2_at_apache.org wrote:
>> Author: stefan2
>> Date: Sat Jul 31 20:46:59 2010
>> New Revision: 981091
>> URL: http://svn.apache.org/viewvc?rev=981091&view=rev
>> Introduce a private file handle cache API and provide an implementation.
>> * subversion/include/private/svn_file_cache.h
>> (svn_file_cache_t, svn_file_cache__handle_t): introduce new opaque
>> data types
>> (svn_file_cache__open, svn_file_cache__has_file,
>> svn_file_cache__get_apr_handle, svn_file_cache__get_name,
>> svn_file_cache__close, svn_file_cache__flush,
>> declare new private API functions
> The name svn_file_cache_t isn't clear that this is a file handle
> cache. A first reading of it suggest that is a local cache for files
> (possibly remote). I suggest renaming it to svn_file_handle_cache_t
> and all other functions.
Done in r981204. Actually, I had the longer name in it before finally
committing the code but it seemed to yield very long identifiers.
Received on 2010-08-01 13:29:35 CEST
This is an archived mail posted to the Subversion Dev