Re: svn commit: rev 1176 - trunk/subversion/svnadmin trunk/subversion/include trunk/subversion/libsvn_fs trunk/subversion/tests/libsvn_fs

cmpilato@tigris.org writes:

> * subversion/include/svn_fs.h
>
> (svn_fs_revisions_changed): New function.
>
> * subversion/libsvn_fs/tree.c
>
> (struct revisions_changed_args): New structure.
> (txn_body_revisions_changed, svn_fs_revisions_changed): New functions.

> @@ -662,6 +663,22 @@
> svn_node_kind_t svn_fs_check_path (svn_fs_root_t *root,
> const char *path,
> apr_pool_t *pool);
> +
> +
> +/* Allocate and return an array *REVS of svn_revnum_t revisions in
> + which PATH under ROOT in FS was modified. Use POOL for all
> + allocations.
> +
> + NOTE: This function uses node-id ancestry alone to determine
> + modifiedness, and therefore does NOT claim that in any of the
> + returned revisions file contents changed, properties changed,
> + directory entries lists changed, etc. */
> +svn_error_t *svn_fs_revisions_changed (apr_array_header_t **revs,
> + svn_fs_t *fs,
> + svn_fs_root_t *root,
> + const char *path,
> + apr_pool_t *pool);
> +
>
> /* Set *IS_DIR to non-zero iff PATH in ROOT is a directory.
> Do any necessary temporary allocation in POOL. */
>

I thought about writing Ruby binding for this function and a
bit surprised to see there're both svn_fs_t and
svn_fs_root_t arguments. Why svn_fs_t is there? I think
tree.c can access inside svn_fs_root_t and it can use
root->fs.

-- 
Yoshiki Hayashi
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org