You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by da...@apache.org on 2011/08/31 00:48:15 UTC

svn commit: r1163442 - /subversion/branches/revprop-packing/subversion/libsvn_fs_fs/fs_fs.c

Author: danielsh
Date: Tue Aug 30 22:48:14 2011
New Revision: 1163442

URL: http://svn.apache.org/viewvc?rev=1163442&view=rev
Log:
On the revprop-packing branch, make a field's width consistent with
that field's values type.

This is a breaking change; any previous repositories created by earlier
incarnations of this branch are now broken.

* subversion/libsvn_fs_fs/fs_fs.c
  (REVPROP_MANIFEST_FIELD_WIDTH):
    Make the field large enough for unsigned 64-bit ints.

Modified:
    subversion/branches/revprop-packing/subversion/libsvn_fs_fs/fs_fs.c

Modified: subversion/branches/revprop-packing/subversion/libsvn_fs_fs/fs_fs.c
URL: http://svn.apache.org/viewvc/subversion/branches/revprop-packing/subversion/libsvn_fs_fs/fs_fs.c?rev=1163442&r1=1163441&r2=1163442&view=diff
==============================================================================
--- subversion/branches/revprop-packing/subversion/libsvn_fs_fs/fs_fs.c (original)
+++ subversion/branches/revprop-packing/subversion/libsvn_fs_fs/fs_fs.c Tue Aug 30 22:48:14 2011
@@ -70,8 +70,10 @@
 #define FSFS_MAX_PATH_LEN 4096
 
 /* Revprop packing uses a fixed-width manifest field size.  This is that
- * width, not including the terminating newline. */
-#define REVPROP_MANIFEST_FIELD_WIDTH 16
+ * width, not including the terminating newline.
+ * 20 == len('%s' % ((1<<64)-1)).
+ */
+#define REVPROP_MANIFEST_FIELD_WIDTH 20
 
 /* The default maximum number of files per directory to store in the
    rev and revprops directory.  The number below is somewhat arbitrary,