subversion-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From danie...@apache.org
Subject svn commit: r1173218 - /subversion/branches/fs-successor-ids/subversion/libsvn_fs_fs/fs_fs.c
Date Tue, 20 Sep 2011 16:09:35 GMT
Author: danielsh
Date: Tue Sep 20 16:09:35 2011
New Revision: 1173218

URL: http://svn.apache.org/viewvc?rev=1173218&view=rev
Log:
On the fs-successor-id branch, update comments to reflect a task as done.

* subversion/libsvn_fs_fs/fs_fs.c
  (read_successor_index_entry): Document r1169855.  No functional change.

Modified:
    subversion/branches/fs-successor-ids/subversion/libsvn_fs_fs/fs_fs.c

Modified: subversion/branches/fs-successor-ids/subversion/libsvn_fs_fs/fs_fs.c
URL: http://svn.apache.org/viewvc/subversion/branches/fs-successor-ids/subversion/libsvn_fs_fs/fs_fs.c?rev=1173218&r1=1173217&r2=1173218&view=diff
==============================================================================
--- subversion/branches/fs-successor-ids/subversion/libsvn_fs_fs/fs_fs.c (original)
+++ subversion/branches/fs-successor-ids/subversion/libsvn_fs_fs/fs_fs.c Tue Sep 20 16:09:35
2011
@@ -5974,7 +5974,7 @@ read_successor_index_entry(apr_uint64_t 
       return SVN_NO_ERROR;
     }
 
-  /* ### TODO(sid): don't constantly re-open the file */
+  /* Open the file or re-use the opened file our caller provides. */
   if (file_p && *file_p)
     file = *file_p;
   else
@@ -6007,6 +6007,7 @@ read_successor_index_entry(apr_uint64_t 
     }
   *revision_offset = ((apr_uint64_t)(ntohl(n)) << 32) | ntohl(m);
 
+  /* Close the file or let our caller stash it. */
   if (file_p && ! *file_p)
     *file_p = file;
   else if (! file_p)



Mime
View raw message