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

[Greg Hudson <gh...@MIT.EDU>]

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