Re: question about range_intersect() and range_contains() functions

On Thu, Jul 06, 2006 at 04:53:31PM +0530, Madan U Sreenivasan wrote:
> In the merge-tracking branch, I notice that the range_intersect() and
> the range_contains() functions in the
> subversion/libsvn_subr/mergeinfo.c file are single line functions. Am
> thinking why not make them a macro. Is there any specific reason these are
> functions?
>

Are these public functions? (I haven't looked). If so, making them
macros would mean that we couldn't change the implementation, which
might be a good reason to keep them as functions.

Regards,
Malcolm

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Jul 6 13:38:31 2006