makl <makl@tigris.org> writes:
> Index: subversion/libsvn_wc/adm_ops.c
> ===================================================================
> --- subversion/libsvn_wc/adm_ops.c (revision 9193)
> +++ subversion/libsvn_wc/adm_ops.c (working copy)
> @@ -200,20 +202,13 @@
> SVN_ERR (svn_wc_adm_retrieve (&dir_access, adm_access, path, pool));
>
> if (! recursive)
> - {
> - svn_boolean_t write_required = FALSE;
> - SVN_ERR (svn_wc_entries_read (&entries, dir_access, TRUE, pool));
> - SVN_ERR (svn_wc__tweak_entry (entries, SVN_WC_ENTRY_THIS_DIR,
> - base_url, new_revision, &write_required,
> - svn_wc_adm_access_pool (dir_access)));
> - if (write_required)
> - SVN_ERR (svn_wc__entries_write (entries, dir_access, pool));
> - }
> + SVN_ERR (tweak_entries (dir_access, base_url, new_revision,
> + NULL, NULL, FALSE,
> + FALSE, pool));
Why is this passing FALSE for remove_missing_dirs (and thus NULL for
the notify stuff)? I'm not saying it's wrong, I just don't understand
it.
> else
> - SVN_ERR (recursively_tweak_entries (dir_access, base_url,
> - new_revision, notify_func,
> - notify_baton, remove_missing_dirs,
> - pool));
> + SVN_ERR (tweak_entries (dir_access, base_url, new_revision,
> + notify_func, notify_baton, remove_missing_dirs,
> + TRUE, pool));
--
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Mar 24 00:31:16 2004