Re: svn commit: rev 1109 - trunk/subversion/svnadmin trunk/subversion/libsvn_fs trunk/subversion/libsvn_wc trunk/subversion/svnlook trunk/subversion/mod_dav_svn trunk/subversion/clients/cmdline trunk/subversion/tests/libsvn_fs trunk/subversion/libsvn_delta trunk/subversion/libsvn_ra_dav

Greg Stein <gstein@lyra.org> writes:
> Probably would have been even better to have a function in libsvn_subr do
> this so that everything is isolated into one place. If we ever want to
> change something aboue marshalling svn_revnum_t back/forth to strings...
>
> Obviously, not a "must have" but if a function was added, then we could
> switch the code whenever we were near it...

Yeah, I thought about that and decided not to for now. svn_revnum_t
is unlikely to change its underlying type, and all these instances
should now be easy to find (oooh... if, that is, we are consistent
about using casted atol in the future... heh, another reason to prefer
a single conversion routine, yes, I see what you mean... I think I'll
just do that now, if we hesitate we are lost).

-K

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:37:01 2006