> >
> > [[[
> > Patch by: Kamesh Jayachandran <kamesh@collab.net>
> >
> > Combining the revert ranges is undefined.
> >
> > * subversion/libsvn_subr/mergeinfo.c
> > (combine_ranges):
> > Combining the revert ranges is undefined.
> > ]]]
> >
> > ------------------------------------------------------------------------
> >
I'm not sure what you think this patch does, but it's not right to
remove these cases.
The cases below that you are removing combine ranges in case you have
the following:
in1 = [0, 10]
in2 = [4, 7]
and
in1= [1, 3]
in2 = [0, 10]
In both cases, one range completely subsumes the other, so we just use
one of the ranges.
It has nothing to do with revert ranges.
> > Index: subversion/libsvn_subr/mergeinfo.c
> > ===================================================================
> > --- subversion/libsvn_subr/mergeinfo.c (revision 21340)
> > +++ subversion/libsvn_subr/mergeinfo.c (working copy)
> > @@ -73,20 +73,6 @@
> > (*output)->end = MAX(in1->end, in2->end);
> > return TRUE;
> > }
> > - else if (in1->start >= in2->start
> > - && in1->end <= in2->end)
> > - {
> > - (*output)->start = in2->start;
> > - (*output)->end = in2->end;
> > - return TRUE;
> > - }
> > - else if (in2->start >= in1->start
> > - && in2->end <= in1->end)
> > - {
> > - (*output)->start = in1->start;
> > - (*output)->end = in1->end;
> > - return TRUE;
> > - }
> >
> > return FALSE;
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Sep 6 16:08:40 2006