Hi Jim, > I did most of this weeks ago and just noticed I never checked it in. > Ok with you, Bruno?
I'll make up my mind on it once you have put an 'mreadlink' module into gnulib. I don't see such a module in gnulib, nor in coreutils. Remember that the difference between xreadlink_with_size and xreadlink was that in one case a (likely good) estimate for the size is already known and in the other case there is no such estimate. Also, about the naming convention: In the *printf family of functions, the prefix 'a' is used to denote a function that allocates memory with malloc: asprintf vs. sprintf. I would prefer to have a single such convention in gnulib, not 'a' for some functions and 'm' for others. And since the 'a' convention comes from glibc, I suggest renaming 'mreadlink_with_size' to 'areadlink_with_size'. Bruno
