Re: svn commit: r921716 - /subversion/trunk/subversion/libsvn_subr/mergeinfo.c

2010-03-12 Thread Julian Foad
I (Julian Foad) wrote: > Yup, "const" would be better. But in order to do that we need to > constify the various functions in this part of the API, such as > svn_rangelist_dup(). > > I'll do a big sweeping change, constifying all apr_array_header_t *'s > that appear as inputs (i.e. logically cons

Re: svn commit: r921716 - /subversion/trunk/subversion/libsvn_subr/mergeinfo.c

2010-03-12 Thread Julian Foad
Greg Stein wrote: > On Thu, Mar 11, 2010 at 03:08, wrote: > >... > > +++ subversion/trunk/subversion/libsvn_subr/mergeinfo.c Thu Mar 11 08:08:06 > > 2010 > >... > > @@ -1311,23 +1309,20 @@ svn_mergeinfo_intersect2(svn_mergeinfo_t > > for (hi = apr_hash_first(apr_hash_pool_get(mergeinfo1), mer

Re: svn commit: r921716 - /subversion/trunk/subversion/libsvn_subr/mergeinfo.c

2010-03-11 Thread Greg Stein
On Thu, Mar 11, 2010 at 03:08, wrote: >... > +++ subversion/trunk/subversion/libsvn_subr/mergeinfo.c Thu Mar 11 08:08:06 > 2010 >... > @@ -1311,23 +1309,20 @@ svn_mergeinfo_intersect2(svn_mergeinfo_t >   for (hi = apr_hash_first(apr_hash_pool_get(mergeinfo1), mergeinfo1); >        hi; hi = apr_h