> >> 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
