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

RE: svn commit: r1403979 - /subversion/trunk/subversion/libsvn_subr/io.c

From: Bert Huijben <bert_at_qqmail.nl>
Date: Wed, 31 Oct 2012 10:33:35 +0100

> -----Original Message-----
> From: danielsh_at_apache.org [mailto:danielsh_at_apache.org]
> Sent: woensdag 31 oktober 2012 05:50
> To: commits_at_subversion.apache.org
> Subject: svn commit: r1403979 -
> /subversion/trunk/subversion/libsvn_subr/io.c
>
> Author: danielsh
> Date: Wed Oct 31 04:50:22 2012
> New Revision: 1403979
>
> URL: http://svn.apache.org/viewvc?rev=1403979&view=rev
> Log:
> No functional change.
>
> * subversion/libsvn_subr/io.c
> (do_io_file_wrapper_cleanup): Move earlier in the file so that it's next
> to a "Forward declarations" header.
>
> Modified:
> subversion/trunk/subversion/libsvn_subr/io.c
>
> Modified: subversion/trunk/subversion/libsvn_subr/io.c
> URL:
> http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_subr/io.
> c?rev=1403979&r1=1403978&r2=1403979&view=diff
> ==========================================================
> ====================
> --- subversion/trunk/subversion/libsvn_subr/io.c (original)
> +++ subversion/trunk/subversion/libsvn_subr/io.c Wed Oct 31 04:50:22 2012
> @@ -2077,11 +2077,6 @@ svn_io_file_lock2(const char *lock_file,
>

> /* Data consistency/coherency operations. */
>
> -static APR_INLINE svn_error_t *
> -do_io_file_wrapper_cleanup(apr_file_t *file, apr_status_t status,
> - const char *msg, const char *msg_no_name,
> - apr_pool_t *pool);
> -
> svn_error_t *svn_io_file_flush_to_disk(apr_file_t *file,
> apr_pool_t *pool)
> {
> @@ -3726,6 +3721,10 @@ svn_io_dir_open(apr_dir_t **new_dir, con
> /* Forward declaration */
> static apr_status_t
> dir_is_empty(const char *dir, apr_pool_t *pool);
> +static APR_INLINE svn_error_t *
> +do_io_file_wrapper_cleanup(apr_file_t *file, apr_status_t status,
> + const char *msg, const char *msg_no_name,
> + apr_pool_t *pool);

Just looking at this patch, isn't this moving it to further in the file?
(Or is it just switching the order and do we see a different diff?)

The log message would indicate that you moved the definition up.

        Bert
Received on 2012-10-31 10:34:31 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.