On Wed, May 12, 2010 at 12:41, <julianf...@apache.org> wrote:
>...
> +++ subversion/trunk/subversion/libsvn_wc/wc_db.c Wed May 12 16:41:18 2010
> @@ -3767,6 +3767,49 @@ svn_wc__db_temp_op_remove_entry(svn_wc__
> }
>
>
> +svn_error_t *
> +svn_wc__db_temp_op_remove_working(svn_wc__db_t *db,
> + const char *local_abspath,
> + apr_pool_t *scratch_pool)
> +{
> + svn_wc__db_pdh_t *pdh;
> + svn_sqlite__stmt_t *stmt;
> + svn_sqlite__db_t *sdb;
> + apr_int64_t wc_id;
> + const char *local_relpath;
> +
> + SVN_ERR_ASSERT(svn_dirent_is_absolute(local_abspath));
> +
> + SVN_ERR(parse_local_abspath(&pdh, &local_relpath, db, local_abspath,
> + svn_sqlite__mode_readwrite,
> + scratch_pool, scratch_pool));
> + VERIFY_USABLE_PDH(pdh);
> +
> + flush_entries(pdh);
> +
> + /* Check if we should remove it from the parent db instead */
Instead? It should be removing it from both places.
>...
> + sdb = pdh->wcroot->sdb;
> + wc_id = pdh->wcroot->wc_id;
Precedent is to just spell it out in the usage, rather than have local
variables for this.
>...
Cheers,
-g
