You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by st...@apache.org on 2015/08/26 14:45:12 UTC
svn commit: r1697912 - /subversion/trunk/subversion/libsvn_ra_svn/marshal.c
Author: stefan2
Date: Wed Aug 26 12:45:12 2015
New Revision: 1697912
URL: http://svn.apache.org/r1697912
Log:
Revert r1694533.
Modified:
subversion/trunk/subversion/libsvn_ra_svn/marshal.c
Modified: subversion/trunk/subversion/libsvn_ra_svn/marshal.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_ra_svn/marshal.c?rev=1697912&r1=1697911&r2=1697912&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_ra_svn/marshal.c (original)
+++ subversion/trunk/subversion/libsvn_ra_svn/marshal.c Wed Aug 26 12:45:12 2015
@@ -1190,20 +1190,14 @@ static svn_error_t *read_item(svn_ra_svn
}
else if (c == '(')
{
- /* On machines with 32 bit pointers, array headers are only 20 bytes
- * which is not enough for our standard 64 bit alignment.
- * So, determine a suitable block size for the APR array header that
- * keeps proper alignment for following structs. */
- const apr_size_t header_size
- = APR_ALIGN_DEFAULT(sizeof(apr_array_header_t));
-
/* Allocate an APR array with room for (initially) 4 items.
* We do this manually because lists are the most frequent protocol
* element, often used to frame a single, optional value. We save
* about 20% of total protocol handling time. */
- char *buffer = apr_palloc(pool,
- header_size + 4 * sizeof(svn_ra_svn_item_t));
- svn_ra_svn_item_t *data = (svn_ra_svn_item_t *)(buffer + header_size);
+ char *buffer = apr_palloc(pool, sizeof(apr_array_header_t)
+ + 4 * sizeof(svn_ra_svn_item_t));
+ svn_ra_svn_item_t *data
+ = (svn_ra_svn_item_t *)(buffer + sizeof(apr_array_header_t));
item->kind = SVN_RA_SVN_LIST;
item->u.list = (apr_array_header_t *)buffer;