[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Fix for issue #2982

From: Kamesh Jayachandran <kamesh_at_collab.net>
Date: 2007-11-13 16:02:56 CET

> [[[
> Fix issue #2982.
> Reflect dropped/renumbered revisions in modified svn:mergeinfo data
> * subversion/svndumpfilter/main.c
> (include): Include svn_mergeinfo.h
No *(include)*, it should be ().

>+/* Skip missing merge sources in mergeinfo. */

Fix the docstring , purpose of this helper function is not matching the
doc string.

> static svn_error_t *
>+skip_missing_merge_sources(const char **mergeinfo_val, const char
>+ struct revision_baton_t *rb, apr_pool_t *pool)

Fix the alignment issue here.

This helper function name is not correct, it should be

>+ /* Renumber mergeinfo rangelist. */
>+ for (i = 0; i < ((apr_array_header_t *)rangelist)->nelts; i++)
>+ {
>+ struct revmap_t *renum_start;
>+ struct revmap_t *renum_end;

Probably you can name this as 'revmap_start' and 'revmap_end', my first
look at it just by name in the later part of the code made me to believe
that it was a revnum which was wrong.

>+ svn_merge_range_t *range;
>+ svn_merge_range_t *newrange = apr_pcalloc(pool, sizeof(*range));

No need to allocate 'newrange', you can just overwrite range's members.

>+ SVN_ERR(svn_mergeinfo_sort(modified_mergeinfo, pool));

I don't think we need to sort here, as the incoming range themselves
should be in a sorted order. Anyways resorting should not hurt.

>+ if (strcmp(name, SVN_PROP_MERGE_INFO) == 0)
>+ {
>+ /* Skip missing merge sources. */
>+ const char *mergeinfo_val;
>+ SVN_ERR(skip_missing_merge_sources(&mergeinfo_val, value->data, rb,
>+ subpool));

Formatting issue with arguments.

With regards
Kamesh Jayachandran

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Nov 13 16:02:53 2007

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.