Re: svn commit: r13631 - in trunk/subversion: include libsvn_fs libsvn_fs_base libsvn_fs_fs libsvn_ra_local libsvn_repos mod_dav_svn svnserve tests/libsvn_fs tests/libsvn_repos

On Thu, 2005-03-24 at 09:50, C. Michael Pilato wrote:
> My fault. I started to make it an apr_time_t, then bailed (but forgot
> to tweak the log). My reasoning was that an apr_time_t is supposed to
> represent a moment in Time, not a span thereof. I dunno. I'd not -1
> a change to apr_time_t if the community wants it.

There's a bit of mental judo involved here: if I want to represent the
difference between two numbers of integral type foo_t, what type should
I use? Well, if the numbers and the difference are known to be
positive, then foo_t will definitely work, even though it's not intended
to represent a difference. Any other type isn't guaranteed to work.

(Sometimes people don't apply this mental judo. For instance, the C
standard chose to introduce ptrdiff_t as the result of subtracting one
pointer from another, even though size_t would have worked fine. I
guess they were just conceptually uncomfortable with the idea of using a
"size" type to represent a distance.)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Mar 24 18:23:48 2005