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

Re: svn commit: r1041102 - /subversion/trunk/subversion/include/svn_io.h

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Thu, 2 Dec 2010 07:55:29 +0200

julianfoad_at_apache.org wrote on Wed, Dec 01, 2010 at 17:44:50 -0000:
> Author: julianfoad
> Date: Wed Dec 1 17:44:49 2010
> New Revision: 1041102
>
> URL: http://svn.apache.org/viewvc?rev=1041102&view=rev
> Log:
> * subversion/include/svn_io.h
> (svn_io_dir_file_copy): Clarify the doc string.
>
> Modified:
> subversion/trunk/subversion/include/svn_io.h
>
> Modified: subversion/trunk/subversion/include/svn_io.h
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/include/svn_io.h?rev=1041102&r1=1041101&r2=1041102&view=diff
> ==============================================================================
> --- subversion/trunk/subversion/include/svn_io.h (original)
> +++ subversion/trunk/subversion/include/svn_io.h Wed Dec 1 17:44:49 2010
> @@ -692,8 +692,11 @@ svn_error_t *
> svn_io_file_flush_to_disk(apr_file_t *file,
> apr_pool_t *pool);
>
> -/** Copy file @a file from location @a src_path to location @a dest_path.
> - * Use @a pool for memory allocations.
> +/** Copy the file whose basename or relative path is @a file within
> + * directory @a src_path to the same basename or relative path within
> + * directory @a dest_path. Overwrite the destination if it already

If you allow relative paths as input for @a file, then shouldn't the
docstring say whether or not this creates intermediate directories?

Example: svn_io_dir_file_copy('A/B', 'A/C', 'E/alpha', NULL).

> + * exists. Set the destination file's permissions to match those of
> + * the source. Use @a pool for memory allocations.
> */
> svn_error_t *
> svn_io_dir_file_copy(const char *src_path,
> const char *dest_path,
> const char *file,
> apr_pool_t *pool)
>
>
Received on 2010-12-02 06:57:35 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.