On Tue, 2011-02-08 at 10:16 +0100, Ondrej Valousek wrote: > > > Yes, and replied on the 11/01, did you miss it? > > > Ok I must have missed it - now I see you have taken a different > approach - you are constructing a single looong string with all ldap > uris and then you parse them again. > I must say I do not like it too much as: > a) it is wasted effort (merging together and then splitting again) > b) potentially can ldap uri contain spaces I think
Don't think so, in this case anyway. The uris used here aren't full uris, they can only have the proto, host name and port. AFAIK, host names can't have spaces in them yet. > > But if you like it more, I am fine with it, too. Yeah, your points above are well taken but had already been done and committed to the repo so to change would be more work with the potential for mistakes. Ian _______________________________________________ autofs mailing list autofs@linux.kernel.org http://linux.kernel.org/mailman/listinfo/autofs
