apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Saxon Druce" <sa...@blocksoftware.com>
Subject Re: [PATCH] filepath.c with Borland C++ Builder
Date Wed, 25 Jun 2003 06:54:13 GMT
> >> 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.

Yup that would be great, thanks :)

Saxon


Mime
View raw message