apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Brian Pane <bri...@apache.org>
Subject Re: [PATCH] filepath.c with Borland C++ Builder
Date Wed, 25 Jun 2003 06:53:55 GMT
Saxon Druce wrote:

>----- Original Message -----
>From: "Brian Pane" <brianp@apache.org>
>To: "APR dev" <dev@apr.apache.org>
>Sent: Wednesday, June 25, 2003 2:42 PM
>Subject: Re: [PATCH] filepath.c with Borland C++ Builder
>

>> why initialize separator only
>>to overwrite it?  I.e., why not:
>>    char separator[2];
>>    const char *delim1;
>>    const char *delim2;
>>
>>    seperator[0] = (flags & APR_FILEPATH_NATIVE) ? '\\' : '/';
>>    separator[1] = 0;
>>    
>>
>
>I copied the method used for the #ifdef NETWARE case further up in the
>function. I can change it though and submit a new patch if that would give
>it a better chance of being applied?
>  
>

No need to send an updated patch; I'll just go ahead and commit
it now if the change is okay with you.

Brian



Mime
View raw message