[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Karl Fogel <kfogel_at_newton.ch.collab.net>
Date: 2002-01-30 23:21:41 CET

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

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.