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

Re: svn commit: r28897 - branches/reintegrate/subversion/libsvn_fs_fs

From: Hyrum K. Wright <hyrum_wright_at_mail.utexas.edu>
Date: Mon, 14 Jan 2008 15:23:16 -0600

dlr_at_tigris.org wrote:
> Author: dlr
> Date: Mon Jan 14 12:07:37 2008
> New Revision: 28897
>
> Log:
> On the reintegrate branch: Add some include statements.
>
> * subversion/libsvn_fs_fs/tree.c
> Add includes for apr_hash.h and apr_pools.h.
>
>
> Modified:
> branches/reintegrate/subversion/libsvn_fs_fs/tree.c
>
> Modified: branches/reintegrate/subversion/libsvn_fs_fs/tree.c
> URL: http://svn.collab.net/viewvc/svn/branches/reintegrate/subversion/libsvn_fs_fs/tree.c?pathrev=28897&r1=28896&r2=28897
> ==============================================================================
> --- branches/reintegrate/subversion/libsvn_fs_fs/tree.c (original)
> +++ branches/reintegrate/subversion/libsvn_fs_fs/tree.c Mon Jan 14 12:07:37 2008
> @@ -33,6 +33,9 @@
> #include <stdlib.h>
> #include <string.h>
> #include <assert.h>
> +#include <apr_pools.h>
> +#include <apr_hash.h>
> +
> #include "svn_private_config.h"
> #include "svn_pools.h"
> #include "svn_error.h"

apr_pools.h would seem a bit redundant with svn_pools.h. In a similar
vein, is there a reason to include apr_hash.h instead of svn_hash.h?

-Hyrum

Received on 2008-01-14 22:23:27 CET

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