Re: [PATCH] svn_fs_revision_prop and friends

Ben Collins-Sussman <sussman@newton.ch.collab.net> writes:

> Yoshiki Hayashi <yoshiki@xemacs.org> writes:
>
> > +/* Return true iff SKEL is an atom whose data is the same as STR. */
> > +int svn_fs__atom_matches_string (skel_t *skel, svn_string_t *str);
>
> Is this really necessary? Can't you just call
>
> svn_fs__matches_atom (skel, string->data)
>
> Remember than svn_string_t's are guaranteed to be null-terminated, so
> you can use them as char *'s effortlessly.

It isn't necessary but it doesn't hurt either. If
string->data is big, saving a call to strlen() might mean
something. If you don't like it, I'll send a new patch
without above change.

-- 
Yoshiki Hayashi