On 7/31/10 1:47 PM, stef...@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
Log:
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, svn_file_cache__create_cache):
   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.

Regards,
Blair

Reply via email to