subversion-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Paul Burba <ptbu...@gmail.com>
Subject Re: svn commit: r1229980 - /subversion/trunk/subversion/libsvn_wc/externals.c
Date Wed, 11 Jan 2012 15:14:14 GMT
On Wed, Jan 11, 2012 at 7:29 AM,  <rhuijben@apache.org> wrote:
> Author: rhuijben
> Date: Wed Jan 11 12:29:49 2012
> New Revision: 1229980
>
> URL: http://svn.apache.org/viewvc?rev=1229980&view=rev
> Log:
> * subversion/libsvn_wc/externals.c
>  (close_file): Following up on r1229975, read the right set of properties in
>    each variable. Use had_props to avoid an unneeded database transaction in
>    some cases.
>
> Modified:
>    subversion/trunk/subversion/libsvn_wc/externals.c
>
> Modified: subversion/trunk/subversion/libsvn_wc/externals.c
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_wc/externals.c?rev=1229980&r1=1229979&r2=1229980&view=diff
> ==============================================================================
> --- subversion/trunk/subversion/libsvn_wc/externals.c (original)
> +++ subversion/trunk/subversion/libsvn_wc/externals.c Wed Jan 11 12:29:49 2012
> @@ -621,11 +621,13 @@ close_file(void *file_baton,
>       {
>         new_checksum = eb->original_checksum;
>
> -        SVN_ERR(svn_wc__db_base_get_props(&actual_props, eb->db,
> -                                          eb->local_abspath,
pool, pool));
> -        SVN_ERR(svn_wc__db_read_pristine_props(&base_props, eb->db,
> -                                               eb->local_abspath,
> -                                               pool, pool));
> +        if (eb->had_props)
> +          SVN_ERR(svn_wc__db_base_get_props(&base_props, eb->db,
> +                                            eb->local_abspath,
> +                                            pool, pool));


Hi Bert,

Why did you replace svn_wc__db_read_pristine_props with
svn_wc__db_base_get_props here?  The log message doesn't touch upon
why you did.  It might be obvious to those with strong wcng-fu, but
I'm a bit puzzled :-)

Paul


> +        SVN_ERR(svn_wc__db_read_props(&actual_props, eb->db,
> +                                      eb->local_abspath, pool,
pool));
>       }
>
>     if (!base_props)
>
>
Mime
View raw message