Author: stefan2
Date: Sun Dec 5 22:38:41 2010
New Revision: 1042479
URL: http://svn.apache.org/viewvc?rev=1042479&view=rev
Log:
Mention the guarantees and non-guarantees associated with the
path that will be returned by svn_fs_fs__path_rev_absolute().
* subversion/libsvn_fs_fs/fs_fs.c
(svn_fs_fs__path_rev_absolute): improve commentary
Modified:
subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c
Modified: subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c
URL:
http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c?rev=1042479&r1=1042478&r2=1042479&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c (original)
+++ subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c Sun Dec 5 22:38:41 2010
@@ -247,7 +247,13 @@ path_rev(svn_fs_t *fs, svn_revnum_t rev,
}
/* Returns the path of REV in FS, whether in a pack file or not.
- Allocate in POOL. */
+ Allocate in POOL.
+ NOTE. The file given by path is only guaranteed to actually
+ exist if it has not been deleted since this FSFS session has
+ been opened. This is, no race condition will occur if the caller
+ holds a write lock to the repository. If it doesn't, it may
+ need to retry the path mapping upon failure to open the file.
+ */
svn_error_t *
svn_fs_fs__path_rev_absolute(const char **path,
svn_fs_t *fs,
