Login | Register
My pages Projects Community openCollabNet

Discussions > dev [DISABLED] > Re: svn commit: r38702 - trunk/subversion/libsvn_client

subversion
Discussion topic

Back to topic list

Re: svn commit: r38702 - trunk/subversion/libsvn_client

Author gstein
Full name Greg Stein
Date 2009-08-12 15:33:03 PDT
Message On Wed, Aug 12, 2009 at 20:16, Hyrum K. Wright<hyrum@hyru​mwright.org> wrote:
>...
> +++ trunk/subversion/lib​svn_client/merge.c      Wed Aug 12 11:16:02 2009        (r38702)
>...
> @@ -3675,8 +3677,8 @@ find_gaps_in_merge_s​ource_history(svn_re​
>   peg_rev.value.number = young_rev;
>   SVN_ERR(svn_client__​get_history_as_merge​info(&implicit_s​rc_mergeinfo, url,
>                                                &peg_rev, young_rev, old_rev,
> -                                               ra_session, NULL,
> -                                               merge_b->ctx, scratch_pool));
> +                                               ra_session, merge_b->ctx,
> +                                               result_pool));

Why did the pool change here?

>...
> +++ trunk/subversion/lib​svn_client/url.c        Wed Aug 12 11:16:02 2009        (r38702)
> @@ -46,9 +46,16 @@ svn_client_url_from_path(const char **ur
>                          apr_pool_t *pool)
>  {
>   svn_opt_revision_t revision;
> +  svn_client_ctx_t *ctx;
> +
> +  SVN_ERR(svn_client_​create_context(&​ctx, pool));
> +
> +  if (!svn_path_is_url(path_or_url))
> +    SVN_ERR(svn_dirent_​get_absolute(&pa​th_or_url, path_or_url, pool));
> +
>   revision.kind = svn_opt_revision_unspecified;
>   return svn_client__derive_l​ocation(url, NULL, path_or_url, &revision,
> -                                     NULL, NULL, NULL, pool);
> +                                     NULL, ctx, pool, pool);
>  }

Seems a bit "dangerous" to leave this ctx around waiting for the pool
to be cleared. Couldn't we end up with some dangling handles on DBs?

>...

Cheers,
-g

« Previous message in topic | 1 of 2 | Next message in topic »

Messages

Show all messages in topic

Re: svn commit: r38702 - trunk/subversion/libsvn_client gstein Greg Stein 2009-08-12 15:33:03 PDT
     Re: svn commit: r38702 - trunk/subversion/libsvn_client hwright Hyrum K. Wright 2009-08-13 08:30:31 PDT
Messages per page: