Saxon Druce wrote:

----- Original Message -----
From: "Brian Pane" <[EMAIL PROTECTED]>
To: "APR dev" <[email protected]>
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




Reply via email to