On Wed, Mar 17, 2010 at 05:37, <[email protected]> wrote: >... > +++ subversion/trunk/subversion/libsvn_subr/mergeinfo.c Wed Mar 17 09:37:11 > 2010 >... > @@ -763,20 +765,31 @@ svn_rangelist_merge(apr_array_header_t * > return SVN_NO_ERROR; > } > > +/* Return TRUE iff the forward revision ranges FIRST and SECOND overlap and > + * (if CONSIDER_INHERITANCE is TRUE) have the same inheritability. */ > static svn_boolean_t > range_intersect(const svn_merge_range_t *first, const svn_merge_range_t > *second, > svn_boolean_t consider_inheritance) > { > + SVN_ERR_ASSERT_NO_RETURN(IS_VALID_FORWARD_RANGE(first)); > + SVN_ERR_ASSERT_NO_RETURN(IS_VALID_FORWARD_RANGE(second));
If somebody manually tweaks an svn:mergeinfo property to include 0, then is our library going to dump core? (I imagine props are checked much earlier, but a new core dump is always something to question :-P ) >... Cheers, -g
