Re: svn commit: r921916 - in /subversion/trunk/subversion: libsvn_repos/fs-wrap.c libsvn_repos/load.c svn/status-cmd.c

[Julian Foad <ju...@wandisco.com>]

On Thu, 2010-03-11, Greg Stein wrote:
> On Thu, Mar 11, 2010 at 11:13,  <ju...@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