[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[PATCH] Use svn_fs_fs__id_unparse() to construct the noderev cache key

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Sun, 31 Oct 2010 03:42:04 +0200

Is there a reason not to apply this "let's not reinvent the wheel" patch?

[[[
Index: subversion/libsvn_fs_fs/fs_fs.c
===================================================================
--- subversion/libsvn_fs_fs/fs_fs.c (revision 1029231)
+++ subversion/libsvn_fs_fs/fs_fs.c (working copy)
@@ -2205,15 +2205,14 @@ read_rep_offsets(representation_t **rep_p,
   return SVN_NO_ERROR;
 }
 
-/* Combine the revision and offset of the ID to a string that will
- * be used as a cache key. Allocations will be made from POOL.
+/* Return a string that uniquely identifies the noderev with the
+ * given ID, for use as a cache key.
  */
 static const char *
 get_noderev_cache_key(const svn_fs_id_t *id, apr_pool_t *pool)
 {
- return svn_fs_fs__combine_two_numbers(svn_fs_fs__id_rev(id),
- svn_fs_fs__id_offset(id),
- pool);
+ const svn_string_t *id_unparsed = svn_fs_fs__id_unparse(id, pool);
+ return id_unparsed->data;
 }
 
 /* Look up the NODEREV_P for ID in FS' node revsion cache. If noderev
]]]

(It does not resolve my test failures.)

Daniel
Received on 2010-10-31 02:44:52 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.