Re: svn commit: r15799 - trunk/subversion/include

Garrett Rooney wrote:

> Branko ÄŒibej wrote:
>
>> But first of all, the correct way to go about this is to use the APR
>> macros if they're available: therefore,
>>
>> #ifndef APR_ARRAY_IDX
>> #define APR_ARRAY_IDX ...
>>
>> Etc.
>
>
> Well, it sure seems like 6 of one, half dozen of the other to me, but
> if you want it done that way I have no objection. Feel free to commit
> the change, or if you don't get around to it I'll take a look tonight
> when I get home from work.

The significant bit you're missing here is that the definitions of the
macros in APR _could_ change in the future, so we certainly want to use
the APR macros if they're available.

I'm horribly busy today, so you'll probably get to make the change
before I do...

-- Brane

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Aug 18 17:26:16 2005