On Thu, 2010-03-11, Greg Stein wrote: > On Thu, Mar 11, 2010 at 11:13, <julianf...@apache.org> wrote: > >... > > +++ subversion/trunk/subversion/libsvn_repos/load.c Thu Mar 11 16:13:39 2010 > > @@ -279,8 +279,8 @@ renumber_mergeinfo_revs(svn_string_t **f > > void *val; > > > > apr_hash_this(hi, &key, NULL, &val); > > - merge_source = (const char *) key; > > - rangelist = (apr_array_header_t *) val; > > + merge_source = key; > > + rangelist = val; > > Wouldn't it be better to just use our hash_index_key/val functions here?
Absolutely, but I was just doing a sweep of type-cast-to-array-* removal and didn't want to stop and make other changes at that time. I'll do it some time if no-one gets there before me. - Julian
