[prev in list] [next in list] [prev in thread] [next in thread] 

List:       subversion-dev
Subject:    Re: svn commit: r1527217 - /subversion/trunk/subversion/libsvn_fs_fs/transaction.c
From:       Ivan Zhakov <ivan () visualsvn ! com>
Date:       2013-09-28 19:22:02
Message-ID: CABw-3Yf4FXcQuufQRCeDoqnDdfh5iptCokFAueGkaH3nsGttsg () mail ! gmail ! com
[Download RAW message or body]

On 28 September 2013 23:11,  <stefan2@apache.org> wrote:
> Author: stefan2
> Date: Sat Sep 28 19:11:23 2013
> New Revision: 1527217
> 
> URL: http://svn.apache.org/r1527217
> Log:
> Skip move-related checks and updates in FSFS if moves are not supported.
> This minimizes the overhead for existing repositories.
> 
> * subversion/libsvn_fs_fs/transaction.c
> (write_final_changed_path_info,
> commit_body): execute fs_move related code only if moves are supported
> by the current FSFS format
> 
> Modified:
> subversion/trunk/subversion/libsvn_fs_fs/transaction.c
> 
> Modified: subversion/trunk/subversion/libsvn_fs_fs/transaction.c
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_fs/transaction.c?rev=1527217&r1=1527216&r2=1527217&view=diff
>  ==============================================================================
> --- subversion/trunk/subversion/libsvn_fs_fs/transaction.c (original)
> +++ subversion/trunk/subversion/libsvn_fs_fs/transaction.c Sat Sep 28 19:11:23 2013
> @@ -2706,15 +2706,16 @@ write_final_changed_path_info(apr_off_t
> SVN_ERR(svn_fs_fs__get_file_offset(&offset, file, pool));
> 
> /* all moves specify the "copy-from-rev" as REV-1 */
> -  for (hi = apr_hash_first(pool, changed_paths); hi; hi = apr_hash_next(hi))
> -    {
> -      svn_fs_path_change2_t *change;
> -      apr_hash_this(hi, NULL, NULL, (void **)&change);
> +  if (svn_fs_fs__supports_move(fs))
Minor nit: explicit statement block ({}) will make code more readable.

> +    for (hi = apr_hash_first(pool, changed_paths); hi; hi = apr_hash_next(hi))
> +      {
> +        svn_fs_path_change2_t *change;
> +        apr_hash_this(hi, NULL, NULL, (void **)&change);
"change = svn__apr_hash_index_key(hi);" ?


-- 
Ivan Zhakov
CTO | VisualSVN | http://www.visualsvn.com


[prev in list] [next in list] [prev in thread] [next in thread]